* Re: [PATCH][RFC] ASoC: wm8350: remove snd_soc_codec
2018-03-28 2:22 [PATCH][RFC] ASoC: wm8350: remove snd_soc_codec Kuninori Morimoto
@ 2018-03-28 9:48 ` Charles Keepax
2018-03-28 23:56 ` Kuninori Morimoto
2018-04-12 16:16 ` Applied "ASoC: wm8350: remove snd_soc_codec" to the asoc tree Mark Brown
` (2 subsequent siblings)
3 siblings, 1 reply; 7+ messages in thread
From: Charles Keepax @ 2018-03-28 9:48 UTC (permalink / raw)
To: Kuninori Morimoto
Cc: Linux-ALSA, patches, Liam Girdwood, Mark Brown, Lee Jones
On Wed, Mar 28, 2018 at 02:22:07AM +0000, Kuninori Morimoto wrote:
>
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> codec is replace to component.
> It seems no-one is using it, Let's remove it.
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
> Hi Mark, Liam
>
> I think no-one is using snd_soc_codec, but please double-check it.
> I added [RFC] on this patch
>
Looks to me like there are still users in
drivers/mfd/cwm8350-core.c:
wm8350_client_dev_register(wm8350, "wm8350-codec",
&(wm8350->codec.pdev));
And arch/arm/mach-imx/mach-mx31ads.c:
wm8350->codec.platform_data = &imx32ads_wm8350_setup;
Also it should be simple enough to build test drivers for patches
like this, you don't need to have the hardware to build the driver.
Thanks,
Charles
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH][RFC] ASoC: wm8350: remove snd_soc_codec
2018-03-28 9:48 ` Charles Keepax
@ 2018-03-28 23:56 ` Kuninori Morimoto
2018-03-29 10:56 ` Charles Keepax
0 siblings, 1 reply; 7+ messages in thread
From: Kuninori Morimoto @ 2018-03-28 23:56 UTC (permalink / raw)
To: Charles Keepax; +Cc: patches, Linux-ALSA, Mark Brown, Lee Jones, Liam Girdwood
Hi Charles
Thank you for your feedback
> > I think no-one is using snd_soc_codec, but please double-check it.
> > I added [RFC] on this patch
> >
>
> Looks to me like there are still users in
> drivers/mfd/cwm8350-core.c:
>
> wm8350_client_dev_register(wm8350, "wm8350-codec",
> &(wm8350->codec.pdev));
>
> And arch/arm/mach-imx/mach-mx31ads.c:
>
> wm8350->codec.platform_data = &imx32ads_wm8350_setup;
>
> Also it should be simple enough to build test drivers for patches
> like this, you don't need to have the hardware to build the driver.
The relationships (and my removed) are
struct wm8350_codec {
struct platform_device *pdev;
- struct snd_soc_codec *codec;
struct wm8350_audio_platform_data *platform_data;
};
struct wm8350 {
...
struct wm8350_codec codec;
...
}
This means I think my removed was
wm8350->codec.codec
not
wm8350->codec
And I think no one is using wm8350->codec.codec.
But can you double check ?
Compile test in my side was OK
Best regards
---
Kuninori Morimoto
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH][RFC] ASoC: wm8350: remove snd_soc_codec
2018-03-28 23:56 ` Kuninori Morimoto
@ 2018-03-29 10:56 ` Charles Keepax
0 siblings, 0 replies; 7+ messages in thread
From: Charles Keepax @ 2018-03-29 10:56 UTC (permalink / raw)
To: Kuninori Morimoto
Cc: patches, Linux-ALSA, Mark Brown, Lee Jones, Liam Girdwood
On Wed, Mar 28, 2018 at 11:56:52PM +0000, Kuninori Morimoto wrote:
>
> Hi Charles
>
> Thank you for your feedback
>
> > > I think no-one is using snd_soc_codec, but please double-check it.
> > > I added [RFC] on this patch
> > >
> >
> > Looks to me like there are still users in
> > drivers/mfd/cwm8350-core.c:
> >
> > wm8350_client_dev_register(wm8350, "wm8350-codec",
> > &(wm8350->codec.pdev));
> >
> > And arch/arm/mach-imx/mach-mx31ads.c:
> >
> > wm8350->codec.platform_data = &imx32ads_wm8350_setup;
> >
> > Also it should be simple enough to build test drivers for patches
> > like this, you don't need to have the hardware to build the driver.
>
> The relationships (and my removed) are
>
> struct wm8350_codec {
> struct platform_device *pdev;
> - struct snd_soc_codec *codec;
> struct wm8350_audio_platform_data *platform_data;
> };
>
> struct wm8350 {
> ...
> struct wm8350_codec codec;
> ...
> }
>
> This means I think my removed was
>
> wm8350->codec.codec
>
> not
>
> wm8350->codec
>
> And I think no one is using wm8350->codec.codec.
> But can you double check ?
Apologies you are of course correct. Looks good from my side:
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Thanks,
Charles
^ permalink raw reply [flat|nested] 7+ messages in thread
* Applied "ASoC: wm8350: remove snd_soc_codec" to the asoc tree
2018-03-28 2:22 [PATCH][RFC] ASoC: wm8350: remove snd_soc_codec Kuninori Morimoto
2018-03-28 9:48 ` Charles Keepax
@ 2018-04-12 16:16 ` Mark Brown
2018-04-13 10:56 ` Mark Brown
2018-04-13 11:24 ` Mark Brown
3 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2018-04-12 16:16 UTC (permalink / raw)
To: Kuninori Morimoto
Cc: alsa-devel, Charles Keepax, patches, Liam Girdwood, Mark Brown,
Lee Jones
The patch
ASoC: wm8350: remove snd_soc_codec
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 d00ef703065abc8d1efe5e548daa923f5411b042 Mon Sep 17 00:00:00 2001
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Date: Wed, 28 Mar 2018 02:22:07 +0000
Subject: [PATCH] ASoC: wm8350: remove snd_soc_codec
codec is replace to component.
It seems no-one is using it, Let's remove it.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
include/linux/mfd/wm8350/audio.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/include/linux/mfd/wm8350/audio.h b/include/linux/mfd/wm8350/audio.h
index bd581c6fa085..0bc41c4c0429 100644
--- a/include/linux/mfd/wm8350/audio.h
+++ b/include/linux/mfd/wm8350/audio.h
@@ -617,11 +617,8 @@ struct wm8350_audio_platform_data {
u32 codec_current_charge:2; /* codec current @ vmid charge */
};
-struct snd_soc_codec;
-
struct wm8350_codec {
struct platform_device *pdev;
- struct snd_soc_codec *codec;
struct wm8350_audio_platform_data *platform_data;
};
--
2.17.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Applied "ASoC: wm8350: remove snd_soc_codec" to the asoc tree
2018-03-28 2:22 [PATCH][RFC] ASoC: wm8350: remove snd_soc_codec Kuninori Morimoto
2018-03-28 9:48 ` Charles Keepax
2018-04-12 16:16 ` Applied "ASoC: wm8350: remove snd_soc_codec" to the asoc tree Mark Brown
@ 2018-04-13 10:56 ` Mark Brown
2018-04-13 11:24 ` Mark Brown
3 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2018-04-13 10:56 UTC (permalink / raw)
To: Kuninori Morimoto
Cc: alsa-devel, Charles Keepax, patches, Liam Girdwood, Mark Brown,
Lee Jones
The patch
ASoC: wm8350: remove snd_soc_codec
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 d00ef703065abc8d1efe5e548daa923f5411b042 Mon Sep 17 00:00:00 2001
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Date: Wed, 28 Mar 2018 02:22:07 +0000
Subject: [PATCH] ASoC: wm8350: remove snd_soc_codec
codec is replace to component.
It seems no-one is using it, Let's remove it.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
include/linux/mfd/wm8350/audio.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/include/linux/mfd/wm8350/audio.h b/include/linux/mfd/wm8350/audio.h
index bd581c6fa085..0bc41c4c0429 100644
--- a/include/linux/mfd/wm8350/audio.h
+++ b/include/linux/mfd/wm8350/audio.h
@@ -617,11 +617,8 @@ struct wm8350_audio_platform_data {
u32 codec_current_charge:2; /* codec current @ vmid charge */
};
-struct snd_soc_codec;
-
struct wm8350_codec {
struct platform_device *pdev;
- struct snd_soc_codec *codec;
struct wm8350_audio_platform_data *platform_data;
};
--
2.17.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Applied "ASoC: wm8350: remove snd_soc_codec" to the asoc tree
2018-03-28 2:22 [PATCH][RFC] ASoC: wm8350: remove snd_soc_codec Kuninori Morimoto
` (2 preceding siblings ...)
2018-04-13 10:56 ` Mark Brown
@ 2018-04-13 11:24 ` Mark Brown
3 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2018-04-13 11:24 UTC (permalink / raw)
To: Kuninori Morimoto
Cc: alsa-devel, Charles Keepax, patches, Liam Girdwood, Mark Brown,
Lee Jones
The patch
ASoC: wm8350: remove snd_soc_codec
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 d00ef703065abc8d1efe5e548daa923f5411b042 Mon Sep 17 00:00:00 2001
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Date: Wed, 28 Mar 2018 02:22:07 +0000
Subject: [PATCH] ASoC: wm8350: remove snd_soc_codec
codec is replace to component.
It seems no-one is using it, Let's remove it.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
include/linux/mfd/wm8350/audio.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/include/linux/mfd/wm8350/audio.h b/include/linux/mfd/wm8350/audio.h
index bd581c6fa085..0bc41c4c0429 100644
--- a/include/linux/mfd/wm8350/audio.h
+++ b/include/linux/mfd/wm8350/audio.h
@@ -617,11 +617,8 @@ struct wm8350_audio_platform_data {
u32 codec_current_charge:2; /* codec current @ vmid charge */
};
-struct snd_soc_codec;
-
struct wm8350_codec {
struct platform_device *pdev;
- struct snd_soc_codec *codec;
struct wm8350_audio_platform_data *platform_data;
};
--
2.17.0
^ permalink raw reply related [flat|nested] 7+ messages in thread