-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Pico W pin names and functions need tweaking? #7017
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
Comments
As far as I understand it, that's incorrect.
SMPS mode should be referring to CYW GPIO 1 already. I think this was a part of #6960. It would be good to fix up the remaining items you mention. I would be happy to see someone else take it on, micropython should have good code to study. |
You're absolutely right, sorry, I completely missed it this time around. It's right there in board and it's a settable |
...but I think the second line: |
Gp23 is not for use of CP code. But old code that used this name to refer to the function will work if this name is provided. If it's better to remove this name that's fine by me. |
I'm not sure, the docs say Non-W Pico code that references special pins by number, (e.g., |
No, you'd just crash things if you could externally reset the wifi chip. |
That's unfortunate. So we probably want to enforce using these special pins by name only on both boards, so code for one runs on the other? Use the named aliases for the pins, and they will (in future) map to the correct non-W pin number, or W pin number or CywPin number. |
I've created a separate issue for the vsys measurement (board.VOLTAGE_MONITOR) aspect: #7020 |
Closes: adafruit#7017 * Remove the 'GP23' alias for CYW1 * Remove the 'CYW0' alias for CYW0 * Switch VBUS_SENSE to CYW2, remove 'GP24' alias Code that wants to use SMPS_MODE, VBUS_SENSE and LED while being portable to the W and non-W variants should use those names, not alias names. * Remove A3 / VOLTAGE_MONITOR Right now this cannot be used. The ability to check the voltage monitor should be added back in some fashion in the future.
Closes: adafruit#7017 * Remove the 'GP23' alias for CYW1 * Remove the 'CYW0' alias for CYW0 * Switch VBUS_SENSE to CYW2, remove 'GP24' alias Code that wants to use SMPS_MODE, VBUS_SENSE and LED while being portable to the W and non-W variants should use those names, not alias names. * Remove A3 / VOLTAGE_MONITOR Right now this cannot be used. The ability to check the voltage monitor should be added back in some fashion in the future.
Uh oh!
There was an error while loading. Please reload this page.
CircuitPython version
Code/REPL
Behavior
Pin names and functions from p.8:
https://datasheets.raspberrypi.com/picow/pico-w-datasheet.pdf
GPIO23 OP wireless power on signal
Not in
microcontroller
. Not sure if it's intended to be read-only, or if it could be used to hard-reset the wifi module from user code.GPIO24 IP wireless SPI data/IRQ
microcontroller.pin.GPIO24
board.GP24 board.VBUS_SENSE
This doesn't seem right.
ValueError: VBUS_SENSE in use
. It isn'tVBUS_SENSE
. Also, user code probably shouldn't have access to this pin?GPIO25 OP wireless SPI CS- when high also enables GPIO29 ADC pin to read VSYS OP
Not in
microcontroller
. Not sure how to use this to readVSYS
since we don't want to mess upwireless SPI
.GPIO29 OP/IP wireless SPI CLK/ADC mode (ADC3) to measure VSYS/3
microcontroller.pin.GPIO29
board.A3 board.VOLTAGE_MONITOR
Yes, but "ValueError: A3 in use" and we can't read it directly w/o prior use of
GPIO25
edit: (p.12)
Then, we also have these pins exposed (to the SDK) from the wifi processor:
WL_GPIO0 OP connected to user LED
Exposed to user and works great!
WL_GPIO1 OP controls the on-board SMPS power save pin (Section 3.4)
This is not exposed to user directly. I suspect it's used in the driver via
cyw43.set_power_management()
WL_GPIO2 IP VBUS sense - high if VBUS is present, else low
This seems to be our
VBUS_SENSE
, but it's not exposed to user code.Description
Additional information
WL_GPIO1
also potentially useful for ADC:I didn't spot this in
analogio
common-hal
, perhaps as a later enhancement, if needed.The text was updated successfully, but these errors were encountered: