[PATCH 1/4] drm/crtc: Remove redundant NULL check before kfree

Sachin Kamat sachin.kamat at linaro.org
Wed Nov 21 07:12:30 PST 2012


Hi Dave,

Please ignore this patch.


On 19 November 2012 15:14, Sachin Kamat <sachin.kamat at linaro.org> wrote:
> kfree() on a NULL input is a no-op. Hence remove the check.
>
> Signed-off-by: Sachin Kamat <sachin.kamat at linaro.org>
> ---
> This series is build tested on the latest linux-next (20121115).
> ---
>  drivers/gpu/drm/drm_crtc.c |    6 ++----
>  1 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
> index 3533609..807336a 100644
> --- a/drivers/gpu/drm/drm_crtc.c
> +++ b/drivers/gpu/drm/drm_crtc.c
> @@ -470,10 +470,8 @@ void drm_crtc_cleanup(struct drm_crtc *crtc)
>  {
>         struct drm_device *dev = crtc->dev;
>
> -       if (crtc->gamma_store) {
> -               kfree(crtc->gamma_store);
> -               crtc->gamma_store = NULL;
> -       }
> +       kfree(crtc->gamma_store);
> +       crtc->gamma_store = NULL;
>
>         drm_mode_object_put(dev, &crtc->base);
>         list_del(&crtc->head);
> --
> 1.7.4.1
>



-- 
With warm regards,
Sachin


More information about the dri-devel mailing list