lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <YIsHCnMm+mDBA+yo@Red>
Date:   Thu, 29 Apr 2021 21:20:42 +0200
From:   LABBE Corentin <clabbe@...libre.com>
To:     Rob Herring <robh@...nel.org>
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

Le Thu, Apr 29, 2021 at 11:12:45AM -0500, Rob Herring a écrit :
> 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".
> 

It is exactly what I needed.
Thanks.

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