Commit 19358d44 authored by Wolfram Sang's avatar Wolfram Sang Committed by Wolfram Sang

i2c: rcar: implement STOP and REP_START according to docs

When doing a REP_START after a read message, the driver used to trigger
a STOP first which would then be overwritten by REP_START. This was the
only stable method found when doing the last refactoring. However, this
was not in accordance with the documentation.

After research from our BSP team and myself, we now can implement a
version which works and is according to the documentation. The new
approach ensures the ICMCR register is only changed when really needed.

Tested on a R-Car Gen2 (H2) and Gen3 with DMA (M3N).
Signed-off-by: default avatarHiromitsu Yamasaki <hiromitsu.yamasaki.ym@renesas.com>
Signed-off-by: default avatarWolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: default avatarUlrich Hecht <uli+renesas@fpond.eu>
Signed-off-by: default avatarWolfram Sang <wsa@the-dreams.de>
parent b07531ac
...@@ -113,9 +113,10 @@ ...@@ -113,9 +113,10 @@
#define ID_ARBLOST (1 << 3) #define ID_ARBLOST (1 << 3)
#define ID_NACK (1 << 4) #define ID_NACK (1 << 4)
/* persistent flags */ /* persistent flags */
#define ID_P_REP_AFTER_RD BIT(29)
#define ID_P_NO_RXDMA BIT(30) /* HW forbids RXDMA sometimes */ #define ID_P_NO_RXDMA BIT(30) /* HW forbids RXDMA sometimes */
#define ID_P_PM_BLOCKED BIT(31) #define ID_P_PM_BLOCKED BIT(31)
#define ID_P_MASK GENMASK(31, 30) #define ID_P_MASK GENMASK(31, 29)
enum rcar_i2c_type { enum rcar_i2c_type {
I2C_RCAR_GEN1, I2C_RCAR_GEN1,
...@@ -345,7 +346,10 @@ static void rcar_i2c_prepare_msg(struct rcar_i2c_priv *priv) ...@@ -345,7 +346,10 @@ static void rcar_i2c_prepare_msg(struct rcar_i2c_priv *priv)
rcar_i2c_write(priv, ICMSR, 0); rcar_i2c_write(priv, ICMSR, 0);
rcar_i2c_write(priv, ICMCR, RCAR_BUS_PHASE_START); rcar_i2c_write(priv, ICMCR, RCAR_BUS_PHASE_START);
} else { } else {
rcar_i2c_write(priv, ICMCR, RCAR_BUS_PHASE_START); if (priv->flags & ID_P_REP_AFTER_RD)
priv->flags &= ~ID_P_REP_AFTER_RD;
else
rcar_i2c_write(priv, ICMCR, RCAR_BUS_PHASE_START);
rcar_i2c_write(priv, ICMSR, 0); rcar_i2c_write(priv, ICMSR, 0);
} }
rcar_i2c_write(priv, ICMIER, read ? RCAR_IRQ_RECV : RCAR_IRQ_SEND); rcar_i2c_write(priv, ICMIER, read ? RCAR_IRQ_RECV : RCAR_IRQ_SEND);
...@@ -550,15 +554,15 @@ static void rcar_i2c_irq_recv(struct rcar_i2c_priv *priv, u32 msr) ...@@ -550,15 +554,15 @@ static void rcar_i2c_irq_recv(struct rcar_i2c_priv *priv, u32 msr)
priv->pos++; priv->pos++;
} }
/* /* If next received data is the _LAST_, go to new phase. */
* If next received data is the _LAST_, go to STOP phase. Might be if (priv->pos + 1 == msg->len) {
* overwritten by REP START when setting up a new msg. Not elegant if (priv->flags & ID_LAST_MSG) {
* but the only stable sequence for REP START I have found so far. rcar_i2c_write(priv, ICMCR, RCAR_BUS_PHASE_STOP);
* If you want to change this code, make sure sending one transfer with } else {
* four messages (WR-RD-WR-RD) works! rcar_i2c_write(priv, ICMCR, RCAR_BUS_PHASE_START);
*/ priv->flags |= ID_P_REP_AFTER_RD;
if (priv->pos + 1 >= msg->len) }
rcar_i2c_write(priv, ICMCR, RCAR_BUS_PHASE_STOP); }
if (priv->pos == msg->len && !(priv->flags & ID_LAST_MSG)) if (priv->pos == msg->len && !(priv->flags & ID_LAST_MSG))
rcar_i2c_next_msg(priv); rcar_i2c_next_msg(priv);
...@@ -626,9 +630,11 @@ static irqreturn_t rcar_i2c_irq(int irq, void *ptr) ...@@ -626,9 +630,11 @@ static irqreturn_t rcar_i2c_irq(int irq, void *ptr)
struct rcar_i2c_priv *priv = ptr; struct rcar_i2c_priv *priv = ptr;
u32 msr, val; u32 msr, val;
/* Clear START or STOP as soon as we can */ /* Clear START or STOP immediately, except for REPSTART after read */
val = rcar_i2c_read(priv, ICMCR); if (likely(!(priv->flags & ID_P_REP_AFTER_RD))) {
rcar_i2c_write(priv, ICMCR, val & RCAR_BUS_MASK_DATA); val = rcar_i2c_read(priv, ICMCR);
rcar_i2c_write(priv, ICMCR, val & RCAR_BUS_MASK_DATA);
}
msr = rcar_i2c_read(priv, ICMSR); msr = rcar_i2c_read(priv, ICMSR);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment