[PATCH v2 5/6] arm: dts: tqma7: integrate tq, eeprom sysinfo driver

[Simon Glass]

Hi Alexander,

On 2026-05-26T12:40:39, Feilke, Alexander
<Alexander.Feilke at ew.tq-group.com> 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 <alexander.feilke at ew.tq-group.com>
>
> arch/arm/dts/imx7s-tqma7-u-boot.dtsi | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)

Reviewed-by: Simon Glass <sjg at chromium.org>

> 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