diff mbox series

ASoC: Intel: cml_rt1011_rt5682: fix codec_conf by removing legacy style

Message ID 20191217165649.12091-1-pierre-louis.bossart@linux.intel.com (mailing list archive)
State New, archived
Headers show
Series ASoC: Intel: cml_rt1011_rt5682: fix codec_conf by removing legacy style | expand

Commit Message

Pierre-Louis Bossart Dec. 17, 2019, 4:56 p.m. UTC
Now that the legacy style is removed, we have to use the new macros
for the codec configuration. This change was missed in the initial
series.

Reported-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Fixes: ee8f537fd8b71c ("ASoC: soc-core: remove legacy style of codec_conf")
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
 sound/soc/intel/boards/cml_rt1011_rt5682.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Randy Dunlap Dec. 17, 2019, 9:49 p.m. UTC | #1
On 12/17/19 8:56 AM, Pierre-Louis Bossart wrote:
> Now that the legacy style is removed, we have to use the new macros
> for the codec configuration. This change was missed in the initial
> series.
> 
> Reported-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> Fixes: ee8f537fd8b71c ("ASoC: soc-core: remove legacy style of codec_conf")
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

Acked-by: Randy Dunlap <rdunlap@infradead.org> # build-tested

Thanks.

> ---
>  sound/soc/intel/boards/cml_rt1011_rt5682.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/sound/soc/intel/boards/cml_rt1011_rt5682.c b/sound/soc/intel/boards/cml_rt1011_rt5682.c
> index a22f97234201..ab1196108d23 100644
> --- a/sound/soc/intel/boards/cml_rt1011_rt5682.c
> +++ b/sound/soc/intel/boards/cml_rt1011_rt5682.c
> @@ -406,19 +406,19 @@ static struct snd_soc_dai_link cml_rt1011_rt5682_dailink[] = {
>  
>  static struct snd_soc_codec_conf rt1011_conf[] = {
>  	{
> -		.dev_name = "i2c-10EC1011:00",
> +		.dlc = COMP_CODEC_CONF("i2c-10EC1011:00"),
>  		.name_prefix = "WL",
>  	},
>  	{
> -		.dev_name = "i2c-10EC1011:01",
> +		.dlc = COMP_CODEC_CONF("i2c-10EC1011:01"),
>  		.name_prefix = "WR",
>  	},
>  	{
> -		.dev_name = "i2c-10EC1011:02",
> +		.dlc = COMP_CODEC_CONF("i2c-10EC1011:02"),
>  		.name_prefix = "TL",
>  	},
>  	{
> -		.dev_name = "i2c-10EC1011:03",
> +		.dlc = COMP_CODEC_CONF("i2c-10EC1011:03"),
>  		.name_prefix = "TR",
>  	},
>  };
>
Kuninori Morimoto Dec. 18, 2019, 1:53 a.m. UTC | #2
Hi Pierre-Louis

> Now that the legacy style is removed, we have to use the new macros
> for the codec configuration. This change was missed in the initial
> series.
> 
> Reported-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> Fixes: ee8f537fd8b71c ("ASoC: soc-core: remove legacy style of codec_conf")
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> ---

Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Thank you for your help !!
Best regards
---
Kuninori Morimoto
diff mbox series

Patch

diff --git a/sound/soc/intel/boards/cml_rt1011_rt5682.c b/sound/soc/intel/boards/cml_rt1011_rt5682.c
index a22f97234201..ab1196108d23 100644
--- a/sound/soc/intel/boards/cml_rt1011_rt5682.c
+++ b/sound/soc/intel/boards/cml_rt1011_rt5682.c
@@ -406,19 +406,19 @@  static struct snd_soc_dai_link cml_rt1011_rt5682_dailink[] = {
 
 static struct snd_soc_codec_conf rt1011_conf[] = {
 	{
-		.dev_name = "i2c-10EC1011:00",
+		.dlc = COMP_CODEC_CONF("i2c-10EC1011:00"),
 		.name_prefix = "WL",
 	},
 	{
-		.dev_name = "i2c-10EC1011:01",
+		.dlc = COMP_CODEC_CONF("i2c-10EC1011:01"),
 		.name_prefix = "WR",
 	},
 	{
-		.dev_name = "i2c-10EC1011:02",
+		.dlc = COMP_CODEC_CONF("i2c-10EC1011:02"),
 		.name_prefix = "TL",
 	},
 	{
-		.dev_name = "i2c-10EC1011:03",
+		.dlc = COMP_CODEC_CONF("i2c-10EC1011:03"),
 		.name_prefix = "TR",
 	},
 };