-
Notifications
You must be signed in to change notification settings - Fork 314
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: DRC: Tune: DRC setup scripts to module directory #9188
Conversation
The purpose of this patch is to consolidate all the files those belong to the DRC module. The top level function names are changed to have sof_ prefix to avoid conflict with possible other similar files. Note: A small fix is needed after similar patch for EQ is merged since some functions for DRC blob are used from there. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
This patch adds checks and errors for fopen() failures. The check is useful with setup scripts paths changes. Aborting the script avoids the errors in blob exports to be missed. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
rmpath(common); | ||
rmpath(eq); | ||
end | ||
end |
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 is fine for the moment (short term), but in the long term we need a script that sets the Matlab/Octave environment and paths up (we have a source script for Zephyr today- so something similar). See https://docs.octave.org/v4.2.1/Startup-Files.html as these mechanism can be used
SOFCI TEST |
@wszypelt Can you check, quick run seems stuck on this PR that should not impact quickbuild tests. |
@kv2019i Intel intel CI system, all good |
No description provided.