add rom_reset_usb_boot_extra which supports >32 pins and ACTIVE_LOW #2084
Conversation
…change existing uses to use this
| * - 2 To disable the USB PICOBOOT Interface | ||
| * \param usb_activity_gpio_pin_active_low Activity GPIO is active low (ignored on RP2040) | ||
| */ | ||
| void __attribute__((noreturn)) rom_reset_usb_boot_extra(int usb_activity_gpio_pin, uint32_t disable_interface_mask, bool usb_activity_gpio_pin_active_low); |
There was a problem hiding this comment.
rom_reset_usb_boot has uint32_t usb_activity_gpio_pin_mask, whereas rom_reset_usb_boot_extra has int usb_activity_gpio_pin. This might be a stupid question, but given the similarity of these two functions, would it make sense to modify rom_reset_usb_boot_extra to accept a pin_mask value too?
Given that the OTP |
they are only used when the corresponding PIN define is set |
I noticed this in passing - this change also adds relevant PICO_CONFIG to specify the ACTIVE_LOW default (note if the ACTIVITY_LED is not specified, the OTP defaults will be used on RP2350)