-
Notifications
You must be signed in to change notification settings - Fork 444
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
Feature Request: Add support for CheckPicoW() for the Pico2/2W #2671
Comments
I commented out the PicoW check on the RP2350 boards because it uses some special analog tests that were valid on the PicoW. I am not certain they work properly on the Pico 2W (don't have one yet) or if it works on boards with the CYW43 RP-WiFi module like the SparkFun Thing Pro RP2350. If the RPi folks could provide some boilerplate code that would be really awesome here. The original code came from one of their appnotes... |
Schematic:
GPIO 25 and 29 circuits looks identical |
I just got my Pico2W's yesterday. I pulled the Edit: A bit more testing shows the ADC check seems to work fine, but the gp25 does not. |
Q1 is FET. I am curious why this code is trying to READ the state of this Gate rather than to set it to HIGH or LOW in order to switch the FET On or Off. Inappropriate GPIO 25 READ value can be caused by this RP2350 errata: |
Check arduino-pico/cores/rp2040/cyw43_wrappers.cpp Lines 114 to 115 in 0061d3f
|
Currently the CheckPicoW() is guarded with
#ifdef PICO_RP2040
- elsetrue
. This will incorrectly returntrue
on a Pico 2 (non-W).It would be great if this runtime check could be made to work with each of the Pico/Pico-W/Pico2/Pico2-W versions.
I appreciate all the time and effort you put into maintaining this project, Thanks!
The text was updated successfully, but these errors were encountered: