Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@kernel.org>
To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Cc: alsa-devel@alsa-project.org, Mark Brown <broonie@kernel.org>,
	Simon <horms@verge.net.au>
Subject: Applied "ASoC: intel: baytrail: replace platform to component" to the asoc tree
Date: Mon, 12 Feb 2018 12:32:27 +0000	[thread overview]
Message-ID: <E1elDHX-0003PJ-MK@debutante> (raw)
In-Reply-To: <87mv1jgb4t.wl%kuninori.morimoto.gx@renesas.com>

The patch

   ASoC: intel: baytrail: replace platform to component

has been applied to the asoc tree at

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git 

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.  

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

>From 81dc5f3d806ad50283b4b58abebd08ef65817f87 Mon Sep 17 00:00:00 2001
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Date: Mon, 29 Jan 2018 02:42:02 +0000
Subject: [PATCH] ASoC: intel: baytrail: replace platform to component

Now platform can be replaced to component, let's do it.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/intel/baytrail/sst-baytrail-pcm.c | 66 +++++++++++------------------
 1 file changed, 25 insertions(+), 41 deletions(-)

diff --git a/sound/soc/intel/baytrail/sst-baytrail-pcm.c b/sound/soc/intel/baytrail/sst-baytrail-pcm.c
index c54529320f07..aabb35bf6b96 100644
--- a/sound/soc/intel/baytrail/sst-baytrail-pcm.c
+++ b/sound/soc/intel/baytrail/sst-baytrail-pcm.c
@@ -23,6 +23,7 @@
 #include "../common/sst-dsp-priv.h"
 #include "../common/sst-dsp.h"
 
