2022-03-13 17:12:49

by Michael Walle

Subject: Re: [PATCH v1 8/8] dt-bindings: pinctrl: convert ocelot-pinctrl to YAML format

Hi,

wow, you're fast!

Am 2022-03-13 16:55, schrieb Krzysztof Kozlowski:

>> + reg: true
>
> maxItems

There are up to two address ranges. The second one is only needed for
particular controllers (the sparx5 and the lan966x).

Is the following snippet the correct form?

reg:
items:
- description: Base address
- description: Extended pin configuration registers
minItems: 1

-michael

2022-03-14 07:33:05

by Krzysztof Kozlowski

Subject: Re: [PATCH v1 8/8] dt-bindings: pinctrl: convert ocelot-pinctrl to YAML format

On 13/03/2022 17:36, Michael Walle wrote:
> Hi,
>
> wow, you're fast!
>
> Am 2022-03-13 16:55, schrieb Krzysztof Kozlowski:
>
>>> + reg: true
>>
>> maxItems
>
> There are up to two address ranges. The second one is only needed for
> particular controllers (the sparx5 and the lan966x).
>
> Is the following snippet the correct form?
>
> reg:
> items:
> - description: Base address
> - description: Extended pin configuration registers
> minItems: 1

Yes, it's correct. Please also add proper "if:then" under "allOf:" (and
move such allOf under "required:") which changes minItems to two for
such controllers, based on compatible.

Best regards,
Krzysztof