Re: [PATCH RFT 2/2] ARM: dts: mxs: imx28: Fix NAND hierarchy description

Miquel Raynal
2023-12-21T10:16:20Z

Hello,

sebastien.szymanski@armadeus.com wrote on Thu, 21 Dec 2023 10:56:09
+0100:

> Hello Stefan,
>
> On 12/18/23 14:06, Stefan Wahren wrote:
> > The size-cells for GPMI are wrong in imx28.dtsi, which causes the
> > following warning:
> >
> > nand-controller@8000c000: #size-cells:0:0: 0 was expected
> >
> > The reason for this is the definition of the partitions directly
> > below the nand controller node. According to nand-controller.yaml
> > the NAND chip must be defined as a child of the controller. Even
> > the fixed partitions must be kept in a partitions container.

Bindings evolve over time, we sometimes deprecate them when they are
problematic but:
- the old representations are still accepted (but marked legacy)
- if you want to support the new representation you probably need to
update the controller driver as well (to support both).

In this case I would expect a deprecation notice rather than a pure
error.

Thanks,
Miquèl