Re: [PATCH] dt-bindings: mfd: syscon: Make ranges required for renesas,r9a08g046-lvds-cmn

[Krzysztof Kozlowski]

On 08/06/2026 12:26, Biju Das wrote:
> Hi Krzysztof Kozlowski,
> 
> Thanks for the feedback.
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
>> Sent: 08 June 2026 11:22
>> Subject: Re: [PATCH] dt-bindings: mfd: syscon: Make ranges required for renesas,r9a08g046-lvds-cmn
>>
>> On 08/06/2026 12:20, Krzysztof Kozlowski wrote:
>>> On Tue, Jun 02, 2026 at 02:13:29PM +0100, Biju wrote:
>>>> From: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
>>>>
>>>> Add a conditional schema rule to the syscon bindings that requires
>>>> the ranges property when the compatible string contains
>>>> renesas,r9a08g046-lvds-cmn. This ensures the LVDS common control
>>>> block on the RZ/G3L SoC correctly declares its address translation,
>>>> as the device has child nodes that need a valid ranges mapping to be
>>>> described in the device tree.
>>>>
>>>> Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
>>>> ---
>>>>  Documentation/devicetree/bindings/mfd/syscon.yaml | 14
>>>> ++++++++++++++
>>>>  1 file changed, 14 insertions(+)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml
>>>> b/Documentation/devicetree/bindings/mfd/syscon.yaml
>>>> index 9c81010d5a74..cbf83a06ae25 100644
>>>> --- a/Documentation/devicetree/bindings/mfd/syscon.yaml
>>>> +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
>>>> @@ -269,6 +269,8 @@ properties:
>>>>    resets:
>>>>      maxItems: 1
>>>>
>>>> +  ranges: true
>>>
>>> There are no children allowed, so ranges property is wrong.
>>>
>>> You are changing binding which DOES NOT allow simple-mfd or any other
>>> children. Ranges is not a problem here.
>>
>>
>> And if you tested it on your DTS, you would see this does not work...
> 
> I don't see any warnings or error. I have done this change based on Rob's comment
> based on sashiko review [1].
> 
> If you agree, I can drop this patch.
> 
> [1]
> https://lore.kernel.org/all/20260601022619.GA3961324-robh@xxxxxxxxxx/

Look:
https://lore.kernel.org/all/20260524194457.479681-2-biju.das.jz@xxxxxxxxxxxxxx/
What compatibles are here?

Now open the binding - what compatibles are allowed for
renesas,r9a08g046-lvds-cmn ?

Best regards,
Krzysztof