On Mon, Jul 1, 2024 at 5:12 AM Krzysztof Kozlowski <krzk@xxxxxxxxxx> 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@xxxxxxxxxxxx> > > --- > > .../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@xxxxxxx> > > + - Jim Quinlan <james.quinlan@xxxxxxxxxxxx> > > > > 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 >
Attachment:
smime.p7s
Description: S/MIME Cryptographic Signature