]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commit
ASoC: Intel: atom: Take the drv->lock mutex before calling sst_send_slot_map()
authorHans de Goede <hdegoede@redhat.com>
Thu, 2 Apr 2020 18:53:57 +0000 (20:53 +0200)
committerStefan Bader <stefan.bader@canonical.com>
Thu, 14 May 2020 08:54:06 +0000 (10:54 +0200)
commitc1a90891dd9498aa6e1fb1edce69f23ac929ca28
tree812af1d5d2b8c88849b762b4c00c060442a61066
parent2e9ada6a1dc2aed6d6ef4acd63f8ad051588c36d
ASoC: Intel: atom: Take the drv->lock mutex before calling sst_send_slot_map()

BugLink: https://bugs.launchpad.net/bugs/1877461
[ Upstream commit 81630dc042af998b9f58cd8e2c29dab9777ea176 ]

sst_send_slot_map() uses sst_fill_and_send_cmd_unlocked() because in some
places it is called with the drv->lock mutex already held.

So it must always be called with the mutex locked. This commit adds missing
locking in the sst_set_be_modules() code-path.

Fixes: 24c8d14192cc ("ASoC: Intel: mrfld: add DSP core controls")
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20200402185359.3424-1-hdegoede@redhat.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
sound/soc/intel/atom/sst-atom-controls.c