| Message ID | 20240407102000.37213-2-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | [1/3] dt-bindings: PCI: cdns,cdns-pcie-host: drop redundant msi-parent and pci-bus.yaml | expand |
On Sun, Apr 7, 2024 at 12:20 PM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > > MT7621 PCI host bridge has children which apparently are also PCI host > bridges, at least that's what the binding suggest. The children have > "reg" property, but do not explicitly define it. Instead they rely on > pci-bus.yaml schema, but that one has "reg" without any constraints. > > Define the "reg" for the children, so the binding will be more specific > and later will allow dropping reference to deprecated pci-bus.yaml > schema. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/pci/mediatek,mt7621-pcie.yaml | 3 +++ > 1 file changed, 3 insertions(+) Acked-by: Sergio Paracuellos <sergio.paracuellos@gmail.com> Thanks, Sergio Paracuellos
On Sun, 07 Apr 2024 12:19:59 +0200, Krzysztof Kozlowski wrote: > MT7621 PCI host bridge has children which apparently are also PCI host > bridges, at least that's what the binding suggest. The children have > "reg" property, but do not explicitly define it. Instead they rely on > pci-bus.yaml schema, but that one has "reg" without any constraints. > > Define the "reg" for the children, so the binding will be more specific > and later will allow dropping reference to deprecated pci-bus.yaml > schema. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/pci/mediatek,mt7621-pcie.yaml | 3 +++ > 1 file changed, 3 insertions(+) > Acked-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml index e63e6458cea8..61d027239910 100644 --- a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml +++ b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml @@ -36,6 +36,9 @@ patternProperties: $ref: /schemas/pci/pci-bus.yaml# properties: + reg: + maxItems: 1 + resets: maxItems: 1
MT7621 PCI host bridge has children which apparently are also PCI host bridges, at least that's what the binding suggest. The children have "reg" property, but do not explicitly define it. Instead they rely on pci-bus.yaml schema, but that one has "reg" without any constraints. Define the "reg" for the children, so the binding will be more specific and later will allow dropping reference to deprecated pci-bus.yaml schema. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../devicetree/bindings/pci/mediatek,mt7621-pcie.yaml | 3 +++ 1 file changed, 3 insertions(+)