michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers/dai/intel/ssp: Skip setting config when playback/capture is active

Browse source 
The reference count, dp->sref, is never incremented for the SSP DAI as
we use the runtime PM API's to probe/remove the DAI during
resume/suspend. Therefore, use the DAI state to check if the DAI is
active in any direction before proceesing to set the config. This fixes
the IO errors seen during simultaenous playback and capture on the same
SSP.

Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
This commit is contained in:
Ranjani Sridharan 2022-11-07 21:38:48 -08:00 • committed by Fabio Baltieri
commit 5f9c20db5a
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
drivers/dai/intel/ssp/ssp.c
View file

@ -1541,9 +1541,9 @@ static int dai_ssp_set_config_blob(struct dai_intel_ssp *dp, const struct dai_co
uint32_t ssc0, sstsa, ssrsa;
/* set config only once for playback or capture */
if (dp->sref > 1) {
if (ssp->state[DAI_DIR_PLAYBACK] > DAI_STATE_READY ||
ssp->state[DAI_DIR_CAPTURE] > DAI_STATE_READY)
return 0;
}
ssc0 = blob->i2s_driver_config.i2s_config.ssc0;
sstsa = blob->i2s_driver_config.i2s_config.sstsa;