]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
ASoC: SOF: Do not send cmd via SHIM register
authorDaniel Baluta <daniel.baluta@gmail.com>
Mon, 3 Jun 2019 16:20:25 +0000 (11:20 -0500)
committerMark Brown <broonie@kernel.org>
Mon, 3 Jun 2019 16:47:49 +0000 (17:47 +0100)
We use shim registers only to notify the other
side that a message was sent. The actual information
for the message is transmitted via mailbox.

cmd information inside shim register is not used by
the DSP, so we remove it to avoid confusion.

Signed-off-by: Daniel Baluta <daniel.baluta@gmail.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sof/intel/byt.c
sound/soc/sof/intel/cnl.c
sound/soc/sof/intel/hda-ipc.c

index 9e4c07eb889b63db5fc2ad2995d5a1a0d04c2a08..8765eedfcf54b366cae51bde96020c74581efbe9 100644 (file)
@@ -373,13 +373,10 @@ static irqreturn_t byt_irq_thread(int irq, void *context)
 
 static int byt_send_msg(struct snd_sof_dev *sdev, struct snd_sof_ipc_msg *msg)
 {
-       u64 cmd = msg->header;
-
        /* send the message */
        sof_mailbox_write(sdev, sdev->host_box.offset, msg->msg_data,
                          msg->msg_size);
-       snd_sof_dsp_write64(sdev, BYT_DSP_BAR, SHIM_IPCX,
-                           cmd | SHIM_BYT_IPCX_BUSY);
+       snd_sof_dsp_write64(sdev, BYT_DSP_BAR, SHIM_IPCX, SHIM_BYT_IPCX_BUSY);
 
        return 0;
 }
index e59d180da7e283dca140d9eb67d93689778a70ca..fd86269b503760aa989e1d994f699d8fc3ae249f 100644 (file)
@@ -151,13 +151,11 @@ static void cnl_ipc_dsp_done(struct snd_sof_dev *sdev)
 static int cnl_ipc_send_msg(struct snd_sof_dev *sdev,
                            struct snd_sof_ipc_msg *msg)
 {
-       u32 cmd = msg->header;
-
        /* send the message */
        sof_mailbox_write(sdev, sdev->host_box.offset, msg->msg_data,
                          msg->msg_size);
        snd_sof_dsp_write(sdev, HDA_DSP_BAR, CNL_DSP_REG_HIPCIDR,
-                         cmd | CNL_DSP_REG_HIPCIDR_BUSY);
+                         CNL_DSP_REG_HIPCIDR_BUSY);
 
        return 0;
 }
index 51b285103394ef31b88831a1cb6ba4ef019ad699..728dd1e075a4e086e4c6946e1cc627f671b61428 100644 (file)
@@ -56,13 +56,11 @@ static void hda_dsp_ipc_dsp_done(struct snd_sof_dev *sdev)
 
 int hda_dsp_ipc_send_msg(struct snd_sof_dev *sdev, struct snd_sof_ipc_msg *msg)
 {
-       u32 cmd = msg->header;
-
        /* send IPC message to DSP */
        sof_mailbox_write(sdev, sdev->host_box.offset, msg->msg_data,
                          msg->msg_size);
        snd_sof_dsp_write(sdev, HDA_DSP_BAR, HDA_DSP_REG_HIPCI,
-                         cmd | HDA_DSP_REG_HIPCI_BUSY);
+                         HDA_DSP_REG_HIPCI_BUSY);
 
        return 0;
 }