Skip to content

Conversation

@BujSet
Copy link
Contributor

@BujSet BujSet commented Jul 23, 2025

Summary

Previously the PAL file was uploaded as a placeholder for Zephyr apps to use. However, building arm_zephyr.o object was unsuccessful. The PAL has been updated and can now be used and linked to zephyr executorch applications.

Test plan

Tested on a simple add model arm executor runner, see here for a concrete example. CI tests are WIP, see here.

@pytorch-bot
Copy link

pytorch-bot bot commented Jul 23, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/12759

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure, 7 Pending

As of commit 9c578db with merge base 503db13 (image):

NEW FAILURE - The following job has failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 23, 2025
@BujSet BujSet self-assigned this Jul 23, 2025
@BujSet
Copy link
Contributor Author

BujSet commented Jul 23, 2025

@pytorchbot label "release notes: none"

@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Jul 23, 2025
@BujSet BujSet marked this pull request as ready for review July 23, 2025 20:58
@BujSet BujSet force-pushed the functional_arm_zephyr_pal branch from de7ee5d to 9c578db Compare July 23, 2025 21:06
@BujSet BujSet merged commit ec9b0c0 into pytorch:main Jul 23, 2025
98 of 99 checks passed
@BujSet BujSet deleted the functional_arm_zephyr_pal branch July 23, 2025 22:11
Conarnar pushed a commit to Conarnar/executorch that referenced this pull request Jul 25, 2025
### Summary
Previously the PAL file was uploaded as a placeholder for Zephyr apps to
use. However, building `arm_zephyr.o` object was unsuccessful. The PAL
has been updated and can now be used and linked to zephyr executorch
applications.

### Test plan
Tested on a simple add model arm executor runner, see
[here](https://github.com/BujSet/zephyr/blob/5801a783883f3cf9bb172213d891c0de596c3dcf/samples/modules/executorch/arm-fvp-tutorials/models/CMakeLists.txt#L61)
for a concrete example. CI tests are WIP, see
[here](pytorch#12734).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants