[RFC PATCH 2/5] ASoC: SOF: Intel: hda: add helper to initialize SoundWire IP

From: Pierre-Louis Bossart
Date: Wed Aug 21 2019 - 16:17:52 EST


The helper calls the SoundWire initializations and sets the resource
structure defined as an interface with the platform driver. The
resource deals with PCI base address, interrupts and the callback for
the stream configuration.

The interrupts are gated/ungated at the top-level with the
ADSPIPC2.SNDW field

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>
---
sound/soc/sof/intel/hda.c | 63 +++++++++++++++++++++++++++++++++++++++
sound/soc/sof/intel/hda.h | 1 +
2 files changed, 64 insertions(+)

diff --git a/sound/soc/sof/intel/hda.c b/sound/soc/sof/intel/hda.c
index c72e9a09eee1..a968890d0754 100644
--- a/sound/soc/sof/intel/hda.c
+++ b/sound/soc/sof/intel/hda.c
@@ -18,7 +18,9 @@
#include <sound/hdaudio_ext.h>
#include <sound/hda_register.h>

+#include <linux/acpi.h>
#include <linux/module.h>
+#include <linux/soundwire/sdw_intel.h>
#include <sound/intel-nhlt.h>
#include <sound/sof.h>
#include <sound/sof/xtensa.h>
@@ -35,6 +37,67 @@
#define IS_CFL(pci) ((pci)->vendor == 0x8086 && (pci)->device == 0xa348)
#define IS_CNL(pci) ((pci)->vendor == 0x8086 && (pci)->device == 0x9dc8)

+#if IS_ENABLED(CONFIG_SOUNDWIRE_INTEL)
+
+static void hda_sdw_int_enable(struct snd_sof_dev *sdev, bool enable)
+{
+ if (enable)
+ snd_sof_dsp_update_bits(sdev, HDA_DSP_BAR,
+ HDA_DSP_REG_ADSPIC2,
+ HDA_DSP_ADSPIC2_SNDW,
+ HDA_DSP_ADSPIC2_SNDW);
+ else
+ snd_sof_dsp_update_bits(sdev, HDA_DSP_BAR,
+ HDA_DSP_REG_ADSPIC2,
+ HDA_DSP_ADSPIC2_SNDW,
+ 0);
+}
+
+static int hda_sdw_init(struct snd_sof_dev *sdev)
+{
+ acpi_handle handle;
+ struct sdw_intel_res res;
+
+ handle = ACPI_HANDLE(sdev->dev);
+
+ memset(&res, 0, sizeof(res));
+
+ res.mmio_base = sdev->bar[HDA_DSP_BAR];
+ res.irq = sdev->ipc_irq;
+ res.parent = sdev->dev;
+
+ hda_sdw_int_enable(sdev, true);
+
+ sdev->sdw = sdw_intel_init(handle, &res);
+ if (!sdev->sdw) {
+ dev_err(sdev->dev, "SDW Init failed\n");
+ return -EIO;
+ }
+
+ return 0;
+}
+
+static int hda_sdw_exit(struct snd_sof_dev *sdev)
+{
+ hda_sdw_int_enable(sdev, false);
+
+ if (sdev->sdw)
+ sdw_intel_exit(sdev->sdw);
+
+ return 0;
+}
+#else
+static int hda_sdw_init(struct snd_sof_dev *sdev)
+{
+ return 0;
+}
+
+static int hda_sdw_exit(struct snd_sof_dev *sdev)
+{
+ return 0;
+}
+#endif
+
/*
* Debug
*/
diff --git a/sound/soc/sof/intel/hda.h b/sound/soc/sof/intel/hda.h
index 5591841a1b6f..c8f93317aeb4 100644
--- a/sound/soc/sof/intel/hda.h
+++ b/sound/soc/sof/intel/hda.h
@@ -211,6 +211,7 @@

#define HDA_DSP_ADSPIC_IPC 1
#define HDA_DSP_ADSPIS_IPC 1
+#define HDA_DSP_ADSPIC2_SNDW BIT(5)

/* Intel HD Audio General DSP Registers */
#define HDA_DSP_GEN_BASE 0x0
--
2.20.1