On Mon, Aug 22, 2022 at 02:48:19AM +0200, Andrew Lunn wrote: > On Sun, Aug 21, 2022 at 03:21:26PM -0500, Rob Herring wrote: > > On Sat, 20 Aug 2022 21:48:01 +0200, Andrew Lunn wrote: > > > This works for Kirkwood, but orion5x has an odd bus structure > > > which results in some warnings. > > > > > > Signed-off-by: Andrew Lunn <andrew@xxxxxxx> > > > --- > > > .../bindings/mtd/marvell,orion-nand.yaml | 85 +++++++++++++++++++ > > > .../devicetree/bindings/mtd/orion-nand.txt | 50 ----------- > > > 2 files changed, 85 insertions(+), 50 deletions(-) > > > create mode 100644 Documentation/devicetree/bindings/mtd/marvell,orion-nand.yaml > > > delete mode 100644 Documentation/devicetree/bindings/mtd/orion-nand.txt > > > > > > > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' > > on your patch (DT_CHECKER_FLAGS is new in v5.13): > > > > yamllint warnings/errors: > > > > dtschema/dtc warnings/errors: > > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mtd/marvell,orion-nand.example.dtb: nand-controller@f4000000: #size-cells:0:0: 0 was expected > > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mtd/nand-controller.yaml > > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mtd/marvell,orion-nand.example.dtb: nand-controller@f4000000: #size-cells:0:0: 0 was expected > > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mtd/marvell,orion-nand.yaml > > Does the bot not apply patches N-1 before testing patch N? In this > case, if patch N-2 was applied, you would not see this warning. It does unless there is a failure and it drops the prior patches. Rob