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.

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

added Xerxes RP2040 board to includes #1459

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

theMladyPan
Copy link
Contributor

added header file for metrotech's xerxes board.

xerxes-rp2040 board is open source used to produce various IoT sensors.

Thank you in advance for considering this addition.

@kilograham kilograham added this to the 1.5.2 milestone Jul 19, 2023
@kilograham kilograham merged commit 4e3951d into raspberrypi:develop Dec 15, 2023
mungewell pushed a commit to mungewell/pico-sdk that referenced this pull request Mar 18, 2024
@lurch
Copy link
Contributor

lurch commented Aug 15, 2024

@theMladyPan I noticed (whilst working on #1814 ) that the metrotech_xerxes_rp2040.h header has a

#ifdef __cplusplus
}
#endif // __cplusplus

section near the bottom of the file, but it doesn't have a corresponding #ifdef __cplusplus section near the top of the file, which seems odd? 😕

@theMladyPan
Copy link
Contributor Author

dear @lurch

Thank you for pointing out this problem. It looks like we forget to remove this code block after some cleanup. It is obviously a error on our side. Could you recommend how should I fix this? Open a new PR?

@lurch
Copy link
Contributor

lurch commented Aug 15, 2024

Yes, opening a new PR would be best, thank you! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants