diff options
| author |   Rui Miguel Silva <rui.silva@linaro.org> | 2016-08-16 22:31:56 +0100 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@google.com> | 2016-08-18 17:34:01 +0200 |
| commit | 5f66d62e8372d8a9c97bd75d772a5a5788925218 (patch) | |
| tree | 44676a02dfa6bdad107f68a85e7a37928dfc1eee /drivers/staging/greybus/power_supply.c | |
| parent | greybus: power_supply: add callback to handle power supply changes (diff) | |
| download | linux-dev-5f66d62e8372d8a9c97bd75d772a5a5788925218.tar.xz linux-dev-5f66d62e8372d8a9c97bd75d772a5a5788925218.zip | |
greybus: power_supply: fix update interval check at request handler
We use the update interval to control the remove path and we set it to
zero when we do not want to have more updates in transit. That means
that the check in the request handler needs to be for interval update
zero to discard the newly received request and not the other way around
like it is.
This will fix the issue that all incoming requests were being discard.
Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/power_supply.c')
| -rw-r--r-- | drivers/staging/greybus/power_supply.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/greybus/power_supply.c b/drivers/staging/greybus/power_supply.c index aeb6a07e43a6..3d6f81017d98 100644 --- a/drivers/staging/greybus/power_supply.c +++ b/drivers/staging/greybus/power_supply.c @@ -982,7 +982,7 @@ static int gb_supplies_request_handler(struct gb_operation *op) * running. For that just check update_interval. */ gbpsy = &supplies->supply[psy_id]; - if (gbpsy->update_interval) { + if (!gbpsy->update_interval) { ret = -ESHUTDOWN; goto out_unlock; } |