* [PATCH] i2c: meson: fix wrong variable usage in meson_i2c_put_data
@ 2017-03-07 20:06 Heiner Kallweit
2017-03-08 9:17 ` Jerome Brunet
2017-03-09 14:41 ` Wolfram Sang
0 siblings, 2 replies; 3+ messages in thread
From: Heiner Kallweit @ 2017-03-07 20:06 UTC (permalink / raw)
To: linus-amlogic
Most likely a copy & paste error.
Signed-off-by: Heiner Kallweit <hkallweit1@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);
--
2.12.0
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] i2c: meson: fix wrong variable usage in meson_i2c_put_data
2017-03-07 20:06 [PATCH] i2c: meson: fix wrong variable usage in meson_i2c_put_data Heiner Kallweit
@ 2017-03-08 9:17 ` Jerome Brunet
2017-03-09 14:41 ` Wolfram Sang
1 sibling, 0 replies; 3+ messages in thread
From: Jerome Brunet @ 2017-03-08 9:17 UTC (permalink / raw)
To: linus-amlogic
On Tue, 2017-03-07 at 21:06 +0100, Heiner Kallweit wrote:
> Most likely a copy & paste error.
>
> Signed-off-by: Heiner Kallweit <hkallweit1@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@baylibre.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] i2c: meson: fix wrong variable usage in meson_i2c_put_data
2017-03-07 20:06 [PATCH] i2c: meson: fix wrong variable usage in meson_i2c_put_data Heiner Kallweit
2017-03-08 9:17 ` Jerome Brunet
@ 2017-03-09 14:41 ` Wolfram Sang
1 sibling, 0 replies; 3+ messages in thread
From: Wolfram Sang @ 2017-03-09 14:41 UTC (permalink / raw)
To: linus-amlogic
On Tue, Mar 07, 2017 at 09:06:38PM +0100, Heiner Kallweit wrote:
> Most likely a copy & paste error.
>
> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Applied to for-current, thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-amlogic/attachments/20170309/27a52d1e/attachment.sig>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-03-09 14:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-07 20:06 [PATCH] i2c: meson: fix wrong variable usage in meson_i2c_put_data Heiner Kallweit
2017-03-08 9:17 ` Jerome Brunet
2017-03-09 14:41 ` Wolfram Sang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®