[U-Boot] [PATCH 1/2] misc: i2c_eeprom: add fixed partitions support

[Heiko Schocher]

Hello Robert,

Am 28.10.2019 um 19:29 schrieb Robert Beckett:
> Add ability to partition eeprom via devicetree bindings
> 
> Signed-off-by: Robert Beckett <bob.beckett at collabora.com>
> ---
> 
>   drivers/misc/i2c_eeprom.c | 98 +++++++++++++++++++++++++++++++++++++++
>   1 file changed, 98 insertions(+)

Reviewed-by: Heiko Schocher <hs at denx.de>

Remark:
I just wondering that there is no MTD support for i2c eeproms in
MTD (also not in linux) ... okay, flash has blocked base access,
while i2c eeprom can have byte accesses ... but does this really
is a show stopper for adding i2c eeproms to MTD layer?

You already use the MTD "fixed-partitions" property, which is fine.

May it s worth to look into using mtd layer also for i2c eeproms?

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