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

[Krzysztof Kozlowski]

On 08/06/2026 13:46, Krzysztof Kozlowski wrote:
> On 08/06/2026 13:22, Biju Das wrote:
>> Hi Krzysztof Kozlowski,
>>
>>> -----Original Message-----
>>> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
>>> Sent: 08 June 2026 12:11
>>> Subject: Re: [PATCH] dt-bindings: mfd: syscon: Make ranges required for renesas,r9a08g046-lvds-cmn
>>>
>>> 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 ?
>>
>> OK, I am missing the fallbacks "simple-mfd", "syscon"; in the compatibles allowed
>> for renesas,r9a08g046-lvds-cmn.
>>
>> OK, I will add those.
> 
> No, maybe, dunno... How anything here could have been tested? dtbs_check
> clearly points errors on your DTS.
> 


You should re-do the commit 51284d8b1dbcd7fa0220c49eeab29b14617e0d88
because it is completely broken. This is schema for syscons, not devices
with children.

You added incomplete, incorrect binding having obvious dtbs_check
failures which you would see if you validated DTS (never posted!!!).

And you keep introducing more issues, by trying to fix non-existing
issues while keeping the main problem - completely mismatched
renesas,r9a08g046-lvds-cmn binding with reality unfixed.

Best regards,
Krzysztof