rtc: ds1302: fix error check in set_time
authorAkinobu Mita <akinobu.mita@gmail.com>
Sun, 10 Apr 2016 14:59:23 +0000 (23:59 +0900)
committerAlexandre Belloni <alexandre.belloni@free-electrons.com>
Fri, 20 May 2016 10:33:51 +0000 (12:33 +0200)
The set_time callback for rtc-ds1302 doesn't write clock registers
because the error check for the return value from spi_write_then_read()
is not correct.  spi_write_then_read() which returns zero on success.

Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Cc: Sergey Yanovich <ynvich@gmail.com>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
drivers/rtc/rtc-ds1302.c

index 7811b357ed705309aae59a5c5cee762684c1e914..283e653fa1896aa9b60bfa91c5b366a22ec9973f 100644 (file)
@@ -53,7 +53,7 @@ static int ds1302_rtc_set_time(struct device *dev, struct rtc_time *time)
 
        status = spi_write_then_read(spi, buf, 2,
                        NULL, 0);
-       if (!status)
+       if (status)
                return status;
 
        /* Write registers starting at the first time/date address. */