Skip to content

Conversation

@btian1
Copy link
Contributor

@btian1 btian1 commented Dec 21, 2023

Copier missed ibs(384)/obs(192) case, and eq-iir missed ibs(384)/obs(384) case, adding it to current CPC config.

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

Thank you @btian1 for fast update! Can you fix the commit prefix to "rimage: mtl: add copier and eq-iir CPC cases"

This will be racing with #8490 , one needs to be udpated after another, FYI @lyakh

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

Wrong button, adding request changes to fix the git commit.

@lgirdwood
Copy link
Member

Thank you @btian1 for fast update! Can you fix the commit prefix to "rimage: mtl: add copier and eq-iir CPC cases"

This will be racing with #8490 , one needs to be udpated after another, FYI @lyakh

@btian1 fyi TOML PR now merged

@btian1 btian1 changed the title Audio: CPC: add missed CPC config for copier and eq-iir rimage: mtl: add copier and eq-iir CPC cases Dec 22, 2023
Copier missed ibs(384)/obs(192) case, and eq-iir missed
ibs(384)/obs(384) case, adding it to current CPC config for mtl.

Signed-off-by: Baofeng Tian <baofeng.tian@intel.com>
@btian1 btian1 requested a review from kv2019i December 22, 2023 07:45
Copy link
Collaborator

@singalsu singalsu left a comment

Choose a reason for hiding this comment

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

Thanks! I can't decode the numbers in my head but looks assuring. Yep I know you got the numbers from tool so they are right :)

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

Thanks!

@kv2019i kv2019i merged commit b617388 into thesofproject:main Dec 22, 2023
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