On Tue, 2017-03-07 at 21:06 +0100, Heiner Kallweit wrote: > Most likely a copy & paste error. > > Signed-off-by: Heiner Kallweit <hkallweit1@xxxxxxxxx> > --- > drivers/i2c/busses/i2c-meson.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/i2c/busses/i2c-meson.c b/drivers/i2c/busses/i2c-meson.c > index 2aa61bbb..73b97c71 100644 > --- a/drivers/i2c/busses/i2c-meson.c > +++ b/drivers/i2c/busses/i2c-meson.c > @@ -175,7 +175,7 @@ static void meson_i2c_put_data(struct meson_i2c *i2c, char *buf, int len) > wdata1 |= *buf++ << ((i - 4) * 8); > > writel(wdata0, i2c->regs + REG_TOK_WDATA0); > - writel(wdata0, i2c->regs + REG_TOK_WDATA1); > + writel(wdata1, i2c->regs + REG_TOK_WDATA1); > > dev_dbg(i2c->dev, "%s: data %08x %08x len %d\n", __func__, > wdata0, wdata1, len); Good catch, Strange it did not show up before. You are just missing the Fixes tag. Fixes: 30021e3707a7 ("i2c: add support for Amlogic Meson I2C controller") Acked-by: Jerome Brunet <jbrunet@xxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html