]> git.karo-electronics.de Git - linux-beck.git/commitdiff
staging: greybus: light: check the correct value of delay_on
authorRui Miguel Silva <rmfrfs@gmail.com>
Fri, 30 Sep 2016 17:26:55 +0000 (18:26 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 2 Oct 2016 13:49:26 +0000 (15:49 +0200)
When checking the value of delay_on to set the channel as active, it was
checked the pointer and not the value, as it should be.

Fixes: cc43368a3c ("greybus: lights: Control runtime pm suspend/resume on AP side")
Signed-off-by: Rui Miguel Silva <rmfrfs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/greybus/light.c

index 80dc4a9d9a0e9f2a8bc4a5250eae8a3e49e9b772..8dffd8a7e762d28d6448650c98f97e7874b6c43b 100644 (file)
@@ -483,7 +483,7 @@ static int gb_blink_set(struct led_classdev *cdev, unsigned long *delay_on,
        if (ret < 0)
                goto out_pm_put;
 
-       if (delay_on)
+       if (*delay_on)
                channel->active = true;
        else
                channel->active = false;