Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for RP2040 with 16MB of flash #59

Open
Fede-26 opened this issue Mar 11, 2024 · 2 comments
Open

Add support for RP2040 with 16MB of flash #59

Fede-26 opened this issue Mar 11, 2024 · 2 comments

Comments

@Fede-26
Copy link

Fede-26 commented Mar 11, 2024

I want to add support for this board: https://it.aliexpress.com/item/1005003657583800.html.
It's a clone of the official pi pico with 16MB of flash.

I can't find how to specify the ROM size like the (pimoroni-pico-lipo-16mb)[https://github.com/rp-rs/rp-hal-boards/tree/main/boards/pimoroni-pico-lipo-16mb].

I believe this is the only change necessary, because when using the C sdk, I just need to copy the official board config and change the ROM size.

@jannic
Copy link
Member

jannic commented Mar 18, 2024

I don't have a board with 16MB flash to test this, but as far as I know, all you need to do is update memory.x accordingly.
However, there's currently no mechanism to have a BSP specific memory.x.

In fact, it should still work if you don't update it. You just wouldn't be able to use all that flash, but only the first 2MB.

@Fede-26
Copy link
Author

Fede-26 commented Mar 19, 2024

Then why the readme (and the specific documentation in the relative directory) says

Note that if you use this crate the compiler will expect the full 16MB flash space, and so it may not work if you only have the 4MB variant.

?

If you can't change the flash size of each different bsp shouldn't the compiler expect a normal 2MB flash?

For now I think you can just change the memory.x file on the pc accordingly to the size of the flash installed.
If that's the only way to change this parameter I believe it should be written in the readme.
In this case I can make a pr later.

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

No branches or pull requests

2 participants