ASoC: SOF: inform DSP that driver is going to be removed
authorMarcin Rajwa <marcin.rajwa@linux.intel.com>
Fri, 15 May 2020 13:59:53 +0000 (16:59 +0300)
committerMark Brown <broonie@kernel.org>
Mon, 18 May 2020 12:24:55 +0000 (13:24 +0100)
This patch invokes the DSP power down notifier to inform
DSP that driver is going to be removed.
The module removal entails DSP power down, disabling of
IRQs and more. Therefore it is highly recommended to inform
the DSP about this upcoming event.

Due to hardware limitations on some Intel platforms it is necessary
to power gate all LPSRAM banks that were enabled prior to controller
reset. Otherwise, an attempt to write LPSRAM control registers may
have no effect.

Signed-off-by: Marcin Rajwa <marcin.rajwa@linux.intel.com>
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Link: https://lore.kernel.org/r/20200515135958.17511-4-kai.vehmanen@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sof/core.c

index ef9be4f45e27cc28cc353c717bb6d5c8ac418a1d..339c4930b0c0a77d0c613a4d3b41ee982bd2a66b 100644 (file)
@@ -343,6 +343,12 @@ int snd_sof_device_remove(struct device *dev)
 {
        struct snd_sof_dev *sdev = dev_get_drvdata(dev);
        struct snd_sof_pdata *pdata = sdev->pdata;
+       int ret;
+
+       ret = snd_sof_dsp_power_down_notify(sdev);
+       if (ret < 0)
+               dev_warn(dev, "error: %d failed to prepare DSP for device removal",
+                        ret);
 
        if (IS_ENABLED(CONFIG_SND_SOC_SOF_PROBE_WORK_QUEUE))
                cancel_work_sync(&sdev->probe_work);