Skip to content

samd: Add the boards files for Adafruit Metro M4 Express Airlift. #11220

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

Merged
merged 1 commit into from
May 23, 2023

Conversation

robert-hh
Copy link
Contributor

The Wifi support by MicroPython is in PR #11219. The board files can be used for both Adafruit Metro M4 variants.

@dpgeorge
Copy link
Member

Is this blocked on #11219, or can it be merged without that?

mp_hal_pin_output(MICROPY_HW_NINA_GPIO0);
#ifdef MICROPY_HW_NINA_GPIO0
mp_hal_pin_input(MICROPY_HW_NINA_GPIO0);
#endif
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please split these changes into a separate commit (so there's one commit for the ninaw10 changes, and one commit to add the new board).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll remove this file from the PR.

@robert-hh
Copy link
Contributor Author

robert-hh commented Apr 11, 2023

Is this blocked on #11219, or can it be merged without that?

The intention was to make it independent. So I'll remove the change to the ninaw10 files. They are anyhow part of PR #11219.

@robert-hh
Copy link
Contributor Author

Updated.

@robert-hh robert-hh force-pushed the samd_metro_m4 branch 4 times, most recently from 155a8a9 to c97e277 Compare April 20, 2023 07:46
The board files can be used for both Adafruit Metro M4 variants.  The Wifi
support is in progress.
@dpgeorge dpgeorge merged commit c3f031a into micropython:master May 23, 2023
@dpgeorge
Copy link
Member

Now merged. WiFi support can be enabled later, once #11219 is done.

@robert-hh
Copy link
Contributor Author

Thanks. I just added the definitions for the WiFi pins.

@robert-hh robert-hh deleted the samd_metro_m4 branch May 23, 2023 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants