[U-Boot] [PATCH] i2c: Fix pca953x endianess issue

From: Dirk Eibach dirk.eibach@gdsys.cc
By reading 2 consecutive bytes from i2c to an u16 value we have an endianess issue.
Signed-off-by: Dirk Eibach dirk.eibach@gdsys.cc ---
drivers/gpio/pca953x.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpio/pca953x.c b/drivers/gpio/pca953x.c index 7371cd4..c8c8637 100644 --- a/drivers/gpio/pca953x.c +++ b/drivers/gpio/pca953x.c @@ -88,8 +88,10 @@ static int pca953x_reg_write(uint8_t chip, uint addr, uint mask, uint data) if (i2c_read(chip, addr << 1, 1, (u8*)&valw, 2)) return -1;
+ valw = le16_to_cpu(valw); valw &= ~mask; valw |= data; + valw = cpu_to_le16(valw);
return i2c_write(chip, addr << 1, 1, (u8*)&valw, 2); } @@ -107,7 +109,7 @@ static int pca953x_reg_read(uint8_t chip, uint addr, uint *data) } else { if (i2c_read(chip, addr << 1, 1, (u8*)&valw, 2)) return -1; - *data = (int)valw; + *data = (uint)le16_to_cpu(valw); } return 0; }

On Thu, Oct 29, 2015 at 01:51:27PM +0100, Dirk Eibach wrote:
From: Dirk Eibach dirk.eibach@gdsys.cc
By reading 2 consecutive bytes from i2c to an u16 value we have an endianess issue.
Signed-off-by: Dirk Eibach dirk.eibach@gdsys.cc
Applied to u-boot/master, thanks!
participants (2)
-
dirk.eibach@gdsys.cc
-
Tom Rini