On Fri, May 27, 2022 at 11:38:39AM +0200, Marc Kleine-Budde wrote: > On 25.05.2022 15:59:46, Rob Herring wrote: > > With unevaluatedProperties issues fixed, 'nand-bus-width' and > > 'nand-ecc-mode' are flagged as undocumented. Removing them from the example > > is the easiest solution to silence the warnings. > > > > Signed-off-by: Rob Herring <robh@xxxxxxxxxx> > > At least the 'nand-bus-width' property is described in > "nand-controller.yaml" and the "mxc-nand.yaml" refers to it. > > | allOf: > | - $ref: "nand-controller.yaml" > > Is this ref broken? Nope, read my note below. 'nand-bus-width' is documented as a nand chip (child node) property, not a nand controller property. > regards, > Marc > > > --- > > These properties may be deprecated, but they are still widely used in > > bindings. They either need to be documented (and marked deprecated) or > > removed from current users (i.e. dts files). ^^^^ > > --- > > Documentation/devicetree/bindings/mtd/mxc-nand.yaml | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/mtd/mxc-nand.yaml b/Documentation/devicetree/bindings/mtd/mxc-nand.yaml > > index 73b86f2226c7..66da1b476ab7 100644 > > --- a/Documentation/devicetree/bindings/mtd/mxc-nand.yaml > > +++ b/Documentation/devicetree/bindings/mtd/mxc-nand.yaml > > @@ -37,6 +37,4 @@ examples: > > compatible = "fsl,imx27-nand"; > > reg = <0xd8000000 0x1000>; > > interrupts = <29>; > > - nand-bus-width = <8>; > > - nand-ecc-mode = "hw"; > > }; > > -- > > 2.34.1 > > > > > > -- > Pengutronix e.K. | Marc Kleine-Budde | > Embedded Linux | https://www.pengutronix.de | > Vertretung West/Dortmund | Phone: +49-231-2826-924 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
