mbox series

[0/4] arm64: renesas: enable ULCB HDMI / ULCB-KF sound

Message ID 87in18e3um.wl-kuninori.morimoto.gx@renesas.com (mailing list archive)
Headers show
Series arm64: renesas: enable ULCB HDMI / ULCB-KF sound | expand

Message

Kuninori Morimoto Nov. 8, 2018, 1:57 a.m. UTC
Hi Simon

These patches adds sound support for KingFisher.
We can enable it on top of v4.20-rc1, but, it is not stable.
We need this patch (= from ASoC for-v4.21 branch) to be stable it.

	223bc10b84970fd772c105b550beeef3ac3502be
	("ASoC: pcm3168a: remove read-only status register from snd_kcontrol_new")

Kuninori Morimoto (4):
  arm64: renesas: ulcb: use audio-graph-card
  arm64: renesas: ulcb: add HDMI sound support
  arm64: renesas: ulcb-kf: add pcm3168 sound codec
  arm64: renesas_defconfig: select Kingfisher Sound related configs

 arch/arm64/boot/dts/renesas/ulcb-kf.dtsi | 151 +++++++++++++++++++++++++++++++
 arch/arm64/boot/dts/renesas/ulcb.dtsi    |  70 ++++++++++----
 arch/arm64/configs/renesas_defconfig     |   2 +
 3 files changed, 206 insertions(+), 17 deletions(-)

Comments

Simon Horman Nov. 8, 2018, 10:48 a.m. UTC | #1
On Thu, Nov 08, 2018 at 01:57:41AM +0000, Kuninori Morimoto wrote:
> 
> Hi Simon
> 
> These patches adds sound support for KingFisher.
> We can enable it on top of v4.20-rc1, but, it is not stable.
> We need this patch (= from ASoC for-v4.21 branch) to be stable it.
> 
> 	223bc10b84970fd772c105b550beeef3ac3502be
> 	("ASoC: pcm3168a: remove read-only status register from snd_kcontrol_new")

Perhaps it is best to wait for that patch to hit an rc release.
Do you know when that will happen?

> 
> Kuninori Morimoto (4):
>   arm64: renesas: ulcb: use audio-graph-card
>   arm64: renesas: ulcb: add HDMI sound support
>   arm64: renesas: ulcb-kf: add pcm3168 sound codec
>   arm64: renesas_defconfig: select Kingfisher Sound related configs
> 
>  arch/arm64/boot/dts/renesas/ulcb-kf.dtsi | 151 +++++++++++++++++++++++++++++++
>  arch/arm64/boot/dts/renesas/ulcb.dtsi    |  70 ++++++++++----
>  arch/arm64/configs/renesas_defconfig     |   2 +
>  3 files changed, 206 insertions(+), 17 deletions(-)
> 
> -- 
> 2.7.4
> 
> 
> 
> Best regards
> ---
> Kuninori Morimoto
>
Kuninori Morimoto Nov. 9, 2018, 12:02 a.m. UTC | #2
Hi Simon

> > These patches adds sound support for KingFisher.
> > We can enable it on top of v4.20-rc1, but, it is not stable.
> > We need this patch (= from ASoC for-v4.21 branch) to be stable it.
> > 
> > 	223bc10b84970fd772c105b550beeef3ac3502be
> > 	("ASoC: pcm3168a: remove read-only status register from snd_kcontrol_new")
> 
> Perhaps it is best to wait for that patch to hit an rc release.
> Do you know when that will happen?

It will be v4.21, not for v4.20.

Best regards
---
Kuninori Morimoto
Simon Horman Nov. 9, 2018, 9:47 a.m. UTC | #3
On Fri, Nov 09, 2018 at 12:02:01AM +0000, Kuninori Morimoto wrote:
> 
> Hi Simon
> 
> > > These patches adds sound support for KingFisher.
> > > We can enable it on top of v4.20-rc1, but, it is not stable.
> > > We need this patch (= from ASoC for-v4.21 branch) to be stable it.
> > > 
> > > 	223bc10b84970fd772c105b550beeef3ac3502be
> > > 	("ASoC: pcm3168a: remove read-only status register from snd_kcontrol_new")
> > 
> > Perhaps it is best to wait for that patch to hit an rc release.
> > Do you know when that will happen?
> 
> It will be v4.21, not for v4.20.

Are the dependencies present in v4.21-rc1?
If so perhaps we could apply this series as
my branches are currently based on v4.21-rc1.

Let me know what you think.
Simon Horman Nov. 19, 2018, 10:15 a.m. UTC | #4
On Fri, Nov 09, 2018 at 10:47:07AM +0100, Simon Horman wrote:
> On Fri, Nov 09, 2018 at 12:02:01AM +0000, Kuninori Morimoto wrote:
> > 
> > Hi Simon
> > 
> > > > These patches adds sound support for KingFisher.
> > > > We can enable it on top of v4.20-rc1, but, it is not stable.
> > > > We need this patch (= from ASoC for-v4.21 branch) to be stable it.
> > > > 
> > > > 	223bc10b84970fd772c105b550beeef3ac3502be
> > > > 	("ASoC: pcm3168a: remove read-only status register from snd_kcontrol_new")
> > > 
> > > Perhaps it is best to wait for that patch to hit an rc release.
> > > Do you know when that will happen?
> > 
> > It will be v4.21, not for v4.20.
> 
> Are the dependencies present in v4.21-rc1?
> If so perhaps we could apply this series as
> my branches are currently based on v4.21-rc1.
> 
> Let me know what you think.

Hi Morimoto-san,

could you let me know your thoughts on this?
Kuninori Morimoto Nov. 20, 2018, 1:06 a.m. UTC | #5
Hi Simon

sorry for my late response

> > > It will be v4.21, not for v4.20.
> > 
> > Are the dependencies present in v4.21-rc1?
> > If so perhaps we could apply this series as
> > my branches are currently based on v4.21-rc1.
> > 
> > Let me know what you think.
> 
> Hi Morimoto-san,
> 
> could you let me know your thoughts on this?

Does your branches are based on v4.21-rc1 ? not v4.20-rc1 ?
The necessary patch is not yet included on v4.20-rc1.
But will be OK on v4.21-rc1

I will re-post it if linus/master branch includes
necessary commit.