[PATCH 0/2] Make AUDIO_CLKOUT optional and drop it from RZ/G3E SMARC

[PATCH 0/2] Make AUDIO_CLKOUT optional and drop it from RZ/G3E SMARC

[Tommaso Merciai]

Dear All,

This series makes #clock-cells optional in the renesas,r9a09g047-sound
binding, since the node acts as a clock provider only on boards that
route AUDIO_CLKOUT, and drops the corresponding AUDIO_CLKOUT properties
from r9a09g047e57-smarc.dts, where the pin is neither pinmuxed nor
consumed. The DA7212 MCLK comes from the on-board versa3.

This fix also addresses the following warning:

sound@13c00000 (renesas,r9a09g047-sound): Unevaluated properties are not
allowed ('clock-frequency' was unexpected)

Spotted running dtbs_check with -W=1.

Kind regards,
Tommaso

Tommaso Merciai (2):
  dt-bindings: sound: renesas,r9a09g047-sound: Drop #clock-cells from
    required
  arm64: dts: renesas: r9a09g047e57-smarc: Drop AUDIO_CLKOUT properties

 .../devicetree/bindings/sound/renesas,r9a09g047-sound.yaml    | 1 -
 arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts            | 4 ----
 2 files changed, 5 deletions(-)

-- 
2.54.0

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

[Tommaso Merciai]

`#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@bp.renesas.com>
---
 .../devicetree/bindings/sound/renesas,r9a09g047-sound.yaml       | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/sound/renesas,r9a09g047-sound.yaml b/Documentation/devicetree/bindings/sound/renesas,r9a09g047-sound.yaml
index 213c1c40b6f0..8b65a5713250 100644
--- 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
-- 
2.54.0

[PATCH 2/2] arm64: dts: renesas: r9a09g047e57-smarc: Drop AUDIO_CLKOUT properties

[Tommaso Merciai]

AUDIO_CLKOUT is neither pinmuxed nor consumed on this board, so the node is
not a clock provider: #clock-cells and clock-frequency describe an output
that does not exist here. The DA7212 MCLK comes from the on-board versa3.

Fixes: 3e23d2548ddb ("arm64: dts: renesas: r9a09g047e57-smarc: Add DA7212 audio codec support")
Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
---
 arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
index 4eed095b683b..d43b50bca4a8 100644
--- a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
+++ b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
@@ -362,10 +362,6 @@ &snd_rzg3e {
 
 	status = "okay";
 
-	/* audio_clkout */
-	#clock-cells = <0>;
-	clock-frequency = <11289600>;
-
 	/* Multi DAI */
 	#sound-dai-cells = <1>;
 
-- 
2.54.0

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

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] Dropping `#clock-cells` from the `required` list violates subsystem guidelines for hardware blocks with provider capabilities.
--

--- Patch [1]: [PATCH 1/2] dt-bindings: sound: renesas,r9a09g047-sound: Drop #clock-cells from required ---
commit b7bf878068dfd3a3e9a8e0e64533cd2629a71799
Author: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>

dt-bindings: sound: renesas,r9a09g047-sound: Drop #clock-cells from required

This commit drops the "#clock-cells" property from the required list in the
renesas,r9a09g047-sound device tree binding schema. It is intended to allow
boards that do not wire AUDIO_CLKOUT to omit the property.

> diff --git a/Documentation/devicetree/bindings/sound/renesas,r9a09g047-sound.yaml b/Documentation/devicetree/bindings/sound/renesas,r9a09g047-sound.yaml
> index d7fa165546985..e91de2db071da 100644
> --- 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"

[Severity: Medium]
Does dropping "#clock-cells" from the required list conflict with the
subsystem guidelines for hardware provider capabilities?

The device tree binding subsystem rules indicate that when a hardware variant
retains clock provider capabilities, the corresponding "#clock-cells" property
must remain in the required list. Will removing it allow incomplete board
configurations to silently pass schema validation?

>    - clocks
>    - clock-names
>    - resets

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260820095742.3094558-1-tommaso.merciai.xr@bp.renesas.com?part=1

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

[Geert Uytterhoeven]

Hi Tommaso,

On Thu, 20 Aug 2026 at 11:58, Tommaso Merciai
<tommaso.merciai.xr@bp.renesas.com> 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@bp.renesas.com>

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.

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

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@linux-m68k.org>
> Sent: Thursday, August 20, 2026 12:14 PM
> To: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
> Cc: Tommaso Merciai <tomm.merciai@gmail.com>; linux-renesas-
> soc@vger.kernel.org; Biju Das <biju.das.jz@bp.renesas.com>; Liam Girdwood
> <lgirdwood@gmail.com>; Mark Brown <broonie@kernel.org>; Rob Herring
> <robh@kernel.org>; Krzysztof Kozlowski <krzk+dt@kernel.org>; Conor Dooley
> <conor+dt@kernel.org>; magnus.damm <magnus.damm@gmail.com>; Kuninori
> Morimoto <kuninori.morimoto.gx@renesas.com>; John Madieu
> <john.madieu.xa@bp.renesas.com>; linux-sound@vger.kernel.org;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org
> 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@bp.renesas.com> 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@bp.renesas.com>
> 
> 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