ASoC: Intel: kbl_da7219_max98927: Fix kabylake_ssp_fixup function
authorLukasz Majczak <lma@semihalf.com>
Thu, 15 Apr 2021 12:43:47 +0000 (14:43 +0200)
committerMark Brown <broonie@kernel.org>
Thu, 15 Apr 2021 17:01:05 +0000 (18:01 +0100)
commita523ef731ac6674dc07574f31bf44cc5bfa14e4d
tree3f2cf1c9c62f2fc26a09bfd8223c16e3187d8dd5
parent858066864a6383d1eecd2fa96a0b8e69935632f8
ASoC: Intel: kbl_da7219_max98927: Fix kabylake_ssp_fixup function

kabylake_ssp_fixup function uses snd_soc_dpcm to identify the
codecs DAIs. The HW parameters are changed based on the codec DAI of the
stream. The earlier approach to get snd_soc_dpcm was using container_of()
macro on snd_pcm_hw_params.

The structures have been modified over time and snd_soc_dpcm does not have
snd_pcm_hw_params as a reference but as a copy. This causes the current
driver to crash when used.

This patch changes the way snd_soc_dpcm is extracted. snd_soc_pcm_runtime
holds 2 dpcm instances (one for playback and one for capture). 2 codecs
on the SSP are dmic (capture) and speakers (playback). Based on the
stream direction, snd_soc_dpcm is extracted from snd_soc_pcm_runtime.

Tested for all use cases of the driver.
Based on similar fix in kbl_rt5663_rt5514_max98927.c
from Harsha Priya <harshapriya.n@intel.com> and
Vamshi Krishna Gopal <vamshi.krishna.gopal@intel.com>

Cc: <stable@vger.kernel.org> # 5.4+
Signed-off-by: Lukasz Majczak <lma@semihalf.com>
Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20210415124347.475432-1-lma@semihalf.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/boards/kbl_da7219_max98927.c