Skip to content

Conversation

@lyakh
Copy link
Collaborator

@lyakh lyakh commented Jun 27, 2025

initial mostly whole-file XTOS support removal

Copy link
Collaborator

@kv2019i kv2019i left a 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.

Copy link
Collaborator

@kv2019i kv2019i left a 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) ).

Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lyakh lyakh marked this pull request as ready for review July 2, 2025 10:44
@lyakh
Copy link
Collaborator Author

lyakh commented Jul 2, 2025

@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) ).

@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 CONFIG_ZEPHYR_NATIVE_DRIVERS?

lyakh added 5 commits July 2, 2025 14:32
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>
@kv2019i
Copy link
Collaborator

kv2019i commented Jul 2, 2025

@lyakh wrote:

@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 CONFIG_ZEPHYR_NATIVE_DRIVERS?

We can leave that to driver owners to decide and do the PR (i.e. not this PR).

@lgirdwood lgirdwood merged commit b8266ae into thesofproject:main Jul 3, 2025
21 of 44 checks passed
@lyakh lyakh deleted the xtos branch July 3, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants