Message ID | 20220208162908.3182581-1-hugo@hugovil.com |
---|---|
State | Accepted |
Headers | show |
Series | rtc: pcf2127: fix bug when reading alarm registers | expand |
On Tue, 8 Feb 2022 11:29:07 -0500, Hugo Villeneuve wrote: > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > The first bug is that reading the 5 alarm registers results in a read > operation of 20 bytes. The reason is because the destination buffer is > defined as an array of "unsigned int", and we use the sizeof() > operator on this array to define the bulk read count. > > [...] Applied, thanks! [1/1] rtc: pcf2127: fix bug when reading alarm registers commit: 73ce05302007eece23a6acb7dc124c92a2209087 Best regards,
diff --git a/drivers/rtc/rtc-pcf2127.c b/drivers/rtc/rtc-pcf2127.c index 823abe2a7147..d88c0fdbe117 100644 --- a/drivers/rtc/rtc-pcf2127.c +++ b/drivers/rtc/rtc-pcf2127.c @@ -374,7 +374,8 @@ static int pcf2127_watchdog_init(struct device *dev, struct pcf2127 *pcf2127) static int pcf2127_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm) { struct pcf2127 *pcf2127 = dev_get_drvdata(dev); - unsigned int buf[5], ctrl2; + u8 buf[5]; + unsigned int ctrl2; int ret; ret = regmap_read(pcf2127->regmap, PCF2127_REG_CTRL2, &ctrl2);