Skip to content

Conversation

@danicampora
Copy link
Member

Simple implementation in-line with the rest of the Micropython ports.

@jonnor
Copy link
Contributor

jonnor commented Mar 3, 2025

Very nice! What hardware have you tested this on? Any particular concerns if I would want to test this on hardware? I think I have an NRF52 available, at least.

@danicampora
Copy link
Member Author

danicampora commented Mar 4, 2025

Thanks! I tested it on the nRF52832 and the nRF5340.

Copy link
Member

@dpgeorge dpgeorge left a comment

Choose a reason for hiding this comment

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

Thanks for this, it's nicely integrated into the common extmod bindings.

@danicampora
Copy link
Member Author

@dpgeorge made the changes you requested and re-pushed.

Simple implementation in-line with the rest of the MicroPython ports

Tested on the nRF52832 and the nRF5340.

Signed-off-by: danicampora <danicampora@gmail.com>
@dpgeorge dpgeorge merged commit 859b6ef into micropython:master Mar 5, 2025
7 checks passed
@dpgeorge
Copy link
Member

dpgeorge commented Mar 5, 2025

Thanks for updating, now merged.

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.

3 participants