Re: [PATCH v1 v1 2/7] dt-bindings: display: Add yamls for JH7110 display system

[Rob Herring <robh@xxxxxxxxxx>]

On Tue, 01 Aug 2023 18:10:25 +0800, Keith Zhao wrote:
> StarFive SoCs JH7110 display system:
> lcd-controller bases verisilicon dc8200 IP,
> and hdmi bases Innosilicon IP.
> Add bindings for them.
> 
> Signed-off-by: Keith Zhao <keith.zhao@xxxxxxxxxxxxxxxx>
> ---
>  .../starfive/starfive,display-subsystem.yaml  |  41 +++++++
>  .../starfive/starfive,jh7110-dc8200.yaml      | 107 ++++++++++++++++++
>  .../starfive/starfive,jh7110-inno-hdmi.yaml   |  92 +++++++++++++++
>  3 files changed, 240 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/starfive/starfive,display-subsystem.yaml
>  create mode 100644 Documentation/devicetree/bindings/display/starfive/starfive,jh7110-dc8200.yaml
>  create mode 100644 Documentation/devicetree/bindings/display/starfive/starfive,jh7110-inno-hdmi.yaml
> 

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/dt-review-ci/linux/Documentation/devicetree/bindings/display/starfive/starfive,display-subsystem.example.dtb: display-subsystem: ports: [[4294967295]] is not of type 'object'
	from schema $id: http://devicetree.org/schemas/display/starfive/starfive,display-subsystem.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/starfive/starfive,display-subsystem.example.dtb: display-subsystem: ports: More than one condition true in oneOf schema:
	{'additionalProperties': False,
	 'oneOf': [{'required': ['port']},
	           {'required': ['#address-cells', '#size-cells']}],
	 'patternProperties': {'^port@[0-9a-f]+$': {'required': ['reg'],
	                                            'type': 'object'},
	                       'pinctrl-[0-9]+': True},
	 'properties': {'#address-cells': {'items': [{'items': [{'const': 1}],
	                                              'maxItems': 1,
	                                              'minItems': 1,
	                                              'type': 'array'}],
	                                   'maxItems': 1,
	                                   'minItems': 1,
	                                   'type': 'array'},
	                '#size-cells': {'items': [{'items': [{'const': 0}],
	                                           'maxItems': 1,
	                                           'minItems': 1,
	                                           'type': 'array'}],
	                                'maxItems': 1,
	                                'minItems': 1,
	                                'type': 'array'},
	                '$nodename': True,
	                'bootph-all': True,
	                'bootph-pre-ram': True,
	                'bootph-pre-sram': True,
	                'bootph-some-ram': True,
	                'bootph-verify': True,
	                'phandle': True,
	                'pinctrl-names': True,
	                'port': {'type': 'object'},
	                'secure-status': True,
	                'status': True},
	 'type': 'object'}
	from schema $id: http://devicetree.org/schemas/display/starfive/starfive,display-subsystem.yaml#

doc reference errors (make refcheckdocs):

See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20230801101030.2040-3-keith.zhao@xxxxxxxxxxxxxxxx

The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.

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 after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.