-
Notifications
You must be signed in to change notification settings - Fork 61
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
gpio: Enable all GPIO for all targets #66
gpio: Enable all GPIO for all targets #66
Conversation
Seems I broke something on the stm32l0x2. I'll check it out tomorrow. |
It looks like even GPIOE and GPIOH are present on the bigger L0x1 chips, so maybe all the |
Ah, you're right, I missed that! |
According to the reference manuals for the 0x1¹, 0x2² and 0x3³ series, the GPIO A/B/C/D/E as well as H[0:1][9:10] are available on all devices (given a class 5 device). ¹ Reference manual RM0377 ² Reference manual RM0376 ³ Reference manual RM0367
6d5f51a
to
9dfb6c5
Compare
It wasn't in my code, but I fixed it anyways.
Verified against the reference manuals and fixed. Do you plan to limit the availability of GPIO pins depending on the package in the future? |
I can't speak to anyone's plans, but I'd like to note there's precedent for going full bananas with packages: https://github.com/stm32-rs/stm32l0xx-hal/blob/68eeba6cd8c71a22b365d15dfe4ae9c4bb62888a/src/adc.rs#L223-#L299 |
This is technically true, but I think there is no released L0x1 chip that actually has those ports, so maybe it doesn't make sense to expose them? |
Hm? There's the STM32L071V8 with an LQFP100 package. |
Oh, never mind then. I was confused by ST's product selector. |
Good catch! Thanks for fixing the serial dma async bug as well. Not sure how that one slipped through the cracks. |
For some reason, GPIOC and GPIOD were disabled for the stm32l0x1, but these GPIO banks are present in the PAC (and in the datasheet).