sof icon indicating copy to clipboard operation
sof copied to clipboard

[BUG] PTL doesn't have an open source toolchain, "xtensa-build-zephyr.py ptl" fails by default

Open andyross opened this issue 1 year ago • 5 comments

The PTL platform merged without a toolchain added to the Zephyr SDK for the ace30_ptl SOC, or any other way to create one. This means that builds from a bare tree checkout are always going to fail without access to the Cadence SDK for the device. Also new work that might impact the platform (c.f https://github.com/thesofproject/sof/pull/9261) can't be validated by the submitter.

At the very least, if there's resistance to adding another toolchain to the already huge Zephyr SDK[1] an overlay file should be added to the SOF xtensa-overlays project ( https://github.com/thesofproject/xtensa-overlay ) so that users can build their own.

[1] There are, to be sure, a lot of Xtensa variants in there now.

andyross avatar Jun 26 '24 17:06 andyross

FWIW: another side band option would be to get partners copies of the relevant Cadence SDK integration files so they could do the work themselves.

andyross avatar Jun 26 '24 17:06 andyross

At the very least, if there's resistance to adding another toolchain to the already huge Zephyr SDK[1]

Do you have this in mind? https://github.com/zephyrproject-rtos/sdk-ng/pull/767

@lgirdwood not sure about the current policy, we should have it also in xtensa-overlay repo or sdk-ng is enough?

abonislawski avatar Jul 02 '24 08:07 abonislawski

At the very least, if there's resistance to adding another toolchain to the already huge Zephyr SDK[1]

Do you have this in mind? zephyrproject-rtos/sdk-ng#767

@lgirdwood not sure about the current policy, we should have it also in xtensa-overlay repo or sdk-ng is enough?

I think CI needs it on SOF side i.e. for GCC front end static analysis for PR testing but IIRC it should all come via Zephyr SDK now.

lgirdwood avatar Jul 05 '24 15:07 lgirdwood

Current GitHub workflows in sof/.github/ use a Zephyr docker image exclusively - because it provides the Zephyr SDK. https://github.com/zephyrproject-rtos/docker-image

  • https://github.com/zephyrproject-rtos/zephyr/issues/63738
  • https://github.com/zephyrproject-rtos/zephyr/pull/63759

marc-hb avatar Aug 14 '24 17:08 marc-hb

https://github.com/zephyrproject-rtos/sdk-ng/pull/767 is now merged, so support exists in Zephyr SDK.

No activity on the xtensa-overlay and "ptl" is still marked in extras, so I'll keep this open until we have more clear concensus (either the Zephyr SDK is sufficient for Zephyr native SOF platform and/or xtensa-overlay is updated).

kv2019i avatar Sep 06 '24 10:09 kv2019i

With #9732 we can now close this.

kv2019i avatar Jan 03 '25 10:01 kv2019i