diff options
author | Suraj Jitindar Singh <sjitindarsingh@gmail.com> | 2016-06-28 23:38:38 -0400 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2016-06-29 03:33:18 -0400 |
commit | d0226d315dba5e401a124b394a1af5e35e082b08 (patch) | |
tree | 1cd2e85e40b4e49bc53c7de0232d56f69f6daadf /drivers/rtc | |
parent | 1ae88fd54c3ac31f68f91e37f719be7e2dbcc810 (diff) |
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/rtc')
-rw-r--r-- | drivers/rtc/rtc-opal.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/rtc/rtc-opal.c b/drivers/rtc/rtc-opal.c index 9c18d6fd8107..ea20f627dabe 100644 --- a/drivers/rtc/rtc-opal.c +++ b/drivers/rtc/rtc-opal.c | |||
@@ -134,7 +134,7 @@ static int opal_get_tpo_time(struct device *dev, struct rtc_wkalrm *alarm) | |||
134 | goto exit; | 134 | goto exit; |
135 | } | 135 | } |
136 | 136 | ||
137 | rc = be64_to_cpu(msg.params[1]); | 137 | rc = opal_get_async_rc(msg); |
138 | if (rc != OPAL_SUCCESS) { | 138 | if (rc != OPAL_SUCCESS) { |
139 | rc = -EIO; | 139 | rc = -EIO; |
140 | goto exit; | 140 | goto exit; |
@@ -181,7 +181,7 @@ static int opal_set_tpo_time(struct device *dev, struct rtc_wkalrm *alarm) | |||
181 | goto exit; | 181 | goto exit; |
182 | } | 182 | } |
183 | 183 | ||
184 | rc = be64_to_cpu(msg.params[1]); | 184 | rc = opal_get_async_rc(msg); |
185 | if (rc != OPAL_SUCCESS) | 185 | if (rc != OPAL_SUCCESS) |
186 | rc = -EIO; | 186 | rc = -EIO; |
187 | 187 | ||