]> git.dujemihanovic.xyz Git - u-boot.git/commit
misc: i2c_eeprom: implement different probe test eeprom offset
authorEugen Hristev <eugen.hristev@microchip.com>
Thu, 7 May 2020 08:53:18 +0000 (11:53 +0300)
committerHeiko Schocher <hs@denx.de>
Thu, 28 May 2020 04:51:06 +0000 (06:51 +0200)
commitb24dc83f156604f253ef6521776444188c5bff9b
treedaf09b5135606ba8eb9023f18042cc4ddba4ec9c
parentda585c3c680add381662417b11cfc7d0ac310ba5
misc: i2c_eeprom: implement different probe test eeprom offset

Because of this commit :
5ae84860b0 ("misc: i2c_eeprom: verify that the chip is functional at probe()")
at probe time, each eeprom is tested for read at offset 0.

The Atmel AT24MAC402 eeprom has different mapping. One i2c slave address is
used for the lower 0x80 bytes and another i2c slave address is used for the
upper 0x80 bytes. Because of this basically the i2c master sees 2 different
slaves. We need the upper bytes because we read the unique MAC address from
this EEPROM area.

However this implies that our slave address will return error on reads
from address 0x0 to 0x80.

To solve this, implemented an offset field inside platform data that is by
default 0 (as it is used now), but can be changed in the compatible table.

The probe function will now read at this offset and use it, instead of blindly
checking offset 0.

This will fix the regression noticed on these EEPROMs since the commit
abovementioned that introduces the probe failed issue.

Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
Reviewed-by: Heiko Schocher <hs@denx.de>
drivers/misc/i2c_eeprom.c