mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-27 21:03:32 +00:00
ASoC: wm_adsp: Fix missing locking in wm_adsp_[read|write]_ctl()
wm_adsp_read_ctl() and wm_adsp_write_ctl() must hold the cs_dsp pwr_lock mutex when calling cs_dsp_coeff_read_ctrl() and cs_dsp_coeff_write_ctrl(). Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com> Link: https://lore.kernel.org/r/20230913160250.3700346-1-rf@opensource.cirrus.com Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
69343ce914
commit
781118bc2f
1 changed files with 11 additions and 2 deletions
|
@ -687,7 +687,10 @@ int wm_adsp_write_ctl(struct wm_adsp *dsp, const char *name, int type,
|
||||||
struct wm_coeff_ctl *ctl;
|
struct wm_coeff_ctl *ctl;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
|
mutex_lock(&dsp->cs_dsp.pwr_lock);
|
||||||
ret = cs_dsp_coeff_write_ctrl(cs_ctl, 0, buf, len);
|
ret = cs_dsp_coeff_write_ctrl(cs_ctl, 0, buf, len);
|
||||||
|
mutex_unlock(&dsp->cs_dsp.pwr_lock);
|
||||||
|
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
return ret;
|
return ret;
|
||||||
|
|
||||||
|
@ -703,8 +706,14 @@ EXPORT_SYMBOL_GPL(wm_adsp_write_ctl);
|
||||||
int wm_adsp_read_ctl(struct wm_adsp *dsp, const char *name, int type,
|
int wm_adsp_read_ctl(struct wm_adsp *dsp, const char *name, int type,
|
||||||
unsigned int alg, void *buf, size_t len)
|
unsigned int alg, void *buf, size_t len)
|
||||||
{
|
{
|
||||||
return cs_dsp_coeff_read_ctrl(cs_dsp_get_ctl(&dsp->cs_dsp, name, type, alg),
|
int ret;
|
||||||
0, buf, len);
|
|
||||||
|
mutex_lock(&dsp->cs_dsp.pwr_lock);
|
||||||
|
ret = cs_dsp_coeff_read_ctrl(cs_dsp_get_ctl(&dsp->cs_dsp, name, type, alg),
|
||||||
|
0, buf, len);
|
||||||
|
mutex_unlock(&dsp->cs_dsp.pwr_lock);
|
||||||
|
|
||||||
|
return ret;
|
||||||
}
|
}
|
||||||
EXPORT_SYMBOL_GPL(wm_adsp_read_ctl);
|
EXPORT_SYMBOL_GPL(wm_adsp_read_ctl);
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue