diff options
| author | Suraj Jitindar Singh <sjitindarsingh@gmail.com> | 2016-06-29 05:38:38 +0200 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2016-06-29 09:33:18 +0200 |
| commit | d0226d315dba5e401a124b394a1af5e35e082b08 (patch) | |
| tree | 1cd2e85e40b4e49bc53c7de0232d56f69f6daadf /drivers/leds | |
| parent | devicetree/bindings: Add binding for operator panel on FSP machines (diff) | |
| download | linux-d0226d315dba5e401a124b394a1af5e35e082b08.tar.xz linux-d0226d315dba5e401a124b394a1af5e35e082b08.zip | |
powerpc/opal: Add inline function to get rc from an ASYNC_COMP opal_msg
An opal_msg of type OPAL_MSG_ASYNC_COMP contains the return code in the
params[1] struct member. However this isn't intuitive or obvious when
reading the code and requires that a user look at the skiboot
documentation or opal-api.h to verify this.
Add an inline function to get the return code from an opal_msg and update
call sites accordingly.
Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Diffstat (limited to 'drivers/leds')
| -rw-r--r-- | drivers/leds/leds-powernv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/leds/leds-powernv.c b/drivers/leds/leds-powernv.c index dfb8bd390125..b2a98c7b521b 100644 --- a/drivers/leds/leds-powernv.c +++ b/drivers/leds/leds-powernv.c @@ -118,7 +118,7 @@ static int powernv_led_set(struct powernv_led_data *powernv_led, goto out_token; } - rc = be64_to_cpu(msg.params[1]); + rc = opal_get_async_rc(msg); if (rc != OPAL_SUCCESS) dev_err(dev, "%s : OAPL async call returned failed [rc=%d]\n", __func__, rc); |