-
Notifications
You must be signed in to change notification settings - Fork 311
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: Multiband-DRC: Move blob configuration scripts to module directory #9216
Conversation
…tory To consolidate the audio moves the scripts are moved to the same directory as the module. This first step only moves the files. The next steps rename the scripts and fix possible issue to directories move. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
The paths to topology and sof-ctl blobs are updated. The paths to other setup scripts (common, crossover, drc, eq) are set with a new helper function multiband_drc_paths.m. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
common = '../../../../tools/tune/common'; | ||
crossover = '../../../../tools/tune/crossover'; | ||
drc = '../../../../tools/tune/drc'; | ||
eq = '../../../../tools/tune/eq'; | ||
# After #9215 merge use this: | ||
# crossover = '../../crossover/tune'; | ||
# After #9188 merge use this: | ||
# drc = '../../drc/tune'; | ||
# After #9187 merge use this: | ||
# eq = '../../eq_iir/tune'; |
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.
Btw, does Matlab/Octave support PATHs via environment variable ? If so, might be easiest to have a script that sets these for the (like Zephyr has convenience script to setup build environment)
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.
Yes, the environment variables can be supported:
octave:1> getenv('SOF_WORKSPACE')
ans = /home/singalsu/work/current/sof
Would above be better e.g. as tune = sprintf('%s/sof/tools/tune', getenv('SOF_WORKSPACE'));
?
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.
Btw, lets track this as a long term feature - ideally long term west would enable this when we configure the environment.
Release reminder - one week to v2.11-rc1. |
No description provided.