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

Add Pico RP2040 (non-mbed) support #36

Merged
merged 4 commits into from
Nov 21, 2023

Conversation

adisbladis
Copy link
Contributor

@adisbladis adisbladis commented Mar 30, 2023

This adds support for RP2040 based boards such as the Pico and the Pico W using arduino-pico.
Note that this is distinctly different from the mbed-OS based platform which would have to be implemented separately.

I have tested this on a Pico W.

Github Actions related changes haven't been tested, only actually running the code on my board.

@adisbladis adisbladis force-pushed the pico-rp2040 branch 3 times, most recently from d9e0410 to 95856b1 Compare March 30, 2023 08:45
@adisbladis adisbladis changed the title Add RP2040 support Add Pico RP2040 (non-mbed) support Mar 30, 2023
@adisbladis
Copy link
Contributor Author

Ping @fabianoriccardi

@adisbladis adisbladis force-pushed the pico-rp2040 branch 3 times, most recently from 0dfeb0a to ecf3b32 Compare November 4, 2023 03:20
@fabianoriccardi
Copy link
Owner

I will test it the next week with the Pico, however I see that some examples don't compile!

@adisbladis
Copy link
Contributor Author

It's very hard to test CI locally because the Github Actions yml files don't exactly correspond to a set of commands that I can run.
The examples compile just fine, it's just hard to figure out the exact set of platform/url/board strings appease the CI.

@adisbladis adisbladis force-pushed the pico-rp2040 branch 3 times, most recently from a508b4b to c2ec43e Compare November 5, 2023 03:38
This adds support for RP2040 based boards such as the Pico and the
Pico W using arduino-pico.

Note that this is distinctly different from the mbed-OS based platform which would have to be implemented separately.

I have tested this on a Pico W.
@fabianoriccardi
Copy link
Owner

Did you tested RPI Pico with PlatformIO?

@adisbladis
Copy link
Contributor Author

Did you tested RPI Pico with PlatformIO?

Yes. My primary use case for this is my espresso machine where I use platformio to build.

@fabianoriccardi
Copy link
Owner

The same issue appears with your project. Here I have opened the issue.

@fabianoriccardi
Copy link
Owner

May I add your name in license of hw_timer_pico.cpp/h files?

@adisbladis
Copy link
Contributor Author

May I add your name in license of hw_timer_pico.cpp/h files?

Feel free.

@fabianoriccardi
Copy link
Owner

I prefer to insert a real name but you don't use it on GitHub...

@adisbladis
Copy link
Contributor Author

My real name is Adam Hoese.

@fabianoriccardi fabianoriccardi merged commit dda54f7 into fabianoriccardi:main Nov 21, 2023
7 checks passed
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.

2 participants