Skip to contentSkip to conversation

dt-bindings: PCI: cdns-ep: Fix 'unevaluatedProperties' warnings

PATCH Applied Rob Herring 1 participant 2 messages Dec 6, 2021 – Dec 14, 2021

Subsystems (4)
Device tree bindings PCI PCI native host bridge and endpoint drivers PCI driver for cadence PCIe IP
lore
Share & tools
lore

Rob Herring

author maintainer
#0 lore raw

PATCH 1/1 dt-bindings: PCI: cdns-ep: Fix 'unevaluatedProperties' warnings

With 'unevaluatedProperties' support implemented, the TI j721e endpoint binding example has a warning:

Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.example.dt.yaml: pcie-ep@d000000: Unevaluated properties are not allowed ('max-link-speed', 'num-lanes', 'max-functions' were unexpected)

Adjust where pci-ep.yaml is referenced so that ti,j721e-pci-ep.yaml will include it.

Cc Tom Joseph <[email protected]>
Cc Bjorn Helgaas <[email protected]>
Cc [email protected]
Cc [email protected]
Signed-off-by Rob Herring <[email protected]>
---
 Documentation/devicetree/bindings/pci/cdns,cdns-pcie-ep.yaml | 1 -
 Documentation/devicetree/bindings/pci/cdns-pcie-ep.yaml      | 1 +
 2 files changed, 1 insertion(+), 1 deletion(-)
Documentation/devicetree/bindings/pci/cdns,cdns-pcie-ep.yaml +0 -1
index 651eee88989d..e6ef1012a580 100644
@@ -11,7 +11,6 @@ maintainers:
 
 allOf:
   - $ref: "cdns-pcie-ep.yaml#"
-  - $ref: "pci-ep.yaml#"
 
 properties:
   compatible:
Documentation/devicetree/bindings/pci/cdns-pcie-ep.yaml +1 -0
index 21e8a8849076..baeafda36ebe 100644
@@ -11,6 +11,7 @@ maintainers:
 
 allOf:
   - $ref: "cdns-pcie.yaml#"
+  - $ref: "pci-ep.yaml#"
 
 properties:
   cdns,max-outbound-regions:
Signature · 1 line
2.32.0
author maintainer Applied
#1 lore raw
Quoted 13 lines · Rob Herring

With 'unevaluatedProperties' support implemented, the TI j721e endpoint binding example has a warning:

Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.example.dt.yaml: pcie-ep@d000000: Unevaluated properties are not allowed ('max-link-speed', 'num-lanes', 'max-functions' were unexpected)

Adjust where pci-ep.yaml is referenced so that ti,j721e-pci-ep.yaml will include it.

Cc Tom Joseph <[email protected]>
Cc Bjorn Helgaas <[email protected]>
Cc [email protected]
Cc [email protected]
Signed-off-by Rob Herring <[email protected]>
Quoted 5 lines
---
 Documentation/devicetree/bindings/pci/cdns,cdns-pcie-ep.yaml | 1 -
 Documentation/devicetree/bindings/pci/cdns-pcie-ep.yaml      | 1 +
 2 files changed, 1 insertion(+), 1 deletion(-)

Applied, thanks!