[U-Boot] [PATCH] i2c: mv_i2c.c: Correct address endianness
Stefan Roese
sr at denx.de
Mon Dec 19 06:56:53 CET 2016
On 13.12.2016 18:49, Bradley Bolen wrote:
> 0c0f719ad2f46c8566a56daee37ebdb7c078c3b1 accidentally changed the
> endianness of the i2c read and write addresses. This was noticable when
> accessing EEPROMs that use 2 byte addressing as the LSB was being sent
> first.
>
> Signed-off-by: Bradley Bolen <bradleybolen at gmail.com>
> ---
> drivers/i2c/mv_i2c.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/i2c/mv_i2c.c b/drivers/i2c/mv_i2c.c
> index 7f52fa2..c780272 100644
> --- a/drivers/i2c/mv_i2c.c
> +++ b/drivers/i2c/mv_i2c.c
> @@ -270,7 +270,7 @@ static int __i2c_read(struct mv_i2c *base, uchar chip, u8 *addr, int alen,
> msg.condition = I2C_COND_NORMAL;
> msg.acknack = I2C_ACKNAK_WAITACK;
> msg.direction = I2C_WRITE;
> - msg.data = *(addr++);
> + msg.data = addr[alen];
> if (i2c_transfer(base, &msg))
> return -1;
> }
> @@ -341,7 +341,7 @@ static int __i2c_write(struct mv_i2c *base, uchar chip, u8 *addr, int alen,
> msg.condition = I2C_COND_NORMAL;
> msg.acknack = I2C_ACKNAK_WAITACK;
> msg.direction = I2C_WRITE;
> - msg.data = *(addr++);
> + msg.data = addr[alen];
> if (i2c_transfer(base, &msg))
> return -1;
> }
>
Thanks for spotting:
Rewiewed-by: Stefan Roese <sr at denx.de>
Thanks,
Stefan
More information about the U-Boot
mailing list