-
Notifications
You must be signed in to change notification settings - Fork 318
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
Add CTC binary and switch controls #9370
Conversation
8747cc8
to
a6e34db
Compare
d966cbd
to
bcf9df2
Compare
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.
I think there's a high likelihood that the fall-through wasn't intended
@@ -243,6 +243,8 @@ static int ctc_free(struct processing_module *mod) | |||
{ | |||
struct google_ctc_audio_processing_comp_data *cd = module_get_private_data(mod); | |||
|
|||
comp_info(mod->dev, "ctc_free()"); |
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.
This 3rd commit could have been a separate PR as well. Not a blocker.
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.
just a couple minor nits left
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.
Please check concerns about the data blob setting process as I stated in comments. Thanks
e2f2e70
to
1a13d57
Compare
Add binary controls of Google CTC component to load different configs. Signed-off-by: Eddy Hsu <eddyhsu@google.com>
Add enablement switch controls of Google CTC component. Signed-off-by: Eddy Hsu <eddyhsu@google.com>
Check CTC state is properly inited and cleanup. Signed-off-by: Eddy Hsu <eddyhsu@google.com>
Add enablement and params controls of Google CTC component.