-
Notifications
You must be signed in to change notification settings - Fork 321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[stable-v2.2] Tools: Topology1: Multiband-DRC: Update setup blobs #8176
[stable-v2.2] Tools: Topology1: Multiband-DRC: Update setup blobs #8176
Conversation
The multiband_drc_coef_default.m4 is updated to what the current setup tool example_multiband_drc() exports. Another blob for pass-through is added with emphasis and all bands processing disabled. Such blob is useful when headphone and speaker paths are shared. Headphone mode should use pass-through. Signed-off-by: Seppo Ingalsuo <[email protected]>
This is same as #8079 but for v2.2. |
"21 successful and 1 failing checks" There is no multiband-drc instance in CI tests. The verify-tplg-binary.sh fail with sof-apl-nocodec.tplg is not related, this happens also with stable-2.2 as such. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Topology2 versions higher priority than topology1
@@ -1,7 +1,7 @@ | |||
# Exported Control Bytes 11-May-2021 | |||
# Exported Control Bytes 16-Aug-2023 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whats the command line to reproduce ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems I need to update #8142 to update this comment for tplg1 too. This change is already merged to main so I'd need to update it there as well. But I don't think the setup tool update is worth to do for v2.2. It would need to be a new middle version of v2.2 and git main without tplg2 part.
The tplg2 is already in order. This tplg1 part was split out from the tplg2 patch.
@singalsu what's the direction with this? |
The tool is still missing the howto add for tplg1. I'll close this PR and update later. @johnylin76 provided a better performing blob that we can take into use. I'll return to this topic with it. |
The multiband_drc_coef_default.m4 is updated to what the current setup tool example_multiband_drc() exports.
Another blob for pass-through is added with emphasis and all bands processing disabled. Such blob is useful when headphone and speaker paths are shared. Headphone mode should use pass-through.