Skip to content
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.

Sign up for GitHub

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

Jump to bottom

PTL: add build testing (includes Zephyr west.yml update) #9732

Merged
merged 4 commits into from
Dec 19, 2024
Merged

PTL: add build testing (includes Zephyr west.yml update) #9732

merged 4 commits into from
Dec 19, 2024

Conversation

lyakh
Copy link
Collaborator

@lyakh lyakh commented Dec 18, 2024

No description provided.

@lyakh
Copy link
Collaborator Author

lyakh commented Dec 18, 2024

needs zephyrproject-rtos/zephyr#83150

lyakh added 4 commits December 19, 2024 07:55
@lyakh
Zephyr: upgrade to include a build fix
85faeaf 
Upgrade to a Zephyr version, containing 6f1fc6e96c60 ("drivers: dai:
ssp: remove unused variables") to fix build failures.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
@lyakh
ci: upgrade to Zephyr SDK 0.17.0
cd3700c 
0.16.4 doesn't support PTL, upgrade to the current 0.17.0.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
@lyakh
ci: add a PTL build test
59477bd 
Now that the Zephyr SDK has support for PTL we can add a build test.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
@lyakh
build: ptl: an open-source toolchain is available for PTL
f2818d9 
Zephyr SDK now supports PTL beginning with 0.16.9, update the build
script.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
@lyakh lyakh marked this pull request as ready for review December 19, 2024 06:59
@kv2019i kv2019i changed the title PTL: add build testing PTL: add build testing (includes Zephyr west.yml update) Dec 19, 2024
kv2019i
kv2019i approved these changes Dec 19, 2024
View reviewed changes
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.

Looks good. This will also close #9262

@lyakh
Copy link
Collaborator Author

lyakh commented Dec 19, 2024 •
edited
Loading

Looks good. This will also close #9262

@kv2019i if we merge this before #9702 then we can just drop the Zephyr update from the latter, and it will also be built-tested for PTL then

@lgirdwood lgirdwood merged commit db6e9cc into thesofproject:main Dec 19, 2024
47 of 49 checks passed
@lyakh lyakh deleted the ptl branch December 19, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kv2019i kv2019i kv2019i approved these changes

@lgirdwood lgirdwood lgirdwood approved these changes

@dbaluta dbaluta Awaiting requested review from dbaluta dbaluta is a code owner

@iuliana-prodan iuliana-prodan Awaiting requested review from iuliana-prodan iuliana-prodan is a code owner

@abonislawski abonislawski Awaiting requested review from abonislawski abonislawski is a code owner

@plbossart plbossart Awaiting requested review from plbossart plbossart is a code owner

@mmaka1 mmaka1 Awaiting requested review from mmaka1 mmaka1 is a code owner

@lbetlej lbetlej Awaiting requested review from lbetlej lbetlej is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lyakh @lgirdwood @kv2019i