ASoC: Intel: atom: Check drv->lock is locked in sst_fill_and_send_cmd_unlocked
authorHans de Goede <hdegoede@redhat.com>
Thu, 2 Apr 2020 18:53:58 +0000 (20:53 +0200)
committerMark Brown <broonie@kernel.org>
Fri, 3 Apr 2020 13:39:57 +0000 (14:39 +0100)
sst_fill_and_send_cmd_unlocked must be called with the drv->lock mutex
locked already. In the past there have been cases where this was not the
case, add a WARN_ON to check for drv->lock being locked.

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-2-hdegoede@redhat.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/atom/sst-atom-controls.c

index df8f7994d3b7a7eb7d04b404e57243acf875708c..69f3af4524abb104c634b8e06511c158a4de48e1 100644 (file)
@@ -50,6 +50,8 @@ static int sst_fill_and_send_cmd_unlocked(struct sst_data *drv,
 {
        int ret = 0;
 
+       WARN_ON(!mutex_is_locked(&drv->lock));
+
        ret = sst_fill_byte_control(drv, ipc_msg,
                                block, task_id, pipe_id, len, cmd_data);
        if (ret < 0)