diff options
| author | Rui Miguel Silva <rmfrfs@gmail.com> | 2016-09-30 17:26:55 (GMT) |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-10-02 13:49:26 (GMT) |
| commit | 5cf62679153ed7014d6952bb6a5094b3ee31428d (patch) | |
| tree | 6700f7b8a17bfd7e85598cbcb5771f737c4758e4 | |
| parent | 544a69443bf069fa52b1b3b5e25b682b710210f5 (diff) | |
| download | linux-5cf62679153ed7014d6952bb6a5094b3ee31428d.tar.xz | |
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>
| -rw-r--r-- | drivers/staging/greybus/light.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/greybus/light.c b/drivers/staging/greybus/light.c index 80dc4a9..8dffd8a 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; |