Re: [PATCH 00/11] dt-bindings: convert various Arm Ltd. IP to DT schema

From: Rob Herring <robh@kernel.org>
To: Andre Przywara <andre.przywara@arm.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Liviu Dudau <liviu.dudau@arm.com>,
	Robin Murphy <robin.murphy@arm.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	Will Deacon <will@kernel.org>
Subject: Re: [PATCH 00/11] dt-bindings: convert various Arm Ltd. IP to DT schema
Date: Wed, 27 Apr 2022 14:29:20 -0500	[thread overview]
Message-ID: <YmmZkAvzRx+uhxo1@robh.at.kernel.org> (raw)
In-Reply-To: <20220427112528.4097815-1-andre.przywara@arm.com>

On Wed, Apr 27, 2022 at 12:25:17PM +0100, Andre Przywara wrote:
> Hi,
> 
> in an effort to get the FVP DTs to pass the automated DT bindings checks,
> this series collects some DT schema binding conversions for various
> Arm Ltd. IP devices.
> This is mostly for old IP, but it's still used by the FVP, for which we
> have a DT in the tree.
> 
> Please have a look!
> 
> Cheers,
> Andre
> 
> Andre Przywara (11):
>   dt-bindings: iommu: arm,smmu-v3: make PRI IRQ optional
>   dt-bindings: arm: spe-pmu: convert to DT schema
>   dt-bindings: arm: sp810: convert to DT schema
>   dt-bindings: sound: add Arm PL041 AACI DT schema
>   dt-bindings: serio: add Arm PL050 DT schema
>   dt-bindings: arm: convert vexpress-sysregs to DT schema
>   dt-bindings: arm: convert vexpress-config to DT schema
>   dt-bindings: display: convert PL110/PL111 to DT schema
>   dt-bindings: display: convert Arm HDLCD to DT schema
>   dt-bindings: display: convert Arm Mali-DP to DT schema
>   dt-bindings: display: convert Arm Komeda to DT schema
> 
>  .../devicetree/bindings/arm/sp810.txt         |  46 ---
>  .../devicetree/bindings/arm/sp810.yaml        |  82 ++++++
>  .../devicetree/bindings/arm/spe-pmu.txt       |  20 --
>  .../devicetree/bindings/arm/spe-pmu.yaml      |  40 +++
>  .../bindings/arm/vexpress-config.yaml         | 274 ++++++++++++++++++
>  .../bindings/arm/vexpress-sysreg.txt          | 103 -------
>  .../bindings/arm/vexpress-sysreg.yaml         |  89 ++++++
>  .../devicetree/bindings/display/arm,hdlcd.txt |  79 -----
>  .../bindings/display/arm,hdlcd.yaml           |  91 ++++++
>  .../bindings/display/arm,komeda.txt           |  78 -----
>  .../bindings/display/arm,komeda.yaml          | 130 +++++++++
>  .../bindings/display/arm,malidp.txt           |  68 -----
>  .../bindings/display/arm,malidp.yaml          | 117 ++++++++
>  .../devicetree/bindings/display/arm,pl11x.txt | 110 -------
>  .../bindings/display/arm,pl11x.yaml           | 174 +++++++++++
>  .../bindings/iommu/arm,smmu-v3.yaml           |  21 +-
>  .../devicetree/bindings/serio/amba-pl050.yaml |  67 +++++
>  .../devicetree/bindings/sound/amba-pl041.yaml |  62 ++++
>  18 files changed, 1142 insertions(+), 509 deletions(-)

>  delete mode 100644 Documentation/devicetree/bindings/arm/sp810.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/sp810.yaml
>  delete mode 100644 Documentation/devicetree/bindings/arm/spe-pmu.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/spe-pmu.yaml

This should be moved to bindings/perf/. I guess arm/pmu.yaml should be 
too.

>  create mode 100644 Documentation/devicetree/bindings/arm/vexpress-config.yaml
>  delete mode 100644 Documentation/devicetree/bindings/arm/vexpress-sysreg.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/vexpress-sysreg.yaml

>  create mode 100644 Documentation/devicetree/bindings/serio/amba-pl050.yaml
>  create mode 100644 Documentation/devicetree/bindings/sound/amba-pl041.yaml

Please use the compatible string as the file name where there's a 
single compatible. Maybe not SPE though. Minimally, use the vendor 
prefix in the file name ('arm,'). 

Rob