rom_reset_usb_boot does not function when Watchdog is enabled #2689
Labels
No labels
bazel
bluetooth
board settings
boot_stage2
bug
build
c++
can't reproduce
clang
clib-interface
cmsis
code cleanliness
code size
compiler version issues
cyw43_driver
debug
discussion
documentation
duplicate
dynamic_clock
elf2uf2
enhancement
errata
filesystem
flash layout
floating point
FreeRTOS
hardware_adc
hardware_claim
hardware_clocks
hardware_dma
hardware_exception
hardware_flash
hardware_gpio
hardware_i2c
hardware_irq
hardware_pio
hardware_pwm
hardware_rcp
hardware_rtc
hardware_spi
hardware_sync
hardware_uart
hardware_vreg
hardware_watchdog
hardware_xosc
help wanted
host
IAR
installation
invalid
linker scripts
lto
lwip
mbedtls
missing API
needs reporter detail
networking
newlib
newlib-multicore
non-c
non-cmake
non-gcc
openocd
orphan PR
OS integration
pico_async_context
pico_base
pico_binary_info
pico_flash
picolibc
pico_lwip
pico_malloc
pico_multicore
pico_printf
pico_runtime
pico_stdio
pico_stdio_usb
pico_sync
pico_time
pico_util
pioasm
port
question
review
review2
rtos interop
runtime init
static library
svd
test
tinyusb
vscode
wifi integration
wontfix
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
gh-raspberrypi/pico-sdk#2689
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
On SDK 2.2.0 if you enable the watchdog then calling rom_reset_usb_boot just halts the calling core but does not trigger bootsel mode. This doesn't seem to be mentioned in the documentation and seems an unusual side effect.
It looks like on RP2350
rom_reset_usb_booteventually callsrom_rebootwhich is documented as "Reboot using the watchdog. Resets the chip and uses the watchdog facility to restart.", so maybe that's the "smoking gun" as to why this isn't working? (But I've no idea if this is a deliberate side-effect or an unintentional side-effect.)Good spot. Would explain it as I have the other RP2350 core resetting the watchdog timer.
With this information I could write a workaround so the core resetting the watchdog is the one to trigger bootsel.
In terms of issue, either the side effect needs to be documented or a fix put in place.
I think the full explanation here is that the core calling
rom_rebootsets the watchdog up for a BOOTSEL reboot, then loads 10ms into the watchdog and goes__wfi()to wait for the reboot. If the other core prevents that watchdog reboot (by eg reloading the watchdog) then the calling core just remains in__wfi(). Fix is to not use the watchdog for 2 things at the same time, as there's only 1 watchdog.Thanks @will-v-pi . I agree with @jondurrant that there ought to be a warning about this in the docs, especially as https://www.raspberrypi.com/documentation/pico-sdk/runtime.html#group_pico_bootrom_1ga1bbe1c1df02daaa7b68a0c95fb9dc545 doesn't mention the watchdog at all.