topology2: tplg-targets-sdca-generic: remove 10ms deep buffer dma buffer#10559
Open
udaymb wants to merge 1 commit intothesofproject:mainfrom
Open
topology2: tplg-targets-sdca-generic: remove 10ms deep buffer dma buffer#10559udaymb wants to merge 1 commit intothesofproject:mainfrom
udaymb wants to merge 1 commit intothesofproject:mainfrom
Conversation
With Dynamic DeepBuffer the host DMA buffer size is calculated based on the requested ALSA period size (with a headroom between the two) using the DEEP_BUFFER token as a maximum limit for the host DMA buffer. This way applications can use the same DeepBuffer enabled PCM for different use cases and still benefit of the power saving of a bigger host DMA Kernel has been updated thesofproject/linux#5673 Signed-off-by: Uday M Bhat <uday.m.bhat@intel.com>
There was a problem hiding this comment.
Pull request overview
This PR removes the hardcoded 10ms deep buffer DMA buffer size from SDCA-generic topology configurations to enable Dynamic DeepBuffer functionality. With this change, the host DMA buffer size will be dynamically calculated based on the requested ALSA period size, using the default DEEP_BUFFER token (100ms) as a maximum limit rather than the restrictive 10ms fixed value. This allows applications to use the same DeepBuffer-enabled PCM for different use cases while benefiting from power savings.
Changes:
- Removed
DEEPBUFFER_FW_DMA_MS=10parameter from 8 SDCA-generic topology configurations - Retained
DEEP_BUF_SPK=trueto keep deep buffer functionality enabled - Configurations now use the default 100ms maximum limit for dynamic buffer calculation
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ujfalusi
approved these changes
Feb 20, 2026
Contributor
ujfalusi
left a comment
ujfalusi
left a comment
There was a problem hiding this comment.
Looks good with combination of the dynamic DeepBuffer in kernel.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
With Dynamic DeepBuffer the host DMA buffer size is calculated based on the requested ALSA period size (with a headroom between the two) using the DEEP_BUFFER token as a maximum limit for the host DMA buffer.
This way applications can use the same DeepBuffer enabled PCM for different use cases and still benefit of the power saving of a bigger host DMA
Kernel has been updated
thesofproject/linux#5673