ASoC: Intel: add I2S function topology support by bardliao · Pull Request #5657 · thesofproject/linux

bardliao · 2026-01-28T02:56:49Z

Currently, function topology is only used by the SoundWire machines. Extend the support to the I2S machines.

ujfalusi

only checked the first patch

ujfalusi · 2026-01-28T07:34:45Z

 	}

-	dev_err(dev, "%s: No match for SSP%d in NHLT table\n", __func__,
+	dev_dbg(dev, "%s: No match for SSP%d in NHLT table\n", __func__,


I would have this as separate patch?

we are also going to look for DMIC blobs in several nhlt, so change those prints as well?

Not sure about the DMIC blobs. Maybe we can change the DMIC prints when we look for DMIC blobs in several nhlt?

but we do after the second patch...

There could be NHLT blobs w/o DMIC (SSP fragment) and we would print error/warn?

intel_nhlt_get_endpoint_blob() will be called for each NHLT blobs, but it uses dev_dbg already.

you need to call out that in preparation for supporting multiple topology fragments per function which could end up calling this function multiple types with NHLT blobs from different fragments, downgrade the error message to debug level.

Commit message updated. But do you mean to move the commit to before the ASoC: SOF: get nhlt from all topologies commit?

ujfalusi · 2026-01-28T07:45:29Z

+								   channel_count, sample_rate,
+								   dir, dev_type);
+				if (cfg)
+					break;


goto out; and remove the cfg check after the loop?

ujfalusi

@bardliao, looks really nice, I have few nitpicks and one bigger question only

ujfalusi · 2026-01-29T06:04:33Z

 	}

