- 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(-)
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
--
2.17.1
s/maintainter/maintainer/ in subject On Fri, Jun 28, 2024 at 04:54:20PM -0400, Jim Quinlan wrote: > - 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. s/Adds/Add/ to be imperative and match "Update".
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
On Mon, Jul 1, 2024 at 5:12 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > On 28/06/2024 22:54, Jim Quinlan wrote: > > - Update maintainer. > > Why? I haven't observed any action or feedback from Nicolas in years. Nicolas, please state your case for being a maintainer because it is not making sense from my perspective. > > > - 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? It is ordered from newest at top to oldest at bottom -- is the convention to put the "new" at the bottom? > > > - 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. My kernel Yaml-fu is weak. I will redo. Regards, Jim Quinlan Broadcom STB/CM > > > Best regards, > Krzysztof >
On 02/07/2024 23:57, Jim Quinlan wrote: > On Mon, Jul 1, 2024 at 5:12 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: >> >> On 28/06/2024 22:54, Jim Quinlan wrote: >>> - Update maintainer. >> >> Why? > > I haven't observed any action or feedback from Nicolas in years. > Nicolas, please > state your case for being a maintainer because it is not making sense from > my perspective. Commit msg should explain 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? > > It is ordered from newest at top to oldest at bottom -- is the > convention to put the "new" at the bottom? Both your proposals lead to conflicts, so of course no. The lists are ordered alphabetically, in most cases. Best regards, Krzysztof
© 2016 - 2025 Red Hat, Inc.