Message ID | 20240628205430.24775-2-james.quinlan@broadcom.com (mailing list archive) |
---|---|
State | New |
Delegated to: | Krzysztof WilczyĆski |
Headers | show |
Series | PCI: brcnstb: Enable STB 7712 SOC | expand |
On 28/06/2024 22:54, Jim Quinlan wrote: > - Update maintainer. Why? > - Adds a driver compatible string for the new STB SOC 7712 > - Adds two new resets for the 7712: "bridge", for the > the bridge between the PCIe core and the memory bus; > and "swinit", the PCIe core reset. > > Signed-off-by: Jim Quinlan <james.quinlan@broadcom.com> > --- > .../bindings/pci/brcm,stb-pcie.yaml | 24 ++++++++++++++++++- > 1 file changed, 23 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml b/Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml > index 11f8ea33240c..f594fef343a1 100644 > --- a/Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml > +++ b/Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml > @@ -7,12 +7,13 @@ $schema: http://devicetree.org/meta-schemas/core.yaml# > title: Brcmstb PCIe Host Controller > > maintainers: > - - Nicolas Saenz Julienne <nsaenzjulienne@suse.de> > + - Jim Quinlan <james.quinlan@broadcom.com> > > properties: > compatible: > items: > - enum: > + - brcm,bcm7712-pcie # STB sibling SOC of Raspberry Pi 5 Why did you place it here? Isn't the list ordered? > - brcm,bcm2711-pcie # The Raspberry Pi 4 > - brcm,bcm4908-pcie > - brcm,bcm7211-pcie # Broadcom STB version of RPi4 > @@ -146,6 +147,27 @@ allOf: > required: > - resets > - reset-names > + - if: > + properties: > + compatible: > + contains: > + const: brcm,bcm7712-pcie > + then: > + properties: > + resets: Fix the binding first - properties should be defined in top level "properties:" and then customized. Where are "resets"? > + items: > + - description: phandle pointing to the RESCAL reset controller Drop redundant text. There is no point in saying that phandle is a phandle. It's obvious. Say something which is not obvious. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml b/Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml index 11f8ea33240c..f594fef343a1 100644 --- a/Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml +++ b/Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml @@ -7,12 +7,13 @@ $schema: http://devicetree.org/meta-schemas/core.yaml# title: Brcmstb PCIe Host Controller maintainers: - - Nicolas Saenz Julienne <nsaenzjulienne@suse.de> + - Jim Quinlan <james.quinlan@broadcom.com> properties: compatible: items: - enum: + - brcm,bcm7712-pcie # STB sibling SOC of Raspberry Pi 5 - brcm,bcm2711-pcie # The Raspberry Pi 4 - brcm,bcm4908-pcie - brcm,bcm7211-pcie # Broadcom STB version of RPi4 @@ -146,6 +147,27 @@ allOf: required: - resets - reset-names + - if: + properties: + compatible: + contains: + const: brcm,bcm7712-pcie + then: + properties: + resets: + items: + - description: phandle pointing to the RESCAL reset controller + - description: phandle pointing to the BRIDGE reset controller + - description: phandle pointing to the core SWINIT reset controller + + reset-names: + items: + - const: rescal + - const: bridge + + required: + - resets + - reset-names unevaluatedProperties: false
- Update maintainer. - Adds a driver compatible string for the new STB SOC 7712 - Adds two new resets for the 7712: "bridge", for the the bridge between the PCIe core and the memory bus; and "swinit", the PCIe core reset. Signed-off-by: Jim Quinlan <james.quinlan@broadcom.com> --- .../bindings/pci/brcm,stb-pcie.yaml | 24 ++++++++++++++++++- 1 file changed, 23 insertions(+), 1 deletion(-)