| Message ID | 20221124081221.1206167-2-mranostay@ti.com (mailing list archive) |
|---|---|
| State | Handled Elsewhere, archived |
| Delegated to: | Rob Herring |
| Headers | show |
| Series | PCI: add 4x lane support for pci-j721e controllers | expand |
On 24/11/2022 09:12, Matt Ranostay wrote: > Add num-lanes schema checks based on compatible string on available lanes > for that platform. > > Signed-off-by: Matt Ranostay <mranostay@ti.com> > --- > .../bindings/pci/ti,j721e-pci-ep.yaml | 28 +++++++++++++++++-- > .../bindings/pci/ti,j721e-pci-host.yaml | 28 +++++++++++++++++-- > 2 files changed, 50 insertions(+), 6 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml > index 10e6eabdff53..1aeea168d3d0 100644 > --- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml > +++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml > @@ -10,9 +10,6 @@ title: TI J721E PCI EP (PCIe Wrapper) > maintainers: > - Kishon Vijay Abraham I <kishon@ti.com> > > -allOf: > - - $ref: "cdns-pcie-ep.yaml#" > - > properties: > compatible: > oneOf: > @@ -65,6 +62,31 @@ properties: > items: > - const: link_state > > +allOf: > + - $ref: "cdns-pcie-ep.yaml#" While moving it, drop the quotes. > + - if: > + properties: > + compatible: > + enum: > + - ti,am64-pcie-ep > + then: > + properties: > + num-lanes: > + minimum: 1 > + maximum: 1 > + > + - if: > + properties: > + compatible: > + enum: > + - ti,j7200-pcie-ep > + - ti,j721e-pcie-ep > + then: > + properties: > + num-lanes: > + minimum: 1 > + maximum: 2 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml > index b0513b197d08..8eca0d08303f 100644 > --- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml > +++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml > @@ -10,9 +10,6 @@ title: TI J721E PCI Host (PCIe Wrapper) > maintainers: > - Kishon Vijay Abraham I <kishon@ti.com> > > -allOf: > - - $ref: "cdns-pcie-host.yaml#" > - > properties: > compatible: > oneOf: > @@ -98,6 +95,31 @@ properties: > interrupts: > maxItems: 1 > > +allOf: > + - $ref: "cdns-pcie-host.yaml#" Same here. > + - if: > + properties: > + compatible: > + enum: > + - ti,am64-pcie-host > + then: > + properties: > + num-lanes: const: 1 Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml index 10e6eabdff53..1aeea168d3d0 100644 --- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml +++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml @@ -10,9 +10,6 @@ title: TI J721E PCI EP (PCIe Wrapper) maintainers: - Kishon Vijay Abraham I <kishon@ti.com> -allOf: - - $ref: "cdns-pcie-ep.yaml#" - properties: compatible: oneOf: @@ -65,6 +62,31 @@ properties: items: - const: link_state +allOf: + - $ref: "cdns-pcie-ep.yaml#" + - if: + properties: + compatible: + enum: + - ti,am64-pcie-ep + then: + properties: + num-lanes: + minimum: 1 + maximum: 1 + + - if: + properties: + compatible: + enum: + - ti,j7200-pcie-ep + - ti,j721e-pcie-ep + then: + properties: + num-lanes: + minimum: 1 + maximum: 2 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml index b0513b197d08..8eca0d08303f 100644 --- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml +++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml @@ -10,9 +10,6 @@ title: TI J721E PCI Host (PCIe Wrapper) maintainers: - Kishon Vijay Abraham I <kishon@ti.com> -allOf: - - $ref: "cdns-pcie-host.yaml#" - properties: compatible: oneOf: @@ -98,6 +95,31 @@ properties: interrupts: maxItems: 1 +allOf: + - $ref: "cdns-pcie-host.yaml#" + - if: + properties: + compatible: + enum: + - ti,am64-pcie-host + then: + properties: + num-lanes: + minimum: 1 + maximum: 1 + + - if: + properties: + compatible: + enum: + - ti,j7200-pcie-host + - ti,j721e-pcie-host + then: + properties: + num-lanes: + minimum: 1 + maximum: 2 + required: - compatible - reg
Add num-lanes schema checks based on compatible string on available lanes for that platform. Signed-off-by: Matt Ranostay <mranostay@ti.com> --- .../bindings/pci/ti,j721e-pci-ep.yaml | 28 +++++++++++++++++-- .../bindings/pci/ti,j721e-pci-host.yaml | 28 +++++++++++++++++-- 2 files changed, 50 insertions(+), 6 deletions(-)