]> git.sur5r.net Git - u-boot/blobdiff - drivers/i2c/mv_i2c.c
Merge branch 'master' of git://git.denx.de/u-boot-uniphier
[u-boot] / drivers / i2c / mv_i2c.c
index 30b986e705a45aa3c28e803958dac8258042307b..8a56ef9a245977d788ca4c29934774bcd6cd5a82 100644 (file)
@@ -1,3 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0+
 /*
  * (C) Copyright 2000
  * Paolo Scaffardi, AIRVENT SAM s.p.a - RIMINI(ITALY), arsenio@tin.it
  * (C) Copyright 2011 Marvell Inc.
  * Lei Wen <leiwen@marvell.com>
  *
- * SPDX-License-Identifier:    GPL-2.0+
- *
  * Back ported to the 8xx platform (from the 8260 platform) by
  * Murray.Jensen@cmst.csiro.au, 27-Jan-01.
  */
 
 #include <common.h>
+#include <dm.h>
 #include <i2c.h>
 #include <asm/io.h>
 #include "mv_i2c.h"
@@ -30,6 +30,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 +51,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 +67,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);
@@ -56,7 +79,8 @@ 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 */
+       /* 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 +234,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 +263,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 +300,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 +310,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 +334,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,6 +368,8 @@ static int __i2c_write(struct mv_i2c *base, uchar chip, uint addr, int alen,
        return 0;
 }
 
+#ifndef CONFIG_DM_I2C
+
 static struct mv_i2c *base_glob;
 
 static void i2c_board_init(struct mv_i2c *base)
@@ -426,6 +425,8 @@ unsigned int i2c_get_bus_num(void)
 /* API Functions */
 void i2c_init(int speed, int slaveaddr)
 {
+       u32 val;
+
 #ifdef CONFIG_I2C_MULTI_BUS
        current_bus = 0;
        base_glob = (struct mv_i2c *)i2c_regs[current_bus];
@@ -433,9 +434,38 @@ void i2c_init(int speed, int slaveaddr)
        base_glob = (struct mv_i2c *)CONFIG_MV_I2C_REG;
 #endif
 
+       if (speed > 100000)
+               val = ICR_FM;
+       else
+               val = ICR_SM;
+       clrsetbits_le32(&base_glob->icr, ICR_MODE_MASK, val);
+
        i2c_board_init(base_glob);
 }
 
+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;
+}
+
 /*
  * i2c_probe: - Test if a chip answers for a given i2c address
  *
@@ -462,7 +492,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 +517,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 > 100000)
+               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 = (void *)devfdt_get_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_alloc_size = sizeof(struct mv_i2c_priv),
+       .ops    = &mv_i2c_ops,
+};
+#endif /* CONFIG_DM_I2C */