Skip to content

Conversation

@udaymb
Copy link
Contributor

@udaymb udaymb commented Dec 23, 2025

SSP BT is supported for sof-ptl-rt721-4ch.tplg as implemented in the Linux kernel code
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/sound/soc/intel/boards/sof_sdw.c?h=v6.18&id=7d0a66e4bb9081d75c82ec4957c50034cb0ea449#n767

Topology name - sof-ptl-rt721-4ch-ssp2-bt-echoref.tplg

@udaymb udaymb requested a review from ranj063 as a code owner December 23, 2025 15:39
Copilot AI review requested due to automatic review settings December 23, 2025 15:39
@udaymb udaymb requested a review from jsarha as a code owner December 23, 2025 15:39
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for SSP BT (Bluetooth) with echo reference capability to the sof-ptl-rt721-4ch topology configuration for Intel Panther Lake platform. The changes align with the kernel implementation referenced in the description.

Key Changes:

  • Added new topology target sof-ptl-rt721-4ch-bt-off-echoref.tplg with BT offload support
  • Configured BT interface on SSP2 with PCM ID 20 and device ID 10

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@udaymb udaymb requested review from bardliao and singalsu December 23, 2025 15:39
# RT721 eval board with PCH-DMIC, sof_sdw_quirk_table with SOC_SDW_PCH_DMIC
# Enable echo reference capture from jack and speaker output
# Enable BT offload
"cavs-sdw\;sof-ptl-rt721-4ch-bt-off-echoref\;PLATFORM=ptl,SDW_DMIC=1,NUM_SDW_AMP_LINKS=1,NUM_DMICS=4,\
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you're enabling BT, why is it called bt-off?

Copy link
Contributor Author

@udaymb udaymb Dec 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ranj063 - renamed to sof-ptl-rt721-4ch-ssp2-bt-echoref.tplg

PREPROCESS_PLUGINS=nhlt,NHLT_BIN=nhlt-sof-ptl-rt721-4ch-echoref.bin,DMIC0_ENHANCED_CAPTURE=true,\
EFX_DMIC0_TDFB_PARAMS=line4_pass,EFX_DMIC0_DRC_PARAMS=dmic_default,\
DEEPBUFFER_FW_DMA_MS=10,DEEP_BUF_SPK=true,SDW_JACK_ECHO_REF=true,SDW_SPK_ECHO_REF=true"

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fix the prefix in the commit title to topology2: development: tplg-targets:....

Also, is the kernel patch link relevant in the commit message? Maybe you can add that in the PR description?

Copy link
Contributor Author

@udaymb udaymb Dec 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ranj063 - Update commit message

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ranj063 - Update commit message

@udaymb I think @ranj063's ask is to change the commit title to topology2: development: Enable SSP BT with echo reference support for sof-ptl-rt721-4ch.tplg as the change is in the development folder.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bardliao - missed this - done the change

# RT721 eval board with PCH-DMIC, sof_sdw_quirk_table with SOC_SDW_PCH_DMIC
# Enable echo reference capture from jack and speaker output
# Enable BT offload
"cavs-sdw\;sof-ptl-rt721-4ch-bt-off-echoref\;PLATFORM=ptl,SDW_DMIC=1,NUM_SDW_AMP_LINKS=1,NUM_DMICS=4,\
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@udaymb How will the tplg file be used? The kernel will not look for the sof-ptl-rt721-4ch-bt-off-echoref.tplg. Will you rename it to sof-ptl-rt721-4ch.tplg in the file system?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bardliao - Yes this will be renamed

@udaymb udaymb force-pushed the alc721-ssp-bt-echo-ref branch from 2982875 to 3d4ea7a Compare December 24, 2025 03:00
@udaymb udaymb changed the title Tools: Topology: Enable SSP BT with echo reference support for sof-ptl-rt721-4ch.tplg topology2: Enable SSP BT with echo reference support for sof-ptl-rt721-4ch.tplg Dec 24, 2025
@udaymb udaymb force-pushed the alc721-ssp-bt-echo-ref branch from 3d4ea7a to f491ebd Compare December 24, 2025 06:33
@udaymb udaymb changed the title topology2: Enable SSP BT with echo reference support for sof-ptl-rt721-4ch.tplg topology2: development: Enable SSP2 BT with echo reference support for sof-ptl-rt721-4ch.tplg Dec 24, 2025
…nce support for sof-ptl-rt721-4ch.tplg

Support for SSP2 BT with echo reference is included in topology

Topology name - sof-ptl-rt721-4ch-ssp2bt-echoref.tplg

Signed-off-by: Uday M Bhat <uday.m.bhat@intel.com>
@udaymb udaymb force-pushed the alc721-ssp-bt-echo-ref branch from f491ebd to 6d5b935 Compare December 24, 2025 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants