rtc: rx8025: drop non-DM support
authorMathew McBride <matt@traverse.com.au>
Fri, 17 Sep 2021 06:46:01 +0000 (06:46 +0000)
committerTom Rini <trini@konsulko.com>
Sun, 3 Oct 2021 18:40:56 +0000 (14:40 -0400)
A search of the tree showed there is only one user
of this driver (soon to be two) - board/socrates

The second user will be the Traverse Ten64 board.

Both these boards have DM_RTC.

Signed-off-by: Mathew McBride <matt@traverse.com.au>
drivers/rtc/Kconfig
drivers/rtc/rx8025.c

index b6692e6..71777cd 100644 (file)
@@ -136,6 +136,7 @@ config RTC_RX8010SJ
 
 config RTC_RX8025
        bool "Enable RX8025 driver"
+       depends on DM_RTC
        help
          Support for Epson RX8025 Real Time Clock devices.
 
index e717dcb..36e5b01 100644 (file)
 #endif
 /*---------------------------------------------------------------------*/
 
-#ifndef CONFIG_SYS_I2C_RTC_ADDR
-# define CONFIG_SYS_I2C_RTC_ADDR       0x32
-#endif
-
-#ifdef CONFIG_DM_RTC
-#define DEV_TYPE struct udevice
-#else
-/* Local udevice */
-struct ludevice {
-       u8 chip;
-};
-
-#define DEV_TYPE struct ludevice
-
-#endif
-
 /*
  * RTC register addresses
  */
@@ -74,39 +58,22 @@ struct ludevice {
  * address in a first cycle that is terminated by
  * a STOP condition. The chips needs a 'restart'
  * sequence (start sequence without a prior stop).
- * This driver has been written for a 4xx board.
- * U-Boot's 4xx i2c driver is currently not capable
- * to generate such cycles to some work arounds
- * are used.
  */
 
-/* static uchar rtc_read (uchar reg); */
-#ifdef CONFIG_DM_RTC
-/*
- * on mpc85xx based board with DM and offset len 1
- * accessing rtc works fine. May we can drop this ?
- */
 #define rtc_read(reg) buf[(reg) & 0xf]
-#else
-#define rtc_read(reg) buf[((reg) + 1) & 0xf]
-#endif
 
-static int rtc_write(DEV_TYPE *dev, uchar reg, uchar val);
+static int rtc_write(struct udevice *dev, uchar reg, uchar val);
 
 /*
  * Get the current time from the RTC
  */
-static int rx8025_rtc_get(DEV_TYPE *dev, struct rtc_time *tmp)
+static int rx8025_rtc_get(struct udevice *dev, struct rtc_time *tmp)
 {
        int rel = 0;
        uchar sec, min, hour, mday, wday, mon, year, ctl2;
        uchar buf[16];
 
-#ifdef CONFIG_DM_RTC
        if (dm_i2c_read(dev, 0, buf, sizeof(buf))) {
-#else
-       if (i2c_read(dev->chip, 0, 0, buf, 16)) {
-#endif
                printf("Error reading from RTC\n");
                return -EIO;
        }
@@ -165,7 +132,7 @@ static int rx8025_rtc_get(DEV_TYPE *dev, struct rtc_time *tmp)
 /*
  * Set the RTC
  */
-static int rx8025_rtc_set(DEV_TYPE *dev, const struct rtc_time *tmp)
+static int rx8025_rtc_set(struct udevice *dev, const struct rtc_time *tmp)
 {
        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,
@@ -201,16 +168,12 @@ static int rx8025_rtc_set(DEV_TYPE *dev, const struct rtc_time *tmp)
 /*
  * Reset the RTC
  */
-static int rx8025_rtc_reset(DEV_TYPE *dev)
+static int rx8025_rtc_reset(struct udevice *dev)
 {
        uchar buf[16];
        uchar ctl2;
 
-#ifdef CONFIG_DM_RTC
        if (dm_i2c_read(dev, 0, buf, sizeof(buf))) {
-#else
-       if (i2c_read(dev->chip, 0, 0, buf, 16)) {
-#endif
                printf("Error reading from RTC\n");
                return -EIO;
        }
@@ -225,17 +188,13 @@ static int rx8025_rtc_reset(DEV_TYPE *dev)
 /*
  * Helper functions
  */
-static int rtc_write(DEV_TYPE *dev, uchar reg, uchar val)
+static int rtc_write(struct udevice *dev, uchar reg, uchar val)
 {
        uchar buf[2];
        buf[0] = reg << 4;
        buf[1] = val;
 
-#ifdef CONFIG_DM_RTC
        if (dm_i2c_write(dev, 0, buf, 2)) {
-#else
-       if (i2c_write(dev->chip, 0, 0, buf, 2) != 0) {
-#endif
                printf("Error writing to RTC\n");
                return -EIO;
        }
@@ -243,7 +202,6 @@ static int rtc_write(DEV_TYPE *dev, uchar reg, uchar val)
        return 0;
 }
 
-#ifdef CONFIG_DM_RTC
 static int rx8025_probe(struct udevice *dev)
 {
        uchar buf[16];
@@ -276,31 +234,3 @@ U_BOOT_DRIVER(rx8010sj_rtc) = {
        .of_match = rx8025_rtc_ids,
        .ops      = &rx8025_rtc_ops,
 };
-#else
-int rtc_get(struct rtc_time *tm)
-{
-       struct ludevice dev = {
-               .chip = CONFIG_SYS_I2C_RTC_ADDR,
-       };
-
-       return rx8025_rtc_get(&dev, tm);
-}
-
-int rtc_set(struct rtc_time *tm)
-{
-       struct ludevice dev = {
-               .chip = CONFIG_SYS_I2C_RTC_ADDR,
-       };
-
-       return rx8025_rtc_set(&dev, tm);
-}
-
-void rtc_reset(void)
-{
-       struct ludevice dev = {
-               .chip = CONFIG_SYS_I2C_RTC_ADDR,
-       };
-
-       rx8025_rtc_reset(&dev);
-}
-#endif