Re: [PATCH v2 10/10] arm64: dts: renesas: r8a77961-salvator-xs: add HDMI Sound support

2020-09-11 Thread Geert Uytterhoeven
On Tue, Sep 8, 2020 at 8:37 AM Laurent Pinchart
 wrote:
> On Tue, Sep 08, 2020 at 03:33:29PM +0900, Kuninori Morimoto wrote:
> > > > From: Kuninori Morimoto 
> > > >
> > > > This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.
> > > >
> > > > This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.
> > >
> > > Which tree can this commit be found in ?
> >
> > Grr, I forgot to remove it from git-log.
> > will fix in v3
>
> No worries :-)
>
> I've applied patch 01 to 04 to my tree and plan to send a pull request
> later today. Could you just let me know if you're fine with the small
> modification to the commit message proposed in 04/10 ?

And I'm queueing patches 6-10 in renesas-devel for v5.10, with the "revert"
line removed.  For patch 6, I took v1, as it matches what we did for M3-W.

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@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
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH v2 10/10] arm64: dts: renesas: r8a77961-salvator-xs: add HDMI Sound support

2020-09-08 Thread Kuninori Morimoto


Hi Laurent

> > > > From: Kuninori Morimoto 
> > > > 
> > > > This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.
> > > > 
> > > > This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.
> > > 
> > > Which tree can this commit be found in ?
> > 
> > Grr, I forgot to remove it from git-log.
> > will fix in v3
> 
> No worries :-)
> 
> I've applied patch 01 to 04 to my tree and plan to send a pull request
> later today. Could you just let me know if you're fine with the small
> modification to the commit message proposed in 04/10 ?

Yes, no objection about it.
Thank you for your help !!

Best regards
---
Kuninori Morimoto
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH v2 10/10] arm64: dts: renesas: r8a77961-salvator-xs: add HDMI Sound support

2020-09-08 Thread Laurent Pinchart
Hi Morimoto-san,

On Tue, Sep 08, 2020 at 03:33:29PM +0900, Kuninori Morimoto wrote:
> 
> Hi Laurent
> 
> Thank you for your review
> 
> > > From: Kuninori Morimoto 
> > > 
> > > This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.
> > > 
> > > This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.
> > 
> > Which tree can this commit be found in ?
> 
> Grr, I forgot to remove it from git-log.
> will fix in v3

No worries :-)

I've applied patch 01 to 04 to my tree and plan to send a pull request
later today. Could you just let me know if you're fine with the small
modification to the commit message proposed in 04/10 ?

-- 
Regards,

Laurent Pinchart
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH v2 10/10] arm64: dts: renesas: r8a77961-salvator-xs: add HDMI Sound support

2020-09-08 Thread Kuninori Morimoto


Hi Laurent

Thank you for your review

> > From: Kuninori Morimoto 
> > 
> > This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.
> > 
> > This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.
> 
> Which tree can this commit be found in ?

Grr, I forgot to remove it from git-log.
will fix in v3

Thank you for your help !!

Best regards
---
Kuninori Morimoto
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH v2 10/10] arm64: dts: renesas: r8a77961-salvator-xs: add HDMI Sound support

2020-09-08 Thread Laurent Pinchart
Hi Morimoto-san,

Thank you for the patch.

On Tue, Sep 08, 2020 at 09:35:25AM +0900, Kuninori Morimoto wrote:
> From: Kuninori Morimoto 
> 
> This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.
> 
> This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.

Which tree can this commit be found in ?

> Signed-off-by: Kuninori Morimoto 
> ---
>  .../boot/dts/renesas/r8a77961-salvator-xs.dts | 29 +++
>  1 file changed, 29 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77961-salvator-xs.dts 
> b/arch/arm64/boot/dts/renesas/r8a77961-salvator-xs.dts
> index ca21a702db54..1e7603365106 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77961-salvator-xs.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77961-salvator-xs.dts
> @@ -51,9 +51,38 @@ rcar_dw_hdmi0_out: endpoint {
>   remote-endpoint = <_con>;
>   };
>   };
> + port@2 {
> + reg = <2>;
> + dw_hdmi0_snd_in: endpoint {
> + remote-endpoint = <_endpoint1>;
> + };
> + };
>   };
>  };
>  
>  _con {
>   remote-endpoint = <_dw_hdmi0_out>;
>  };
> +
> +_sound {
> + ports {
> + /* rsnd_port0 is on salvator-common */
> + rsnd_port1: port@1 {
> + reg = <1>;
> + rsnd_endpoint1: endpoint {
> + remote-endpoint = <_hdmi0_snd_in>;
> +
> + dai-format = "i2s";
> + bitclock-master = <_endpoint1>;
> + frame-master = <_endpoint1>;
> +
> + playback = <>;
> + };
> + };
> + };
> +};
> +
> +_card {
> + dais = <_port0 /* ak4613 */
> + _port1>;   /* HDMI0  */
> +};

-- 
Regards,

Laurent Pinchart
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH v2 10/10] arm64: dts: renesas: r8a77961-salvator-xs: add HDMI Sound support

2020-09-07 Thread Kuninori Morimoto
From: Kuninori Morimoto 

This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.

This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.

Signed-off-by: Kuninori Morimoto 
---
 .../boot/dts/renesas/r8a77961-salvator-xs.dts | 29 +++
 1 file changed, 29 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a77961-salvator-xs.dts 
b/arch/arm64/boot/dts/renesas/r8a77961-salvator-xs.dts
index ca21a702db54..1e7603365106 100644
--- a/arch/arm64/boot/dts/renesas/r8a77961-salvator-xs.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77961-salvator-xs.dts
@@ -51,9 +51,38 @@ rcar_dw_hdmi0_out: endpoint {
remote-endpoint = <_con>;
};
};
+   port@2 {
+   reg = <2>;
+   dw_hdmi0_snd_in: endpoint {
+   remote-endpoint = <_endpoint1>;
+   };
+   };
};
 };
 
 _con {
remote-endpoint = <_dw_hdmi0_out>;
 };
+
+_sound {
+   ports {
+   /* rsnd_port0 is on salvator-common */
+   rsnd_port1: port@1 {
+   reg = <1>;
+   rsnd_endpoint1: endpoint {
+   remote-endpoint = <_hdmi0_snd_in>;
+
+   dai-format = "i2s";
+   bitclock-master = <_endpoint1>;
+   frame-master = <_endpoint1>;
+
+   playback = <>;
+   };
+   };
+   };
+};
+
+_card {
+   dais = <_port0 /* ak4613 */
+   _port1>;   /* HDMI0  */
+};
-- 
2.25.1

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel