Re: [PATCH] ASoC: Intel: cml_rt1011_rt5682: fix codec_conf by removing legacy style

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxx>
> Cc: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> Fixes: ee8f537fd8b71c ("ASoC: soc-core: remove legacy style of codec_conf")
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>

Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> # 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",
>  	},
>  };
> 


-- 
~Randy

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux