mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v2 0/2] Drop clock-frequency from RZ/G3E SMARC snd_rzg3e dts node
@ 2026-10-20 11:11 Tommaso Merciai
  2026-10-20 11:11 ` [PATCH v2 1/2] arm64: dts: renesas: r9a09g047: Move #clock-cells to the RZ/G3E SoC dtsi Tommaso Merciai
  2026-10-20 11:11 ` [PATCH v2 2/2] arm64: dts: renesas: r9a09g047e57-smarc: Drop AUDIO_CLKOUT clock-frequency Tommaso Merciai
  0 siblings, 2 replies; 4+ messages in thread
From: Tommaso Merciai @ 2026-10-20 11:11 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Tommaso Merciai,
	Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, John Madieu, devicetree,
	linux-kernel

Dear All,

This series move #clock-cells from r9a09g047e57-smarc.dts to the snd_rzg3e
node in r9a09g047.dtsi. Also AUDIO_CLKOUT is neither pinmuxed nor consumed
on this board, then clock-frequency into r9a09g047e57-smarc.dts describe
the frequency of an output that does not exist here. The DA7212 MCLK comes
from the on-board versa3 clock generator, then drop this entry from the
r9a09g047e57-smarc.dts.

This fix also addresses the following warning:

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

Found running dtbs_check with -W=1.

Kind regards,
Tommaso

v1->v2:
 - Improved/update cover letter subject and body.
 - Drop v1 patch 1/2 that Drop #clock-cells from required into the bindings
   and instead move #clock-cells to the snd_rzg3e node in r9a09g047.dtsi and
   drop only the clock-frequency entry from r9a09g047e57-smarc.dts.
 - Link to v1: https://lore.kernel.org/all/20260820095742.3094558-1-tommaso.merciai.xr@bp.renesas.com

Tommaso Merciai (2):
  arm64: dts: renesas: r9a09g047: Move #clock-cells to the RZ/G3E SoC
    dtsi
  arm64: dts: renesas: r9a09g047e57-smarc: Drop AUDIO_CLKOUT
    clock-frequency

 arch/arm64/boot/dts/renesas/r9a09g047.dtsi         | 1 +
 arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts | 4 ----
 2 files changed, 1 insertion(+), 4 deletions(-)

-- 
2.54.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH v2 1/2] arm64: dts: renesas: r9a09g047: Move #clock-cells to the RZ/G3E SoC dtsi
  2026-10-20 11:11 [PATCH v2 0/2] Drop clock-frequency from RZ/G3E SMARC snd_rzg3e dts node Tommaso Merciai
@ 2026-10-20 11:11 ` Tommaso Merciai
  2026-10-20 11:11 ` [PATCH v2 2/2] arm64: dts: renesas: r9a09g047e57-smarc: Drop AUDIO_CLKOUT clock-frequency Tommaso Merciai
  1 sibling, 0 replies; 4+ messages in thread
From: Tommaso Merciai @ 2026-10-20 11:11 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Tommaso Merciai,
	Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, John Madieu, devicetree,
	linux-kernel

