Fix 16 bit address byte order for i2c transactions

[chriskeeser]

Why the change?

When setting the read/write pointer for an EEPROM that uses 16 bit addressing to access the internal memory, the bytes are sent [high byte] [low byte]. This commit fixes the byte ordering when sending the address to the device.

Problem example

I used an I2C EEPROM with 16 bit addressing and stored a string into the memory for recall within u-boot. Here is the information stored in the eeprom, and being read out again from within linux to show that is stored correctly:

but, when I read it from u-boot, the data was incorrect

using an i2c protocol analyzer to observe the transaction showed the problem. The code is reading 16 bytes at a time, and setting the address after each read of 16 bytes. The address is incorrect (high and low bytes are swapped)

I can further illustrate the problem by forcing the swap of the bytes manually in the command

The fix

As shown in the code, the fix is to properly send the address when doing a read or a write. After making the change, the data can be read correctly from the i2c device.