Add support for Raspberry Pi Pico 2, Pico 2 W #377
Conversation
chrisib
added
firmware
|
Adding the Assuming that testing goes well, I'll remove the |
There is already a note about clones:
I can add another line about the Pico 2 being usable with additional configuration. |
Ah my bad, I forgot about the original BOM note. That addition is perfect! |
|
Had to disable the temperature check in the diagnostic script because of micropython/micropython#15687 Hopefully we can re-enable this feature later. |
…actual frequency, add "pico2" as a new value for PICO_MODEL
… I'm waiting for mine to ship and this is all considered placeholder for now.
…r linter warnings
…t hang on the RP2350. Hopefully this can be reverted with a future RP2350 Micropython implementation (breaks in 1.24.0-preview.201)
…o 2; it's not currently usable and causes an unhandled exception when used
…g new ones. Ask for 5 and 10V for the simple calibration, but support skipping if the rack can't supply one or the other (most cases should be able to produce at least one). Allow skipping voltages during advanced calibration too
…strings, remove unused functions. Print the desired voltage with 1 decimal place and units
…just use the ones from europi.py
… to resolve the issue with _thread we were previously encountering
|
Rebased to latest A few issues, like the temperature sensor and USB-connected detectors, are fixed in the Calibration update (#379), so I'll keep this as a draft until that gets merged. |
…date branch into here, since they're needed for Pico2 support
|
@roryjamesallen: I've made a few changes since you gave your approval. Rather than tying this up waiting for the calibration update, I've copied the relevant changes from over there into this branch. I think we're in a state where we can merge this in; I've been using one of my EuroPi modules with the RP2350 in it full-time for a few months without any issues. I'll hold off merging until you have a chance to re-approve since there have been changes though. |
software/firmware/europi.py
Outdated
| @return The current temperature in Celsius, or None if the hardware did not initialze properly | ||
| """ | ||
| if self.pin: | ||
| # see the pico's datasheet for the details of this calculation |
There was a problem hiding this comment.
[Spelling]
Capital letter for See and Pico
There was a problem hiding this comment.
Done, though this comment was copied verbatim from the diagnostic script. How picky are we in general about punctuation and caps in explanatory comments?
…; any slower than this and CPU-intensive scripts (e.g. Pam's) may start having performance issues.
…riant (pico2 was the exception before). Add Pico 2W to the models (treated identically to Pico 2)
|
Since this is still open I've made a few formatting changes (more named constants, fewer string literals), added support for underclocking, and added the Pico 2W to the supported models. |
chrisib
changed the title
pico2toeuropi_config.PICO_MODELoptions