Merged
Conversation
soburi
changed the title
soburi
force-pushed
the
variant_name
branch
14 times, most recently
from
67ea236 to
194d31a
Compare
soburi
requested review from
DhruvaG2000,
beriberikix and
szczys
as code owners
soburi
force-pushed
the
variant_name
branch
12 times, most recently
from
c960140 to
584d40f
Compare
soburi
force-pushed
the
variant_name
branch
7 times, most recently
from
17f4c2e to
3fe9c4a
Compare
soburi
changed the title
soburi
changed the title
Use the checkpatch.pl included with Zephyr to perform the check. Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Changed boards to be specified by their normalized names. Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Modify to use west.yml included in the PR version. Accordingly, the initialization process has been changed. Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Added cmsis_6 to the repository in line with the update of Zephyr. Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Apply newest zephyr version Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Get the variant name (NORMALIZED_BOARD_TARGET) from the Zephyr build system as early as possible. This allows to have per-target build directories. Note that a bug with the shield specifiers does not currently allow to reuse the same build directory multiple times. Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com> Co-authored-by: Luca Burelli <l.burelli@arduino.cc>
Member
Author
|
Could you take a look, please? |
Ayush1325
reviewed
Aug 25, 2025
Member
There was a problem hiding this comment.
Can you explain why the changes to license_check are required?
Member
Author
There was a problem hiding this comment.
The github action is stuck with the current version.
It was originally brought over from the Zephyr main body, so I applied the current one.
Updated commit message.
Ayush1325
approved these changes
Aug 26, 2025
soburi
force-pushed
the
variant_name
branch
2 times, most recently
from
484a5ea to
eb778d1
Compare
Member
Author
|
@DhruvaG2000 Could you take a look, please? |
DhruvaG2000
approved these changes
Sep 7, 2025
Member
DhruvaG2000
left a comment
DhruvaG2000
left a comment
There was a problem hiding this comment.
Alot has changed of recent in zephyr seems like.
Thanks for fixing CI!
soburi
pushed a commit
to soburi/ArduinoCore-zephyr
that referenced
this pull request
Sep 24, 2025
…update Upate Camera support to reflect changes in API
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI is failing for several reasons.
We will fix these.
Also, west.yml references the main repository,
not the one in the PR, so we will change it to use the one in the PR.
This workflow uses the Linux version of
checkpatch.pl.We change it to zephyr's version.