-
Notifications
You must be signed in to change notification settings - Fork 349
Stable v1.9 refine #5608
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
Stable v1.9 refine #5608
Conversation
This reverts commit 5e89b58. Signed-off-by: Libin Yang <libin.yang@intel.com>
…& NOAMP support" This reverts commit f892b31. Signed-off-by: Libin Yang <libin.yang@intel.com>
…description" This reverts commit ee3631a. Signed-off-by: Libin Yang <libin.yang@intel.com>
…TION" This reverts commit 3904756. Signed-off-by: Libin Yang <libin.yang@intel.com>
…arts" This reverts commit 748b3ca. Signed-off-by: Libin Yang <libin.yang@intel.com>
…rmat" This reverts commit 93bf384. Signed-off-by: Libin Yang <libin.yang@intel.com>
This reverts commit 22b57a1. Signed-off-by: Libin Yang <libin.yang@intel.com>
NO_AGGREGATION is never used now. Let's remove it. Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Signed-off-by: Libin Yang <libin.yang@intel.com>
The links are dynamically assigned. The descriptions should be changed correspondingly. Signed-off-by: Libin Yang <libin.yang@intel.com>
…support add sof-icl-rt711-rt1308-rt715-hdmi NO_LOCAL_MIC & NOAMP support Signed-off-by: Libin Yang <libin.yang@intel.com>
Add sof-adl-rt711-l2-rt1316-l01 support. This tplg doesn't include any local MICs. Signed-off-by: Libin Yang <libin.yang@intel.com>
lgirdwood
left a comment
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.
@libinyang pls let me know when this backport PR is validated.
|
@lgirdwood As the CI system fails build the old tplg, I built the tplg and trigger the CI system manually. The test passes. |
|
Topologies build fix submitted in #5632 |
Building in parallel is much faster but it makes logs unreadable and build failures impossible to understand. This is especially true when building with recent ALSA that produces of deprecation warnings, see examples below. For test topologies the problem is even worse: its XARGS parallel build provides no log at all. To find what actually fails, it is required to fall back on a single threaded and verbose build and this is achieved with the variables USE_XARGS, NO_PROCESSORS and VERBOSE. Pass these through docker-run.sh and CMake. Examples from thesofproject#5608 https://github.com/thesofproject/sof/runs/5717553555?check_suite_focus=true https://sof-ci.01.org/sofpr/PR5608/build12556/build/tools.txt ALSA pcm.c:1523:(parse_hw_config) deprecated fsync value 'codec_slave',... ALSA pcm.c:1471:(parse_hw_config) deprecated bclk value 'codec_slave',... ALSA pcm.c:1523:(parse_hw_config) deprecated fsync value 'codec_slave',... ALSA pcm.c:1471:(parse_hw_config) deprecated bclk value 'codec_slave',... ALSA pcm.c:1523:(parse_hw_config) deprecated fsync value 'codec_slave',... Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Building in parallel is much faster but it makes logs unreadable and build failures impossible to understand. This is especially true when building with recent ALSA that produces of deprecation warnings, see examples below. For test topologies the problem is even worse: its XARGS parallel build provides no log at all. To find what actually fails, it is required to fall back on a single threaded and verbose build and this is achieved with the variables USE_XARGS, NO_PROCESSORS and VERBOSE. Pass these through docker-run.sh and CMake. Examples from thesofproject#5608 https://github.com/thesofproject/sof/runs/5717553555?check_suite_focus=true https://sof-ci.01.org/sofpr/PR5608/build12556/build/tools.txt ALSA pcm.c:1523:(parse_hw_config) deprecated fsync value 'codec_slave',... ALSA pcm.c:1471:(parse_hw_config) deprecated bclk value 'codec_slave',... ALSA pcm.c:1523:(parse_hw_config) deprecated fsync value 'codec_slave',... ALSA pcm.c:1471:(parse_hw_config) deprecated bclk value 'codec_slave',... ALSA pcm.c:1523:(parse_hw_config) deprecated fsync value 'codec_slave',... Signed-off-by: Marc Herbert <marc.herbert@intel.com>
|
It's impossible to find what exactly fails to build in https://github.com/thesofproject/sof/runs/5717552879 and others. Fixed by small but impactful change #5640, please review. |
Building in parallel is much faster but it makes logs unreadable and build failures impossible to understand. This is especially true when building with recent ALSA that produces of deprecation warnings, see examples below. For test topologies the problem is even worse: its XARGS parallel build provides no log at all. To find what actually fails, it is required to fall back on a single threaded and verbose build and this is achieved with the variables USE_XARGS, NO_PROCESSORS and VERBOSE. Pass these through docker-run.sh and CMake. Examples from #5608 https://github.com/thesofproject/sof/runs/5717553555?check_suite_focus=true https://sof-ci.01.org/sofpr/PR5608/build12556/build/tools.txt ALSA pcm.c:1523:(parse_hw_config) deprecated fsync value 'codec_slave',... ALSA pcm.c:1471:(parse_hw_config) deprecated bclk value 'codec_slave',... ALSA pcm.c:1523:(parse_hw_config) deprecated fsync value 'codec_slave',... ALSA pcm.c:1471:(parse_hw_config) deprecated bclk value 'codec_slave',... ALSA pcm.c:1523:(parse_hw_config) deprecated fsync value 'codec_slave',... Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Add the support for NO MIC platforms.
The previous PR missed some patches when backing port, which caused compile failure.
Revert the patches and resubmit them.
This PR passes the compile and the BE ID matches expectation.