Skip to content

For tac5272 - #5911

Open
niranjanhyti wants to merge 2 commits into
thesofproject:topic/sof-devfrom
niranjanhyti:for-tac5272
Open

For tac5272#5911
niranjanhyti wants to merge 2 commits into
thesofproject:topic/sof-devfrom
niranjanhyti:for-tac5272

Conversation

@niranjanhyti

Copy link
Copy Markdown

No description provided.

@sofci

sofci commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

reply test this please to run this test once

@bardliao
bardliao requested a lite review from Copilot September 3, 2026 02:36
@bardliao

bardliao commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

test this please

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changes are small, internally consistent (DAI naming/streams match existing UAJ patterns), and scoped to adding TAC5272 support without impacting other part IDs.

Pull request overview

Adds SoundWire/ASoC support for the TI TAC5272 by introducing a TAC5272-specific codec info entry for SoundWire machine helpers and a matching codec driver registration path in the TAC5xx2 SoundWire codec driver.

Changes:

  • Add TAC5272 entry to codec_info_list[] with a single UAJ (jack) DAI configuration.
  • Add TAC5272-specific DAI driver definition and component-driver template selection in tac5xx2-sdw.
  • Gate SmartAmp/SmartMic SDCA init sequences behind TAC5272 capability flags and add the TAC5272 SDW device ID.
File summaries
File Description
sound/soc/sdw_utils/soc_sdw_utils.c Adds TAC5272 codec info entry for UAJ-only SoundWire endpoint/dailink construction.
sound/soc/codecs/tac5xx2-sdw.c Registers TAC5272 as a supported SDW device with a dedicated DAI set and component template, and skips SA/SM init for TAC5272.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

/* UAJ only - no speaker or DMIC */
.direction = {true, true},
.dai_name = "tac5xx2-aif3",
.component_name = "tac5272",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

component_name is currently for the amp only. But it is harmless to add it.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks, I realized this during the ucm2 file creation.

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.

4 participants