Commit 62201937 authored by Nicolin Chen's avatar Nicolin Chen Committed by Mark Brown

ASoC: cs53l30: Fix a bug for TDM slot location validation

The maximum slot number of CS53L30 is 4 while it should support
the situation that's less than 4 channels based on the rx_mask.

So when the driver validates the last slot location, it should
check the last active slot instead of always the 4th one.
Signed-off-by: default avatarNicolin Chen <nicoleotsuka@gmail.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 05f33bc5
...@@ -809,8 +809,8 @@ static int cs53l30_set_dai_tdm_slot(struct snd_soc_dai *dai, ...@@ -809,8 +809,8 @@ static int cs53l30_set_dai_tdm_slot(struct snd_soc_dai *dai,
return -EINVAL; return -EINVAL;
} }
/* Validate the last CS53L30 slot */ /* Validate the last active CS53L30 slot */
slot_next = loc[CS53L30_TDM_SLOT_MAX - 1] + slot_step - 1; slot_next = loc[i - 1] + slot_step - 1;
if (slot_next > 47) { if (slot_next > 47) {
dev_err(dai->dev, "slot selection out of bounds: %u\n", dev_err(dai->dev, "slot selection out of bounds: %u\n",
slot_next); slot_next);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment