Hi Krzysztof Kozlowski,

> -----Original Message-----
> From: Krzysztof Kozlowski <[email protected]>
> Sent: 08 June 2026 14:13
> Subject: Re: [PATCH v3 1/2] dt-bindings: display: bridge: Document Renesas 
> RZ/G3L LVDS encoder
> 
> On 26/05/2026 09:47, Biju wrote:
> > +
> > +examples:
> > +  - |
> > +    #include <dt-bindings/clock/renesas,r9a08g046-cpg.h>
> > +
> > +    lvds-cmn@108a0000 {
> > +        compatible = "renesas,r9a08g046-lvds-cmn",
> > +                     "simple-mfd", "syscon";
> > +        reg = <0x108a0000 0x10000>;
> 
> In current form patch is not correct. MUST NOT BE APPLIED.

I completely agree.

> 
> If you apply it, it will introduce new warnings.

Yes, I know. Currently the mfd binding skip the checks, which found during the 
testing
today.

> 
> Reviews could stay if not the fact that dependency was incorrectly described 
> and to do it properly
> this patch should be changed. Thus please drop all review tags and re-do the 
> patches.
> 
> NAK

Sure, it will be taken care in [1].

[1] 
https://lore.kernel.org/all/ty3pr01mb113466f579f4093217daa6b7086...@ty3pr01mb11346.jpnprd01.prod.outlook.com/

Cheers,
Biju

Reply via email to