[U-Boot] [PATCH] i2c: mv_i2c.c: Correct address endianness

Heiko Schocher hs at denx.de
Mon Dec 19 07:22:04 CET 2016


Hello Stefan,

Am 19.12.2016 um 06:56 schrieb Stefan Roese:
> 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>

Typo ... I fix this when applying this patch.

Thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany


More information about the U-Boot mailing list