All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
To: Marek Szyprowski <m.szyprowski@samsung.com>
Cc: 'Linux Samsung SOC' <linux-samsung-soc@vger.kernel.org>,
	Linux-ALSA <alsa-devel@alsa-project.org>,
	Mark Brown <broonie@kernel.org>,
	Krzysztof Kozlowski <krzk@kernel.org>,
	Sylwester Nawrocki <s.nawrocki@samsung.com>
Subject: Re: [PATCH resend 25/47] ASoC: samsung: snow: don't select unnecessary Platform
Date: 26 Jun 2019 08:44:47 +0900	[thread overview]
Message-ID: <87ef3hz0yn.wl-kuninori.morimoto.gx@renesas.com> (raw)
In-Reply-To: <c4bb4599-e5d5-926b-75d5-d5c349ed8076@samsung.com>


Hi Marek

Thank you for your test.
And so sorry to bother you.
I will fixup this issue.

> >>> I have no idea so far, but one capability is that if your system
> >>> used / based dummy platform, and if your CPU driver didn't use/have
> >>> snd_soc_set_runtime_hwparams() / snd_pcm_lib_ioctl,
> >>> it might be happen, but this patch seems not related to it...
> >> Reverting this patch fixes the issue, so it is definitely related somehow.
> > Ahh... OK now I think I could understand your issue.
> > Your CPU is using devm_snd_dmaengine_pcm_register() with same *dev.
> > In such case, indeed this patch breaks sound card.
> >
> > Can I confirm it ?
> > In local email, I asked printing component name list.
> > Can you please replace it to below ?
> > I think disappear component is "snd_dmaengine_pcm".
> > If so, above my patch is the bug.
> >
> > ---------------------------
> > diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
> > index 4cd77cd..4b1f8ae 100644
> > --- a/sound/soc/soc-core.c
> > +++ b/sound/soc/soc-core.c
> > @@ -313,6 +313,7 @@ static int snd_soc_rtdcom_add(struct snd_soc_pcm_runtime *rtd,
> >   	new_rtdcom->component = component;
> >   	INIT_LIST_HEAD(&new_rtdcom->list);
> >   
> > +	printk("-----%s : %s\n", component->name, component->driver->name);
> >   	list_add_tail(&new_rtdcom->list, &rtd->component_list);
> >   
> >   	return 0;
> > ---------------------------
> >
> > Thank you for your help !!
> 
> pure next-20190624:
> 
> [    3.549712] -----3830000.i2s : samsung-i2s
> [    3.767732] -----max98095.7-0011 : (null)
> [    3.986370] -----hdmi-audio-codec.3.auto : (null)
> 
> next-20190624 + Revert "ASoC: samsung: snow: don't select unnecessary 
> Platform":
> 
> [    3.508391] -----3830000.i2s : samsung-i2s
> [    3.726332] -----max98095.7-0011 : (null)
> [    3.944906] -----hdmi-audio-codec.3.auto : (null)
> [    4.376639] -----3830000.i2s : snd_dmaengine_pcm
> 
> 
> Best regards
> -- 
> Marek Szyprowski, PhD
> Samsung R&D Institute Poland
> 

  reply	other threads:[~2019-06-25 23:44 UTC|newest]

Thread overview: 109+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-19  1:13 [PATCH resend 00/47] ASoC: no Platform support Kuninori Morimoto
2019-06-19  1:14 ` [PATCH resend 01/47] ASoC: soc-core: allow no Platform on dai_link Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: soc-core: allow no Platform on dai_link" to the asoc tree Mark Brown
2019-06-19  1:14 ` [PATCH resend 02/47] ASoC: vc4: vc4_hdmi: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: vc4: vc4_hdmi: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:14 ` [PATCH resend 03/47] ASoC: atmel: atmel-classd: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: atmel: atmel-classd: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:14 ` [PATCH resend 04/47] ASoC: atmel: atmel-pdmic: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: atmel: atmel-pdmic: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:14 ` [PATCH resend 05/47] ASoC: atmel: atmel_wm8904: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: atmel: atmel_wm8904: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:14 ` [PATCH resend 05/47] ASoC: atmel: atmel_wm8904: don't select unnecessary Platform Kuninori Morimoto
2019-06-19  1:15 ` [PATCH resend 06/47] ASoC: atmel: mikroe-proto: " Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: atmel: mikroe-proto: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:15 ` [PATCH resend 07/47] ASoC: atmel: sam9g20_wm8731: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: atmel: sam9g20_wm8731: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:15 ` [PATCH resend 07/47] ASoC: atmel: sam9g20_wm8731: don't select unnecessary Platform Kuninori Morimoto
2019-06-19  1:16 ` [PATCH resend 08/47] ASoC: atmel: sam9x5_wm8731: " Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: atmel: sam9x5_wm8731: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:16 ` [PATCH resend 09/47] ASoC: atmel: tse850-pcm5142: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: atmel: tse850-pcm5142: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:16 ` [PATCH resend 10/47] ASoC: fsl: eukrea-tlv320: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: fsl: eukrea-tlv320: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:16 ` [PATCH resend 11/47] ASoC: fsl: fsl-asoc-card: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: fsl: fsl-asoc-card: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:16 ` [PATCH resend 12/47] ASoC: fsl: imx-es8328: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: fsl: imx-es8328: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:16 ` [PATCH resend 13/47] ASoC: fsl: imx-sgtl5000: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: fsl: imx-sgtl5000: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:17 ` [PATCH resend 14/47] ASoC: fsl: imx-spdif: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: fsl: imx-spdif: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:17 ` [PATCH resend 15/47] ASoC: fsl: imx-audmix: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: fsl: imx-audmix: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:17 ` [PATCH resend 16/47] ASoC: kirkwood: armada-370-db: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: kirkwood: armada-370-db: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:17 ` [PATCH resend 17/47] ASoC: mxs: mxs-sgtl5000: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: mxs: mxs-sgtl5000: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:17 ` [PATCH resend 18/47] ASoC: qcom: apq8016_sbc: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: qcom: apq8016_sbc: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:17 ` [PATCH resend 19/47] ASoC: qcom: storm: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: qcom: storm: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:17 ` [PATCH resend 20/47] ASoC: rockchip: rk3288_hdmi_analog: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: rockchip: rk3288_hdmi_analog: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:17 ` [PATCH resend 21/47] ASoC: rockchip: rockchip_max98090: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: rockchip: rockchip_max98090: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 22/47] ASoC: rockchip: rockchip_rt5645: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: rockchip: rockchip_rt5645: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 23/47] ASoC: samsung: arndale_rt5631: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: samsung: arndale_rt5631: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 24/47] ASoC: samsung: smdk_wm8994: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: samsung: smdk_wm8994: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 25/47] ASoC: samsung: snow: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: samsung: snow: don't select unnecessary Platform" to the asoc tree Mark Brown
     [not found]   ` <CGME20190624131905eucas1p2cf4335b4d1483b7e62d0c9e7b5223a3c@eucas1p2.samsung.com>
2019-06-24 13:19     ` [PATCH resend 25/47] ASoC: samsung: snow: don't select unnecessary Platform Marek Szyprowski
2019-06-25  0:51       ` Kuninori Morimoto
2019-06-25  5:36         ` Marek Szyprowski
2019-06-25  8:25           ` Kuninori Morimoto
2019-06-25  8:37             ` Marek Szyprowski
2019-06-25 23:44               ` Kuninori Morimoto [this message]
2019-06-26  1:15                 ` Kuninori Morimoto
2019-06-26  6:41                   ` Marek Szyprowski
2019-06-26  6:48                     ` Marek Szyprowski
2019-06-26  7:20                       ` Kuninori Morimoto
2019-06-27 12:44                       ` Marek Szyprowski
2019-06-27 23:53                         ` Kuninori Morimoto
2019-06-19  1:18 ` [PATCH resend 26/47] ASoC: samsung: tm2_wm5110: " Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: samsung: tm2_wm5110: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 27/47] ASoC: sirf: sirf-audio: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: sirf: sirf-audio: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 28/47] ASoC: sunxi: sun4i-codec: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: sunxi: sun4i-codec: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 29/47] ASoC: tegra: tegra_alc5632: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: tegra: tegra_alc5632: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 30/47] ASoC: tegra: tegra_max98090: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: tegra: tegra_max98090: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 31/47] ASoC: tegra: tegra_rt5640: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: tegra: tegra_rt5640: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 32/47] ASoC: tegra: tegra_rt5677: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: tegra: tegra_rt5677: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 33/47] ASoC: tegra: tegra_sgtl5000: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: tegra: tegra_sgtl5000: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 34/47] ASoC: tegra: tegra_wm8753: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: tegra: tegra_wm8753: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:18 ` [PATCH resend 35/47] ASoC: tegra: tegra_wm8903: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: tegra: tegra_wm8903: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 36/47] ASoC: tegra: tegra_wm9712: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: tegra: tegra_wm9712: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 37/47] ASoC: tegra: trimslice: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:12   ` Applied "ASoC: tegra: trimslice: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 38/47] ASoC: ti: davinci-evm: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: ti: davinci-evm: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 39/47] ASoC: ti: omap-abe-twl6040: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: ti: omap-abe-twl6040: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 40/47] ASoC: ti: omap-hdmi: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: ti: omap-hdmi: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 41/47] ASoC: ti: omap-twl4030: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: ti: omap-twl4030: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 42/47] ASoC: ti: rx51: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: ti: rx51: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 43/47] ASoC: ux500: mop500: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: ux500: mop500: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 44/47] ASoC: simple-card-utils: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: simple-card-utils: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 45/47] ASoC: qcom: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: qcom: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 46/47] ASoC: rockchip: rk3399_gru_sound: don't select unnecessary Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: rockchip: rk3399_gru_sound: don't select unnecessary Platform" to the asoc tree Mark Brown
2019-06-19  1:19 ` [PATCH resend 47/47] ASoC: soc-utils: remove dummy Platform Kuninori Morimoto
2019-06-19 12:11   ` Applied "ASoC: soc-utils: remove dummy Platform" to the asoc tree Mark Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87ef3hz0yn.wl-kuninori.morimoto.gx@renesas.com \
    --to=kuninori.morimoto.gx@renesas.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=krzk@kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    --cc=s.nawrocki@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.