]> Git Repo - J-u-boot.git/blobdiff - drivers/i2c/mv_i2c.c
i2c: Remove CFG_SYS_I2C_DIRECT_BUS
[J-u-boot.git] / drivers / i2c / mv_i2c.c
index 30b986e705a45aa3c28e803958dac8258042307b..949cc45d308d46411e6d6b032f85d8873aa889bd 100644 (file)
@@ -1,3 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0+
 /*
  * (C) Copyright 2000
  * Paolo Scaffardi, AIRVENT SAM s.p.a - RIMINI(ITALY), [email protected]
  * (C) Copyright 2011 Marvell Inc.
  * Lei Wen <[email protected]>
  *
- * SPDX-License-Identifier:    GPL-2.0+
- *
  * Back ported to the 8xx platform (from the 8260 platform) by
  * [email protected], 27-Jan-01.
  */
 
-#include <common.h>
+#include <dm.h>
 #include <i2c.h>
+#include <log.h>
 #include <asm/io.h>
+#include <linux/delay.h>
 #include "mv_i2c.h"
 
 /* All transfers are described by this data structure */
@@ -30,6 +31,16 @@ struct mv_i2c_msg {
        u8 data;
 };
 
+#ifdef CONFIG_ARMADA_3700
+/* Armada 3700 has no padding between the registers */
+struct mv_i2c {
+       u32 ibmr;
+       u32 idbr;
+       u32 icr;
+       u32 isr;
+       u32 isar;
+};
+#else
 struct mv_i2c {
        u32 ibmr;
        u32 pad0;
@@ -41,6 +52,15 @@ struct mv_i2c {
        u32 pad3;
        u32 isar;
 };
+#endif
+
+/*
+ * Dummy implementation that can be overwritten by a board
+ * specific function
+ */
+__weak void i2c_clk_enable(void)
+{
+}
 
 /*
  * i2c_reset: - reset the host controller
@@ -48,6 +68,10 @@ struct mv_i2c {
  */
 static void i2c_reset(struct mv_i2c *base)
 {
+       u32 icr_mode;
+
+       /* Save bus mode (standard or fast speed) for later use */
+       icr_mode = readl(&base->icr) & ICR_MODE_MASK;
        writel(readl(&base->icr) & ~ICR_IUE, &base->icr); /* disable unit */
        writel(readl(&base->icr) | ICR_UR, &base->icr);   /* reset the unit */
        udelay(100);
@@ -55,8 +79,9 @@ static void i2c_reset(struct mv_i2c *base)
 
        i2c_clk_enable();
 
-       writel(CONFIG_SYS_I2C_SLAVE, &base->isar); /* set our slave address */
-       writel(I2C_ICR_INIT, &base->icr); /* set control reg values */
+       writel(0x0, &base->isar); /* set our slave address */
+       /* set control reg values */
+       writel(I2C_ICR_INIT | icr_mode, &base->icr);
        writel(I2C_ISR_INIT, &base->isr); /* set clear interrupt bits */
        writel(readl(&base->icr) | ICR_IUE, &base->icr); /* enable unit */
        udelay(100);
@@ -210,37 +235,18 @@ i2c_transfer_finish:
        return ret;
 }
 
-static int __i2c_probe_chip(struct mv_i2c *base, uchar chip)
-{
-       struct mv_i2c_msg msg;
-
-       i2c_reset(base);
-
-       msg.condition = I2C_COND_START;
-       msg.acknack   = I2C_ACKNAK_WAITACK;
-       msg.direction = I2C_WRITE;
-       msg.data      = (chip << 1) + 1;
-       if (i2c_transfer(base, &msg))
-               return -1;
-
-       msg.condition = I2C_COND_STOP;
-       msg.acknack   = I2C_ACKNAK_SENDNAK;
-       msg.direction = I2C_READ;
-       msg.data      = 0x00;
-       if (i2c_transfer(base, &msg))
-               return -1;
-
-       return 0;
-}
-
-static int __i2c_read(struct mv_i2c *base, uchar chip, uint addr, int alen,
+static int __i2c_read(struct mv_i2c *base, uchar chip, u8 *addr, int alen,
                      uchar *buffer, int len)
 {
        struct mv_i2c_msg msg;
-       u8 addr_bytes[3]; /* lowest...highest byte of data address */
 
        debug("i2c_read(chip=0x%02x, addr=0x%02x, alen=0x%02x, "
-             "len=0x%02x)\n", chip, addr, alen, len);
+             "len=0x%02x)\n", chip, *addr, alen, len);
+
+       if (len == 0) {
+               printf("reading zero byte is invalid\n");
+               return -EINVAL;
+       }
 
        i2c_reset(base);
 
@@ -258,17 +264,13 @@ static int __i2c_read(struct mv_i2c *base, uchar chip, uint addr, int alen,
         * send memory address bytes;
         * alen defines how much bytes we have to send.
         */
-       /*addr &= ((1 << CONFIG_SYS_EEPROM_PAGE_WRITE_BITS)-1); */
-       addr_bytes[0] = (u8)((addr >>  0) & 0x000000FF);
-       addr_bytes[1] = (u8)((addr >>  8) & 0x000000FF);
-       addr_bytes[2] = (u8)((addr >> 16) & 0x000000FF);
-
        while (--alen >= 0) {
-               debug("i2c_read: send memory word address byte %1d\n", alen);
+               debug("i2c_read: send address byte %02x (alen=%d)\n",
+                     *addr, alen);
                msg.condition = I2C_COND_NORMAL;
                msg.acknack   = I2C_ACKNAK_WAITACK;
                msg.direction = I2C_WRITE;
-               msg.data      = addr_bytes[alen];
+               msg.data      = addr[alen];
                if (i2c_transfer(base, &msg))
                        return -1;
        }
@@ -299,8 +301,8 @@ static int __i2c_read(struct mv_i2c *base, uchar chip, uint addr, int alen,
                        return -1;
 
                *buffer = msg.data;
-               debug("i2c_read: reading byte (0x%08x)=0x%02x\n",
-                     (unsigned int)buffer, *buffer);
+               debug("i2c_read: reading byte (%p)=0x%02x\n",
+                     buffer, *buffer);
                buffer++;
        }
 
@@ -309,14 +311,13 @@ static int __i2c_read(struct mv_i2c *base, uchar chip, uint addr, int alen,
        return 0;
 }
 
-static int __i2c_write(struct mv_i2c *base, uchar chip, uint addr, int alen,
+static int __i2c_write(struct mv_i2c *base, uchar chip, u8 *addr, int alen,
                       uchar *buffer, int len)
 {
        struct mv_i2c_msg msg;
-       u8 addr_bytes[3]; /* lowest...highest byte of data address */
 
        debug("i2c_write(chip=0x%02x, addr=0x%02x, alen=0x%02x, "
-             "len=0x%02x)\n", chip, addr, alen, len);
+             "len=0x%02x)\n", chip, *addr, alen, len);
 
        i2c_reset(base);
 
@@ -334,24 +335,21 @@ static int __i2c_write(struct mv_i2c *base, uchar chip, uint addr, int alen,
         * send memory address bytes;
         * alen defines how much bytes we have to send.
         */
-       addr_bytes[0] = (u8)((addr >>  0) & 0x000000FF);
-       addr_bytes[1] = (u8)((addr >>  8) & 0x000000FF);
-       addr_bytes[2] = (u8)((addr >> 16) & 0x000000FF);
-
        while (--alen >= 0) {
-               debug("i2c_write: send memory word address\n");
+               debug("i2c_read: send address byte %02x (alen=%d)\n",
+                     *addr, alen);
                msg.condition = I2C_COND_NORMAL;
                msg.acknack   = I2C_ACKNAK_WAITACK;
                msg.direction = I2C_WRITE;
-               msg.data      = addr_bytes[alen];
+               msg.data      = addr[alen];
                if (i2c_transfer(base, &msg))
                        return -1;
        }
 
        /* write bytes; send NACK at last byte */
        while (len--) {
-               debug("i2c_write: writing byte (0x%08x)=0x%02x\n",
-                     (unsigned int)buffer, *buffer);
+               debug("i2c_write: writing byte (%p)=0x%02x\n",
+                     buffer, *buffer);
 
                if (len == 0)
                        msg.condition = I2C_COND_STOP;
@@ -371,69 +369,45 @@ static int __i2c_write(struct mv_i2c *base, uchar chip, uint addr, int alen,
        return 0;
 }
 
+#if !CONFIG_IS_ENABLED(DM_I2C)
+
 static struct mv_i2c *base_glob;
 
-static void i2c_board_init(struct mv_i2c *base)
+/* API Functions */
+void i2c_init(int speed, int slaveaddr)
 {
-#ifdef CONFIG_SYS_I2C_INIT_BOARD
-       u32 icr;
-       /*
-        * call board specific i2c bus reset routine before accessing the
-        * environment, which might be in a chip on that bus. For details
-        * about this problem see doc/I2C_Edge_Conditions.
-        *
-        * disable I2C controller first, otherwhise it thinks we want to
-        * talk to the slave port...
-        */
-       icr = readl(&base->icr);
-       writel(readl(&base->icr) & ~(ICR_SCLE | ICR_IUE), &base->icr);
+       u32 val;
 
-       i2c_init_board();
+       base_glob = (struct mv_i2c *)CONFIG_MV_I2C_REG;
 
-       writel(icr, &base->icr);
-#endif
+       if (speed > I2C_SPEED_STANDARD_RATE)
+               val = ICR_FM;
+       else
+               val = ICR_SM;
+       clrsetbits_le32(&base_glob->icr, ICR_MODE_MASK, val);
 }
 
-#ifdef CONFIG_I2C_MULTI_BUS
-static unsigned long i2c_regs[CONFIG_MV_I2C_NUM] = CONFIG_MV_I2C_REG;
-static unsigned int bus_initialized[CONFIG_MV_I2C_NUM];
-static unsigned int current_bus;
-
-int i2c_set_bus_num(unsigned int bus)
+static int __i2c_probe_chip(struct mv_i2c *base, uchar chip)
 {
-       if ((bus < 0) || (bus >= CONFIG_MV_I2C_NUM)) {
-               printf("Bad bus: %d\n", bus);
-               return -1;
-       }
-
-       base_glob = (struct mv_i2c *)i2c_regs[bus];
-       current_bus = bus;
-
-       if (!bus_initialized[current_bus]) {
-               i2c_board_init(base_glob);
-               bus_initialized[current_bus] = 1;
-       }
+       struct mv_i2c_msg msg;
 
-       return 0;
-}
+       i2c_reset(base);
 
-unsigned int i2c_get_bus_num(void)
-{
-       return current_bus;
-}
-#endif
+       msg.condition = I2C_COND_START;
+       msg.acknack   = I2C_ACKNAK_WAITACK;
+       msg.direction = I2C_WRITE;
+       msg.data      = (chip << 1) + 1;
+       if (i2c_transfer(base, &msg))
+               return -1;
 
-/* API Functions */
-void i2c_init(int speed, int slaveaddr)
-{
-#ifdef CONFIG_I2C_MULTI_BUS
-       current_bus = 0;
-       base_glob = (struct mv_i2c *)i2c_regs[current_bus];
-#else
-       base_glob = (struct mv_i2c *)CONFIG_MV_I2C_REG;
-#endif
+       msg.condition = I2C_COND_STOP;
+       msg.acknack   = I2C_ACKNAK_SENDNAK;
+       msg.direction = I2C_READ;
+       msg.data      = 0x00;
+       if (i2c_transfer(base, &msg))
+               return -1;
 
-       i2c_board_init(base_glob);
+       return 0;
 }
 
 /*
@@ -462,7 +436,14 @@ int i2c_probe(uchar chip)
  */
 int i2c_read(uchar chip, uint addr, int alen, uchar *buffer, int len)
 {
-       return __i2c_read(base_glob, chip, addr, alen, buffer, len);
+       u8 addr_bytes[4];
+
+       addr_bytes[0] = (addr >> 0) & 0xFF;
+       addr_bytes[1] = (addr >> 8) & 0xFF;
+       addr_bytes[2] = (addr >> 16) & 0xFF;
+       addr_bytes[3] = (addr >> 24) & 0xFF;
+
+       return __i2c_read(base_glob, chip, addr_bytes, alen, buffer, len);
 }
 
 /*
@@ -480,5 +461,88 @@ int i2c_read(uchar chip, uint addr, int alen, uchar *buffer, int len)
  */
 int i2c_write(uchar chip, uint addr, int alen, uchar *buffer, int len)
 {
-       return __i2c_write(base_glob, chip, addr, alen, buffer, len);
+       u8 addr_bytes[4];
+
+       addr_bytes[0] = (addr >> 0) & 0xFF;
+       addr_bytes[1] = (addr >> 8) & 0xFF;
+       addr_bytes[2] = (addr >> 16) & 0xFF;
+       addr_bytes[3] = (addr >> 24) & 0xFF;
+
+       return __i2c_write(base_glob, chip, addr_bytes, alen, buffer, len);
+}
+
+#else /* CONFIG_DM_I2C */
+
+struct mv_i2c_priv {
+       struct mv_i2c *base;
+};
+
+static int mv_i2c_xfer(struct udevice *bus, struct i2c_msg *msg, int nmsgs)
+{
+       struct mv_i2c_priv *i2c = dev_get_priv(bus);
+       struct i2c_msg *dmsg, *omsg, dummy;
+
+       memset(&dummy, 0, sizeof(struct i2c_msg));
+
+       /*
+        * We expect either two messages (one with an offset and one with the
+        * actual data) or one message (just data or offset/data combined)
+        */
+       if (nmsgs > 2 || nmsgs == 0) {
+               debug("%s: Only one or two messages are supported.", __func__);
+               return -1;
+       }
+
+       omsg = nmsgs == 1 ? &dummy : msg;
+       dmsg = nmsgs == 1 ? msg : msg + 1;
+
+       if (dmsg->flags & I2C_M_RD)
+               return __i2c_read(i2c->base, dmsg->addr, omsg->buf,
+                                 omsg->len, dmsg->buf, dmsg->len);
+       else
+               return __i2c_write(i2c->base, dmsg->addr, omsg->buf,
+                                  omsg->len, dmsg->buf, dmsg->len);
+}
+
+static int mv_i2c_set_bus_speed(struct udevice *bus, unsigned int speed)
+{
+       struct mv_i2c_priv *priv = dev_get_priv(bus);
+       u32 val;
+
+       if (speed > I2C_SPEED_STANDARD_RATE)
+               val = ICR_FM;
+       else
+               val = ICR_SM;
+       clrsetbits_le32(&priv->base->icr, ICR_MODE_MASK, val);
+
+       return 0;
 }
+
+static int mv_i2c_probe(struct udevice *bus)
+{
+       struct mv_i2c_priv *priv = dev_get_priv(bus);
+
+       priv->base = dev_read_addr_ptr(bus);
+
+       return 0;
+}
+
+static const struct dm_i2c_ops mv_i2c_ops = {
+       .xfer           = mv_i2c_xfer,
+       .set_bus_speed  = mv_i2c_set_bus_speed,
+};
+
+static const struct udevice_id mv_i2c_ids[] = {
+       { .compatible = "marvell,armada-3700-i2c" },
+       { }
+};
+
+U_BOOT_DRIVER(i2c_mv) = {
+       .name   = "i2c_mv",
+       .id     = UCLASS_I2C,
+       .of_match = mv_i2c_ids,
+       .probe  = mv_i2c_probe,
+       .priv_auto      = sizeof(struct mv_i2c_priv),
+       .ops    = &mv_i2c_ops,
+};
+#endif /* CONFIG_DM_I2C */
This page took 0.041438 seconds and 4 git commands to generate.