ChibiOS Free Embedded RTOS

Hi,

I discovered a bug (or at least a not expected behaviour) in rp2040's watchdog driver. At the driver initialization it calculates the LOAD register's value from the config's `lrl` value (delay in ms) with a certain formula coming from the PICO SDK, but at the reseting it directly applies the `lrl` value (delay in ms) and causes inconsistencies. I have a commit which fixes this, I wonder if you accept it somewhere as a PR or rather fix it yourself?
Here is the commit, it helps to understand what I am talking about: https://github.com/ChibiOS/ChibiOS/compare/master...zsigmondszilveszter:ChibiOS:pr/RP-WDG-bugfix

Hi,
Thanks for the report.
Slightly different fix now applied to trunk on SF.
Please give it a test (don't have a Pico board handy at the moment).

--
Bob

Hi,
I am not that familiar with SVN. How and from where could I pull the code ?

Szilveszter

Here you go...
It will turn up on the GIT mirror also (whenever it syncs)

Also note there is now a patch submission site which can be found at…
https://review.chibios.org

The 'wdgp' has to be passed to the function or the global 'WDGD1' should be used.

The watchdog driver also needs to set something in the the PSM's (Power-on State Machine) WDSEL register, otherwise the watchdog doesn't reset anything.
Something like this:


Here is my Commit for an example.

Thanks for working on this.
I will find my RP2040s and set up for test soon.
In the mean time here is an updated driver.

--
Bob

It works now as expected. Thank you for the fix.

Szilveszter

Trunk now updated.
This report can be closed.