Hi Alexander, On 2026-05-26T12:40:39, Feilke, Alexander <[email protected]> wrote: > arm: dts: tqma7: integrate tq,eeprom sysinfo driver > > TQMa7 has board-information located in EEPROM at offset 0x20. > Add necessary nodes and properties for tq,eeprom sysinfo driver. > > Signed-off-by: Alexander Feilke <[email protected]> > > arch/arm/dts/imx7s-tqma7-u-boot.dtsi | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+)
Reviewed-by: Simon Glass <[email protected]> > diff --git a/arch/arm/dts/imx7s-tqma7-u-boot.dtsi > b/arch/arm/dts/imx7s-tqma7-u-boot.dtsi > @@ -9,6 +9,26 @@ > +&m24c64 { > + nvmem-layout { > + compatible = 'fixed-partitions'; Should this be 'fixed-layout' ? See nvmem_cell_get_by_index() which only walks up past the layout node when its parent is compatible with 'fixed-layout', so I suspect that the cell lookup will not find the EEPROM device? Your binding doc in patch 3 uses 'fixed-layout', for example. Regards, Simon
