-
Notifications
You must be signed in to change notification settings - Fork 321
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
audio: aria: Add set_large_config op #8338
audio: aria: Add set_large_config op #8338
Conversation
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.
The board config changes need to be split out.
This patch is needed to make the module work with linux and avoid firmware crash. This patch sets the attenuation parameters with the bytes control since the setup parameters can't be passed in init() as in Windows OS. Signed-off-by: Ranjani Sridharan <[email protected]> Signed-off-by: Seppo Ingalsuo <[email protected]>
Signed-off-by: Ranjani Sridharan <[email protected]> Signed-off-by: Seppo Ingalsuo <[email protected]>
43672c6
to
eabd5fb
Compare
@abonislawski Does this add look OK to you? |
@@ -30,6 +30,7 @@ CONFIG_COMP_VOLUME_WINDOWS_FADE=y | |||
CONFIG_COMP_DRC=y |
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.
@singalsu checkpatch does not like commits without description. as other checks are green, I think I'll let this pass through, but note for future.
Ignoring "sof-ci/jenkins/pr-tools-build", rest of CI checks are good, proceeding with merge. |
This patch is needed to make the module work with linux and avoid firmware crash. This patch sets the attenuation parameters with the bytes control since the setup parameters can't be passed in init() as in Windows OS.