[PATCH] i2c: mvtwsi: Swab the register address if its size is > 1

Heiko Schocher hs at denx.de
Mon Dec 20 11:12:48 CET 2021


Hello Stefan,

On 18.12.21 14:42, Stefan Roese wrote:
> Hi Heiko,
> 
> On 11/18/21 09:18, Stefan Roese wrote:
>> Testing on Armada XP with an EEPROM using register address with size
>> of 2 has shown, that the register address bytes are sent to the I2C
>> EEPROM in the incorrect order. This patch swabs the address bytes so
>> that the correct address is transferred to the I2C device.
>>
>> BTW: This worked without any issues before migrating Armada XP to
>> DM I2C.
>>
>> Signed-off-by: Stefan Roese <sr at denx.de>
>> Cc: Heiko Schocher <hs at denx.de>
>> Cc: Samuel Holland <samuel at sholland.org>
>> Cc: Baruch Siach <baruch at tkos.co.il>
>> Cc: Pali Rohár <pali at kernel.org>
>> Cc: Marek Behún <marek.behun at nic.cz>
>> ---
>> It would be good if other users of this I2C driver could test this change
>> with e.g. I2C EEPROM devices using 2 bytes (or more) for addressing.
> 
> Could you and other please take a look at this? Would be great, if this
> could be pulled in the next merge window.

Applied to u-boot-i2c master

Thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de


More information about the U-Boot mailing list