[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAL_JsqJab4wmfDSyz-uQpp24Qcus8j=Vks8Yfk472rgXyd5U6g@mail.gmail.com>
Date: Thu, 29 Apr 2021 11:12:45 -0500
From: Rob Herring <robh@...nel.org>
To: LABBE Corentin <clabbe@...libre.com>
Cc: devicetree@...r.kernel.org,
Linus Walleij <linus.walleij@...aro.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM"
<dmaengine@...r.kernel.org>, Vinod <vkoul@...nel.org>
Subject: Re: [PATCH RFC] dt-bindings: dma: convert arm-pl08x to yaml
On Thu, Apr 29, 2021 at 9:57 AM LABBE Corentin <clabbe@...libre.com> wrote:
>
> Le Wed, Apr 28, 2021 at 05:15:09PM -0500, Rob Herring a écrit :
> > On Wed, 28 Apr 2021 05:57:50 +0000, Corentin Labbe wrote:
> > > Converts dma/arm-pl08x.txt to yaml.
> > > In the process, I add an example for the faraday variant.
> > >
> > > Signed-off-by: Corentin Labbe <clabbe@...libre.com>
> > > ---
> > > .../devicetree/bindings/dma/arm-pl08x.txt | 59 --------
> > > .../devicetree/bindings/dma/arm-pl08x.yaml | 127 ++++++++++++++++++
> > > 2 files changed, 127 insertions(+), 59 deletions(-)
> > > delete mode 100644 Documentation/devicetree/bindings/dma/arm-pl08x.txt
> > > create mode 100644 Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > >
> >
> > 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:
> > ./Documentation/devicetree/bindings/dma/arm-pl08x.yaml:19:9: [warning] wrong indentation: expected 10 but found 8 (indentation)
> > ./Documentation/devicetree/bindings/dma/arm-pl08x.yaml:22:9: [warning] wrong indentation: expected 10 but found 8 (indentation)
> > ./Documentation/devicetree/bindings/dma/arm-pl08x.yaml:25:9: [warning] wrong indentation: expected 10 but found 8 (indentation)
> > ./Documentation/devicetree/bindings/dma/arm-pl08x.yaml:66:9: [warning] wrong indentation: expected 6 but found 8 (indentation)
> > ./Documentation/devicetree/bindings/dma/arm-pl08x.yaml:78:9: [warning] wrong indentation: expected 6 but found 8 (indentation)
> >
>
> Hello
>
> I have installed yamllint, so this kind of error should no happen again
>
> > dtschema/dtc warnings/errors:
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/arm,sp805.example.dt.yaml: watchdog@...90000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/arm,sp805.example.dt.yaml: watchdog@...90000: $nodename:0: 'watchdog@...90000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/arm,sp805.example.dt.yaml: watchdog@...90000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,sp805', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/arm,sp805.example.dt.yaml: watchdog@...90000: clocks: [[4294967295], [4294967295]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/arm,sp805.example.dt.yaml: watchdog@...90000: clock-names:0: 'apb_pclk' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/arm,sp805.example.dt.yaml: watchdog@...90000: clock-names: ['wdog_clk', 'apb_pclk'] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/arm,sp805.example.dt.yaml: watchdog@...90000: clock-names: Additional items are not allowed ('apb_pclk' was unexpected)
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/arm,sp805.example.dt.yaml: watchdog@...90000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/spi-pl022.example.dt.yaml: spi@...00000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/spi-pl022.example.dt.yaml: spi@...00000: $nodename:0: 'spi@...00000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/spi-pl022.example.dt.yaml: spi@...00000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,pl022', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/spi-pl022.example.dt.yaml: spi@...00000: 'clocks' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/spi-pl022.example.dt.yaml: spi@...00000: 'clock-names' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/spi/spi-pl022.example.dt.yaml: spi@...00000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/serial/pl011.example.dt.yaml: serial@...20000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/serial/pl011.example.dt.yaml: serial@...20000: $nodename:0: 'serial@...20000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/serial/pl011.example.dt.yaml: serial@...20000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,pl011', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/serial/pl011.example.dt.yaml: serial@...20000: clocks: [[4294967295], [4294967295]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/serial/pl011.example.dt.yaml: serial@...20000: clock-names:0: 'apb_pclk' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/serial/pl011.example.dt.yaml: serial@...20000: clock-names: ['uartclk', 'apb_pclk'] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/serial/pl011.example.dt.yaml: serial@...20000: clock-names: Additional items are not allowed ('apb_pclk' was unexpected)
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/serial/pl011.example.dt.yaml: serial@...20000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...20000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...20000: $nodename:0: 'cti@...20000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...20000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,coresight-cti', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...20000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...20000: 'oneOf' conditional failed, one must be fixed:
> > 'interrupts' is a required property
> > 'interrupts-extended' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: $nodename:0: 'cti@...000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,coresight-cti-v8-arch', 'arm,coresight-cti', 'arm,primecell'] is too long
> > Additional items are not allowed ('arm,primecell' was unexpected)
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > 'arm,primecell' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: 'oneOf' conditional failed, one must be fixed:
> > 'interrupts' is a required property
> > 'interrupts-extended' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: $nodename:0: 'cti@...000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,coresight-cti', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...000: 'oneOf' conditional failed, one must be fixed:
> > 'interrupts' is a required property
> > 'interrupts-extended' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...10000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...10000: $nodename:0: 'cti@...10000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...10000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,coresight-cti', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...10000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/coresight-cti.example.dt.yaml: cti@...10000: 'oneOf' conditional failed, one must be fixed:
> > 'interrupts' is a required property
> > 'interrupts-extended' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...0: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...0: $nodename:0: 'mmc@...0' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...0: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,pl180', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...0: clocks: [[4294967295], [4294967295]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...0: clock-names:0: 'apb_pclk' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...0: clock-names: ['mclk', 'apb_pclk'] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...0: clock-names: Additional items are not allowed ('apb_pclk' was unexpected)
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...0: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...26000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...26000: $nodename:0: 'mmc@...26000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...26000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,pl18x', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...26000: clocks: [[4294967295, 1, 5], [4294967295, 1, 5]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...26000: clock-names:0: 'apb_pclk' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...26000: clock-names: ['sdi', 'apb_pclk'] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...26000: clock-names: Additional items are not allowed ('apb_pclk' was unexpected)
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...26000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...f6000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...f6000: $nodename:0: 'mmc@...f6000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...f6000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,pl18x', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...f6000: clocks: [[4294967295], [4294967295]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...f6000: clock-names:0: 'apb_pclk' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...f6000: clock-names: ['mclk', 'apb_pclk'] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...f6000: clock-names: Additional items are not allowed ('apb_pclk' was unexpected)
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...f6000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...07000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...07000: $nodename:0: 'mmc@...07000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...07000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,pl18x', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mmc/arm,pl18x.example.dt.yaml: mmc@...07000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/timer/arm,sp804.example.dt.yaml: timer@...00000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/timer/arm,sp804.example.dt.yaml: timer@...00000: $nodename:0: 'timer@...00000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/timer/arm,sp804.example.dt.yaml: timer@...00000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,sp804', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/timer/arm,sp804.example.dt.yaml: timer@...00000: interrupts: [[0, 0, 4], [0, 1, 4]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/timer/arm,sp804.example.dt.yaml: timer@...00000: clocks: [[4294967295], [4294967295], [4294967295]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/timer/arm,sp804.example.dt.yaml: timer@...00000: clock-names:0: 'apb_pclk' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/timer/arm,sp804.example.dt.yaml: timer@...00000: clock-names: ['timer1', 'timer2', 'apb_pclk'] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/timer/arm,sp804.example.dt.yaml: timer@...00000: clock-names: Additional items are not allowed ('timer2', 'apb_pclk' were unexpected)
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/timer/arm,sp804.example.dt.yaml: timer@...00000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: $nodename:0: 'mailbox@...f0000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,mhu', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: interrupts: [[0, 36, 4], [0, 35, 4], [0, 37, 4]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: $nodename:0: 'mailbox@...f0000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,mhu-doorbell', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: interrupts: [[0, 36, 4], [0, 35, 4], [0, 37, 4]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhu.example.dt.yaml: mailbox@...f0000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhuv2.example.dt.yaml: mailbox@...f0000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhuv2.example.dt.yaml: mailbox@...f0000: $nodename:0: 'mailbox@...f0000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhuv2.example.dt.yaml: mailbox@...f0000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,mhuv2-tx', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhuv2.example.dt.yaml: mailbox@...f0000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhuv2.example.dt.yaml: mailbox@...f1000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhuv2.example.dt.yaml: mailbox@...f1000: $nodename:0: 'mailbox@...f1000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhuv2.example.dt.yaml: mailbox@...f1000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,mhuv2-rx', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mailbox/arm,mhuv2.example.dt.yaml: mailbox@...f1000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/bus/arm,integrator-ap-lm.example.dt.yaml: serial@...000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/bus/arm,integrator-ap-lm.example.dt.yaml: serial@...000: $nodename:0: 'serial@...000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/bus/arm,integrator-ap-lm.example.dt.yaml: serial@...000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,pl011', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/bus/arm,integrator-ap-lm.example.dt.yaml: serial@...000: 'clocks' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/bus/arm,integrator-ap-lm.example.dt.yaml: serial@...000: 'clock-names' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/bus/arm,integrator-ap-lm.example.dt.yaml: serial@...000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.example.dt.yaml: mmc@...18000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.example.dt.yaml: mmc@...18000: $nodename:0: 'mmc@...18000' does not match '^dma-controller(@.*)?$'
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.example.dt.yaml: mmc@...18000: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['arm,pl18x', 'arm,primecell'] is too short
> > 'arm,pl080' was expected
> > 'arm,pl081' was expected
> > 'faraday,ftdma020' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.example.dt.yaml: mmc@...18000: clocks: [[4294967295, 0], [4294967295, 1]] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.example.dt.yaml: mmc@...18000: clock-names:0: 'apb_pclk' was expected
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.example.dt.yaml: mmc@...18000: clock-names: ['mclk', 'apb_pclk'] is too long
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.example.dt.yaml: mmc@...18000: clock-names: Additional items are not allowed ('apb_pclk' was unexpected)
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.example.dt.yaml: mmc@...18000: '#dma-cells' is a required property
> > From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
> > Error: Documentation/devicetree/bindings/dma/arm-pl08x.example.dts:56.27-28 syntax error
> > FATAL ERROR: Unable to parse input tree
> > make[1]: *** [scripts/Makefile.lib:377: Documentation/devicetree/bindings/dma/arm-pl08x.example.dt.yaml] Error 1
> > make[1]: *** Waiting for unfinished jobs....
> > make: *** [Makefile:1414: dt_binding_check] Error 2
> >
>
> Most of thoses error came from the fact the compatible end with "arm,primecell".
> So it try to match against the wrong dt-binding.
>
> Does there is any correct way to limit/prevent this ?
You need a custom 'select' schema. Look at the 'select' schema for
other users of "arm,primecell".
Rob
Powered by blists - more mailing lists