-
Notifications
You must be signed in to change notification settings - Fork 8.4k
doc: develop: manifests: external: add arduino core #97726
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.
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
base: main
Are you sure you want to change the base?
Conversation
4d9735b to
81d384c
Compare
e43b494 to
09b3789
Compare
982677b to
7676c2f
Compare
| #. Complete the core setup by linking the API folder from the ArduinoCore-API repository into | ||
| the arduinocore-zephyr folder: | ||
|
|
||
| .. code-block:: bash | ||
|
|
||
| git clone https://github.com/arduino/ArduinoCore-API # Any location | ||
| cd /<path>/<to>/<zephyrproject>/modules/lib/arduinocore-zephyr | ||
| ln -s /<your>/<location>/arduinocore-zephyr/api cores/arduino/. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this supposed the be the "manual steps" mentioned in the previous bullet point? If yes it probably needs to be a sub-bullet or a .. note or something since otherwise it's not clear users who've run the install script can skip this
What's more, it's supposed to be instructions for non Linux users and yet it uses ln -s which won't be available to windows users?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's more, it's supposed to be instructions for non Linux users and yet it uses ln -s which won't be available to windows users
I haven't really tried using Arduino Core Zephyr on windows, I meant these steps for people with non standard paths/ environments. I will try to reword all of it to reduce ambiguity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kartben can you take a look at the note I added now, does it make sense?
Thanks for all the reviews so far!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recommend that you follow the getting started guide's method of providing platform-specific installation instruction tabs instead of this. The current approach feels kind of hacky by comparison and the infrastructure to do a better job is already there in the docs build system.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I said above too,
will update my docs according to zephyrproject-rtos/arduino-core-zephyr#139 now, we can just use a single west command and all this confusion should go away.
Thanks @soburi !
Add documentation for the Arduino Core API for zephyr which sits as an external module as of today. Signed-off-by: Dhruva Gole <[email protected]>
7676c2f to
c8db58a
Compare
|
| =============================================== | ||
|
|
||
| #. To pull in the Arduino Core for Zephyr as a Zephyr module, either add it as | ||
| a West project in the west.yaml file or pull it in by adding a submanifest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
west.yml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ack, will fix
| #. To pull in the Arduino Core for Zephyr as a Zephyr module, either add it as | ||
| a West project in the west.yaml file or pull it in by adding a submanifest | ||
| (e.g. ``zephyr/submanifests/arduino-core.yaml``) file with the following content | ||
| and run west update: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line duplicates the next step.
| #. Complete the core setup by linking the API folder from the ArduinoCore-API repository into | ||
| the arduinocore-zephyr folder: | ||
|
|
||
| .. code-block:: bash | ||
|
|
||
| git clone https://github.com/arduino/ArduinoCore-API # Any location | ||
| cd /<path>/<to>/<zephyrproject>/modules/lib/arduinocore-zephyr | ||
| ln -s /<your>/<location>/arduinocore-zephyr/api cores/arduino/. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recommend that you follow the getting started guide's method of providing platform-specific installation instruction tabs instead of this. The current approach feels kind of hacky by comparison and the infrastructure to do a better job is already there in the docs build system.
| Supported Boards | ||
| ================ | ||
|
|
||
| The Arduino Core API module currently has variants for these boards: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is going to go stale. What is your maintenance plan?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm you're right, I might rather add a link to the variants folder of the module directly, that might make more sense @mbolivar ?
|
|
||
| #. Create a new folder in the ``variants/`` directory with your board's name | ||
| #. Add an overlay file and a pinmap header file that match the board name | ||
| #. Add your new headerfile to an ``#ifdef`` statement in the ``variant.h`` file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
header file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ack, will fix



Add documentation for the Arduino Core API for zephyr which sits as an external module as of today.