Tools: Topology2: Add 24 kHz rate to topologies with SRC by singalsu · Pull Request #10597 · thesofproject/sof

singalsu · 2026-03-04T12:41:33Z

This patch adds the missing 24 kHz rate that is used with MPEG-2 standard based and other codecs. The support started with Linux kernel 6.12.

Note: Build of these topologies needs alsa-lib version TBD.

singalsu · 2026-03-04T13:25:03Z

Topologies build in SOF CI fails with:

ALSA lib pcm.c:366:(split_rate) unsupported stream rate 24000

Need to have alsa-lib with patch alsa-project/alsa-lib#497

singalsu · 2026-03-04T17:41:06Z

This PR also depends on #10598 . We can't update alsa-lib with 24 kHz rate support without fixing the topology build issue.

lgirdwood · 2026-03-30T17:17:28Z

This PR also depends on #10598 . We can't update alsa-lib with 24 kHz rate support without fixing the topology build issue.

Merged. Non draft ?

singalsu · 2026-04-09T08:14:33Z

This PR also depends on #10598 . We can't update alsa-lib with 24 kHz rate support without fixing the topology build issue.

Merged. Non draft ?

I need to wait for Perex to merge the alsa-lib patch.

lgirdwood · 2026-04-13T12:03:54Z

This PR also depends on #10598 . We can't update alsa-lib with 24 kHz rate support without fixing the topology build issue.

Merged. Non draft ?

I need to wait for Perex to merge the alsa-lib patch.

Fyi @singalsu now merged.

singalsu · 2026-04-17T15:28:33Z

This PR also depends on #10598 . We can't update alsa-lib with 24 kHz rate support without fixing the topology build issue.

Merged. Non draft ?

I need to wait for Perex to merge the alsa-lib patch.

Fyi @singalsu now merged.

