Hi Tommaso,
On Thu, 3 Sept 2026 at 13:01, Tommaso Merciai
<[email protected]> wrote:
> On Wed, Sep 02, 2026 at 04:20:51PM +0200, Geert Uytterhoeven wrote:
> > On Wed, 2 Sept 2026 at 16:03, Tommaso Merciai
> > <[email protected]> wrote:
> > > On Wed, Sep 02, 2026 at 08:37:06AM +0200, Krzysztof Kozlowski wrote:
> > > > On Fri, Aug 28, 2026 at 02:21:04PM +0200, Tommaso Merciai wrote:
> > > > > The RZ/G3E Soc has 2 LCD controller (LCDC), contain a Frame
> > > > > Compression
> > > > > Processor (FCPVD), a Video Signal Processor (VSPD), Video Signal
> > > > > Processor (VSPD), and Display Unit (DU).
> > > > >
> > > > > - LCDC0 supports DSI and LVDS (single or dual-channel) outputs.
> > > > > - LCDC1 supports DSI, LVDS (single-channel), and RGB outputs.
> > > > >
> > > > > Add new SoC-specific compatible string 'renesas,r9a09g047-du'.
> > > > >
> > > > > Signed-off-by: Tommaso Merciai <[email protected]>
> >
> > > > > --- a/Documentation/devicetree/bindings/display/renesas,rzg2l-du.yaml
> > > > > +++ b/Documentation/devicetree/bindings/display/renesas,rzg2l-du.yaml
> > > > > @@ -21,6 +21,7 @@ properties:
> > > > > - renesas,r9a07g043u-du # RZ/G2UL
> > > > > - renesas,r9a07g044-du # RZ/G2{L,LC}
> > > > > - renesas,r9a08g046-du # RZ/G3L
> > > > > + - renesas,r9a09g047-du # RZ/G3E
> > > > > - renesas,r9a09g057-du # RZ/V2H(P)
> > > > > - renesas,r9a09g077-du # RZ/T2H
> > > > > - items:
> > > > > @@ -35,25 +36,51 @@ properties:
> > > > > - const: renesas,r9a09g077-du # RZ/T2H fallback
> > > > >
> > > > > reg:
> > > > > - maxItems: 1
> > > > > + minItems: 1
> > > > > + maxItems: 2
> > > > > +
> > > > > + reg-names:
> > > > > + items:
> > > > > + - const: du.0
> > > > > + - const: du.1
> > > >
> > > > du is the name of the device, thus calling items "0" and "1" is pretty
> > > > pointless - indices already define that. Please drop the reg-names.
> > >
> > > Will drop this in v8.
> > >
> > > > > interrupts:
> > > > > - maxItems: 1
> > > > > + minItems: 1
> > > > > + maxItems: 2
> > > > > +
> > > > > + interrupt-names:
> > > > > + items:
> > > > > + - const: du.0
> > > > > + - const: du.1
> > > >
> > > > Same here
> > >
> > > Same, thanks.
> > >
> > > > > clocks:
> > > > > + minItems: 3
> > > > > items:
> > > > > - description: Main clock
> > > > > - description: Register access clock
> > > > > - description: Video clock
> > > > > + - description: Main clock for DU1
> > > > > + - description: Register access clock for DU1
> > > > > + - description: Video clock for DU1
> > > > >
> > > > > clock-names:
> > > > > + minItems: 3
> > > > > items:
> > > > > - const: aclk
> > > > > - const: pclk
> > > > > - const: vclk
> > > > > + - const: aclk1
> > > > > + - const: pclk1
> > > > > + - const: vclk1
> > > > >
> > > > > resets:
> > > > > - maxItems: 1
> > > > > + minItems: 1
> > > > > + maxItems: 2
> > > > > +
> > > > > + reset-names:
> > > > > + items:
> > > > > + - const: resetn
> > > > > + - const: resetn1
> > > >
> > > > Drop reset-names
> > >
> > > For reset-names, I got the the following comment from Philipp in v7 [1].
> > >
> > > Dropping reset-names would force the driver back to an index-based
> > > lookup, which is what that comment explicitly asked me to avoid.
> > >
> > > [1]
> > > https://lore.kernel.org/all/[email protected]/
> > >
> > > Philipp, Krzysztof, could you agree on which way you'd prefer?
> > > I'll follow whatever you decide.
> >
> > Until we get a variant with a third interrupt (or reset or reg),
> > which is not related to the number of channels...
>
> Just to make sure I follow: are you referring to variants like R-Car H3
> and M3-N, where the position in the array no longer matches the channel
> index? From renesas,du.yaml:
Not directly, as these don't have any non-"du.X" resets and clocks.
But yes, these cases are similar.
> [renesas,du-r8a7795: 4 channels, two resets] [1]
> reset-names:
> items:
> - const: du.0
> - const: du.2
>
> [renesas,du-r8a77965: 3 channels, du.2 (channel 2) does not exist] [2]
> clock-names:
> items:
> - const: du.0
> - const: du.1
> - const: du.3
> reset-names:
> items:
> - const: du.0
> - const: du.3
>
> If so, I agree: the names are what makes these unambiguous, and an
> index-based lookup could not express them.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
