On Wed, Jul 01, 2026 at 02:53:11PM -0500, Rob Herring (Arm) wrote: > > On Wed, 01 Jul 2026 18:08:13 +0200, Thierry Reding wrote: > > From: Thierry Reding <[email protected]> > > > > Add the memory-region and memory-region-names properties to the bindings > > for the display controllers and the host1x engine found on various Tegra > > generations. These memory regions are used to access firmware-provided > > framebuffer memory as well as the video protection region. > > > > Signed-off-by: Thierry Reding <[email protected]> > > --- > > Changes in v3: > > - document properties for VIC > > --- > > .../devicetree/bindings/display/tegra/nvidia,tegra124-vic.yaml | 8 > > ++++++++ > > .../devicetree/bindings/display/tegra/nvidia,tegra186-dc.yaml | 10 > > ++++++++++ > > .../devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml | 10 > > +++++++++- > > .../bindings/display/tegra/nvidia,tegra20-host1x.yaml | 7 +++++++ > > 4 files changed, 34 insertions(+), 1 deletion(-) > > > > My bot found errors running 'make dt_binding_check' on your patch: > > yamllint warnings/errors: > > dtschema/dtc warnings/errors: > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml: > properties:memory-region-names: 'anyOf' conditional failed, one must be > fixed: > 'maxitems' is not one of ['$ref', 'additionalItems', > 'additionalProperties', 'allOf', 'anyOf', 'const', 'contains', 'default', > 'dependencies', 'dependentRequired', 'dependentSchemas', 'deprecated', > 'description', 'else', 'enum', 'exclusiveMaximum', 'exclusiveMinimum', > 'items', 'if', 'minItems', 'minimum', 'maxItems', 'maximum', 'multipleOf', > 'not', 'oneOf', 'pattern', 'patternProperties', 'properties', 'required', > 'then', 'typeSize', 'unevaluatedProperties', 'uniqueItems'] > 'type' was expected > from schema $id: http://devicetree.org/meta-schemas/keywords.yaml > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml: > properties:memory-region-names:items: {'enum': ['framebuffer', 'protected']} > is not of type 'array' > from schema $id: http://devicetree.org/meta-schemas/string-array.yaml > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml: > properties:memory-region-names: Additional properties are not allowed > ('maxitems' was unexpected) > from schema $id: http://devicetree.org/meta-schemas/string-array.yaml > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml: > properties:memory-region-names:items: {'enum': ['framebuffer', 'protected']} > is not of type 'array' > from schema $id: http://devicetree.org/meta-schemas/string-array.yaml > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml: > properties:memory-region-names: Additional properties are not allowed > ('maxitems' was unexpected) > from schema $id: http://devicetree.org/meta-schemas/string-array.yaml
Ugh... looks like these are all because I mistyped maxItems as maxitems. Oh well. Sorry for the noise. Thierry
signature.asc
Description: PGP signature
