ASoC: Intel: sof_sdw: Remove no longer supported quirk

The SOF_SDW_NO_AGGREGATION quirk is mostly for debug and no longer works
correctly with the current state of the machine driver. Remove it from
the code and add an error message if someone uses it.

Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://msgid.link/r/20240326160429.13560-17-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 1253a2dc33d22881f16ddea6d764a0458d583be2..fdedc32291f017abd8a289110ee81630db992b25 100644 (file)
--- a/sound/soc/intel/boards/sof_sdw.c
+++ b/sound/soc/intel/boards/sof_sdw.c
@@ -36,7 +36,7 @@ static void log_quirks(struct device *dev)
                dev_dbg(dev, "SSP port %ld\n",
                        SOF_SSP_GET_PORT(sof_sdw_quirk));
        if (sof_sdw_quirk & SOF_SDW_NO_AGGREGATION)
-               dev_dbg(dev, "quirk SOF_SDW_NO_AGGREGATION enabled\n");
+               dev_err(dev, "quirk SOF_SDW_NO_AGGREGATION enabled but no longer supported\n");
 }
 
 static int sof_sdw_quirk_cb(const struct dmi_system_id *id)
@@ -1136,11 +1136,9 @@ static int get_dailink_info(struct device *dev,
                            int *sdw_be_num, int *codecs_num)
 {
        bool group_visited[SDW_MAX_GROUPS];
-       bool no_aggregation;
        int i;
        int j;
 
-       no_aggregation = sof_sdw_quirk & SOF_SDW_NO_AGGREGATION;
        *sdw_be_num  = 0;
 
        if (!adr_link)
@@ -1187,7 +1185,7 @@ static int get_dailink_info(struct device *dev,
                                                continue;
 
                                        /* count BE for each non-aggregated slave or group */
-                                       if (!endpoint->aggregated || no_aggregation ||
+                                       if (!endpoint->aggregated ||
                                            !group_visited[endpoint->group_id])
                                                (*sdw_be_num)++;
                                }
@@ -1393,10 +1391,9 @@ static int get_slave_info(const struct snd_soc_acpi_link_adr *adr_link,
                          int *codec_num, unsigned int *group_id,
                          int adr_index)
 {
-       bool no_aggregation = sof_sdw_quirk & SOF_SDW_NO_AGGREGATION;
        int i;
 
-       if (!adr_link->adr_d[adr_index].endpoints->aggregated || no_aggregation) {
+       if (!adr_link->adr_d[adr_index].endpoints->aggregated) {
                cpu_dai_id[0] = ffs(adr_link->mask) - 1;
                *cpu_dai_num = 1;
                *codec_num = 1;
@@ -1758,7 +1755,6 @@ static int sof_card_dai_links_create(struct snd_soc_card *card)
        struct mc_private *ctx = snd_soc_card_get_drvdata(card);
        struct snd_soc_acpi_mach_params *mach_params = &mach->mach_params;
        const struct snd_soc_acpi_link_adr *adr_link = mach_params->links;
-       bool aggregation = !(sof_sdw_quirk & SOF_SDW_NO_AGGREGATION);
        struct snd_soc_codec_conf *codec_conf;
        struct sof_sdw_codec_info *codec_info;
        struct sof_sdw_codec_info *ssp_info;
@@ -1896,7 +1892,7 @@ out:
                                        be_id = current_be_id;
                        }
 
-                       if (aggregation && endpoint->aggregated)
+                       if (endpoint->aggregated)
                                group_generated[endpoint->group_id] = true;
                }
        }

diff --git a/sound/soc/intel/boards/sof_sdw_common.h b/sound/soc/intel/boards/sof_sdw_common.h
index 1daf98a5849f7e74c0194e31fa51ff8ae79a500e..e21ef79126b298c981ff556a8bdfb06157729d2c 100644 (file)
--- a/sound/soc/intel/boards/sof_sdw_common.h
+++ b/sound/soc/intel/boards/sof_sdw_common.h
@@ -50,6 +50,7 @@ enum {
 #define SOF_SDW_PCH_DMIC               BIT(6)
 #define SOF_SSP_PORT(x)                (((x) & GENMASK(5, 0)) << 7)
 #define SOF_SSP_GET_PORT(quirk)        (((quirk) >> 7) & GENMASK(5, 0))
+/* Deprecated and no longer supported by the code */
 #define SOF_SDW_NO_AGGREGATION         BIT(14)
 
 /* BT audio offload: reserve 3 bits for future */