rtc: rewrite isl1208 to support DM

Adds devicemodel support to the ISL1208 driver.
This patch drops the non-dm API as no board was using it anyway.
Also add it to Kconfig.

Signed-off-by: Klaus Goger <klaus.goger@theobroma-systems.com>
Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
Reviewed-by: Simon Glass <sjg@chromium.org>

diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
index 95ac0312439deb83eb3b10e126ae9fcb639b563c..277dc3de737c518c841a848e4317953c9dfbc6fa 100644 (file)
--- a/drivers/rtc/Kconfig
+++ b/drivers/rtc/Kconfig
@@ -30,6 +30,18 @@ config RTC_DS1307
          Support for Dallas Semiconductor (now Maxim) DS1307 and DS1338/9 and
          compatible Real Time Clock devices.
 
+config RTC_ISL1208
+       bool "Enable ISL1208 driver"
+       depends on DM_RTC
+       help
+         The Renesas (formerly Intersil) ISL1208 is a I2C Real Time Clock (RTC) and
+         calendar with automatic leap year correction, 2-byte battery backed SRAM,
+         automatic power switch-over, alarm function and 15 selectable frequency
+         outputs.
+
+         This driver supports reading and writing the RTC/calendar and detects
+         total power failures.
+
 config RTC_RX8010SJ
        bool "Enable RX8010SJ driver"
        depends on DM_RTC

diff --git a/drivers/rtc/isl1208.c b/drivers/rtc/isl1208.c
index 807e2e404e11dc2a0e193f3a56510246f89cdf5c..fa1178a36e9bf0ac5b3691e002d4cf90e2bfd683 100644 (file)
--- a/drivers/rtc/isl1208.c
+++ b/drivers/rtc/isl1208.c
@@ -14,6 +14,7 @@
 
 #include <common.h>
 #include <command.h>
+#include <dm.h>
 #include <rtc.h>
 #include <i2c.h>
 
@@ -51,45 +52,38 @@
 #define RTC_STAT_BIT_BAT       0x02    /* BATTERY BIT */
 #define RTC_STAT_BIT_RTCF      0x01    /* REAL TIME CLOCK FAIL BIT */
 
-static uchar rtc_read (uchar reg);
-static void rtc_write (uchar reg, uchar val);
-
 /*
  * Get the current time from the RTC
  */
 
-int rtc_get (struct rtc_time *tmp)
+static int isl1208_rtc_get(struct udevice *dev, struct rtc_time *tmp)
 {
-       int rel = 0;
-       uchar sec, min, hour, mday, wday, mon, year, status;
-
-       status = rtc_read (RTC_STAT_REG_ADDR);
-       sec = rtc_read (RTC_SEC_REG_ADDR);
-       min = rtc_read (RTC_MIN_REG_ADDR);
-       hour = rtc_read (RTC_HR_REG_ADDR);
-       wday = rtc_read (RTC_DAY_REG_ADDR);
-       mday = rtc_read (RTC_DATE_REG_ADDR);
-       mon = rtc_read (RTC_MON_REG_ADDR);
-       year = rtc_read (RTC_YR_REG_ADDR);
-
-       DEBUGR ("Get RTC year: %02x mon: %02x mday: %02x wday: %02x "
-               "hr: %02x min: %02x sec: %02x status: %02x\n",
-               year, mon, mday, wday, hour, min, sec, status);
-
-       if (status & RTC_STAT_BIT_RTCF) {
+       int ret;
+       uchar buf[8], val;
+
+       ret = dm_i2c_read(dev, 0, buf, sizeof(buf));
+       if (ret < 0)
+               return ret;
+
+       if (buf[RTC_STAT_REG_ADDR] & RTC_STAT_BIT_RTCF) {
                printf ("### Warning: RTC oscillator has stopped\n");
-               rtc_write(RTC_STAT_REG_ADDR,
-                       rtc_read(RTC_STAT_REG_ADDR) &~ (RTC_STAT_BIT_BAT|RTC_STAT_BIT_RTCF));
-               rel = -1;
+               ret = dm_i2c_read(dev, RTC_STAT_REG_ADDR, &val, sizeof(val));
+               if (ret < 0)
+                       return ret;
+
+               val = val & ~(RTC_STAT_BIT_BAT | RTC_STAT_BIT_RTCF);
+               ret = dm_i2c_write(dev, RTC_STAT_REG_ADDR, &val, sizeof(val));
+               if (ret < 0)
+                       return ret;
        }
 
-       tmp->tm_sec  = bcd2bin (sec & 0x7F);
-       tmp->tm_min  = bcd2bin (min & 0x7F);
-       tmp->tm_hour = bcd2bin (hour & 0x3F);
-       tmp->tm_mday = bcd2bin (mday & 0x3F);
-       tmp->tm_mon  = bcd2bin (mon & 0x1F);
-       tmp->tm_year = bcd2bin (year)+2000;
-       tmp->tm_wday = bcd2bin (wday & 0x07);
+       tmp->tm_sec  = bcd2bin(buf[RTC_SEC_REG_ADDR] & 0x7F);
+       tmp->tm_min  = bcd2bin(buf[RTC_MIN_REG_ADDR] & 0x7F);
+       tmp->tm_hour = bcd2bin(buf[RTC_HR_REG_ADDR] & 0x3F);
+       tmp->tm_mday = bcd2bin(buf[RTC_DATE_REG_ADDR] & 0x3F);
+       tmp->tm_mon  = bcd2bin(buf[RTC_MON_REG_ADDR] & 0x1F);
+       tmp->tm_year = bcd2bin(buf[RTC_YR_REG_ADDR]) + 2000;
+       tmp->tm_wday = bcd2bin(buf[RTC_DAY_REG_ADDR] & 0x07);
        tmp->tm_yday = 0;
        tmp->tm_isdst= 0;
 
@@ -97,51 +91,88 @@ int rtc_get (struct rtc_time *tmp)
                tmp->tm_year, tmp->tm_mon, tmp->tm_mday, tmp->tm_wday,
                tmp->tm_hour, tmp->tm_min, tmp->tm_sec);
 
-       return rel;
+       return 0;
 }
 
 /*
  * Set the RTC
  */