He has before closed the PR when merged (alsa-project/alsa-lib#497) but it's still open and I can't see "24 kHz" in latest commit titles.

singalsu · 2026-04-22T09:10:17Z

He has before closed the PR when merged (alsa-project/alsa-lib#497) but it's still open and I can't see "24 kHz" in latest commit titles.

I pinged today Perex on the alsa-lib PR, hope he responds.

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

This change updates to Release v1.2.16, needed for extended sample rates add to SOF topologies. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>

Copilot

Pull request overview

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

This patch updates the commits to Release v1.2.16. It is needed to build topologies with extended sample rates. The autoconf install is needed due to dependency from alsa-lib. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>

lgirdwood · 2026-06-04T15:20:49Z

LGTM @singalsu , but do we have the wrong ALSA commit id ?

ALSA lib pcm.c:366:(split_rate) unsupported stream rate 24000
ALSA lib parser.c:211:(tplg_load_config) failed to parse topology
Unable to load configuration: Invalid argument
FAILED: topology/topology2/development/sof-mtl-sdw-benchmark-sound_dose16-sdw0.tplg /home/sof/work/sof.git/tools/build_tools/topology/topology2/development/sof-mtl-sdw-benchmark-sound_dose16-sdw0.tplg 
cd /home/sof/work/sof.git/tools/build_tools/topology/topology2/development && cat /home/sof/work/sof.git/tools/build_tools/topology/topology2/development/../abi.conf /home/sof/work/sof.git/tools/topology/topology2/development/../cavs-benchmark-sdw.conf > sof-mtl-sdw-benchmark-sound_dose16-sdw0.conf && ALSA_CONFIG_DIR=/home/sof/work/sof.git/tools/topology/topology2 /home/sof/work/sof.git/tools/../../tools/bin/alsatplg ${VERBOSE:+-v 1} -I /home/sof/work/sof.git/tools/topology/topology2/development/../ -D 'PLATFORM=mtl,BENCH_MODULE_FORMAT=s16,BENCH_CONFIG=sound_dose16,BENCH_SOUND_DOSE_PARAMS=default' -p -c sof-mtl-sdw-benchmark-sound_dose16-sdw0.conf -o sof-mtl-sdw-benchmark-sound_dose16-sdw0.tplg
ninja: build stopped: subcommand failed.
Error: Process completed with exit code 1.

singalsu · 2026-06-05T15:04:59Z

LGTM @singalsu , but do we have the wrong ALSA commit id ?

Not wrong but I need to replace the build-tools docker container (no more maintainer for it) based test tools.yml with native version as e.g. testbench.yml.

The pre-built thesofproject/sof docker image carries an older alsa-lib / alsa-utils that no longer matches the versions required to build the current SOF user-space tools (in particular alsatplg used by the topology2 build). This patch moves the user-space tools build off docker and build it directly on the ubuntu-24.04 runner. - install the toolchain via apt - bootstrap autoconf 2.72 (Ubuntu 24.04 ships 2.71) - build the SOF forks of alsa-lib and alsa-utils at the same pinned commits used by testbench.yml - run scripts/build-tools.sh with PKG_CONFIG_PATH / LD_LIBRARY_PATH / PATH pointed at the freshly built ALSA tree The SOF-alsa-plugin job in the same workflow was already native and is left unchanged. The only consumer of this reusable workflow (daily-tests.yml) is unaffected by the internal switch. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>

This patch adds the missing 24 kHz rate that is used with MPEG-2 standard based and other codecs. The support started with Linux kernel 6.12. Note: Build of these topologies needs alsa-lib version TBD. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>

singalsu · 2026-06-05T15:43:50Z

LGTM @singalsu , but do we have the wrong ALSA commit id ?

Not wrong but I need to replace the build-tools docker container (no more maintainer for it) based test tools.yml with native version as e.g. testbench.yml.

@lgirdwood The build-tools step passed with the added workflow commit. Please review it too.

singalsu mentioned this pull request Apr 9, 2026

Tools: Topology: Add to nocodec port2 24 kHz rate capability #10234

Closed

singalsu marked this pull request as ready for review May 29, 2026 15:37

singalsu requested a review from ranj063 as a code owner May 29, 2026 15:37

Copilot AI review requested due to automatic review settings May 29, 2026 15:37

singalsu requested a review from jsarha as a code owner May 29, 2026 15:37

Copilot AI reviewed May 29, 2026

View reviewed changes

singalsu force-pushed the tplg_add_24k_rate_to_src_pcms branch from 7dd260d to bc9a8fc Compare June 2, 2026 14:06

singalsu requested review from dbaluta, kv2019i, lbetlej, lgirdwood, mmaka1 and plbossart as code owners June 2, 2026 14:06

scripts: build-alsa-tools.sh: Update alsa-lib and alsa-utils

c4785ba

This change updates to Release v1.2.16, needed for extended sample rates add to SOF topologies. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>

singalsu force-pushed the tplg_add_24k_rate_to_src_pcms branch from bc9a8fc to 44c5302 Compare June 2, 2026 14:14

singalsu requested a review from Copilot June 2, 2026 14:15

Copilot started reviewing on behalf of singalsu June 2, 2026 14:15 View session

Copilot AI reviewed Jun 2, 2026

View reviewed changes

singalsu force-pushed the tplg_add_24k_rate_to_src_pcms branch from 44c5302 to 50f0f06 Compare June 2, 2026 14:36

singalsu force-pushed the tplg_add_24k_rate_to_src_pcms branch from 50f0f06 to 02bdca6 Compare June 2, 2026 15:27

lgirdwood approved these changes Jun 4, 2026

View reviewed changes

singalsu added 2 commits June 5, 2026 18:32

singalsu force-pushed the tplg_add_24k_rate_to_src_pcms branch from 02bdca6 to c8f9145 Compare June 5, 2026 15:33

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Tools: Topology2: Add 24 kHz rate to topologies with SRC#10597

Tools: Topology2: Add 24 kHz rate to topologies with SRC#10597
singalsu wants to merge 4 commits into
thesofproject:mainfrom
singalsu:tplg_add_24k_rate_to_src_pcms

singalsu commented Mar 4, 2026

Uh oh!

singalsu commented Mar 4, 2026

Uh oh!

singalsu commented Mar 4, 2026

Uh oh!

lgirdwood commented Mar 30, 2026

Uh oh!

singalsu commented Apr 9, 2026

Uh oh!

lgirdwood commented Apr 13, 2026

Uh oh!

singalsu commented Apr 17, 2026

Uh oh!

singalsu commented Apr 22, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

lgirdwood commented Jun 4, 2026

Uh oh!

singalsu commented Jun 5, 2026

Uh oh!

singalsu commented Jun 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

singalsu commented Mar 4, 2026

Uh oh!

singalsu commented Mar 4, 2026

Uh oh!

singalsu commented Mar 4, 2026

Uh oh!

lgirdwood commented Mar 30, 2026

Uh oh!

singalsu commented Apr 9, 2026

Uh oh!

lgirdwood commented Apr 13, 2026

Uh oh!

singalsu commented Apr 17, 2026

Uh oh!

singalsu commented Apr 22, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

lgirdwood commented Jun 4, 2026

Uh oh!

singalsu commented Jun 5, 2026

Uh oh!

singalsu commented Jun 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants