Skip to contentSkip to conversation

i2c: mt76xx: add I2C_FUNC_NOSTART and I2C_M_REV_DIR_ADDR

PATCH Superseded [email protected] 2 participants 2 messages Oct 19, 2021 – Oct 22, 2021

Subsystems (3)

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 i2c: mt76xx: add I2C_FUNC_NOSTART and I2C_M_REV_DIR_ADDR

 This patch adds I2C_FUNC_NOSTART and I2C_M_REV_DIR_ADDR support for MediaTek MT7621/7628/7688. This is useful for some I2C slave chips, such as 'drivers/input/joystick/as5011.c'.

---
 drivers/i2c/busses/i2c-mt7621.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)
drivers/i2c/busses/i2c-mt7621.c +8 -1
index 45fe4a7fe0c0..3d2763d9c090 100644
@@ -150,6 +150,9 @@ static int mtk_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs,
 	for (i = 0; i < num; i++) {
 		pmsg = &msgs[i];
 
+		if (pmsg->flags & I2C_M_NOSTART)
+			goto nostart;
+
 		/* wait hardware idle */
 		ret = mtk_i2c_wait_idle(i2c);
 		if (ret)
@@ -174,6 +177,8 @@ static int mtk_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs,
 		} else {
 			/* 7 bits address */
 			addr = i2c_8bit_addr_from_msg(pmsg);
+			if (pmsg->flags & I2C_M_REV_DIR_ADDR)
+				addr ^= 1;
 			iowrite32(addr, i2c->base + REG_SM0D0_REG);
 			ret = mtk_i2c_master_cmd(i2c, SM0CTL1_WRITE, 1);
 			if (ret)
@@ -187,6 +192,7 @@ static int mtk_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs,
 				goto err_ack;
 		}
 
+nostart:
 		/* transfer data */
 		for (len = pmsg->len, j = 0; len > 0; len -= 8, j += 8) {
 			page_len = (len >= 8) ? 8 : len;
@@ -242,7 +248,8 @@ static int mtk_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs,
 
 static u32 mtk_i2c_func(struct i2c_adapter *a)
 {
-	return I2C_FUNC_I2C | I2C_FUNC_SMBUS_EMUL | I2C_FUNC_PROTOCOL_MANGLING;
+	return I2C_FUNC_I2C | I2C_FUNC_SMBUS_EMUL | I2C_FUNC_10BIT_ADDR
+		| I2C_FUNC_PROTOCOL_MANGLING | I2C_FUNC_NOSTART;
 }
 
 static const struct i2c_algorithm mtk_i2c_algo = {
Signature · 6 lines
2.25.1

_______________________________________________
Linux-mediatek mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-mediatek
maintainer Reviewed-by on v1 1/1
#1 lore raw
Quoted 41 lines · On 19.10.21 18:07, [email protected] wrote:

 This patch adds I2C_FUNC_NOSTART and I2C_M_REV_DIR_ADDR support for
MediaTek MT7621/7628/7688. This is useful for some I2C slave chips, such as 'drivers/input/joystick/as5011.c'.

---
  drivers/i2c/busses/i2c-mt7621.c | 9 ++++++++-
  1 file changed, 8 insertions(+), 1 deletion(-)
drivers/i2c/busses/i2c-mt7621.c +6 -0
index 45fe4a7fe0c0..3d2763d9c090 100644
@@ -150,6 +150,9 @@ static int mtk_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs,
  	for (i = 0; i < num; i++) {
  		pmsg = &msgs[i];
 
+		if (pmsg->flags & I2C_M_NOSTART)
+			goto nostart;
+
  		/* wait hardware idle */
  		ret = mtk_i2c_wait_idle(i2c);
  		if (ret)
@@ -174,6 +177,8 @@ static int mtk_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs,
  		} else {
  			/* 7 bits address */
  			addr = i2c_8bit_addr_from_msg(pmsg);
+			if (pmsg->flags & I2C_M_REV_DIR_ADDR)
+				addr ^= 1;
  			iowrite32(addr, i2c->base + REG_SM0D0_REG);
  			ret = mtk_i2c_master_cmd(i2c, SM0CTL1_WRITE, 1);
  			if (ret)
@@ -187,6 +192,7 @@ static int mtk_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs,
  				goto err_ack;
  		}
 
+nostart:
  		/* transfer data */
  		for (len = pmsg->len, j = 0; len > 0; len -= 8, j += 8) {
  			page_len = (len >= 8) ? 8 : len;

drivers/i2c/busses/i2c-mt7621.c static int mtk_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs, · line 252

Quoted 6 lines
drivers/i2c/busses/i2c-mt7621.c +2 -1
@@ -242,7 +248,8 @@ static int mtk_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs,
 
  static u32 mtk_i2c_func(struct i2c_adapter *a)
  {
-	return I2C_FUNC_I2C | I2C_FUNC_SMBUS_EMUL | I2C_FUNC_PROTOCOL_MANGLING;
+	return I2C_FUNC_I2C | I2C_FUNC_SMBUS_EMUL | I2C_FUNC_10BIT_ADDR
+		| I2C_FUNC_PROTOCOL_MANGLING | I2C_FUNC_NOSTART;

Nit: I prefer to have the "|" at the end of the upper line instead of adding it in the 2nd line.

Other that this:

Thanks,
Stefan

_______________________________________________
Linux-mediatek mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-mediatek