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

[Krzysztof Kozlowski <krzk@kernel.org>]

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@bp.renesas.com>
>>
>> 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@bp.renesas.com>
>> ---
>>  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...

Best regards,
Krzysztof