Re: [PATCH v3 02/16] dt-bindings: bus: imx-weim: convert to YAML

From: Krzysztof Kozlowski
Date: Mon Feb 19 2024 - 02:11:13 EST


On 19/02/2024 02:59, Sebastian Reichel wrote:
> Hi Krzysztof,
>
> On Sat, Feb 17, 2024 at 09:39:52AM +0100, Krzysztof Kozlowski wrote:
>> On 16/02/2024 23:34, Sebastian Reichel wrote:
>>> +properties:
>>> + reg: true
>>
>> Your pattern in weim schema suggests you have here minItems: 2.
>> maxItems: can be 2 or 255.
>
> Which part suggests minItems: 2?
>
> It's minItems: 1 with each item consisting of two address cells and
> one size cell. Most in-tree users actually have exactly one item.
> The only exception seems to be arch/arm/boot/dts/nxp/imx/imx1-apf9328.dts,
> which has "davicom,dm9000" using two reg items. Considering this is an
> external bus, any number is possible in theory. Should I keep it as is,
> or use minItems: 1, maxItems: 255 instead?

Hm, indeed, existing 'true' is fine.

Best regards,
Krzysztof