load topology for each device #5209

bardliao · 2024-10-18T13:32:53Z

Get device information from dai links. load topology for each device.

This should not impact the existing devices.

bardliao · 2024-10-21T05:55:03Z

SOFCI TEST

sound/soc/sof/topology.c

ujfalusi · 2024-11-14T09:34:28Z

sound/soc/sof/topology.c

+		ret = request_firmware(&fw, tplg_files[i].file, scomp->dev);
+		if (ret < 0) {
+			if (i == 0) {
+				dev_dbg(scomp->dev,"Fail back to %s\n", tplg_name);


If it was a topology w/ DMIC then you have dropped the -2ch in a previous loop, you are not going to load the topology you supposed to be loading.

Yes, you are right. I need to restore the original topology name.

ujfalusi · 2024-11-14T09:35:19Z

sound/soc/sof/topology.c

+				tplg_name = strremove(tplg_name, "-2ch");
+			} else if (strstr(file, "-4ch")) {
+				tplg_device = "dmic-4ch";
+				tplg_name = strremove(tplg_name, "-4ch");


why are you dropping the 2ch/4ch from the original name?

Because I would like to load the topology without dmic. For example, sof-hda-generic-2ch.tplg -> sof-hda-generic.tplg or similar.

But the tplg_name is used only for loading the fallback, legacy, monolithic topology, in which case the fragments are not loaded. On a machine which have DMIC, you will load the topology w/o DMIC

Not exactly, tplg_name topology will be also loaded if not all dai links are handled. For example, if the monolithic topology is sof-lnl-rt711-4ch, then sof-lnl-rt711 and sof-lnl-dmic-4ch-id5 will be loaded

ujfalusi · 2024-11-14T09:39:14Z

sound/soc/sof/topology.c

+			if (strstr(file, "hda-generic")) {
+				tplg_device = "idisp";
+			} else {
+				tplg_device = "sdca-hdmi";


this is HDMI audio using SDW link or HDMI audio using HDA link, but used with SDW machine?

This is the HDMI audio used with SDW codecs. I use different name for different codec interfaces is because that the existing topologies use different PCM id for different codec interfaces. For example, the HDMI PCM ids are 5,6,7 for SDW machines and 3,4,5 for HDA machines.

00-00: Jack Out (*) : : playback 1 00-01: Jack In (*) : : capture 1 00-02: Speaker (*) : : playback 1 00-03: Amp feedback (*) : : capture 1 00-04: Microphone (*) : : capture 1 00-05: HDMI1 (*) : : playback 1 00-06: HDMI2 (*) : : playback 1 00-07: HDMI3 (*) : : playback 1 00-31: Deepbuffer Jack Out (*) : : playback 1

00-00: HDA Analog (*) : : playback 1 : capture 1 00-03: HDMI1 (*) : : playback 1 00-04: HDMI2 (*) : : playback 1 00-05: HDMI3 (*) : : playback 1 00-06: DMIC Raw (*) : : capture 1 00-31: Deepbuffer HDA Analog (*) : : playback 1

ujfalusi · 2024-11-14T09:40:36Z

sound/soc/sof/topology.c

+		} else if (strstr(dai_link->name, "iDisp")) {
+			tplg_dev = TPLG_DEVICE_HDMI;
+			if (strstr(file, "hda-generic")) {
+				tplg_device = "idisp";


If I read the function right, this is not going to be used as the hda-generic will fall under load_default_tplg = true, the HDA dai_link->name is not handled?

load_default_tplg = true means there are still some remaining pcms that are not loaded in the sparated topologies. And we need to load the renamed topology at line 2620.

but can you load the idisp.tplg and the sof-hda-generic.tplg which also contains idisp definitions?

but can you load the idisp.tplg and the sof-hda-generic.tplg which also contains idisp definitions?

No, so we need to make sure there is no idisp in sof-hda-generic.tplg.

ujfalusi · 2024-11-14T09:44:26Z

sound/soc/sof/topology.c

+						     "sdca-%damp", dai_link->num_cpus);
+		} else if (strstr(dai_link->name, "SmartMic")) {
+			tplg_dev = TPLG_DEVICE_SDW_MIC;
+			tplg_device = "sdca-mic";


Here and for the other checks: we need to update this match list every time a new component got introduced to products and we have new tplg fragment?
Is it going to scale?

We need to update this match list when a new dai links got introduced. Given that we didn't update dai links in our machine drivers. I think we will only update the match list occasionally.

ujfalusi · 2024-11-14T09:49:08Z

sound/soc/sof/topology.c

+						    tplg_files[i].device,
+						    tplg_files[i].be_id);
+		dev_dbg(scomp->dev, "Requesting %d %s\n", i, tplg_files[i].file);
+		ret = request_firmware(&fw, tplg_files[i].file, scomp->dev);


the sof_create_ipc_file_profile() will check if the firmware and the topology file (the monolithic one!) is in place.
The means that you always need to have the monolithic tplg and the split components installed, otherwise the profile is rejected and if not fallback IPC version is available then we will fail.

I have a feeling that this split tplg handling has to touch the fw-file-profile.c as well.

Yeah, I am thinking this open, too. My idea is that we don't touch the monolithic topology. I.e. the monolithic topology will coexist with the split topologies. So that sof_create_ipc_file_profile() will still valid.

ujfalusi · 2024-11-14T09:50:29Z

sound/soc/sof/topology.c

+						    "%s/sof-%s-%s-id%d.tplg",
+						    sof_pdata->tplg_filename_prefix, platform,
+						    tplg_files[i].device,
+						    tplg_files[i].be_id);


a clean documentation as commit message and comments to explain clearly the expected file naming and also an update to sof-docs must be done.

Totally agree.

bardliao · 2024-11-20T11:25:08Z

Topology PR: thesofproject/sof#9668. I would like to start with SDCA codecs only. SDCA is more urgent than others. If we can create dai links based on the supported SDCA functions, we don't need to use quirks for enabling/disabling specific dai links.

The existing code point tuples[*num_copied_tuples].value.s to elem->string which works fine if there is only one topology is handled and all the tuples are handled before the topology fw is released. However, we will handle multiple topologies and the tuples may be used after the topology fw is release. Ues devm_kasprintf to allocate the string for the tuples to avoid invalid access. Signed-off-by: Bard Liao <[email protected]>

No need to convert the return value of snd_soc_tplg_component_load(). Signed-off-by: Bard Liao <[email protected]>

Get device information from dai links and load topology for each device. This allow user create a topology for single device. The driver will select the needed topologies and we don't need to create topologies for each product. Signed-off-by: Bard Liao <[email protected]>

bardliao requested a review from ujfalusi October 18, 2024 13:32

bardliao force-pushed the test-multi-topologies branch from 7eb5975 to ffadfbb Compare October 21, 2024 08:38

bardliao force-pushed the test-multi-topologies branch from ffadfbb to 1deb725 Compare October 28, 2024 10:47

bardliao force-pushed the test-multi-topologies branch from 1deb725 to 3264fc3 Compare November 13, 2024 05:44

ujfalusi reviewed Nov 14, 2024

View reviewed changes

bardliao force-pushed the test-multi-topologies branch 2 times, most recently from 6259f1d to 78eed41 Compare November 20, 2024 06:55

bardliao force-pushed the test-multi-topologies branch from 78eed41 to 95da27b Compare November 20, 2024 12:09

bardliao added 2 commits November 22, 2024 16:14

ASoC: SOF: topology: don't convert error code

b0a6bab

No need to convert the return value of snd_soc_tplg_component_load(). Signed-off-by: Bard Liao <[email protected]>

bardliao force-pushed the test-multi-topologies branch from 95da27b to 9641e65 Compare November 22, 2024 08:14

bardliao changed the title ~~TEST: load topology for each device~~ load topology for each device Nov 27, 2024

bardliao marked this pull request as ready for review November 27, 2024 01:50

bardliao requested a review from ranj063 as a code owner November 27, 2024 01:50

bardliao force-pushed the test-multi-topologies branch from 9641e65 to 788e9bf Compare November 27, 2024 02:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

load topology for each device #5209

load topology for each device #5209

bardliao commented Oct 18, 2024

bardliao commented Oct 21, 2024

ujfalusi Nov 14, 2024

bardliao Nov 15, 2024

ujfalusi Nov 14, 2024

bardliao Nov 15, 2024

ujfalusi Nov 15, 2024

bardliao Nov 15, 2024

ujfalusi Nov 14, 2024

bardliao Nov 15, 2024

ujfalusi Nov 14, 2024

bardliao Nov 15, 2024

ujfalusi Nov 15, 2024

bardliao Nov 15, 2024

ujfalusi Nov 14, 2024

bardliao Nov 15, 2024

ujfalusi Nov 14, 2024

bardliao Nov 15, 2024

ujfalusi Nov 14, 2024

bardliao Nov 15, 2024

bardliao commented Nov 20, 2024

load topology for each device #5209

Are you sure you want to change the base?

load topology for each device #5209

Conversation

bardliao commented Oct 18, 2024

bardliao commented Oct 21, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

bardliao commented Nov 20, 2024