ASoC: Intel: sof_sdw: add mutual exclusion between PCH DMIC and RT715

When external RT714/715 devices are used for capture, we don't want
the PCH DMICs to be used.

Any information provided by the SOF platform driver or DMI quirks will
be overridden.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Reviewed-by: Libin Yang <libin.yang@intel.com>
Link: https://lore.kernel.org/r/20210505163705.305616-5-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/intel/boards/sof_sdw.c b/sound/soc/intel/boards/sof_sdw.c
index ecd3f90f4bbea263695e2906b3aee653158a8c57..85a2797c255030427892a6bf0ce661aa2435b635 100644 (file)
--- a/sound/soc/intel/boards/sof_sdw.c
+++ b/sound/soc/intel/boards/sof_sdw.c
@@ -353,6 +353,7 @@ static struct sof_sdw_codec_info codec_info_list[] = {
                .part_id = 0x714,
                .version_id = 3,
                .direction = {false, true},
+               .ignore_pch_dmic = true,
                .dai_name = "rt715-aif2",
                .init = sof_sdw_rt715_sdca_init,
        },
@@ -360,6 +361,7 @@ static struct sof_sdw_codec_info codec_info_list[] = {
                .part_id = 0x715,
                .version_id = 3,
                .direction = {false, true},
+               .ignore_pch_dmic = true,
                .dai_name = "rt715-aif2",
                .init = sof_sdw_rt715_sdca_init,
        },
@@ -367,6 +369,7 @@ static struct sof_sdw_codec_info codec_info_list[] = {
                .part_id = 0x714,
                .version_id = 2,
                .direction = {false, true},
+               .ignore_pch_dmic = true,
                .dai_name = "rt715-aif2",
                .init = sof_sdw_rt715_init,
        },
@@ -374,6 +377,7 @@ static struct sof_sdw_codec_info codec_info_list[] = {
                .part_id = 0x715,
                .version_id = 2,
                .direction = {false, true},
+               .ignore_pch_dmic = true,
                .dai_name = "rt715-aif2",
                .init = sof_sdw_rt715_init,
        },
@@ -729,7 +733,8 @@ static int create_sdw_dailink(struct device *dev, int *be_index,
                              int *cpu_id, bool *group_generated,
                              struct snd_soc_codec_conf *codec_conf,
                              int codec_count,
-                             int *codec_conf_index)
+                             int *codec_conf_index,
+                             bool *ignore_pch_dmic)
 {
        const struct snd_soc_acpi_link_adr *link_next;
        struct snd_soc_dai_link_component *codecs;
@@ -782,6 +787,9 @@ static int create_sdw_dailink(struct device *dev, int *be_index,
        if (codec_index < 0)
                return codec_index;
 
+       if (codec_info_list[codec_index].ignore_pch_dmic)
+               *ignore_pch_dmic = true;
+
        cpu_dai_index = *cpu_id;
        for_each_pcm_streams(stream) {
                char *name, *cpu_name;
@@ -913,6 +921,7 @@ static int sof_card_dai_links_create(struct device *dev,
        const struct snd_soc_acpi_link_adr *adr_link;
        struct snd_soc_dai_link_component *cpus;
        struct snd_soc_codec_conf *codec_conf;
+       bool ignore_pch_dmic = false;
        int codec_conf_count;
        int codec_conf_index = 0;
        bool group_generated[SDW_MAX_GROUPS];
@@ -1019,7 +1028,8 @@ static int sof_card_dai_links_create(struct device *dev,
                                         sdw_cpu_dai_num, cpus, adr_link,
                                         &cpu_id, group_generated,
                                         codec_conf, codec_conf_count,
-                                        &codec_conf_index);
+                                        &codec_conf_index,
+                                        &ignore_pch_dmic);
                if (ret < 0) {
                        dev_err(dev, "failed to create dai link %d", be_id);
                        return -ENOMEM;
@@ -1087,6 +1097,10 @@ SSP:
 DMIC:
        /* dmic */
        if (dmic_num > 0) {
+               if (ignore_pch_dmic) {
+                       dev_warn(dev, "Ignoring PCH DMIC\n");
+                       goto HDMI;
+               }
                cpus[cpu_id].dai_name = "DMIC01 Pin";
                init_dai_link(dev, links + link_id, be_id, "dmic01",
                              0, 1, // DMIC only supports capture
@@ -1105,6 +1119,7 @@ DMIC:
                INC_ID(be_id, cpu_id, link_id);
        }
 
+HDMI:
        /* HDMI */
        if (hdmi_num > 0) {
                idisp_components = devm_kcalloc(dev, hdmi_num,

diff --git a/sound/soc/intel/boards/sof_sdw_common.h b/sound/soc/intel/boards/sof_sdw_common.h
index f3cb6796363e7fcb8ce024d437be7018d936e25c..ea60e8ed215c551c0445c66c956dfd68beff01cb 100644 (file)
--- a/sound/soc/intel/boards/sof_sdw_common.h
+++ b/sound/soc/intel/boards/sof_sdw_common.h
@@ -56,6 +56,7 @@ struct sof_sdw_codec_info {
        int amp_num;
        const u8 acpi_id[ACPI_ID_LEN];
        const bool direction[2]; // playback & capture support
+       const bool ignore_pch_dmic;
        const char *dai_name;
        const struct snd_soc_ops *ops;