diff options
| author |   Rui Miguel Silva <rui.silva@linaro.org> | 2016-01-12 14:38:21 +0000 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@google.com> | 2016-01-12 11:21:35 -0800 |
| commit | adb57cff54dd4418f4d5b17f03a976dfdc670226 (patch) | |
| tree | 2ca439069ec4375a4208190a626b9261b06c88af /drivers/staging/greybus/power_supply.c | |
| parent | greybus: APBridge: move APBridge request protocol to a common .h file (diff) | |
| download | linux-dev-adb57cff54dd4418f4d5b17f03a976dfdc670226.tar.xz linux-dev-adb57cff54dd4418f4d5b17f03a976dfdc670226.zip | |
greybus: power_supply: fix check for configured supply
The correct check for if the power supply is ready to receive event is
wrong and it should check for the registered flag.
Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org>
Reported-by: Johan Hovold <johan@hovoldconsulting.com>
Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
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 | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/staging/greybus/power_supply.c b/drivers/staging/greybus/power_supply.c index d985e13b5a0d..2dc193a081b0 100644 --- a/drivers/staging/greybus/power_supply.c +++ b/drivers/staging/greybus/power_supply.c @@ -697,7 +697,8 @@ static int gb_power_supply_event_recv(u8 type, struct gb_operation *op) payload = request->payload; psy_id = payload->psy_id; mutex_lock(&supplies->supplies_lock); - if (psy_id >= supplies->supplies_count || !&supplies->supply[psy_id]) { + if (psy_id >= supplies->supplies_count || + !supplies->supply[psy_id].registered) { dev_err(&connection->bundle->dev, "Event received for unconfigured power_supply id: %d\n", psy_id); |