RE: [cip-dev] [PATCH 6.12.y-cip 02/23] dt-bindings: pinctrl: renesas: Document RZ/G3L SoC

<[email protected]> Thu, 16 Jul 2026 01:19:56 +0000
Newsgroups org.cip-project.lists.cip-dev
Message-ID <TY7PR01MB148181F049410E17227C1645ACDC72@TY7PR01MB14818.jpnprd01.prod.outlook.com>
Hi,

> -----Original Message-----
> From: [email protected] <[email protected]> On
> Behalf Of Pavel Machek via lists.cip-project.org
> Sent: Wednesday, July 15, 2026 7:12 PM
> To: Biju <[email protected]>
> Cc: [email protected]; iwamatsu nobuhiro(岩松 信洋 □DITC○
> CPT) <[email protected]>; Pavel Machek
> <[email protected]>; Biju Das <[email protected]>; Lad
> Prabhakar <[email protected]>
> Subject: Re: [cip-dev] [PATCH 6.12.y-cip 02/23] dt-bindings: pinctrl: renesas:
> Document RZ/G3L SoC
> 
> Hi!
> 
> There seems to be something wrong here:
> 
> > Add documentation for the pin controller found on the Renesas RZ/G3L
> > (R9A08G046) SoC. The RZ/G3L PFC is similar to the RZ/G3S SoC but has
> > more pins.
> ...
> > Document renesas,clonech property for controlling clone channel
> > control register located on SYSC IP block on RZ/G3L SoC.
> 
> Ok, no mention of deleting constraints for *57.
> 
> > +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.
> > +++ yaml
> > @@ -154,15 +165,10 @@ allOf:
> >        properties:
> >          compatible:
> >            contains:
> > -            const: renesas,r9a09g057-pinctrl
> > +            const: renesas,r9a08g046-pinctrl
> >      then:
> > -      properties:
> > -        resets:
> > -          maxItems: 2
> > -    else:
> > -      properties:
> > -        resets:
> > -          minItems: 3
> > +      required:
> > +        - renesas,clonech
> >
> >    - if:
> >        properties:
> 
> But now we delete constraints for *57, and add constraints for *46. That looks
> wrong?

This is a side effect by backporting.
Depending on the order of commits, the contents of the commit log may not match those of the patch.

> 
> Best regards,
> 								Pavel

Best regards,
  Nobuhiro