Messages in this thread | | | Date | Wed, 31 Mar 2021 09:18:35 +0200 | From | Wolfram Sang <> | Subject | Re: [PATCH] I2C: JZ4780: Fix bug for Ingenic X1000. |
| |
Hi,
> > Any write operation? I wonder then why nobody noticed before? > > > The standard I2C communication should look like this: > > Read: > > device_addr + w, reg_addr, device_addr + r, data; > > Write: > > device_addr + w, reg_addr, data; > > > But without this patch, it looks like this: > > Read: > > device_addr + w, reg_addr, device_addr + r, data; > > Write: > > device_addr + w, reg_addr, device_addr + w, data; > > This is clearly not correct.
Thanks for the additional information! I understand now. I added a bit of this to the commit message of v2 to explain the situation.
[unhandled content-type:application/pgp-signature] | |