+#define DRV_NAME "byt-dai"
 #define BYT_PCM_COUNT		2
 
 static const struct snd_pcm_hardware sst_byt_pcm_hardware = {
@@ -69,8 +70,8 @@ static int sst_byt_pcm_hw_params(struct snd_pcm_substream *substream,
 				 struct snd_pcm_hw_params *params)
 {
 	struct snd_soc_pcm_runtime *rtd = substream->private_data;
-	struct sst_byt_priv_data *pdata =
-		snd_soc_platform_get_drvdata(rtd->platform);
+	struct snd_soc_component *component = snd_soc_rtdcom_lookup(rtd, DRV_NAME);
+	struct sst_byt_priv_data *pdata = snd_soc_component_get_drvdata(component);
 	struct sst_byt_pcm_data *pcm_data = &pdata->pcm[substream->stream];
 	struct sst_byt *byt = pdata->byt;
 	u32 rate, bits;
@@ -141,8 +142,8 @@ static int sst_byt_pcm_hw_free(struct snd_pcm_substream *substream)
 static int sst_byt_pcm_restore_stream_context(struct snd_pcm_substream *substream)
 {
 	struct snd_soc_pcm_runtime *rtd = substream->private_data;
-	struct sst_byt_priv_data *pdata =
-		snd_soc_platform_get_drvdata(rtd->platform);
+	struct snd_soc_component *component = snd_soc_rtdcom_lookup(rtd, DRV_NAME);
+	struct sst_byt_priv_data *pdata = snd_soc_component_get_drvdata(component);
 	struct sst_byt_pcm_data *pcm_data = &pdata->pcm[substream->stream];
 	struct sst_byt *byt = pdata->byt;
 	int ret;
@@ -174,8 +175,8 @@ static void sst_byt_pcm_work(struct work_struct *work)
 static int sst_byt_pcm_trigger(struct snd_pcm_substream *substream, int cmd)
 {
 	struct snd_soc_pcm_runtime *rtd = substream->private_data;
-	struct sst_byt_priv_data *pdata =
-		snd_soc_platform_get_drvdata(rtd->platform);
+	struct snd_soc_component *component = snd_soc_rtdcom_lookup(rtd, DRV_NAME);
+	struct sst_byt_priv_data *pdata = snd_soc_component_get_drvdata(component);
 	struct sst_byt_pcm_data *pcm_data = &pdata->pcm[substream->stream];
 	struct sst_byt *byt = pdata->byt;
 
@@ -216,8 +217,8 @@ static u32 byt_notify_pointer(struct sst_byt_stream *stream, void *data)
 	struct snd_pcm_substream *substream = pcm_data->substream;
 	struct snd_pcm_runtime *runtime = substream->runtime;
 	struct snd_soc_pcm_runtime *rtd = substream->private_data;
-	struct sst_byt_priv_data *pdata =
-		snd_soc_platform_get_drvdata(rtd->platform);
+	struct snd_soc_component *component = snd_soc_rtdcom_lookup(rtd, DRV_NAME);
+	struct sst_byt_priv_data *pdata = snd_soc_component_get_drvdata(component);
 	struct sst_byt *byt = pdata->byt;
 	u32 pos, hw_pos;
 
@@ -238,8 +239,8 @@ static snd_pcm_uframes_t sst_byt_pcm_pointer(struct snd_pcm_substream *substream
 {
 	struct snd_soc_pcm_runtime *rtd = substream->private_data;
 	struct snd_pcm_runtime *runtime = substream->runtime;
-	struct sst_byt_priv_data *pdata =
-		snd_soc_platform_get_drvdata(rtd->platform);
+	struct snd_soc_component *component = snd_soc_rtdcom_lookup(rtd, DRV_NAME);
+	struct sst_byt_priv_data *pdata = snd_soc_component_get_drvdata(component);
 	struct sst_byt_pcm_data *pcm_data = &pdata->pcm[substream->stream];
 
 	dev_dbg(rtd->dev, "PCM: DMA pointer %u bytes\n", pcm_data->hw_ptr);
@@ -250,8 +251,8 @@ static snd_pcm_uframes_t sst_byt_pcm_pointer(struct snd_pcm_substream *substream
 static int sst_byt_pcm_open(struct snd_pcm_substream *substream)
 {
 	struct snd_soc_pcm_runtime *rtd = substream->private_data;
-	struct sst_byt_priv_data *pdata =
-		snd_soc_platform_get_drvdata(rtd->platform);
+	struct snd_soc_component *component = snd_soc_rtdcom_lookup(rtd, DRV_NAME);
+	struct sst_byt_priv_data *pdata = snd_soc_component_get_drvdata(component);
 	struct sst_byt_pcm_data *pcm_data = &pdata->pcm[substream->stream];
 	struct sst_byt *byt = pdata->byt;
 
@@ -278,8 +279,8 @@ static int sst_byt_pcm_open(struct snd_pcm_substream *substream)
 static int sst_byt_pcm_close(struct snd_pcm_substream *substream)
 {
 	struct snd_soc_pcm_runtime *rtd = substream->private_data;
-	struct sst_byt_priv_data *pdata =
-		snd_soc_platform_get_drvdata(rtd->platform);
+	struct snd_soc_component *component = snd_soc_rtdcom_lookup(rtd, DRV_NAME);
+	struct sst_byt_priv_data *pdata = snd_soc_component_get_drvdata(component);
 	struct sst_byt_pcm_data *pcm_data = &pdata->pcm[substream->stream];
 	struct sst_byt *byt = pdata->byt;
 	int ret;
@@ -324,8 +325,8 @@ static int sst_byt_pcm_new(struct snd_soc_pcm_runtime *rtd)
 {
 	struct snd_pcm *pcm = rtd->pcm;
 	size_t size;
-	struct snd_soc_platform *platform = rtd->platform;
-	struct sst_pdata *pdata = dev_get_platdata(platform->dev);
+	struct snd_soc_component *component = snd_soc_rtdcom_lookup(rtd, DRV_NAME);
+	struct sst_pdata *pdata = dev_get_platdata(component->dev);
 	int ret = 0;
 
 	if (pcm->streams[SNDRV_PCM_STREAM_PLAYBACK].substream ||
@@ -366,21 +367,21 @@ static struct snd_soc_dai_driver byt_dais[] = {
 	},
 };
 
-static int sst_byt_pcm_probe(struct snd_soc_platform *platform)
+static int sst_byt_pcm_probe(struct snd_soc_component *component)
 {
-	struct sst_pdata *plat_data = dev_get_platdata(platform->dev);
+	struct sst_pdata *plat_data = dev_get_platdata(component->dev);
 	struct sst_byt_priv_data *priv_data;
 	int i;
 
 	if (!plat_data)
 		return -ENODEV;
 
-	priv_data = devm_kzalloc(platform->dev, sizeof(*priv_data),
+	priv_data = devm_kzalloc(component->dev, sizeof(*priv_data),
 				 GFP_KERNEL);
 	if (!priv_data)
 		return -ENOMEM;
 	priv_data->byt = plat_data->dsp;
-	snd_soc_platform_set_drvdata(platform, priv_data);
+	snd_soc_component_set_drvdata(component, priv_data);
 
 	for (i = 0; i < BYT_PCM_COUNT; i++) {
 		mutex_init(&priv_data->pcm[i].mutex);
@@ -390,22 +391,13 @@ static int sst_byt_pcm_probe(struct snd_soc_platform *platform)
 	return 0;
 }
 
-static int sst_byt_pcm_remove(struct snd_soc_platform *platform)
-{
-	return 0;
-}
-
-static const struct snd_soc_platform_driver byt_soc_platform = {
+static const struct snd_soc_component_driver byt_dai_component = {
+	.name		= DRV_NAME,
 	.probe		= sst_byt_pcm_probe,
-	.remove		= sst_byt_pcm_remove,
 	.ops		= &sst_byt_pcm_ops,
 	.pcm_new	= sst_byt_pcm_new,
 };
 
-static const struct snd_soc_component_driver byt_dai_component = {
-	.name		= "byt-dai",
-};
-
 #ifdef CONFIG_PM
 static int sst_byt_pcm_dev_suspend_late(struct device *dev)
 {
@@ -461,19 +453,13 @@ static int sst_byt_pcm_dev_probe(struct platform_device *pdev)
 	if (ret < 0)
 		return -ENODEV;
 
-	ret = snd_soc_register_platform(&pdev->dev, &byt_soc_platform);
-	if (ret < 0)
-		goto err_plat;
-
-	ret = snd_soc_register_component(&pdev->dev, &byt_dai_component,
+	ret = devm_snd_soc_register_component(&pdev->dev, &byt_dai_component,
 					 byt_dais, ARRAY_SIZE(byt_dais));
 	if (ret < 0)
-		goto err_comp;
+		goto err_plat;
 
 	return 0;
 
-err_comp:
-	snd_soc_unregister_platform(&pdev->dev);
 err_plat:
 	sst_byt_dsp_free(&pdev->dev, sst_pdata);
 	return ret;
@@ -483,8 +469,6 @@ static int sst_byt_pcm_dev_remove(struct platform_device *pdev)
 {
 	struct sst_pdata *sst_pdata = dev_get_platdata(&pdev->dev);
 
-	snd_soc_unregister_platform(&pdev->dev);
-	snd_soc_unregister_component(&pdev->dev);
 	sst_byt_dsp_free(&pdev->dev, sst_pdata);
 
 	return 0;
-- 
2.16.1

  reply	other threads:[~2018-02-12 12:32 UTC|newest]

Thread overview: 79+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-12  0:41 [PATCH 00/38] ASoC: replace platform to component Kuninori Morimoto
2018-01-12  0:42 ` [PATCH 01/38] ASoC: remove rtd->platform checck Kuninori Morimoto
2018-01-12  0:42 ` [PATCH 02/38] ASoC: soc-utils: replace platform to component Kuninori Morimoto
2018-01-12  0:43 ` [PATCH 03/38] ASoC: soc-generic-dmaengine-pcm: " Kuninori Morimoto
2018-02-12 12:32   ` Applied "ASoC: soc-generic-dmaengine-pcm: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:43 ` [PATCH 04/38] ASoC: intel: atom: replace platform to component Kuninori Morimoto
2018-01-12  0:43 ` [PATCH 05/38] ASoC: intel: skylake: " Kuninori Morimoto
2018-02-12 12:32   ` Applied "ASoC: intel: skylake: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:44 ` [PATCH 06/38] ASoC: intel: baytrail: replace platform to component Kuninori Morimoto
2018-02-12 12:32   ` Mark Brown [this message]
2018-01-12  0:44 ` [PATCH 07/38] ASoC: intel: haswell: " Kuninori Morimoto
2018-02-12 12:32   ` Applied "ASoC: intel: haswell: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:44 ` [PATCH 08/38] ASoC: bcm: cygnus: replace platform to component Kuninori Morimoto
2018-01-12  0:45 ` [PATCH 09/38] ASoC: sh: rsnd: " Kuninori Morimoto
2018-02-12 12:31   ` Applied "ASoC: sh: rsnd: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:45 ` [PATCH 10/38] ASoC: sh: dma-sh7760: replace platform to component Kuninori Morimoto
2018-02-12 12:31   ` Applied "ASoC: sh: dma-sh7760: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:45 ` [PATCH 11/38] ASoC: sh: fsi: replace platform to component Kuninori Morimoto
2018-01-12  0:46 ` [PATCH 12/38] ASoC: sh: siu: " Kuninori Morimoto
2018-01-12  0:46 ` [PATCH 13/38] ASoC: atmel: " Kuninori Morimoto
2018-02-12 12:30   ` Applied "ASoC: atmel: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:46 ` [PATCH 14/38] ASoC: amd: replace platform to component Kuninori Morimoto
2018-01-24  0:46   ` Kuninori Morimoto
2018-01-24  9:15     ` Mukunda,Vijendar
2018-02-12 12:30   ` Applied "ASoC: amd: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:47 ` [PATCH 15/38] ASoC: cs47l24: replace platform to component Kuninori Morimoto
2018-01-24 14:23   ` Charles Keepax
2018-02-12 12:30   ` Applied "ASoC: cs47l24: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:47 ` [PATCH 16/38] ASoC: rt5514-spi: replace platform to component Kuninori Morimoto
2018-02-12 12:30   ` Applied "ASoC: rt5514-spi: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:47 ` [PATCH 17/38] ASoC: wm5110: replace platform to component Kuninori Morimoto
2018-01-24 14:24   ` Charles Keepax
2018-02-12 12:30   ` Applied "ASoC: wm5110: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:48 ` [PATCH 18/38] ASoC: wm5102: replace platform to component Kuninori Morimoto
2018-01-24 14:25   ` Charles Keepax
2018-02-12 12:29   ` Applied "ASoC: wm5102: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:48 ` [PATCH 19/38] ASoC: davinci-i2s: replace platform to component Kuninori Morimoto
2018-02-12 12:29   ` Applied "ASoC: davinci-i2s: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:48 ` [PATCH 20/38] ASoC: fsl: dma: replace platform to component Kuninori Morimoto
2018-02-12 12:29   ` Applied "ASoC: fsl: dma: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:50 ` [PATCH 21/38] ASoC: fsl: asrc: replace platform to component Kuninori Morimoto
2018-01-16  0:55   ` Nicolin Chen
2018-02-12 12:29   ` Applied "ASoC: fsl: asrc: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:50 ` [PATCH 22/38] ASoC: fsl: imx: replace platform to component Kuninori Morimoto
2018-02-12 12:29   ` Applied "ASoC: fsl: imx: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:50 ` [PATCH 23/38] ASoC: fsl: mpc5200: replace platform to component Kuninori Morimoto
2018-02-12 12:29   ` Applied "ASoC: fsl: mpc5200: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:51 ` [PATCH 24/38] ASoC: dwc-pcm: replace platform to component Kuninori Morimoto
2018-02-12 12:29   ` Applied "ASoC: dwc-pcm: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:51 ` [PATCH 25/38] ASoC: kirkwood: replace platform to component Kuninori Morimoto
2018-02-12 12:29   ` Applied "ASoC: kirkwood: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:51 ` [PATCH 26/38] ASoC: pxa: mmp: replace platform to component Kuninori Morimoto
2018-02-12 12:29   ` Applied "ASoC: pxa: mmp: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:52 ` [PATCH 27/38] ASoC: pxa: pxa2xx: replace platform to component Kuninori Morimoto
2018-02-12 12:29   ` Applied "ASoC: pxa: pxa2xx: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:52 ` [PATCH 28/38] ASoC: qcom: lpass-platform: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: qcom: lpass-platform: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:52 ` [PATCH 29/38] ASoC: samsung: idma: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: samsung: idma: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:53 ` [PATCH 30/38] ASoC: xtfpga-i2s: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: xtfpga-i2s: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:53 ` [PATCH 31/38] ASoC: blackfin: bf5xx-ac97-pcm: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: blackfin: bf5xx-ac97-pcm: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:53 ` [PATCH 32/38] ASoC: blackfin: bf5xx-i2s-pcm: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: blackfin: bf5xx-i2s-pcm: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:54 ` [PATCH 33/38] ASoC: omap-pcm: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: omap-pcm: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:54 ` [PATCH 34/38] ASoC: au1x: dbdma2: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: au1x: dbdma2: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:54 ` [PATCH 35/38] ASoC: au1x: dma: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: au1x: dma: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:54 ` [PATCH 36/38] ASoC: nuc900-pcm: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: nuc900-pcm: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:55 ` [PATCH 37/38] ASoC: txx9aclc: replace platform to component Kuninori Morimoto
2018-02-12 12:28   ` Applied "ASoC: txx9aclc: replace platform to component" to the asoc tree Mark Brown
2018-01-12  0:55 ` [PATCH 38/38] ASoC: mt2701/mt8173: replace platform to component Kuninori Morimoto
2018-01-12  9:46 ` [PATCH 00/38] ASoC: " Liam Girdwood
2018-01-12 11:28   ` Mark Brown
2018-01-14 23:51     ` Kuninori Morimoto

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=E1elDHX-0003PJ-MK@debutante \
    --to=broonie@kernel.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=horms@verge.net.au \
    --cc=kuninori.morimoto.gx@renesas.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox