-
Notifications
You must be signed in to change notification settings - Fork 349
build: start removing XTOS support #10075
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
Conversation
kv2019i
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.
Cmocka is broken and needs some files to be kept (and/or converted). Otherwise this looks good. I added a link to the enhancement #9983 so people can get the context why this is done.
kv2019i
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.
@lyakh drivers can be mostly removed as well. the xtos ipc drivers are used by many targets also in Zephry builds, so that has to be kept. This requires work on Zephyr side (see #9983 (comment) ).
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.
LGTM
@kv2019i so we can remove all drivers except IPC and clock? Wouldn't platforms when migrating to Zephyr first continue using their existing SOF drivers and only then gradually migrate to enabling |
Remove GitHub XTOS build tests. We'll be removing XTOS support completely, which obviously will lead to all XTOS tests failing. Note, that this also removes a QEMU boot test. It should be fixed to work with a Zephyr build and then re-added. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
xtensa-build-all.sh is only used for XTOS builds, which are now being removed. This also breaks test-repro-build.sh which we don't want to remove to later convert to use Zephyr. To preserve it until then we add a check for test-repro-build.sh. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Travis isn't used any more, remove its configuration. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Remove some obviously exclusively XTOS files. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Only the src/arch/xtensa/drivers/cache_attr.c file under the src/arch/xtensa directory is used in Zephyr builds, remove the rest. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
|
@lyakh wrote:
We can leave that to driver owners to decide and do the PR (i.e. not this PR). |
initial mostly whole-file XTOS support removal