Message ID | 20211112130627.3682795-2-thierry.reding@gmail.com |
---|---|
State | Changes Requested |
Headers | show |
Series | memory: tegra: Add Tegra234 support | expand |
On Fri, 12 Nov 2021 14:06:23 +0100, Thierry Reding wrote: > From: Thierry Reding <treding@nvidia.com> > > The #interconnect-cells properties are required to hook up memory > clients to the MC/EMC in interconnects properties. Add a description for > these properties. > > Also, allow multiple reg and interrupt entries required by Tegra194 and > later. > > Signed-off-by: Thierry Reding <treding@nvidia.com> > --- > .../memory-controllers/nvidia,tegra186-mc.yaml | 12 +++++++++--- > 1 file changed, 9 insertions(+), 3 deletions(-) > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13): yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.example.dt.yaml: memory-controller@2c00000: reg: [[0, 46137344, 0, 720896]] is too short From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.example.dt.yaml: memory-controller@2c00000: interrupts: [[0, 223, 4]] is too short From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.example.dt.yaml: memory-controller@2c00000: external-memory-controller@2c60000:reg: [[0, 46530560, 0, 327680]] is too short From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/patch/1554376 This check can fail if there are any dependencies. The base for a patch series is generally the most recent rc1. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit.
On 12/11/2021 14:06, Thierry Reding wrote: > From: Thierry Reding <treding@nvidia.com> > > The #interconnect-cells properties are required to hook up memory > clients to the MC/EMC in interconnects properties. Add a description for > these properties. > > Also, allow multiple reg and interrupt entries required by Tegra194 and > later. > > Signed-off-by: Thierry Reding <treding@nvidia.com> > --- > .../memory-controllers/nvidia,tegra186-mc.yaml | 12 +++++++++--- > 1 file changed, 9 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml > index 611bda38d187..f6e4af4e86cf 100644 > --- a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml > +++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml > @@ -33,10 +33,10 @@ properties: > - nvidia,tegra194-mc > > reg: > - maxItems: 1 > + maxItems: 3 > > interrupts: > - maxItems: 1 > + maxItems: 2 All these here and reg below might need if-else to define when one item is allowed, when not. For example - can nvidia,tegra234-mc come with only one reg? Except this and Rob's DT-checker-bot rest of patches look ok to me. Best regards, Krzysztof > > "#address-cells": > const: 2 > @@ -48,6 +48,9 @@ properties: > > dma-ranges: true > > + "#interconnect-cells": > + const: 1 > + > patternProperties: > "^external-memory-controller@[0-9a-f]+$": > description: > @@ -65,7 +68,7 @@ patternProperties: > - nvidia,tegra194-emc > > reg: > - maxItems: 1 > + maxItems: 2 > > interrupts: > maxItems: 1 > @@ -78,6 +81,9 @@ patternProperties: > items: > - const: emc Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml index 611bda38d187..f6e4af4e86cf 100644 --- a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml +++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml @@ -33,10 +33,10 @@ properties: - nvidia,tegra194-mc reg: - maxItems: 1 + maxItems: 3 interrupts: - maxItems: 1 + maxItems: 2 "#address-cells": const: 2 @@ -48,6 +48,9 @@ properties: dma-ranges: true + "#interconnect-cells": + const: 1 + patternProperties: "^external-memory-controller@[0-9a-f]+$": description: @@ -65,7 +68,7 @@ patternProperties: - nvidia,tegra194-emc reg: - maxItems: 1 + maxItems: 2 interrupts: maxItems: 1 @@ -78,6 +81,9 @@ patternProperties: items: - const: emc + "#interconnect-cells": + const: 0 + nvidia,bpmp: $ref: /schemas/types.yaml#/definitions/phandle description: