Skip to content

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

Merged
dpgeorge merged 1 commit into
micropython:masterfrom
robert-hh:samd_metro_m4
May 23, 2023
Merged

samd: Add the boards files for Adafruit Metro M4 Express Airlift.#11220
dpgeorge merged 1 commit into
micropython:masterfrom
robert-hh:samd_metro_m4

Conversation

@robert-hh
Copy link
Copy Markdown
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
Copy Markdown
Member

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

Comment thread drivers/ninaw10/nina_wifi_bsp.c Outdated
mp_hal_pin_output(MICROPY_HW_NINA_GPIO0);
#ifdef MICROPY_HW_NINA_GPIO0
mp_hal_pin_input(MICROPY_HW_NINA_GPIO0);
#endif
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

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

@robert-hh
Copy link
Copy Markdown
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