ASoC: Intel: avs-ssm4567: remove redundant dapm routes

Four routes "Left Playback<-sspX Tx", "Right Playback<-sspX Tx",
"sspX Rx<-Left Capture Sense", and "sspX Rx<-Right Capture Sense" are
created by snd_soc_dapm_connect_dai_link_widgets() automatically.
Remove the duplicate routes.

Signed-off-by: Brent Lu <brent.lu@intel.com>
Link: https://lore.kernel.org/r/20230612110958.592674-13-brent.lu@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/intel/avs/boards/ssm4567.c b/sound/soc/intel/avs/boards/ssm4567.c
index 2b7f5ad92aca7b885ba0686f829fe9262067f6da..7324869d61327446d990b66fbc05e922e712fdf5 100644 (file)
--- a/sound/soc/intel/avs/boards/ssm4567.c
+++ b/sound/soc/intel/avs/boards/ssm4567.c
@@ -129,59 +129,14 @@ static int avs_create_dai_link(struct device *dev, const char *platform_name, in
        return 0;
 }
 
-static int avs_create_dapm_routes(struct device *dev, int ssp_port,
-                                 struct snd_soc_dapm_route **routes, int *num_routes)
-{
-       struct snd_soc_dapm_route *dr;
-       const int num_base = ARRAY_SIZE(card_base_routes);
-       const int num_dr = num_base + 4;
-       int idx;
-
-       dr = devm_kcalloc(dev, num_dr, sizeof(*dr), GFP_KERNEL);
-       if (!dr)
-               return -ENOMEM;
-
-       memcpy(dr, card_base_routes, num_base * sizeof(*dr));
-
-       idx = num_base;
-       dr[idx].sink = devm_kasprintf(dev, GFP_KERNEL, "Left Playback");
-       dr[idx].source = devm_kasprintf(dev, GFP_KERNEL, "ssp%d Tx", ssp_port);
-       if (!dr[idx].sink || !dr[idx].source)
-               return -ENOMEM;
-
-       idx++;
-       dr[idx].sink = devm_kasprintf(dev, GFP_KERNEL, "Right Playback");
-       dr[idx].source = devm_kasprintf(dev, GFP_KERNEL, "ssp%d Tx", ssp_port);
-       if (!dr[idx].sink || !dr[idx].source)
-               return -ENOMEM;
-
-       idx++;
-       dr[idx].sink = devm_kasprintf(dev, GFP_KERNEL, "ssp%d Rx", ssp_port);
-       dr[idx].source = devm_kasprintf(dev, GFP_KERNEL, "Left Capture Sense");
-       if (!dr[idx].sink || !dr[idx].source)
-               return -ENOMEM;
-
-       idx++;
-       dr[idx].sink = devm_kasprintf(dev, GFP_KERNEL, "ssp%d Rx", ssp_port);
-       dr[idx].source = devm_kasprintf(dev, GFP_KERNEL, "Right Capture Sense");
-       if (!dr[idx].sink || !dr[idx].source)
-               return -ENOMEM;
-
-       *routes = dr;
-       *num_routes = num_dr;
-
-       return 0;
-}
-
 static int avs_ssm4567_probe(struct platform_device *pdev)
 {
-       struct snd_soc_dapm_route *routes;
        struct snd_soc_dai_link *dai_link;
        struct snd_soc_acpi_mach *mach;
        struct snd_soc_card *card;
        struct device *dev = &pdev->dev;
        const char *pname;
-       int num_routes, ssp_port, ret;
+       int ssp_port, ret;
 
        mach = dev_get_platdata(dev);
        pname = mach->mach_params.platform;
@@ -193,12 +148,6 @@ static int avs_ssm4567_probe(struct platform_device *pdev)
                return ret;
        }
 
-       ret = avs_create_dapm_routes(dev, ssp_port, &routes, &num_routes);
-       if (ret) {
-               dev_err(dev, "Failed to create dapm routes: %d", ret);
-               return ret;
-       }
-
        card = devm_kzalloc(dev, sizeof(*card), GFP_KERNEL);
        if (!card)
                return -ENOMEM;
@@ -214,8 +163,8 @@ static int avs_ssm4567_probe(struct platform_device *pdev)
        card->num_controls = ARRAY_SIZE(card_controls);
        card->dapm_widgets = card_widgets;
        card->num_dapm_widgets = ARRAY_SIZE(card_widgets);
-       card->dapm_routes = routes;
-       card->num_dapm_routes = num_routes;
+       card->dapm_routes = card_base_routes;
+       card->num_dapm_routes = ARRAY_SIZE(card_base_routes);
        card->fully_routed = true;
        card->disable_route_checks = true;