From: Rui Miguel Silva <rmfrfs@gmail.com>
Date: Fri, 30 Sep 2016 17:26:55 +0000 (+0100)
Subject: staging: greybus: light: check the correct value of delay_on
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=5cf62679153ed7014d6952bb6a5094b3ee31428d;p=linux-beck.git

staging: greybus: light: check the correct value of delay_on

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>
---

diff --git a/drivers/staging/greybus/light.c b/drivers/staging/greybus/light.c
index 80dc4a9d9a0e..8dffd8a7e762 100644
--- a/drivers/staging/greybus/light.c
+++ b/drivers/staging/greybus/light.c
@@ -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;