[PATCH] i2c: meson: fix wrong variable usage in meson_i2c_put_data
Jerome Brunet
jbrunet at baylibre.com
Wed Mar 8 01:17:03 PST 2017
On Tue, 2017-03-07 at 21:06 +0100, Heiner Kallweit wrote:
> Most likely a copy & paste error.
>
> Signed-off-by: Heiner Kallweit <hkallweit1 at gmail.com>
> ---
> 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 at baylibre.com>
More information about the linux-amlogic
mailing list