summaryrefslogtreecommitdiffstats
path: root/drivers/rtc
diff options
context:
space:
mode:
authorEmmanuel Gil Peyrot <linkmauve@linkmauve.fr>2022-08-23 15:07:02 +0200
committerAlexandre Belloni <alexandre.belloni@bootlin.com>2022-08-23 22:27:58 +0200
commit509451ac03eb3afa4c4a32d4c11b1938f08de8e4 (patch)
tree3181eea199c128fab74776008b5dd3cc6a0126de /drivers/rtc
parent1e2585b49d849196f359bbf86677943fe2d80afe (diff)
downloadlinux-509451ac03eb3afa4c4a32d4c11b1938f08de8e4.tar.gz
linux-509451ac03eb3afa4c4a32d4c11b1938f08de8e4.tar.bz2
linux-509451ac03eb3afa4c4a32d4c11b1938f08de8e4.zip
rtc: gamecube: Always reset HW_SRNPROT after read
This register would fail to be reset if reading the RTC bias failed for whichever reason. This commit reorganises the code around to unconditionally write it back to its previous value, unmap it, and return the result of regmap_read(), which makes it both simpler and more correct in the error case. Signed-off-by: Emmanuel Gil Peyrot <linkmauve@linkmauve.fr> Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com> Link: https://lore.kernel.org/r/20220823130702.1046-1-linkmauve@linkmauve.fr
Diffstat (limited to 'drivers/rtc')
-rw-r--r--drivers/rtc/rtc-gamecube.c11
1 files changed, 5 insertions, 6 deletions
diff --git a/drivers/rtc/rtc-gamecube.c b/drivers/rtc/rtc-gamecube.c
index c2717bb52b2b..c828bc8e05b9 100644
--- a/drivers/rtc/rtc-gamecube.c
+++ b/drivers/rtc/rtc-gamecube.c
@@ -265,18 +265,17 @@ static int gamecube_rtc_read_offset_from_sram(struct priv *d)
* SRAM address as on previous consoles.
*/
ret = regmap_read(d->regmap, RTC_SRAM_BIAS, &d->rtc_bias);
- if (ret) {
- pr_err("failed to get the RTC bias\n");
- iounmap(hw_srnprot);
- return -1;
- }
/* Reset SRAM access to how it was before, our job here is done. */
if (old != 0x7bf)
iowrite32be(old, hw_srnprot);
+
iounmap(hw_srnprot);
- return 0;
+ if (ret)
+ pr_err("failed to get the RTC bias\n");
+
+ return ret;
}
static const struct regmap_range rtc_rd_ranges[] = {