[PATCH] ASoC: Intel: broadwell: Revert SSP0 link to use dummy components

Pierre-Louis Bossart pierre-louis.bossart at linux.intel.com
Thu Mar 19 23:20:32 CET 2020



On 3/19/20 4:12 PM, Cezary Rojewski wrote:
> Commit:
> "ASoC: Intel: broadwell: change cpu_dai and platform components for SOF

checkpatch.pl probably reports the format doesn't follow conventions?

> 
> while enabling bdw on SOF, changed behavior for non-SOF solutions. In
> essence replacing platform 'dummy' with actual 'platform' causes
> redundant stream initialization to occur during audio start. hw_params
> for haswell-pcm destroys initial stream right after its creation - only
> to recreate it again from proceed from there.
> 
> While harmless so far, this flow ain't right and should be corrected.

isn't

> The actual need for dummy components for SSP0 link is questionable but
> that issue is subject for another series.

If we want to be consistent, the same use of a dummy platform component 
needs to be applied to bdw-rt5677.c and bdw-rt5650.c

> 
> Link to first message in conversation:
> https://lkml.org/lkml/2020/3/18/54
> 
> Reported-by: Dominik Brodowski <linux at dominikbrodowski.net>
> Fixes: 64df6afa0dab ("ASoC: Intel: broadwell: change cpu_dai and platform components for SOF")
> Signed-off-by: Cezary Rojewski <cezary.rojewski at intel.com>
> ---
>   sound/soc/intel/boards/broadwell.c | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/sound/soc/intel/boards/broadwell.c b/sound/soc/intel/boards/broadwell.c
> index 0776ea2d4f36..9b639475e99f 100644
> --- a/sound/soc/intel/boards/broadwell.c
> +++ b/sound/soc/intel/boards/broadwell.c
> @@ -167,9 +167,6 @@ SND_SOC_DAILINK_DEF(codec,
>   #if IS_ENABLED(CONFIG_SND_SOC_SOF_BROADWELL)
>   SND_SOC_DAILINK_DEF(ssp0_port,
>   	    DAILINK_COMP_ARRAY(COMP_CPU("ssp0-port")));
> -#else
> -SND_SOC_DAILINK_DEF(ssp0_port,
> -	    DAILINK_COMP_ARRAY(COMP_DUMMY()));
>   #endif
>   
>   /* broadwell digital audio interface glue - connects codec <--> CPU */
> @@ -225,7 +222,11 @@ static struct snd_soc_dai_link broadwell_rt286_dais[] = {
>   		.ops = &broadwell_rt286_ops,
>   		.dpcm_playback = 1,
>   		.dpcm_capture = 1,
> +#if !IS_ENABLED(CONFIG_SND_SOC_SOF_BROADWELL)
> +		SND_SOC_DAILINK_REG(dummy, codec, dummy),
> +#else
>   		SND_SOC_DAILINK_REG(ssp0_port, codec, platform),
> +#endif
>   	},
>   };
>   
> 


More information about the Alsa-devel mailing list