* [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections
@ 2021-03-22 16:37 Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 1/8] ASoC: SOF: Intel: HDA: fix core status verification Pierre-Louis Bossart
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: Pierre-Louis Bossart @ 2021-03-22 16:37 UTC (permalink / raw)
To: alsa-devel; +Cc: tiwai, broonie, Pierre-Louis Bossart
This patchset fixes a logical flow in the core status, improves
shutdown support on Intel SOF platforms with an HDaudio controller and
cleans-up ElkhartLake support.
Guennadi Liakhovetski (1):
ASoC: SOF: Intel: HDA: fix core status verification
Libin Yang (5):
ASoC: SOF: Intel: TGL: fix EHL ops
ASoC: SOF: Intel: TGL: set shutdown callback to hda_dsp_shutdown
ASoC: SOF: Intel: ICL: set shutdown callback to hda_dsp_shutdown
ASoC: SOF: Intel: CNL: set shutdown callback to hda_dsp_shutdown
ASoC: SOF: Intel: APL: set shutdown callback to hda_dsp_shutdown
Pierre-Louis Bossart (2):
ASoC: SOF: core: harden shutdown helper
ASoC: SOF: Intel: move ELH chip info
sound/soc/sof/core.c | 8 +++++++-
sound/soc/sof/intel/apl.c | 3 ++-
sound/soc/sof/intel/cnl.c | 19 ++-----------------
sound/soc/sof/intel/hda-dsp.c | 21 +++++++++++++++++----
sound/soc/sof/intel/hda.h | 1 +
sound/soc/sof/intel/icl.c | 3 ++-
sound/soc/sof/intel/pci-tgl.c | 2 +-
sound/soc/sof/intel/tgl.c | 18 +++++++++++++++++-
8 files changed, 49 insertions(+), 26 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/8] ASoC: SOF: Intel: HDA: fix core status verification
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
@ 2021-03-22 16:37 ` Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 2/8] ASoC: SOF: core: harden shutdown helper Pierre-Louis Bossart
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Pierre-Louis Bossart @ 2021-03-22 16:37 UTC (permalink / raw)
To: alsa-devel
Cc: Guennadi Liakhovetski, Kai Vehmanen, tiwai, Ranjani Sridharan,
Pierre-Louis Bossart, broonie
From: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
When checking for enabled cores it isn't enough to check that
some of the requested cores are running, we have to check that
all of them are.
Fixes: 747503b1813a ("ASoC: SOF: Intel: Add Intel specific HDA DSP HW operations")
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/sof/intel/hda-dsp.c | 15 +++++++++++----
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/sound/soc/sof/intel/hda-dsp.c b/sound/soc/sof/intel/hda-dsp.c
index 6e24e1cb13f9..1f1ce7036764 100644
--- a/sound/soc/sof/intel/hda-dsp.c
+++ b/sound/soc/sof/intel/hda-dsp.c
@@ -226,10 +226,17 @@ bool hda_dsp_core_is_enabled(struct snd_sof_dev *sdev,
val = snd_sof_dsp_read(sdev, HDA_DSP_BAR, HDA_DSP_REG_ADSPCS);
- is_enable = (val & HDA_DSP_ADSPCS_CPA_MASK(core_mask)) &&
- (val & HDA_DSP_ADSPCS_SPA_MASK(core_mask)) &&
- !(val & HDA_DSP_ADSPCS_CRST_MASK(core_mask)) &&
- !(val & HDA_DSP_ADSPCS_CSTALL_MASK(core_mask));
+#define MASK_IS_EQUAL(v, m, field) ({ \
+ u32 _m = field(m); \
+ ((v) & _m) == _m; \
+})
+
+ is_enable = MASK_IS_EQUAL(val, core_mask, HDA_DSP_ADSPCS_CPA_MASK) &&
+ MASK_IS_EQUAL(val, core_mask, HDA_DSP_ADSPCS_SPA_MASK) &&
+ !(val & HDA_DSP_ADSPCS_CRST_MASK(core_mask)) &&
+ !(val & HDA_DSP_ADSPCS_CSTALL_MASK(core_mask));
+
+#undef MASK_IS_EQUAL
dev_dbg(sdev->dev, "DSP core(s) enabled? %d : core_mask %x\n",
is_enable, core_mask);
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/8] ASoC: SOF: core: harden shutdown helper
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 1/8] ASoC: SOF: Intel: HDA: fix core status verification Pierre-Louis Bossart
@ 2021-03-22 16:37 ` Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 3/8] ASoC: SOF: Intel: TGL: fix EHL ops Pierre-Louis Bossart
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Pierre-Louis Bossart @ 2021-03-22 16:37 UTC (permalink / raw)
To: alsa-devel
Cc: Libin Yang, Kai Vehmanen, tiwai, Pierre-Louis Bossart,
Ranjani Sridharan, broonie
When the probe is handled in a workqueue, we must use
cancel_work_sync() in the shutdown helper to avoid possible race
conditions.
We must also take care of possible errors happening in a probe
workqueue or during pm_runtime resume (called e.g. before shutdown for
PCI devices). We should really only try to access hardware registers
and initiate IPCs if the DSP is fully booted.
Fixes: daff7f1478e12 ("ASoC: SOF: add snd_sof_device_shutdown() helper for shutdown")
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Reviewed-by: Libin Yang <libin.yang@intel.com>
---
sound/soc/sof/core.c | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/sound/soc/sof/core.c b/sound/soc/sof/core.c
index 495295a34c3a..7005353602c4 100644
--- a/sound/soc/sof/core.c
+++ b/sound/soc/sof/core.c
@@ -400,7 +400,13 @@ int snd_sof_device_shutdown(struct device *dev)
{
struct snd_sof_dev *sdev = dev_get_drvdata(dev);
- return snd_sof_shutdown(sdev);
+ if (IS_ENABLED(CONFIG_SND_SOC_SOF_PROBE_WORK_QUEUE))
+ cancel_work_sync(&sdev->probe_work);
+
+ if (sdev->fw_state == SOF_FW_BOOT_COMPLETE)
+ return snd_sof_shutdown(sdev);
+
+ return 0;
}
EXPORT_SYMBOL(snd_sof_device_shutdown);
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/8] ASoC: SOF: Intel: TGL: fix EHL ops
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 1/8] ASoC: SOF: Intel: HDA: fix core status verification Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 2/8] ASoC: SOF: core: harden shutdown helper Pierre-Louis Bossart
@ 2021-03-22 16:37 ` Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 4/8] ASoC: SOF: Intel: TGL: set shutdown callback to hda_dsp_shutdown Pierre-Louis Bossart
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Pierre-Louis Bossart @ 2021-03-22 16:37 UTC (permalink / raw)
To: alsa-devel
Cc: tiwai, Libin Yang, broonie, Ranjani Sridharan, Pierre-Louis Bossart
From: Libin Yang <libin.yang@intel.com>
EHL is derived from TGL, not CNL, so we shall use the TGL ops.
Fixes: 8d4ba1be3d22 ("ASoC: SOF: pci: split PCI into different drivers")
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Signed-off-by: Libin Yang <libin.yang@intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/sof/intel/pci-tgl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/sof/intel/pci-tgl.c b/sound/soc/sof/intel/pci-tgl.c
index 485607471181..38bc353f7313 100644
--- a/sound/soc/sof/intel/pci-tgl.c
+++ b/sound/soc/sof/intel/pci-tgl.c
@@ -65,7 +65,7 @@ static const struct sof_dev_desc ehl_desc = {
.default_tplg_path = "intel/sof-tplg",
.default_fw_filename = "sof-ehl.ri",
.nocodec_tplg_filename = "sof-ehl-nocodec.tplg",
- .ops = &sof_cnl_ops,
+ .ops = &sof_tgl_ops,
};
static const struct sof_dev_desc adls_desc = {
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 4/8] ASoC: SOF: Intel: TGL: set shutdown callback to hda_dsp_shutdown
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
` (2 preceding siblings ...)
2021-03-22 16:37 ` [PATCH 3/8] ASoC: SOF: Intel: TGL: fix EHL ops Pierre-Louis Bossart
@ 2021-03-22 16:37 ` Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 5/8] ASoC: SOF: Intel: ICL: " Pierre-Louis Bossart
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Pierre-Louis Bossart @ 2021-03-22 16:37 UTC (permalink / raw)
To: alsa-devel
Cc: Libin Yang, tiwai, Pan Xiuli, Ranjani Sridharan,
Pierre-Louis Bossart, broonie
From: Libin Yang <libin.yang@intel.com>
According to hardware spec and PMC FW requirement, the DSP must be
in D3 state before entering S5. Define the shutdown function to use
snd_sof_suspend as shutdown callback to make sure DSP is in D3 state.
Fixes: 44a4cfad8d78 ("ASoC: SOF: Intel: tgl: do thorough remove at .shutdown() callback")
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
Signed-off-by: Libin Yang <libin.yang@intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/sof/intel/hda-dsp.c | 6 ++++++
sound/soc/sof/intel/hda.h | 1 +
sound/soc/sof/intel/tgl.c | 2 +-
3 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/sound/soc/sof/intel/hda-dsp.c b/sound/soc/sof/intel/hda-dsp.c
index 1f1ce7036764..09a5d9040a90 100644
--- a/sound/soc/sof/intel/hda-dsp.c
+++ b/sound/soc/sof/intel/hda-dsp.c
@@ -896,6 +896,12 @@ int hda_dsp_suspend(struct snd_sof_dev *sdev, u32 target_state)
return snd_sof_dsp_set_power_state(sdev, &target_dsp_state);
}
+int hda_dsp_shutdown(struct snd_sof_dev *sdev)
+{
+ sdev->system_suspend_target = SOF_SUSPEND_S3;
+ return snd_sof_suspend(sdev->dev);
+}
+
int hda_dsp_set_hw_params_upon_resume(struct snd_sof_dev *sdev)
{
#if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
diff --git a/sound/soc/sof/intel/hda.h b/sound/soc/sof/intel/hda.h
index 7c7579daee7f..ae80725b0e33 100644
--- a/sound/soc/sof/intel/hda.h
+++ b/sound/soc/sof/intel/hda.h
@@ -517,6 +517,7 @@ int hda_dsp_resume(struct snd_sof_dev *sdev);
int hda_dsp_runtime_suspend(struct snd_sof_dev *sdev);
int hda_dsp_runtime_resume(struct snd_sof_dev *sdev);
int hda_dsp_runtime_idle(struct snd_sof_dev *sdev);
+int hda_dsp_shutdown(struct snd_sof_dev *sdev);
int hda_dsp_set_hw_params_upon_resume(struct snd_sof_dev *sdev);
void hda_dsp_dump(struct snd_sof_dev *sdev, u32 flags);
void hda_ipc_dump(struct snd_sof_dev *sdev);
diff --git a/sound/soc/sof/intel/tgl.c b/sound/soc/sof/intel/tgl.c
index 419f05ba1920..3e46fac53f78 100644
--- a/sound/soc/sof/intel/tgl.c
+++ b/sound/soc/sof/intel/tgl.c
@@ -25,7 +25,7 @@ const struct snd_sof_dsp_ops sof_tgl_ops = {
/* probe/remove/shutdown */
.probe = hda_dsp_probe,
.remove = hda_dsp_remove,
- .shutdown = hda_dsp_remove,
+ .shutdown = hda_dsp_shutdown,
/* Register IO */
.write = sof_io_write,
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 5/8] ASoC: SOF: Intel: ICL: set shutdown callback to hda_dsp_shutdown
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
` (3 preceding siblings ...)
2021-03-22 16:37 ` [PATCH 4/8] ASoC: SOF: Intel: TGL: set shutdown callback to hda_dsp_shutdown Pierre-Louis Bossart
@ 2021-03-22 16:37 ` Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 6/8] ASoC: SOF: Intel: CNL: " Pierre-Louis Bossart
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Pierre-Louis Bossart @ 2021-03-22 16:37 UTC (permalink / raw)
To: alsa-devel
Cc: tiwai, Libin Yang, broonie, Ranjani Sridharan, Pierre-Louis Bossart
From: Libin Yang <libin.yang@intel.com>
According to hardware spec and PMC FW requirement, the DSP must be
in D3 state before entering S5. Set shutdown call to hda_dsp_shutdown.
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Signed-off-by: Libin Yang <libin.yang@intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/sof/intel/icl.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/sound/soc/sof/intel/icl.c b/sound/soc/sof/intel/icl.c
index e9d5a0a58504..88a74be8a0c1 100644
--- a/sound/soc/sof/intel/icl.c
+++ b/sound/soc/sof/intel/icl.c
@@ -26,9 +26,10 @@ static const struct snd_sof_debugfs_map icl_dsp_debugfs[] = {
/* Icelake ops */
const struct snd_sof_dsp_ops sof_icl_ops = {
- /* probe and remove */
+ /* probe/remove/shutdown */
.probe = hda_dsp_probe,
.remove = hda_dsp_remove,
+ .shutdown = hda_dsp_shutdown,
/* Register IO */
.write = sof_io_write,
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 6/8] ASoC: SOF: Intel: CNL: set shutdown callback to hda_dsp_shutdown
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
` (4 preceding siblings ...)
2021-03-22 16:37 ` [PATCH 5/8] ASoC: SOF: Intel: ICL: " Pierre-Louis Bossart
@ 2021-03-22 16:37 ` Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 7/8] ASoC: SOF: Intel: APL: " Pierre-Louis Bossart
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Pierre-Louis Bossart @ 2021-03-22 16:37 UTC (permalink / raw)
To: alsa-devel
Cc: tiwai, Libin Yang, broonie, Ranjani Sridharan, Pierre-Louis Bossart
From: Libin Yang <libin.yang@intel.com>
According to hardware spec and PMC FW requirement, the DSP must be
in D3 state before entering S5. Set shutdown call to hda_dsp_shutdown.
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Signed-off-by: Libin Yang <libin.yang@intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/sof/intel/cnl.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/sound/soc/sof/intel/cnl.c b/sound/soc/sof/intel/cnl.c
index e38db519f38d..094cde17a1b7 100644
--- a/sound/soc/sof/intel/cnl.c
+++ b/sound/soc/sof/intel/cnl.c
@@ -232,9 +232,10 @@ void cnl_ipc_dump(struct snd_sof_dev *sdev)
/* cannonlake ops */
const struct snd_sof_dsp_ops sof_cnl_ops = {
- /* probe and remove */
+ /* probe/remove/shutdown */
.probe = hda_dsp_probe,
.remove = hda_dsp_remove,
+ .shutdown = hda_dsp_shutdown,
/* Register IO */
.write = sof_io_write,
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 7/8] ASoC: SOF: Intel: APL: set shutdown callback to hda_dsp_shutdown
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
` (5 preceding siblings ...)
2021-03-22 16:37 ` [PATCH 6/8] ASoC: SOF: Intel: CNL: " Pierre-Louis Bossart
@ 2021-03-22 16:37 ` Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 8/8] ASoC: SOF: Intel: move ELH chip info Pierre-Louis Bossart
2021-03-23 22:12 ` [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Mark Brown
8 siblings, 0 replies; 10+ messages in thread
From: Pierre-Louis Bossart @ 2021-03-22 16:37 UTC (permalink / raw)
To: alsa-devel
Cc: tiwai, Libin Yang, broonie, Ranjani Sridharan, Pierre-Louis Bossart
From: Libin Yang <libin.yang@intel.com>
According to hardware spec and PMC FW requirement, the DSP must be
in D3 state before entering S5. Set shutdown call to hda_dsp_shutdown.
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Signed-off-by: Libin Yang <libin.yang@intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/sof/intel/apl.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/sound/soc/sof/intel/apl.c b/sound/soc/sof/intel/apl.c
index fc29b91b8932..c7ed2b3d6abc 100644
--- a/sound/soc/sof/intel/apl.c
+++ b/sound/soc/sof/intel/apl.c
@@ -27,9 +27,10 @@ static const struct snd_sof_debugfs_map apl_dsp_debugfs[] = {
/* apollolake ops */
const struct snd_sof_dsp_ops sof_apl_ops = {
- /* probe and remove */
+ /* probe/remove/shutdown */
.probe = hda_dsp_probe,
.remove = hda_dsp_remove,
+ .shutdown = hda_dsp_shutdown,
/* Register IO */
.write = sof_io_write,
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 8/8] ASoC: SOF: Intel: move ELH chip info
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
` (6 preceding siblings ...)
2021-03-22 16:37 ` [PATCH 7/8] ASoC: SOF: Intel: APL: " Pierre-Louis Bossart
@ 2021-03-22 16:37 ` Pierre-Louis Bossart
2021-03-23 22:12 ` [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Mark Brown
8 siblings, 0 replies; 10+ messages in thread
From: Pierre-Louis Bossart @ 2021-03-22 16:37 UTC (permalink / raw)
To: alsa-devel
Cc: tiwai, broonie, Pierre-Louis Bossart, Kai Vehmanen, Rander Wang
ELH is a derivative of TGL, so it should be exposed in tgl.c for
consistency.
No functional change.
Reviewed-by: Rander Wang <rander.wang@intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/sof/intel/cnl.c | 16 ----------------
sound/soc/sof/intel/tgl.c | 16 ++++++++++++++++
2 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/sound/soc/sof/intel/cnl.c b/sound/soc/sof/intel/cnl.c
index 094cde17a1b7..821f25fbcf08 100644
--- a/sound/soc/sof/intel/cnl.c
+++ b/sound/soc/sof/intel/cnl.c
@@ -350,22 +350,6 @@ const struct sof_intel_dsp_desc cnl_chip_info = {
};
EXPORT_SYMBOL_NS(cnl_chip_info, SND_SOC_SOF_INTEL_HDA_COMMON);
-const struct sof_intel_dsp_desc ehl_chip_info = {
- /* Elkhartlake */
- .cores_num = 4,
- .init_core_mask = 1,
- .host_managed_cores_mask = BIT(0),
- .ipc_req = CNL_DSP_REG_HIPCIDR,
- .ipc_req_mask = CNL_DSP_REG_HIPCIDR_BUSY,
- .ipc_ack = CNL_DSP_REG_HIPCIDA,
- .ipc_ack_mask = CNL_DSP_REG_HIPCIDA_DONE,
- .ipc_ctl = CNL_DSP_REG_HIPCCTL,
- .rom_init_timeout = 300,
- .ssp_count = ICL_SSP_COUNT,
- .ssp_base_offset = CNL_SSP_BASE_OFFSET,
-};
-EXPORT_SYMBOL_NS(ehl_chip_info, SND_SOC_SOF_INTEL_HDA_COMMON);
-
const struct sof_intel_dsp_desc jsl_chip_info = {
/* Jasperlake */
.cores_num = 2,
diff --git a/sound/soc/sof/intel/tgl.c b/sound/soc/sof/intel/tgl.c
index 3e46fac53f78..54ba1b88ba86 100644
--- a/sound/soc/sof/intel/tgl.c
+++ b/sound/soc/sof/intel/tgl.c
@@ -156,6 +156,22 @@ const struct sof_intel_dsp_desc tglh_chip_info = {
};
EXPORT_SYMBOL_NS(tglh_chip_info, SND_SOC_SOF_INTEL_HDA_COMMON);
+const struct sof_intel_dsp_desc ehl_chip_info = {
+ /* Elkhartlake */
+ .cores_num = 4,
+ .init_core_mask = 1,
+ .host_managed_cores_mask = BIT(0),
+ .ipc_req = CNL_DSP_REG_HIPCIDR,
+ .ipc_req_mask = CNL_DSP_REG_HIPCIDR_BUSY,
+ .ipc_ack = CNL_DSP_REG_HIPCIDA,
+ .ipc_ack_mask = CNL_DSP_REG_HIPCIDA_DONE,
+ .ipc_ctl = CNL_DSP_REG_HIPCCTL,
+ .rom_init_timeout = 300,
+ .ssp_count = ICL_SSP_COUNT,
+ .ssp_base_offset = CNL_SSP_BASE_OFFSET,
+};
+EXPORT_SYMBOL_NS(ehl_chip_info, SND_SOC_SOF_INTEL_HDA_COMMON);
+
const struct sof_intel_dsp_desc adls_chip_info = {
/* Alderlake-S */
.cores_num = 2,
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
` (7 preceding siblings ...)
2021-03-22 16:37 ` [PATCH 8/8] ASoC: SOF: Intel: move ELH chip info Pierre-Louis Bossart
@ 2021-03-23 22:12 ` Mark Brown
8 siblings, 0 replies; 10+ messages in thread
From: Mark Brown @ 2021-03-23 22:12 UTC (permalink / raw)
To: Pierre-Louis Bossart, alsa-devel; +Cc: tiwai, Mark Brown
On Mon, 22 Mar 2021 11:37:20 -0500, Pierre-Louis Bossart wrote:
> This patchset fixes a logical flow in the core status, improves
> shutdown support on Intel SOF platforms with an HDaudio controller and
> cleans-up ElkhartLake support.
>
> Guennadi Liakhovetski (1):
> ASoC: SOF: Intel: HDA: fix core status verification
>
> [...]
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/8] ASoC: SOF: Intel: HDA: fix core status verification
commit: 927280909fa7d8e61596800d82f18047c6cfbbe4
[2/8] ASoC: SOF: core: harden shutdown helper
commit: 91ec48f540f83022377723a774a0a37a630801af
[3/8] ASoC: SOF: Intel: TGL: fix EHL ops
commit: 3c429f861ed483517a0a352281a16503bcc60b55
[4/8] ASoC: SOF: Intel: TGL: set shutdown callback to hda_dsp_shutdown
commit: 22aa9e021ad1ee7ce640270e75f4bdccff65d287
[5/8] ASoC: SOF: Intel: ICL: set shutdown callback to hda_dsp_shutdown
commit: 4939e49ea5804f89941df86d35f1a1e1cd8b435b
[6/8] ASoC: SOF: Intel: CNL: set shutdown callback to hda_dsp_shutdown
commit: b0503e8410e5ee43da116772576dbdeb2a414e0b
[7/8] ASoC: SOF: Intel: APL: set shutdown callback to hda_dsp_shutdown
commit: d3aa96bf349882763b9903e5800d2e83fc086886
[8/8] ASoC: SOF: Intel: move ELH chip info
commit: 8bb84ca873d2222ca220e58a097090775b1fd8df
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2021-03-23 22:13 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-22 16:37 [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 1/8] ASoC: SOF: Intel: HDA: fix core status verification Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 2/8] ASoC: SOF: core: harden shutdown helper Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 3/8] ASoC: SOF: Intel: TGL: fix EHL ops Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 4/8] ASoC: SOF: Intel: TGL: set shutdown callback to hda_dsp_shutdown Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 5/8] ASoC: SOF: Intel: ICL: " Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 6/8] ASoC: SOF: Intel: CNL: " Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 7/8] ASoC: SOF: Intel: APL: " Pierre-Louis Bossart
2021-03-22 16:37 ` [PATCH 8/8] ASoC: SOF: Intel: move ELH chip info Pierre-Louis Bossart
2021-03-23 22:12 ` [PATCH 0/8] ASoC: SOF: Intel: shutdown and core handling corrections Mark Brown
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.