aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/rtc
diff options
context:
space:
mode:
authorSuraj Jitindar Singh <sjitindarsingh@gmail.com>2016-06-28 23:38:38 -0400
committerMichael Ellerman <mpe@ellerman.id.au>2016-06-29 03:33:18 -0400
commitd0226d315dba5e401a124b394a1af5e35e082b08 (patch)
tree1cd2e85e40b4e49bc53c7de0232d56f69f6daadf /drivers/rtc
parent1ae88fd54c3ac31f68f91e37f719be7e2dbcc810 (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.c4
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