-	dev_err(dev, "%s: No match for SSP%d in NHLT table\n", __func__,
+	dev_dbg(dev, "%s: No match for SSP%d in NHLT table\n", __func__,


but we do after the second patch...

There could be NHLT blobs w/o DMIC (SSP fragment) and we would print error/warn?

ujfalusi · 2026-01-29T06:16:23Z

-						      dai_index);
-		if (dev_type < 0)
+		list_for_each_entry(nhlt, &ipc4_data->nhlt_list, list) {
+			dev_type = intel_nhlt_ssp_device_type(sdev->dev, nhlt->nhlt,


nitpick: nhlt->nhlt, can we do entry->nhlt instead? the outer nhlt is not nhlt as such.

ujfalusi · 2026-01-29T06:30:01Z

+		if (!(*tplg_files)[tplg_num])
+			return -ENOMEM;
+		tplg_num++;
+	}


To my untrained eyes this looks like identical to what sof_sdw_get_tplg_files() would do, just that this lacks the SDW part, no?
Why not rename that function ad use it for all types as universal tool?

Yeah, I thought about it. And yes, they look quite similar, but the existing topologies may use different PCM IDs for different machines. I think it would be better to use separated function to keep the flexibility to have different function topology names for different machines.

CBala21 · 2026-01-30T12:05:50Z

Tested HDMI in capture and I2S functional. observed the expected behavior.

ranj063 · 2026-01-30T16:54:36Z

 	}

-	dev_err(dev, "%s: No match for SSP%d in NHLT table\n", __func__,
+	dev_dbg(dev, "%s: No match for SSP%d in NHLT table\n", __func__,


you need to call out that in preparation for supporting multiple topology fragments per function which could end up calling this function multiple types with NHLT blobs from different fragments, downgrade the error message to debug level.

lgirdwood

LGTM, one nitpick would be it could do with more inline commentary when it comes to how we are searching for files.

ranj063

LGTM thanks @bardliao

ujfalusi

I think this looks good, but not really sure how it is planned to be used in practice ;)

lgirdwood · 2026-02-05T16:13:53Z

CI needs kicked.

lgirdwood · 2026-02-05T16:14:00Z

SOFCI TEST

bardliao · 2026-02-06T13:41:53Z

change:
fix checkpatch issues.

lgirdwood

LGTM, just one question.

Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

 		case SOF_MANIFEST_DATA_TYPE_NHLT:
-			/* no NHLT in BIOS, so use the one from topology manifest */
-			if (ipc4_data->nhlt)
-				break;
-			ipc4_data->nhlt = devm_kmemdup(sdev->dev, manifest_tlv->data,
+			struct snd_ipc4_nhlt *tplg_nhlt;
+
+			/* Get the nhlt from topology manifest */
+			tplg_nhlt = devm_kzalloc(sdev->dev, sizeof(*tplg_nhlt), GFP_KERNEL);
+			if (!tplg_nhlt)
+				return -ENOMEM;


The existing code assumes there is only one NHLT blob either from BIOS or topology. As function topologies are used and each function topology contains a NHLT blob section, we need to search the matching NHLT blob from all the NHLT blobs. The commit suggests adding a list of NHLTs and search the NHLTs from the list. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

To support multiple topology fragments per function, this function could be called multiple types with different NHLT blobs from different fragments. Downgrade the error message to debug level since the SSP device type could be found from one of the nhlt. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.

bardliao · 2026-07-09T05:42:24Z

+static int get_platform_name(struct snd_soc_card *card,
+			     const struct snd_soc_acpi_mach *mach, char *platform)
+{
+	int ret;
+
+	ret = sscanf(mach->sof_tplg_filename, "sof-%3s-*.tplg", platform);
+	if (ret != 1) {
+		dev_err(card->dev, "Invalid platform name of tplg %s\n",
+			mach->sof_tplg_filename);
+		return -EINVAL;
+	}
+
+	return 0;
+}


The suffix will be added at run time.

+	/* Check presence of sub-topologies */
+	if (all_tplg_files_exist(card->dev, tplg_files, tplg_num))
+		return tplg_num;
+
+	/* return 0 to use monolithic topology */
+	return 0;


+struct snd_ipc4_nhlt {
+	struct list_head list;
+	void *nhlt;
+	bool from_acpi;
+};


The existing code supports get_function_tplg_files callback for SoundWire machine driver only. Some common sections can be used to extend the support to other machines. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.

+		(*tplg_files)[tplg_num] = get_tplg_filename(card->dev, prefix, platform,
+							    tplg_dev_name, dai_link->id,
+							    tplg_dev);
+		(*tplg_files)[tplg_num] = tplg_file;


Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.

+{
+	int ret;
+
+	ret = sscanf(mach->sof_tplg_filename, "sof-%3s-*.tplg", platform);


+		(*tplg_files)[tplg_num] = get_tplg_filename(card->dev, prefix, platform,
+							    tplg_dev_name, dai_link->id,
+							    tplg_dev);
+		(*tplg_files)[tplg_num] = tplg_file;
+		tplg_num++;


Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.

bardliao · 2026-07-09T12:30:23Z

+	ret = sscanf(mach->sof_tplg_filename, "sof-%3s-*.tplg", platform);
+	if (ret != 1) {
+		dev_err(card->dev, "Invalid platform name of tplg %s\n",
+			mach->sof_tplg_filename);
+		return -EINVAL;
+	}


The 'tplg' suffix will be added in run time

+		if (!tplg_files_exist(card->dev, tplg_file)) {
+			devm_kfree(card->dev, tplg_file);
+			if (best_effort)


+		if (!tplg_files_exist(card->dev, tplg_file)) {
+			devm_kfree(card->dev, tplg_file);
+			if (best_effort)


+struct snd_ipc4_nhlt {
+	struct list_head list;
+	void *nhlt;
+	bool from_acpi;
+};


best_effort mode Currently we will fallback to the monolithic topology if any function topology is missing. But the monolithic topology may not exist if best_effort is set. We should load the existing function topologies in the case. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

…et_tplg_files The Intel SOF SDW machine drive also supports I2S interface. Add related supports for the sof_sdw_get_tplg_files() callback. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

… I2S machines Add sof_i2s_get_tplg_files() callback for Intel SOF I2S machines. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

Use sof_i2s_get_tplg_files() for SOF es83x6 machines. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

+{
+	int ret;
+
+	ret = sscanf(mach->sof_tplg_filename, "sof-%3s-*.tplg", platform);


bardliao requested review from CBala21, ranj063 and ujfalusi January 28, 2026 02:56

bardliao requested review from dbaluta, kv2019i, lgirdwood and plbossart as code owners January 28, 2026 02:56

ujfalusi reviewed Jan 28, 2026

View reviewed changes

bardliao force-pushed the get-i2s-tplg branch 2 times, most recently from a0fd522 to 8d4c0c3 Compare January 29, 2026 03:30

ujfalusi reviewed Jan 29, 2026

View reviewed changes

bardliao force-pushed the get-i2s-tplg branch from 8d4c0c3 to 1af3d92 Compare January 29, 2026 14:19

ranj063 reviewed Jan 29, 2026

View reviewed changes

Comment thread sound/soc/sof/ipc4-topology.c Outdated

ranj063 reviewed Jan 30, 2026

View reviewed changes

bardliao force-pushed the get-i2s-tplg branch from 1af3d92 to 9fdc58f Compare February 2, 2026 05:30

lgirdwood reviewed Feb 3, 2026

View reviewed changes

bardliao force-pushed the get-i2s-tplg branch from 9fdc58f to fd2f6cc Compare February 4, 2026 14:05

ranj063 previously approved these changes Feb 4, 2026

View reviewed changes

ujfalusi previously approved these changes Feb 5, 2026

View reviewed changes

Comment thread sound/soc/intel/common/soc-acpi-intel-ptl-match.c

lgirdwood previously approved these changes Feb 5, 2026

View reviewed changes

bardliao dismissed stale reviews from lgirdwood, ujfalusi, and ranj063 via e6325cc February 6, 2026 13:37

bardliao force-pushed the get-i2s-tplg branch from fd2f6cc to e6325cc Compare February 6, 2026 13:37

lgirdwood reviewed Feb 10, 2026

View reviewed changes

Comment thread sound/soc/sof/intel/hda-dai.c

ranj063 previously approved these changes Feb 11, 2026

View reviewed changes

bardliao force-pushed the get-i2s-tplg branch from 1ba0b97 to 8b59d78 Compare July 8, 2026 02:16

Copilot started reviewing on behalf of bardliao July 8, 2026 02:17 View session

Copilot AI reviewed Jul 8, 2026

View reviewed changes

bardliao added 2 commits July 8, 2026 10:39

Copilot AI review requested due to automatic review settings July 8, 2026 02:39

bardliao force-pushed the get-i2s-tplg branch from 8b59d78 to 0cb240e Compare July 8, 2026 02:39

Copilot started reviewing on behalf of bardliao July 8, 2026 02:40 View session

Copilot AI reviewed Jul 8, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings July 9, 2026 06:50

bardliao force-pushed the get-i2s-tplg branch from 0cb240e to 3983b6b Compare July 9, 2026 06:50

Copilot started reviewing on behalf of bardliao July 9, 2026 06:56 View session

Copilot AI reviewed Jul 9, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings July 9, 2026 08:18

bardliao force-pushed the get-i2s-tplg branch from 3983b6b to 8c7b5d3 Compare July 9, 2026 08:18

Copilot started reviewing on behalf of bardliao July 9, 2026 08:19 View session

Copilot AI reviewed Jul 9, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings July 9, 2026 10:56

bardliao force-pushed the get-i2s-tplg branch from 8c7b5d3 to a1e77fc Compare July 9, 2026 10:56

Copilot started reviewing on behalf of bardliao July 9, 2026 11:02 View session

Copilot AI reviewed Jul 9, 2026

View reviewed changes

bardliao added 4 commits July 9, 2026 20:33

ASoC: Intel: sof-function-topology-lib: add I2S support for sof_sdw_g…

3ea688a

…et_tplg_files The Intel SOF SDW machine drive also supports I2S interface. Add related supports for the sof_sdw_get_tplg_files() callback. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

ASoC: Intel: sof-function-topology-lib: add get_function_topology for…

5dacb58

… I2S machines Add sof_i2s_get_tplg_files() callback for Intel SOF I2S machines. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

ASoC: soc-acpi-intel-ptl-match: I2S machines: use function topology

2a4bfb7

Use sof_i2s_get_tplg_files() for SOF es83x6 machines. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

Copilot AI review requested due to automatic review settings July 9, 2026 12:36

bardliao force-pushed the get-i2s-tplg branch from a1e77fc to 2a4bfb7 Compare July 9, 2026 12:36

Copilot started reviewing on behalf of bardliao July 9, 2026 12:37 View session

Copilot AI reviewed Jul 9, 2026

View reviewed changes

Comment thread sound/soc/intel/common/sof-function-topology-lib.c

{

int ret;

ret = sscanf(mach->sof_tplg_filename, "sof-%3s-*.tplg", platform);

Uh oh!

Conversation

bardliao commented Jan 28, 2026

Uh oh!

ujfalusi left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ujfalusi Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ujfalusi left a comment

Choose a reason for hiding this comment

Uh oh!

ujfalusi Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

CBala21 commented Jan 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lgirdwood left a comment

Choose a reason for hiding this comment

Uh oh!

ranj063 left a comment

Choose a reason for hiding this comment

Uh oh!

ujfalusi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lgirdwood commented Feb 5, 2026

Uh oh!

lgirdwood commented Feb 5, 2026

Uh oh!

bardliao commented Feb 6, 2026

Uh oh!

lgirdwood left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

ujfalusi Jan 29, 2026 •

edited

Loading

ujfalusi Jan 29, 2026 •

edited

Loading

CBala21 commented Jan 30, 2026 •

edited

Loading