diff options
| author | Charles Keepax <ckeepax@opensource.cirrus.com> | 2022-06-23 12:51:19 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2022-06-24 17:22:01 +0200 |
| commit | 980555e95f7cabdc9c80a07107622b097ba23703 (patch) | |
| tree | 0c121bf1ddc463b12d24572f9e1bba652f4192d1 /sound | |
| parent | ASoC: madera: Fix event generation for OUT1 demux (diff) | |
| download | linux-980555e95f7cabdc9c80a07107622b097ba23703.tar.xz linux-980555e95f7cabdc9c80a07107622b097ba23703.zip | |
ASoC: madera: Fix event generation for rate controls
madera_adsp_rate_put always returns zero regardless of if the control
value was updated. This results in missing notifications to user-space
of the control change. Update the handling to return 1 when the
value is changed.
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20220623105120.1981154-5-ckeepax@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
| -rw-r--r-- | sound/soc/codecs/madera.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sound/soc/codecs/madera.c b/sound/soc/codecs/madera.c index 8095a87117cf..b9f19fbd2911 100644 --- a/sound/soc/codecs/madera.c +++ b/sound/soc/codecs/madera.c @@ -899,7 +899,7 @@ static int madera_adsp_rate_put(struct snd_kcontrol *kcontrol, struct soc_enum *e = (struct soc_enum *)kcontrol->private_value; const int adsp_num = e->shift_l; const unsigned int item = ucontrol->value.enumerated.item[0]; - int ret; + int ret = 0; if (item >= e->items) return -EINVAL; @@ -916,10 +916,10 @@ static int madera_adsp_rate_put(struct snd_kcontrol *kcontrol, "Cannot change '%s' while in use by active audio paths\n", kcontrol->id.name); ret = -EBUSY; - } else { + } else if (priv->adsp_rate_cache[adsp_num] != e->values[item]) { /* Volatile register so defer until the codec is powered up */ priv->adsp_rate_cache[adsp_num] = e->values[item]; - ret = 0; + ret = 1; } mutex_unlock(&priv->rate_lock); |