Skip to content
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

Conversation

singalsu
Copy link
Collaborator

@singalsu singalsu commented Sep 7, 2023

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.

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]>
@singalsu
Copy link
Collaborator Author

singalsu commented Sep 7, 2023

This is same as #8079 but for v2.2.

@singalsu singalsu marked this pull request as ready for review September 8, 2023 12:51
@singalsu
Copy link
Collaborator Author

singalsu commented Sep 8, 2023

"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.

Copy link
Member

@lgirdwood lgirdwood left a 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
Copy link
Member

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 ?

Copy link
Collaborator Author

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.

@kv2019i kv2019i changed the title Tools: Topology1: Multiband-DRC: Update setup blobs [stable-v2.2] Tools: Topology1: Multiband-DRC: Update setup blobs Oct 20, 2023
@kv2019i
Copy link
Collaborator

kv2019i commented Oct 20, 2023

@singalsu what's the direction with this?

@singalsu
Copy link
Collaborator Author

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.

@singalsu singalsu closed this Oct 20, 2023
@singalsu singalsu deleted the tplg1_update_multiband_drc_blob_v22 branch February 13, 2024 08:57
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