Skip to content

RP2350A-A2 USB BOOT activity LED workaround#2971

Merged
kilograham merged 10 commits into
developfrom
rom-reboot-docs
Jun 3, 2026
Merged

RP2350A-A2 USB BOOT activity LED workaround#2971
kilograham merged 10 commits into
developfrom
rom-reboot-docs

Conversation

@kilograham

Copy link
Copy Markdown
Contributor

Add docs and optional work around to rom_reset_usb_boot* for RP2350A2 QFN60 Arm to reboot into RISC-V when acitivity LED is requested

… QFN60 Arm to reboot into RISC-V when acitivity LED is requested

Also renamed rp2040_rom_version rp2350_rom_version for consistency (it doesn't appear to be used)
@kilograham kilograham added this to the 2.2.1 milestone May 26, 2026
@kilograham kilograham requested review from lurch and will-v-pi May 26, 2026 22:36
@dpgeorge

Copy link
Copy Markdown
Contributor

Looks reasonable to me! (I did not test it though.)

Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h Outdated
Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h Outdated
Comment thread src/rp2350/pico_platform/include/pico/platform.h
Comment thread src/rp2_common/pico_bootrom/bootrom.c Outdated
Comment thread src/rp2_common/pico_bootrom/bootrom.c Outdated
@lurch

lurch commented May 27, 2026

Copy link
Copy Markdown
Contributor

As you're editing these files anyway, feel free to roll #2622 into this PR 😀

Comment thread src/rp2_common/pico_bootrom/bootrom.c
Comment thread src/rp2350/pico_platform/include/pico/platform.h Outdated
@anoshyn

anoshyn commented May 27, 2026

Copy link
Copy Markdown
Contributor

Tested on my Pico 2 / RP2350A2 setup.

I built the repro with PICO_BOOTROM_WORKAROUND_A2_ACTIVITY_LED_BUG=1 and triggered the BOOTSEL vendor reset request on reset interface 2 with:

  • bmRequestType = 0x21
  • bRequest = 0x01
  • wValue = 0x3300
  • wIndex = 2

The device reset from app mode (2e8a:0009) into BOOTSEL (2e8a:000f), the RP2350 mass-storage volume appeared, and GPIO 25 activity LED was active in BOOTSEL.

So the workaround works for my setup.

Co-authored-by: Andrew Scheller <lurch@durge.org>
Co-authored-by: will-v-pi <108662275+will-v-pi@users.noreply.github.com>
Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h Outdated
@kilograham kilograham requested review from lurch and will-v-pi and removed request for will-v-pi May 28, 2026 20:45
Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h
Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h Outdated
Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h Outdated
Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h Outdated
Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h Outdated
@will-v-pi

Copy link
Copy Markdown
Contributor

This has inspired me to add functionality to use the BOOTSEL activity LED to picotool, to make it easier to test this and the other recent fix in this area - raspberrypi/picotool#316

Comment thread src/rp2_common/pico_bootrom/bootrom.c Outdated
kilograham and others added 2 commits June 1, 2026 13:07
Co-authored-by: will-v-pi <108662275+will-v-pi@users.noreply.github.com>
Co-authored-by: Andrew Scheller <lurch@durge.org>
…AROUND_RP2350_A2_ACTIVITY_LED_BUG and default it to 1 on RP2350
Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h Outdated
Comment thread src/rp2_common/pico_bootrom/include/pico/bootrom.h Outdated
Co-authored-by: will-v-pi <108662275+will-v-pi@users.noreply.github.com>
@kilograham kilograham requested a review from will-v-pi June 3, 2026 16:47
@kilograham kilograham merged commit 3d63b1d into develop Jun 3, 2026
13 checks passed
@kilograham kilograham deleted the rom-reboot-docs branch June 3, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants