Skip to content

flash/nor/spi: add boyamicro byq16b #126

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

Open
wants to merge 1 commit into
base: sdk-2.0.0
Choose a base branch
from

Conversation

krisjdev
Copy link

@krisjdev krisjdev commented Mar 5, 2025

Adds support for BYQ16B, otherwise OpenOCD throws an error: Error: Unknown flash device (ID 0x00154068)

Chip markings: BYT BYQ16B MI2406

chip marking: BYT BYQ16B MI2406
@lurch
Copy link

lurch commented Mar 6, 2025

I suspect you probably want to add this to the upstream OpenOCD repo, rather than Raspberry Pi's downstream fork?
Or do you have some custom board based around RP2040 or RP2350 that happens to use this flash chip?

@krisjdev
Copy link
Author

krisjdev commented Mar 6, 2025

Not a custom board or anything like that, but I actually found it on a Pico clone. I ran into some issues with debugprobe not working so I thought the Raspberry Pi repo would be a good fit for this change, happy to try to move it to the OpenOCD repo if you think it would be better.

@lurch
Copy link

lurch commented Mar 6, 2025

but I actually found it on a Pico clone.

Interesting! Do you have a web-link to that product?

I thought the Raspberry Pi repo would be a good fit for this change

Yes, if there are Pico boards using this Flash chip, then the PR to this repo was the correct thing to do 👍 (see also e.g. #84 )

@krisjdev
Copy link
Author

krisjdev commented Mar 6, 2025

Do you have a web-link to that product?

Yeah! I got the USB C variants from here:

Although they don't actually mention the flash chip model anywhere in the description, I guess uf2 flashing just works out of the box and the majority of people end up using that anyways.

then the PR to this repo was the correct thing to do

Awesome :) Hope this change helps someone else at least

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

Successfully merging this pull request may close these issues.

2 participants