ASoC: SOF: Intel: add min/max channels for SSP on Baytrail/Broadwell
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Fri, 17 Apr 2020 17:20:14 +0000 (12:20 -0500)
committerMark Brown <broonie@kernel.org>
Fri, 17 Apr 2020 18:37:15 +0000 (19:37 +0100)
commit8c05246c0b58cbe80580ea4be05f6d51228af8a9
treea3a564e647a6ba929e16178c5dd91cdaa6af5509
parente2bcb65782f91390952e849e21b82ed7cb05697f
ASoC: SOF: Intel: add min/max channels for SSP on Baytrail/Broadwell

Major regressions were detected by SOF CI on CherryTrail and Broadwell:

[   25.705750]  SSP2-Codec: ASoC: no backend playback stream
[   27.923378]  SSP2-Codec: ASoC: no users playback at close - state

This is root-caused to the introduction of the DAI capability checks
with snd_soc_dai_stream_valid(). Its use in soc-pcm.c makes it a
requirement for all DAIs to report at least a non-zero min_channels
field.

For some reason the SSP structures used for SKL+ did provide this
information but legacy platforms didn't.

Fixes: 9b5db059366ae2 ("ASoC: soc-pcm: dpcm: Only allow playback/capture if supported")
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Link: https://lore.kernel.org/r/20200417172014.11760-1-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sof/intel/bdw.c
sound/soc/sof/intel/byt.c