-int rtc_set (struct rtc_time *tmp)
+static int isl1208_rtc_set(struct udevice *dev, const struct rtc_time *tmp)
 {
+       int ret;
+       uchar val, buf[7];
+
        DEBUGR ("Set DATE: %4d-%02d-%02d (wday=%d)  TIME: %2d:%02d:%02d\n",
                tmp->tm_year, tmp->tm_mon, tmp->tm_mday, tmp->tm_wday,
                tmp->tm_hour, tmp->tm_min, tmp->tm_sec);
 
+       if (tmp->tm_year < 2000 || tmp->tm_year > 2099)
+               printf("WARNING: year should be between 2000 and 2099!\n");
+
        /* enable write */
-       rtc_write(RTC_STAT_REG_ADDR,
-               rtc_read(RTC_STAT_REG_ADDR) | RTC_STAT_BIT_WRTC);
+       ret = dm_i2c_read(dev, RTC_STAT_REG_ADDR, &val, sizeof(val));
+       if (ret < 0)
+               return ret;
+
+       val = val | RTC_STAT_BIT_WRTC;
+
+       ret = dm_i2c_write(dev, RTC_STAT_REG_ADDR, &val, sizeof(val));
+       if (ret < 0)
+               return ret;
+
+       buf[RTC_YR_REG_ADDR] = bin2bcd(tmp->tm_year % 100);
+       buf[RTC_MON_REG_ADDR] = bin2bcd(tmp->tm_mon);
+       buf[RTC_DAY_REG_ADDR] = bin2bcd(tmp->tm_wday);
+       buf[RTC_DATE_REG_ADDR] = bin2bcd(tmp->tm_mday);
+       buf[RTC_HR_REG_ADDR] = bin2bcd(tmp->tm_hour) | 0x80; /* 24h clock */
+       buf[RTC_MIN_REG_ADDR] = bin2bcd(tmp->tm_min);
+       buf[RTC_SEC_REG_ADDR] = bin2bcd(tmp->tm_sec);
 
-       rtc_write (RTC_YR_REG_ADDR, bin2bcd (tmp->tm_year % 100));
-       rtc_write (RTC_MON_REG_ADDR, bin2bcd (tmp->tm_mon));
-       rtc_write (RTC_DAY_REG_ADDR, bin2bcd (tmp->tm_wday));
-       rtc_write (RTC_DATE_REG_ADDR, bin2bcd (tmp->tm_mday));
-       rtc_write (RTC_HR_REG_ADDR, bin2bcd (tmp->tm_hour) | 0x80 ); /* 24h clock */
-       rtc_write (RTC_MIN_REG_ADDR, bin2bcd (tmp->tm_min));
-       rtc_write (RTC_SEC_REG_ADDR, bin2bcd (tmp->tm_sec));
+       ret = dm_i2c_write(dev, 0, buf, sizeof(buf));
+       if (ret < 0)
+               return ret;
 
        /* disable write */
-       rtc_write(RTC_STAT_REG_ADDR,
-               rtc_read(RTC_STAT_REG_ADDR) & ~RTC_STAT_BIT_WRTC);
+       ret = dm_i2c_read(dev, RTC_STAT_REG_ADDR, &val, sizeof(val));
+       if (ret < 0)
+               return ret;
+
+       val = val & ~RTC_STAT_BIT_WRTC;
+       ret = dm_i2c_write(dev, RTC_STAT_REG_ADDR, &val, sizeof(val));
+       if (ret < 0)
+               return ret;
 
        return 0;
 }
 
-void rtc_reset (void)
+static int isl1208_rtc_reset(struct udevice *dev)
 {
+       return 0;
 }
 
-/*
- * Helper functions
- */
-
-static uchar rtc_read (uchar reg)
+static int isl1208_probe(struct udevice *dev)
 {
-       return (i2c_reg_read (CONFIG_SYS_I2C_RTC_ADDR, reg));
-}
+       i2c_set_chip_flags(dev, DM_I2C_CHIP_RD_ADDRESS |
+                          DM_I2C_CHIP_WR_ADDRESS);
 
-static void rtc_write (uchar reg, uchar val)
-{
-       i2c_reg_write (CONFIG_SYS_I2C_RTC_ADDR, reg, val);
+       return 0;
 }
+
+static const struct rtc_ops isl1208_rtc_ops = {
+       .get = isl1208_rtc_get,
+       .set = isl1208_rtc_set,
+       .reset = isl1208_rtc_reset,
+};
+
+static const struct udevice_id isl1208_rtc_ids[] = {
+       { .compatible = "isil,isl1208" },
+       { }
+};
+
+U_BOOT_DRIVER(rtc_isl1208) = {
+       .name   = "rtc-isl1208",
+       .id     = UCLASS_RTC,
+       .probe  = isl1208_probe,
+       .of_match = isl1208_rtc_ids,
+       .ops    = &isl1208_rtc_ops,
+};