greybus: lights: remove unnecessary checks
authorRui Miguel Silva <rui.silva@linaro.org>
Wed, 17 Feb 2016 14:32:51 +0000 (14:32 +0000)
committerGreg Kroah-Hartman <gregkh@google.com>
Wed, 17 Feb 2016 23:24:09 +0000 (15:24 -0800)
We do not need to check for channels and lights as they can never be
NULL as a big memory array elements.

Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org>
Reported-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
drivers/staging/greybus/light.c

index 70fcade451dc036cb209c993cc8439729eeb9590..6c5b9994457cb28d0666eedc5927bbc8e917c770 100644 (file)
@@ -1026,9 +1026,6 @@ static void gb_lights_channel_free(struct gb_channel *channel)
 
 static void gb_lights_channel_release(struct gb_channel *channel)
 {
-       if (!channel)
-               return;
-
        channel->releasing = true;
 
        gb_lights_channel_unregister(channel);
@@ -1041,8 +1038,6 @@ static void gb_lights_light_release(struct gb_light *light)
        int i;
        int count;
 
-       if (!light)
-               return;
 
        count = light->channels_count;