[alsa-devel] [PATCH] ASoC: Intel: bxt_da7219_max98357a: update dai_link platform name

[Pierre-Louis Bossart]

On 10/28/19 10:35 PM, mac.chiang at intel.com wrote:
> From: Mac Chiang <mac.chiang at intel.com>
> 
> redefine soc platform name "0000:00:1f.3" if cml core

we already have code that can update the platform name:

	/* override plaform name, if required */
	mach = (&pdev->dev)->platform_data;
	platform_name = mach->mach_params.platform;

	ret = snd_soc_fixup_dai_links_platform_name(&broxton_audio_card,
						    platform_name);

Are you sure this additional assignment is required?

> 
> Signed-off-by: Mac Chiang <mac.chiang at intel.com>
> Signed-off-by: Sathya Prakash M R <sathya.prakash.m.r at intel.com>
> ---
>   sound/soc/intel/boards/bxt_da7219_max98357a.c | 8 ++++++++
>   1 file changed, 8 insertions(+)
> 
> diff --git a/sound/soc/intel/boards/bxt_da7219_max98357a.c b/sound/soc/intel/boards/bxt_da7219_max98357a.c
> index ac1dea5..38b7dbe 100644
> --- a/sound/soc/intel/boards/bxt_da7219_max98357a.c
> +++ b/sound/soc/intel/boards/bxt_da7219_max98357a.c
> @@ -40,6 +40,13 @@ struct bxt_card_private {
>   	struct list_head hdmi_pcm_list;
>   };
>   
> +static struct snd_soc_dai_link_component platform_component[] = {
> +	{
> +		/* name might be overridden during probe */
> +		.name = "0000:00:1f.3"
> +	}
> +};
> +
>   enum {
>   	BXT_DPCM_AUDIO_PB = 0,
>   	BXT_DPCM_AUDIO_CP,
> @@ -708,6 +715,7 @@ static int broxton_audio_probe(struct platform_device *pdev)
>   				broxton_dais[i].name = "SSP0-Codec";
>   				broxton_dais[i].cpus->dai_name = "SSP0 Pin";
>   			}
> +			broxton_dais[i].platforms = platform_component;
>   		}
>   	}
>   
>