RE: [PATCH 1/2] dt-bindings: sound: renesas,r9a09g047-sound: Drop #clock-cells from required

[Tommaso Merciai]

Hi Geert,
Thanks for your review!

> -----Original Message-----
> From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Sent: Thursday, August 20, 2026 12:14 PM
> To: Tommaso Merciai <tommaso.merciai.xr@xxxxxxxxxxxxxx>
> Cc: Tommaso Merciai <tomm.merciai@xxxxxxxxx>; linux-renesas-
> soc@xxxxxxxxxxxxxxx; Biju Das <biju.das.jz@xxxxxxxxxxxxxx>; Liam Girdwood
> <lgirdwood@xxxxxxxxx>; Mark Brown <broonie@xxxxxxxxxx>; Rob Herring
> <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley
> <conor+dt@xxxxxxxxxx>; magnus.damm <magnus.damm@xxxxxxxxx>; Kuninori
> Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>; John Madieu
> <john.madieu.xa@xxxxxxxxxxxxxx>; linux-sound@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 1/2] dt-bindings: sound: renesas,r9a09g047-sound: Drop
> #clock-cells from required
> 
> Hi Tommaso,
> 
> On Thu, 20 Aug 2026 at 11:58, Tommaso Merciai
> <tommaso.merciai.xr@xxxxxxxxxxxxxx> wrote:
> > `#clock-cells` declares the node as a clock provider for AUDIO_CLKOUT,
> > which only makes sense on boards that route that pin.
> >
> > Drop it from the required list so that boards which do not wire
> > AUDIO_CLKOUT can leave it out.
> >
> > Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@xxxxxxxxxxxxxx>
> 
> Thanks for your patch!
> 
> > ---
> > a/Documentation/devicetree/bindings/sound/renesas,r9a09g047-sound.yaml
> > +++ b/Documentation/devicetree/bindings/sound/renesas,r9a09g047-sound.
> > +++ yaml
> > @@ -298,7 +298,6 @@ required:
> >    - reg
> >    - reg-names
> >    - "#sound-dai-cells"
> > -  - "#clock-cells"
> >    - clocks
> >    - clock-names
> >    - resets
> 
> I disagree: this is a property of the sound block, not of board wiring.

Maybe better to drop this patch.
Move #clock-cells = <0>; from r9a09g047e57-smarc.dts to r9a09g047.dtsi
Drop clock-frequency = <11289600>; from r9a09g047e57-smarc.dts

What do you think?

Kind regards,
Tommaso

> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
> 
> 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