rtc: r9701: stop setting RWKCNT
authorAlexandre Belloni <alexandre.belloni@bootlin.com>
Thu, 15 Oct 2020 19:11:33 +0000 (21:11 +0200)
committerAlexandre Belloni <alexandre.belloni@bootlin.com>
Mon, 19 Oct 2020 20:48:55 +0000 (22:48 +0200)
tm_wday is never checked for validity and it is not read back in
r9701_get_datetime. Avoid setting it to stop tripping static checkers:

        drivers/rtc/rtc-r9701.c:109 r9701_set_datetime()
        error: undefined (user controlled) shift '1 << dt->tm_wday'

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Link: https://lore.kernel.org/r/20201015191135.471249-4-alexandre.belloni@bootlin.com
drivers/rtc/rtc-r9701.c

index 4b688e9c41921fa8d3936e361f29e0682f5b2192..183c5a0fe78c36252fb2e9b227ef97bbce531c5f 100644 (file)
@@ -100,7 +100,6 @@ static int r9701_set_datetime(struct device *dev, struct rtc_time *dt)
        ret = ret ? ret : write_reg(dev, RDAYCNT, bin2bcd(dt->tm_mday));
        ret = ret ? ret : write_reg(dev, RMONCNT, bin2bcd(dt->tm_mon + 1));
        ret = ret ? ret : write_reg(dev, RYRCNT, bin2bcd(dt->tm_year - 100));
-       ret = ret ? ret : write_reg(dev, RWKCNT, 1 << dt->tm_wday);
 
        return ret;
 }