Skip to content

Tools: Topology2: Add widget class RTNR#8449

Merged
kv2019i merged 2 commits intothesofproject:mainfrom
singalsu:tplg2_rtnr_add
Nov 16, 2023
Merged

Tools: Topology2: Add widget class RTNR#8449
kv2019i merged 2 commits intothesofproject:mainfrom
singalsu:tplg2_rtnr_add

Conversation

@singalsu
Copy link
Collaborator

@singalsu singalsu commented Nov 6, 2023

No description provided.

This patch allows to build topologies to use the RTNR
component. Two bytes control blobs are converted from
topology1 to IPC4 format.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
This patch adds build of hda-generic development topologies to
test RTNR component with all s16/s24/s32 formats.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
@singalsu singalsu marked this pull request as ready for review November 8, 2023 08:35
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.

LGTM, but @ranj063 needs to approve.

@lgirdwood
Copy link
Member

@kv2019i any comments ?

@lgirdwood lgirdwood added this to the v2.8 milestone Nov 13, 2023
@kv2019i
Copy link
Collaborator

kv2019i commented Nov 13, 2023

@ranj063 @jsarha ping, will merge later this week if no further comments.

Copy link
Contributor

@jsarha jsarha left a comment

Choose a reason for hiding this comment

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

Looks syntactically correct. Would be helpful, if it would at least once be written somewhere what RTNR stands for. I had to use Google to know what this for 😅.

@kv2019i kv2019i merged commit 2ab6fe5 into thesofproject:main Nov 16, 2023
@singalsu
Copy link
Collaborator Author

Looks syntactically correct. Would be helpful, if it would at least once be written somewhere what RTNR stands for. I had to use Google to know what this for 😅.

True, I could have added same pointer as in firmware. This was min. effort to get component to run with supplied stubs library. We have now two NR solutions in SOF, though closed library licensing is needed to use them. Hope the companies can contribute to these files e.g. links to their products web pages for their new customers.

@singalsu singalsu deleted the tplg2_rtnr_add branch January 4, 2024 10:35
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