Re: [PATCH v4 2/4] dt-bindings: soc: Add i.MX6SX General Purpose Register

[Fabio Estevam <festevam@xxxxxxx>]

Hi Conor,

On 19/05/2023 16:23, Conor Dooley wrote:

> > +  bridge@18:
> > +    type: object
> > +    $ref: /schemas/display/bridge/fsl,ldb.yaml#
> > +    unevaluatedProperties: false
>
> How come you need to upgrade the additionaProperties: false, and the
> "plain old" $ref is not sufficient?

I am passing unevaluatedProperties: false, not additionaProperties: 
false here,
so I am not sure I understood your comment.

If additionaProperties: false is not present:

$ make dt_binding_check DT_SCHEMA_FILES=fsl,imx6sx-gpr.yaml
  LINT    Documentation/devicetree/bindings
  CHKDT   Documentation/devicetree/bindings/processed-schema.json
/home/fabio/linux-next/Documentation/devicetree/bindings/soc/imx/fsl,imx6sx-gpr.yaml: 
'oneOf' conditional failed, one must be fixed:
	'unevaluatedProperties' is a required property
	'additionalProperties' is a required property
	hint: Either unevaluatedProperties or additionalProperties must be 
present
	from schema $id: http://devicetree.org/meta-schemas/core.yaml#
  SCHEMA  Documentation/devicetree/bindings/processed-schema.json
  DTEX    
Documentation/devicetree/bindings/soc/imx/fsl,imx6sx-gpr.example.dts
  DTC_CHK 
Documentation/devicetree/bindings/soc/imx/fsl,imx6sx-gpr.example.dtb

So I will keep it like this.

I have taken all of your other feedback into account and will submit a 
v5.

Thanks,

Fabio Estevam