Messages in this thread Patch in this message | | | From | Conor Dooley <> | Subject | [PATCH 3/4] dt-bindings: PCI: microchip,pcie-host: fix incorrect child node name | Date | Thu, 11 Aug 2022 21:33:06 +0100 |
| |
From: Conor Dooley <conor.dooley@microchip.com>
v2022.08 of dt-schema improved checking of unevaluatedProperties, and exposed a previously unseen warning for the PCIe controller's interrupt controller node name:
arch/riscv/boot/dts/microchip/mpfs-icicle-kit.dtb: pcie@2000000000: Unevaluated properties are not allowed ('clock-names', 'clocks', 'legacy-interrupt-controller', 'microchip,axi-m-atr0' were unexpected) From schema: Documentation/devicetree/bindings/pci/microchip,pcie-host.yaml
Make the property in the binding match the node name actually used in the dts.
Fixes: dcd49679fb3a ("dt-bindings: PCI: Fix 'unevaluatedProperties' warnings") Signed-off-by: Conor Dooley <conor.dooley@microchip.com> --- This is another one Rob where I feel like I'm doing the wrong thing. The Linux driver gets the child node without using the name, but another OS etc could in theory (or reality), right? --- .../devicetree/bindings/pci/microchip,pcie-host.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/pci/microchip,pcie-host.yaml b/Documentation/devicetree/bindings/pci/microchip,pcie-host.yaml index 2a2166f09e2c..9b123bcd034c 100644 --- a/Documentation/devicetree/bindings/pci/microchip,pcie-host.yaml +++ b/Documentation/devicetree/bindings/pci/microchip,pcie-host.yaml @@ -71,7 +71,7 @@ properties: msi-parent: description: MSI controller the device is capable of using. - interrupt-controller: + legacy-interrupt-controller: type: object properties: '#address-cells': @@ -125,7 +125,7 @@ examples: msi-controller; bus-range = <0x00 0x7f>; ranges = <0x03000000 0x0 0x78000000 0x0 0x78000000 0x0 0x04000000>; - pcie_intc0: interrupt-controller { + pcie_intc0: legacy-interrupt-controller { #address-cells = <0>; #interrupt-cells = <1>; interrupt-controller; -- 2.37.1
| |