ASoC: SOF: topology: skip feature topologies without matching BE DAI links

Before loading a user-specified feature topology, verify that the
card has a matching BE DAI link. Feature topologies extend specific
codec function types (amp, jack, mic) and reference widgets that
only exist when the corresponding BE DAI links are present.

Without this check, loading an amplifier feature topology on a card
without SmartAmp DAI links would fail with -EINVAL due to missing
route endpoints, killing the entire card probe.

This is needed for multi-card configurations where cards are split
by function type, but also provides a safety net in single-card
mode when feature topologies don't match the hardware.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>

Author: ujfalusi

sound/soc/sof/topology.c

@@ -2516,6 +2516,47 @@ static const struct snd_soc_tplg_ops sof_dspless_tplg_ops = {
 	.bytes_ext_ops_count = ARRAY_SIZE(sof_dspless_bytes_ext_ops),
 };
 
+/*
+ * Check if a feature topology is compatible with the current card by
+ * verifying that matching BE DAI links exist.
+ *
+ * Feature topologies extend specific codec function types. If the card
+ * doesn't have the corresponding BE DAI links, the topology load would
+ * fail due to missing routes/widgets. This works transparently for both
+ * multi-card and single-card configurations.
+ */
+static bool
+sof_feature_tplg_matches_card(struct snd_soc_component *scomp,
+			      const char *feature_tplg)
+{
+	static const struct {
+		const char *tplg_key;	/* substring in feature topology filename */
+		const char *dai_key;	/* substring in BE DAI link name */
+	} match_table[] = {
+		{ "amp",  "SmartAmp" },
+		{ "jack", "SimpleJack" },
+		{ "mic",  "SmartMic" },
+	};
+	struct snd_soc_dai_link *dai_link;
+	bool needs_match = false;
+	int i, j;
+
+	for (i = 0; i < ARRAY_SIZE(match_table); i++) {
+		if (!strstr(feature_tplg, match_table[i].tplg_key))
+			continue;
+
+		needs_match = true;
+
+		for_each_card_prelinks(scomp->card, j, dai_link) {
+			if (strstr(dai_link->name, match_table[i].dai_key))
+				return true;
+		}
+	}
+
+	/* Unknown feature type: load it (backwards compatible) */
+	return !needs_match;
+}
+
 int snd_sof_load_topology(struct snd_soc_component *scomp, const char *file)
 {
 	struct sof_client_dev *cdev = snd_sof_component_get_cdev(scomp);
@@ -2612,11 +2653,25 @@ int snd_sof_load_topology(struct snd_soc_component *scomp, const char *file)
 		}
 	}
 
-	/* Loading user defined topologies */
+	/*
+	 * Loading user defined feature topologies.
+	 * Each feature topology extends a specific function type (amp, jack,
+	 * mic, etc.). Only load if the card has a matching BE DAI link,
+	 * otherwise the routes would fail due to missing widgets.
+	 */
 	for (i = 0; i < feature_tplg_cnt; i++) {
-		const char *feature_topology = devm_kasprintf(scomp->dev, GFP_KERNEL, "%s/%s",
-							   tplg_filename_prefix,
-							   feature_topologies[i]);
+		const char *feature_topology;
+
+		if (!sof_feature_tplg_matches_card(scomp, feature_topologies[i])) {
+			dev_dbg(scomp->dev,
+				"skip feature topology %s: no matching BE DAI link\n",
+				feature_topologies[i]);
+			continue;
+		}
+
+		feature_topology = devm_kasprintf(scomp->dev, GFP_KERNEL, "%s/%s",
+						  tplg_filename_prefix,
+						  feature_topologies[i]);
 
 		if (!feature_topology) {
 			ret = -ENOMEM;