ChibiOS Free Embedded RTOS

@xyzzy I applied a modified version of your change today. The mask is used only for an osalDebug so for non-debug builds no overhead. I might just remove it completely though as I tend to agree with your point that we should just always trust the engineer knows what they are doing. PTAL and let me ...

So far I've avoided adding flashing in any of the Makefiles as I think its too opinionated and should be left outside of ChibiOS. Putting it in ChibiOS means getting opinionated about things like BOOTSEL + PicoBoot, BOOTSEL + UF2 copy, debug probe, production flash programming, etc.

(Their is a ...

I will take a look tomorrow, goal for this weekend is to make a refactor of both PAL and PIO LLDs.

- E

I will take a look tomorrow after I finish cleaning up the PAL LLD.

- E

Yes, the PIO LLD is missing a few functions, a patch would be great. If not I will make a pass at is likely later this week.

The existing implementation was primarily motivated as a POC for the CMSIS-DAP example. But that also means most of the support is currently catered to output.

So the new RP2350 and updated RP2040 ports are taking shape and you've started to use it for your project. Suddenly you find an inconsistency or something that simply doesn't work. Well, it's probably my fault. The work has been a pretty significant set of changes and there are likely some latent ...

Thx, was offsite working occasionally and didn't see your changes. All looks good, thank you!

os/common/ports/ARMv8-M-ML-ALT/chcore.h
Ok, I think the safest fix is to keep the existing defaulting of CH_DBG_ENABLE_STACK_CHECK to true on RP2350. I do worry that a user will see DBG and may think they can disable it which wont work on the RP2350. Maybe we could allow PORT_SAVE_PSPLIM to be set ...

I should say even with this issue I think my approach to handling PSPLIM might be better as it never disables PSPLIM if CH_DBG_ENABLE_STACK_CHECK is true and should work happily if PSPLIM is set in CRT0.

Ok, let me think about this for a bit. Basically the current approach is broken on non-unified memory architectures like the PR2350 but works well for unified ones. The current 'fix' for RP2350 is forcing CH_DBG_ENABLE_STACK_CHECK always on, which forces PORT_SAVE_PSPLIM to true. It works but then ...