rtc: st-lpc: make it robust against y2038/2106 bug
authorBenjamin Gaignard <benjamin.gaignard@linaro.org>
Mon, 19 Jun 2017 09:36:22 +0000 (11:36 +0200)
committerAlexandre Belloni <alexandre.belloni@free-electrons.com>
Sun, 9 Jul 2017 20:32:18 +0000 (22:32 +0200)
Make driver use u64 variables and functions to be sure that
it will support dates after year 2038.

Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
Acked-by: Patrice Chotard <patrice.chotard@st.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
drivers/rtc/rtc-st-lpc.c

index 74c0a336ceea0650965701f93399814299275bd1..82b0af159a28221cff367964b27206e8b402385d 100644 (file)
@@ -99,7 +99,7 @@ static int st_rtc_read_time(struct device *dev, struct rtc_time *tm)
 
        lpt = ((unsigned long long)lpt_msb << 32) | lpt_lsb;
        do_div(lpt, rtc->clkrate);
-       rtc_time_to_tm(lpt, tm);
+       rtc_time64_to_tm(lpt, tm);
 
        return 0;
 }
@@ -107,13 +107,10 @@ static int st_rtc_read_time(struct device *dev, struct rtc_time *tm)
 static int st_rtc_set_time(struct device *dev, struct rtc_time *tm)
 {
        struct st_rtc *rtc = dev_get_drvdata(dev);
-       unsigned long long lpt;
-       unsigned long secs, flags;
-       int ret;
+       unsigned long long lpt, secs;
+       unsigned long flags;
 
-       ret = rtc_tm_to_time(tm, &secs);
-       if (ret)
-               return ret;
+       secs = rtc_tm_to_time64(tm);
 
        lpt = (unsigned long long)secs * rtc->clkrate;
 
@@ -161,13 +158,13 @@ static int st_rtc_set_alarm(struct device *dev, struct rtc_wkalrm *t)
 {
        struct st_rtc *rtc = dev_get_drvdata(dev);
        struct rtc_time now;
-       unsigned long now_secs;
-       unsigned long alarm_secs;
+       unsigned long long now_secs;
+       unsigned long long alarm_secs;
        unsigned long long lpa;
 
        st_rtc_read_time(dev, &now);
-       rtc_tm_to_time(&now, &now_secs);
-       rtc_tm_to_time(&t->time, &alarm_secs);
+       now_secs = rtc_tm_to_time64(&now);
+       alarm_secs = rtc_tm_to_time64(&t->time);
 
        /* Invalid alarm time */
        if (now_secs > alarm_secs)