TODO: Review the content and ensure it is consistent with rp235x

[ithinuel]

This repo is an adaptation of the rp2040-project-template.
I tried to make sure all the content is in line with the requirements for rp235x but I likely missed points.

If you find some, please add them here.

[chaosprint]

perhaps keep an eye on JoNil/elf2uf2-rs#39

[steverpalmer]

memory.x includes "2 MiB is a safe default here, although a Pico 2 has 4MiB", and then sets the length of the flash to "2048K".

This is not wrong. However, is there any reason why 4MiB is unsafe for a Pico 2 / RP235x?

(I've added the comment here, since this feels like a "non-adaption" from the rp2040 Pico case.)

[steverpalmer]

memory.x includes comments about RAM banks 8 and 9 (which are correct), but then defines SRAM4 and SRAM5. Certainly, the RP2040 does identify these are SRAM4 and SRAM5, and there may be good reasons for keeping these names consistent. However, if it doesn't break too much, then might some better names be used?

As the comment states, these blocks of memory are fair candidates for separate stacks for core0 and core1, but the name might not presupposed that this is what they will be used for. Perhaps "SRAM_DIRECT_A" and "SRAM_DIRECT_B" since the data-sheet uses the term "map directly". I've suggested "A" and "B" rather than "0" and "1" to avoid any presumption off relations ship to cores 0 and 1, but this could be a bad choice.