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

[Krzysztof Kozlowski]

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.

Best regards,
Krzysztof