summaryrefslogtreecommitdiffstats
path: root/sound
diff options
context:
space:
mode:
authorKeyon Jie <yang.jie@linux.intel.com>2019-04-30 18:09:28 -0500
committerMark Brown <broonie@kernel.org>2019-05-03 15:00:32 +0900
commit6d60a39e5b787ddea5def220d6288cbdf6d63eb8 (patch)
tree0cd1b7077a51814ebc7d8ee712b08da8b9bdd1a7 /sound
parentff758e9f73a0fcb93a53349e8fbace4ac019d031 (diff)
downloadlinux-6d60a39e5b787ddea5def220d6288cbdf6d63eb8.tar.gz
linux-6d60a39e5b787ddea5def220d6288cbdf6d63eb8.tar.bz2
linux-6d60a39e5b787ddea5def220d6288cbdf6d63eb8.zip
ASoC: SOF: Intel: hda-ipc: simplify handling of IPC IRQ
When using a shared IRQ between IPC interrupt and stream IOC interrupt, the interrupt handlers need to check the interrupt source before scheduling their respective IRQ threads. In the case of IPC handler, it should check if it is an IPC interrupt before waking up the IPC IRQ thread. The IPC IRQ thread, once scheduled, does not need to check the IRQ source again. So, remove the superfluous check in the thread. Remove the irq_status field from snd_sof_dev struct also as it is no longer needed. Signed-off-by: Keyon Jie <yang.jie@linux.intel.com> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
-rw-r--r--sound/soc/sof/intel/cnl.c4
-rw-r--r--sound/soc/sof/intel/hda-ipc.c13
-rw-r--r--sound/soc/sof/sof-priv.h1
3 files changed, 5 insertions, 13 deletions
diff --git a/sound/soc/sof/intel/cnl.c b/sound/soc/sof/intel/cnl.c
index 3afd96d9c925..d128839b2450 100644
--- a/sound/soc/sof/intel/cnl.c
+++ b/sound/soc/sof/intel/cnl.c
@@ -39,10 +39,6 @@ static irqreturn_t cnl_ipc_irq_thread(int irq, void *context)
u32 msg_ext;
irqreturn_t ret = IRQ_NONE;
- /* here we handle IPC interrupts only */
- if (!(sdev->irq_status & HDA_DSP_ADSPIS_IPC))
- return ret;
-
hipcida = snd_sof_dsp_read(sdev, HDA_DSP_BAR, CNL_DSP_REG_HIPCIDA);
hipcctl = snd_sof_dsp_read(sdev, HDA_DSP_BAR, CNL_DSP_REG_HIPCCTL);
hipctdr = snd_sof_dsp_read(sdev, HDA_DSP_BAR, CNL_DSP_REG_HIPCTDR);
diff --git a/sound/soc/sof/intel/hda-ipc.c b/sound/soc/sof/intel/hda-ipc.c
index a938f568dbb1..73ead7070cde 100644
--- a/sound/soc/sof/intel/hda-ipc.c
+++ b/sound/soc/sof/intel/hda-ipc.c
@@ -145,10 +145,6 @@ irqreturn_t hda_dsp_ipc_irq_thread(int irq, void *context)
u32 msg;
u32 msg_ext;
- /* here we handle IPC interrupts only */
- if (!(sdev->irq_status & HDA_DSP_ADSPIS_IPC))
- return IRQ_NONE;
-
/* read IPC status */
hipcie = snd_sof_dsp_read(sdev, HDA_DSP_BAR,
HDA_DSP_REG_HIPCIE);
@@ -234,19 +230,20 @@ irqreturn_t hda_dsp_ipc_irq_handler(int irq, void *context)
{
struct snd_sof_dev *sdev = context;
int ret = IRQ_NONE;
+ u32 irq_status;
spin_lock(&sdev->hw_lock);
/* store status */
- sdev->irq_status = snd_sof_dsp_read(sdev, HDA_DSP_BAR,
- HDA_DSP_REG_ADSPIS);
+ irq_status = snd_sof_dsp_read(sdev, HDA_DSP_BAR, HDA_DSP_REG_ADSPIS);
+ dev_vdbg(sdev->dev, "irq handler: irq_status:0x%x\n", irq_status);
/* invalid message ? */
- if (sdev->irq_status == 0xffffffff)
+ if (irq_status == 0xffffffff)
goto out;
/* IPC message ? */
- if (sdev->irq_status & HDA_DSP_ADSPIS_IPC) {
+ if (irq_status & HDA_DSP_ADSPIS_IPC) {
/* disable IPC interrupt */
snd_sof_dsp_update_bits_unlocked(sdev, HDA_DSP_BAR,
HDA_DSP_REG_ADSPIC,
diff --git a/sound/soc/sof/sof-priv.h b/sound/soc/sof/sof-priv.h
index 675bb10c82f5..bbc285018f9a 100644
--- a/sound/soc/sof/sof-priv.h
+++ b/sound/soc/sof/sof-priv.h
@@ -366,7 +366,6 @@ struct snd_sof_dev {
struct snd_sof_mailbox host_box; /* Host initiated IPC */
struct snd_sof_mailbox stream_box; /* Stream position update */
struct snd_sof_ipc_msg *msg;
- u64 irq_status;
int ipc_irq;
u32 next_comp_id; /* monotonic - reset during S3 */