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

Some boot_stage2 of RP2350 still reference to SSI and its registers #2126

Open
matsobdev opened this issue Dec 5, 2024 · 4 comments
Open

Comments

@matsobdev
Copy link

Those files are:
https://github.com/raspberrypi/pico-sdk/blob/master/src/rp2350/boot_stage2/boot2_at25sf128a.S
https://github.com/raspberrypi/pico-sdk/blob/master/src/rp2350/boot_stage2/boot2_is25lp080.S
https://github.com/raspberrypi/pico-sdk/blob/master/src/rp2350/boot_stage2/boot2_w25x10cl.S

I guess, those devices are not commonly used and I think hardware RP2350's flash support might take care of them or at least boot2_generic_03h will be compatible, so I would call it a minor issue.

@lurch
Copy link
Contributor

lurch commented Dec 5, 2024

I guess this wasn't noticed earlier, because RP2350 builds don't use a boot_stage2 by default; you have to explicitly request it.

@matsobdev
Copy link
Author

I've just noticed, they have:

// todo port to QMI

so I think it is on somebody's mind already :D

@michael-ring
Copy link

michael-ring commented Dec 7, 2024 via email

@lurch
Copy link
Contributor

lurch commented Dec 8, 2024

boot_stage2 was used as default for RP2040, what is used now for RP2350?

On page 649 of https://datasheets.raspberrypi.com/pico/raspberry-pi-pico-c-sdk.pdf in the release-notes for SDK 2.0.0 it says:

boot_stage2

  • There are now separate implementations for RP2040 and RP2350.
  • A boot_stage2 is not needed on RP2350, but one can be included via the define PICO_EMBED_XIP_SETUP=1.

You'll probably want to have a look at Chapter 5 and Section 12.14 of https://datasheets.raspberrypi.com/rp2350/rp2350-datasheet.pdf for more information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants