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

[Biju Das]

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/

Cheers,
Biju