Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork8.2k
Add Pico 2 W port and update Pico SDK version#16313
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
I've split these changes out from this PR#16057 |
codecovbot commentedNov 27, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@## master #16313 +/- ##======================================= Coverage 98.57% 98.57% ======================================= Files 164 164 Lines 21349 21349 ======================================= Hits 21045 21045 Misses 304 304 ☔ View full report in Codecov by Sentry. |
github-actionsbot commentedNov 27, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Code size report:
|
There was a problem hiding this 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 a relatively concise PR!
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
ea6014a
to2d4b02b
CompareFor anyone that wants to test but doesn't want to build the firmware, here is azipped UF2 file for the ARM build of2d4b02b. |
lgb-this commentedDec 12, 2024
I have a project with different topics (webserver, I2C, buttons, asynio, WLAN, MQTT, neo led, logging, ...) and this firmware runs stable for me. I could run the project without any modifications from the Pico W. |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
projectgus left a comment• edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
These changes look good to me, thanks@peterharperuk . I don't have a 2 W to run this code myself, but I've ordered one.
Do I understand correctly that this pico-sdk version still has a buggy alarm pool, but after we update to 2.2.0 then we'd be able to switch the soft timer implementation back to the alarm pool if we wish? (Thanks to your PRraspberrypi/pico-sdk#2127)
Uh oh!
There was an error while loading.Please reload this page.
I did a full test of this PR on:
Everything seems to work as it should except the following:
|
Should we include unittest in the list of installed modules? To avoid having to remember to manually install it to run the tests |
I'd rather not do that. The unittest package is pretty large when frozen in to firmware and not all ports have the space for it. Also, it's not a very common package for users to use, so takes up flash space unnecessarily. If you're using a particular board for running the MicroPython test suite, it's relatively easy to do |
Brings in support for Pico 2 W, among other things.Signed-off-by: Peter Harper <peter.harper@raspberrypi.com>
Signed-off-by: Peter Harper <peter.harper@raspberrypi.com>
Signed-off-by: Peter Harper <peter.harper@raspberrypi.com>
The cyw43 pins used to be hardcoded and `CYW43_PIN_WL_HOST_WAKE` and`CYW43_PIN_WL_REG_ON` were in the `pico_w.h` board header.This has been changed so the board header just defines the "defaultversion of the pins, e.g. `CYW43_DEFAULT_PIN_WL_HOST_WAKE`,`CYW43_DEFAULT_PIN_WL_REG_ON` etc.Set the pin values in `cyw43_configport.`h so `cyw43-driver` sees them andallow them to be changed at runtime (dynamic) if required.Signed-off-by: Peter Harper <peter.harper@raspberrypi.com>
As cyw43 pins might be dynamic, add a function that returns if a pin isreserved. This is used by `MICROPY_HW_PIN_RESERVED` to prevent the pin IRQfrom being reset across a soft-reset.Signed-off-by: Peter Harper <peter.harper@raspberrypi.com>
Signed-off-by: Peter Harper <peter.harper@raspberrypi.com>
Build with: make BOARD=RPI_PICO2_W BOARD_VARIANT=RISCVSigned-off-by: Peter Harper <peter.harper@raspberrypi.com>
I squashed down the last two changes you made@peterharperuk . This looks good now, it's very clean thank you! |
a70a90c
intomicropython:masterUh oh!
There was an error while loading.Please reload this page.
Summary
Add Pico 2 W port and changes made in pico-sdk 2.1.0. This fixes issues with lightsleep associated with sev handling and alarm pool problems.
Testing
Ran all the micropython tests