Skip to content

topology2: sof-sdca-jack: change deep buffer DMA buffer to 10ms#10548

Open
bardliao wants to merge 1 commit intothesofproject:mainfrom
bardliao:topology2-deep-buf-dma
Open

topology2: sof-sdca-jack: change deep buffer DMA buffer to 10ms#10548
bardliao wants to merge 1 commit intothesofproject:mainfrom
bardliao:topology2-deep-buf-dma

Conversation

@bardliao
Copy link
Collaborator

To align with the existing Chrome topologies.

To align with the existing Chrome topologies.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
@bardliao bardliao requested a review from ujfalusi February 13, 2026 11:40
Copilot AI review requested due to automatic review settings February 13, 2026 11:40
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 modifies the deep buffer DMA configuration for the sof-sdca-jack topology to use a 10ms buffer size, aligning it with Chrome platform conventions. The change brings this jack-only configuration in line with other SDCA topologies and Chrome-specific topologies that already use the 10ms setting.

Changes:

  • Added DEEPBUFFER_FW_DMA_MS=10 parameter to sof-sdca-jack-id0 topology configuration

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

# Split topologies
"cavs-sdw\;sof-sdca-jack-id0\;SDW_JACK_OUT_STREAM=Playback-SimpleJack,\
SDW_JACK_IN_STREAM=Capture-SimpleJack,NUM_HDMIS=0"
SDW_JACK_IN_STREAM=Capture-SimpleJack,NUM_HDMIS=0,DEEPBUFFER_FW_DMA_MS=10"
Copy link
Contributor

Choose a reason for hiding this comment

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

10ms is a ShallowBuffer ;)
Note that the default host buffer is 4ms...

Copy link
Contributor

@ujfalusi ujfalusi Feb 13, 2026

Choose a reason for hiding this comment

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

and monolithic topologies will still have 100ms host buffer, as well the speaker.

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.

2 participants