All of lore.kernel.org
 help / color / mirror / Atom feed
* [RFC PATCH] arm64: dts: renesas: ebisu: add port node to ak4613
@ 2022-06-28 12:23 Wolfram Sang
  2022-06-29  0:19 ` Kuninori Morimoto
  0 siblings, 1 reply; 4+ messages in thread
From: Wolfram Sang @ 2022-06-28 12:23 UTC (permalink / raw)
  To: linux-renesas-soc; +Cc: Kuninori Morimoto, Wolfram Sang

My Ebisu complains about the missing node when booting:

 OF: graph: no port node found in /soc/i2c@e66d0000/codec@10
 ------------[ cut here ]------------
 WARNING: CPU: 1 PID: 37 at sound/soc/codecs/ak4613.c:873 ak4613_i2c_probe+0x274/0x2a0
 CPU: 1 PID: 37 Comm: kworker/u4:2 Not tainted 5.19.0-rc3-arm64-renesas-00003-g57c83727d651 #50
 Hardware name: Renesas Ebisu board based on r8a77990 (DT)

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

Hi Morimoto-san, I have never done any sound stuff with R-Car. Is this
the correct approach which I copied over from Salvator boards?

All the best,

   Wolfram

 arch/arm64/boot/dts/renesas/ebisu.dtsi | 26 +++++++++++++++++++++-----
 1 file changed, 21 insertions(+), 5 deletions(-)

diff --git a/arch/arm64/boot/dts/renesas/ebisu.dtsi b/arch/arm64/boot/dts/renesas/ebisu.dtsi
index 8fc03491a11c..cec23a3287c0 100644
--- a/arch/arm64/boot/dts/renesas/ebisu.dtsi
+++ b/arch/arm64/boot/dts/renesas/ebisu.dtsi
@@ -478,6 +478,12 @@ ak4613: codec@10 {
 		asahi-kasei,out4-single-end;
 		asahi-kasei,out5-single-end;
 		asahi-kasei,out6-single-end;
+
+		port {
+			ak4613_endpoint: endpoint {
+				remote-endpoint = <&rsnd_endpoint0>;
+			};
+		};
 	};
 
 	cs2000: clk-multiplier@4f {
@@ -708,13 +714,23 @@ &rcar_sound {
 		 <&audio_clk_a>, <&cs2000>, <&audio_clk_c>,
 		 <&cpg CPG_CORE R8A77990_CLK_ZA2>;
 
-	rcar_sound,dai {
-		dai0 {
-			playback = <&ssi0>, <&src0>, <&dvc0>;
-			capture = <&ssi1>, <&src1>, <&dvc1>;
+	ports {
+		#address-cells = <1>;
+		#size-cells = <0>;
+		rsnd_port0: port@0 {
+			reg = <0>;
+			rsnd_endpoint0: endpoint {
+				remote-endpoint = <&ak4613_endpoint>;
+
+				dai-format = "left_j";
+				bitclock-master = <&rsnd_endpoint0>;
+				frame-master = <&rsnd_endpoint0>;
+
+				playback = <&ssi0>, <&src0>, <&dvc0>;
+				capture = <&ssi1>, <&src1>, <&dvc1>;
+			};
 		};
 	};
-
 };
 
 &rpc {
-- 
2.35.1


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

* Re: [RFC PATCH] arm64: dts: renesas: ebisu: add port node to ak4613
  2022-06-28 12:23 [RFC PATCH] arm64: dts: renesas: ebisu: add port node to ak4613 Wolfram Sang
@ 2022-06-29  0:19 ` Kuninori Morimoto
  2022-06-29  6:31   ` Wolfram Sang
  0 siblings, 1 reply; 4+ messages in thread
From: Kuninori Morimoto @ 2022-06-29  0:19 UTC (permalink / raw)
  To: Wolfram Sang; +Cc: linux-renesas-soc


Hi Wolfram

Thank you for contacting me,
and sorry for my bug.

> My Ebisu complains about the missing node when booting:
> 
>  OF: graph: no port node found in /soc/i2c@e66d0000/codec@10
>  ------------[ cut here ]------------
>  WARNING: CPU: 1 PID: 37 at sound/soc/codecs/ak4613.c:873 ak4613_i2c_probe+0x274/0x2a0
>  CPU: 1 PID: 37 Comm: kworker/u4:2 Not tainted 5.19.0-rc3-arm64-renesas-00003-g57c83727d651 #50
>  Hardware name: Renesas Ebisu board based on r8a77990 (DT)
> 
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
> 
> Hi Morimoto-san, I have never done any sound stuff with R-Car. Is this
> the correct approach which I copied over from Salvator boards?

Could you try this patch ?
It should goto linus tree, but it seems not yet ??

	https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=ec3ad554b956d5dbefa1962c419f164ba223e6b3

Thank you for your help !!

Best regards
---
Kuninori Morimoto

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

* Re: [RFC PATCH] arm64: dts: renesas: ebisu: add port node to ak4613
  2022-06-29  0:19 ` Kuninori Morimoto
@ 2022-06-29  6:31   ` Wolfram Sang
  2022-07-01  8:06     ` Geert Uytterhoeven
  0 siblings, 1 reply; 4+ messages in thread
From: Wolfram Sang @ 2022-06-29  6:31 UTC (permalink / raw)
  To: Kuninori Morimoto; +Cc: linux-renesas-soc

[-- Attachment #1: Type: text/plain, Size: 396 bytes --]

Hi Morimoto-san,

> Thank you for contacting me,
> and sorry for my bug.

No worries, bugs happen all the time to all of us.

> Could you try this patch ?

Thank you! It works \o/

> It should goto linus tree, but it seems not yet ??

It is also not in renesas-drivers, this is why I am still seeing this
issue. Now I know.

> Thank you for your help !!

Thank YOU! :D

All the best,

   Wolfram

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* Re: [RFC PATCH] arm64: dts: renesas: ebisu: add port node to ak4613
  2022-06-29  6:31   ` Wolfram Sang
@ 2022-07-01  8:06     ` Geert Uytterhoeven
  0 siblings, 0 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2022-07-01  8:06 UTC (permalink / raw)
  To: Wolfram Sang, Kuninori Morimoto, Linux-Renesas

Hi Wolfram,

On Wed, Jun 29, 2022 at 8:34 AM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> > Thank you for contacting me,
> > and sorry for my bug.
>
> No worries, bugs happen all the time to all of us.
>
> > Could you try this patch ?
>
> Thank you! It works \o/
>
> > It should goto linus tree, but it seems not yet ??
>
> It is also not in renesas-drivers, this is why I am still seeing this
> issue. Now I know.

Strange, I thought it was included.
"git show --pretty=fuller" shows it was committed by Mark Bown on
June 8.  Looks like the branch with the commit was not included in
sound-asoc/for-next, only in sound-asoc/fixes, when I made last
renesas-drivers.

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

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

end of thread, other threads:[~2022-07-01  8:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-28 12:23 [RFC PATCH] arm64: dts: renesas: ebisu: add port node to ak4613 Wolfram Sang
2022-06-29  0:19 ` Kuninori Morimoto
2022-06-29  6:31   ` Wolfram Sang
2022-07-01  8:06     ` Geert Uytterhoeven

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.