The renesas,r9a09g047-sound bindings specified #clock-cells as 0, this
is a SoC property rather than of the board.
The commit 3e23d2548ddb ("arm64: dts: renesas: r9a09g047e57-smarc:
Add DA7212 audio codec support") place this entry into the RZ/G3E SMARC
EVK board dts which is wrong.

Move #clock-cells from r9a09g047e57-smarc.dts to the snd_rzg3e node in
r9a09g047.dtsi.

Fixes: 3e23d2548ddb ("arm64: dts: renesas: r9a09g047e57-smarc: Add DA7212 audio codec support")
Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
---
v1->v2:
 - New patch.

 arch/arm64/boot/dts/renesas/r9a09g047.dtsi         | 1 +
 arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts | 1 -
 2 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/renesas/r9a09g047.dtsi b/arch/arm64/boot/dts/renesas/r9a09g047.dtsi
index 737bf1c60451..314298a6996d 100644
--- a/arch/arm64/boot/dts/renesas/r9a09g047.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a09g047.dtsi
@@ -970,6 +970,7 @@ snd_rzg3e: sound@13c00000 {
 				      "adg-ssi-6", "adg-ssi-7",
 				      "adg-ssi-8", "adg-ssi-9",
 				      "audmapp", "adg";
+			#clock-cells = <0>;
 			power-domains = <&cpg>;
 			resets = <&cpg 0xe1>,
 				 <&cpg 0xe2>, <&cpg 0xe3>,
diff --git a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
index 095acb87c20c..d242f2976296 100644
--- a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
+++ b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
@@ -379,7 +379,6 @@ &snd_rzg3e {
 	status = "okay";
 
 	/* audio_clkout */
-	#clock-cells = <0>;
 	clock-frequency = <11289600>;
 
 	/* Multi DAI */
-- 
2.54.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH v2 2/2] arm64: dts: renesas: r9a09g047e57-smarc: Drop AUDIO_CLKOUT clock-frequency
  2026-10-20 11:11 [PATCH v2 0/2] Drop clock-frequency from RZ/G3E SMARC snd_rzg3e dts node Tommaso Merciai
  2026-10-20 11:11 ` [PATCH v2 1/2] arm64: dts: renesas: r9a09g047: Move #clock-cells to the RZ/G3E SoC dtsi Tommaso Merciai
@ 2026-10-20 11:11 ` Tommaso Merciai
  2026-10-23  6:44   ` Tommaso Merciai
  1 sibling, 1 reply; 4+ messages in thread
From: Tommaso Merciai @ 2026-10-20 11:11 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Tommaso Merciai,
	Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, John Madieu, devicetree,
	linux-kernel

AUDIO_CLKOUT is neither pinmuxed nor consumed on this board,
clock-frequency describe the frequency of an output that does not
exist here. The DA7212 MCLK comes from the on-board versa3 clock
generator.

Drop the clock-frequency entry from the snd_rzg3e node.

This fix also the following dtbs_check warning:

sound@13c00000 (renesas,r9a09g047-sound): Unevaluated properties are not allowed ('clock-frequency' was unexpected)
	from schema $id: http://devicetree.org/schemas/sound/renesas,r9a09g047-sound.yaml#

Fixes: 3e23d2548ddb ("arm64: dts: renesas: r9a09g047e57-smarc: Add DA7212 audio codec support")
Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
---
v1->v2:
 - New patch.

 arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts | 3 ---
 1 file changed, 3 deletions(-)

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 2/2] arm64: dts: renesas: r9a09g047e57-smarc: Drop AUDIO_CLKOUT clock-frequency
  2026-10-20 11:11 ` [PATCH v2 2/2] arm64: dts: renesas: r9a09g047e57-smarc: Drop AUDIO_CLKOUT clock-frequency Tommaso Merciai
@ 2026-10-23  6:44   ` Tommaso Merciai
  0 siblings, 0 replies; 4+ messages in thread
From: Tommaso Merciai @ 2026-10-23  6:44 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Geert Uytterhoeven, Magnus Damm,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, John Madieu,
	devicetree, linux-kernel

Hi,

On Tue, Oct 20, 2026 at 01:11:07PM +0200, Tommaso Merciai wrote:
> AUDIO_CLKOUT is neither pinmuxed nor consumed on this board,
> clock-frequency describe the frequency of an output that does not
> exist here. The DA7212 MCLK comes from the on-board versa3 clock
> generator.
> 
> Drop the clock-frequency entry from the snd_rzg3e node.
> 
> This fix also the following dtbs_check warning:
> 
> sound@13c00000 (renesas,r9a09g047-sound): Unevaluated properties are not allowed ('clock-frequency' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/sound/renesas,r9a09g047-sound.yaml#
> 
> Fixes: 3e23d2548ddb ("arm64: dts: renesas: r9a09g047e57-smarc: Add DA7212 audio codec support")
> Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>

8KHz arecord is failing with this change and currently investigating the issue.
Will update you the findings later.


Kind regards,
Tommaso

> ---
> v1->v2:
>  - New patch.
> 
>  arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
> index d242f2976296..6c0053735904 100644
> --- a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
> +++ b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
> @@ -378,9 +378,6 @@ &snd_rzg3e {
>  
>  	status = "okay";
>  
> -	/* audio_clkout */
> -	clock-frequency = <11289600>;
> -
>  	/* Multi DAI */
>  	#sound-dai-cells = <1>;
>  
> -- 
> 2.54.0
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-10-23  6:44 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-10-20 11:11 [PATCH v2 0/2] Drop clock-frequency from RZ/G3E SMARC snd_rzg3e dts node Tommaso Merciai
2026-10-20 11:11 ` [PATCH v2 1/2] arm64: dts: renesas: r9a09g047: Move #clock-cells to the RZ/G3E SoC dtsi Tommaso Merciai
2026-10-20 11:11 ` [PATCH v2 2/2] arm64: dts: renesas: r9a09g047e57-smarc: Drop AUDIO_CLKOUT clock-frequency Tommaso Merciai
2026-10-23  6:44   ` Tommaso Merciai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®