Skip to content

Conversation

@MingJenTai
Copy link
Contributor

This PR modifies RTNR to work with KWD on TGL and ADL, and removes unused RTNR 16kHz topology.
It also adds RTNR support to max98390 on ADL.

Modify RTNR to work with KWD on TGL and ADL.
Remove unused RTNR 16kHz topology.
Add RTNR support to max98390 on ADL.

Signed-off-by: Ming Jen Tai <mingjen_tai@realtek.com>
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.

Looks like this needs to be 3 patches since the commit message say we are doing 3 different things.
@macchian @plbossart any comments. ?

@macchian
Copy link
Contributor

Looks like this needs to be 3 patches since the commit message say we are doing 3 different things. @macchian @plbossart any comments. ?

@lgirdwood , same thoughts.
remove 16k RTNR topology unused relevant codes as one,
then another: add 'RTNR' support in intel-generic-dmic-kwd.m4,
the other: add 'RTNR' macro for TGL and ADL.
@plbossart , any comments?

@MingJenTai
Copy link
Contributor Author

Hi, @lgirdwood and @macchian thanks for the opinion. I created 3 separate PRs to make it clearer. Please review them instead, thank you!
#5008
#5009
#5010

@lgirdwood
Copy link
Member

Zephyr build failure already fixed.

@lgirdwood lgirdwood merged commit d959089 into thesofproject:main Nov 23, 2021
@marc-hb marc-hb mentioned this pull request Nov 23, 2021
@ujfalusi ujfalusi mentioned this pull request Nov 24, 2021
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