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 to nucleo h563zi #800

Merged
merged 4 commits into from
Feb 17, 2025
Merged

Conversation

patrickelectric
Copy link
Contributor

Fix #730

Signed-off-by: Patrick José Pereira <[email protected]>
@patrickelectric
Copy link
Contributor Author

ping @valeros

@valeros
Copy link
Member

valeros commented Sep 23, 2024

Hi @patrickelectric, please review my comments above.

@patrickelectric
Copy link
Contributor Author

@valeros check now, everything should be fixed

@valeros
Copy link
Member

valeros commented Sep 24, 2024

check now, everything should be fixed

Thanks, have you tested this board with PlatformIO? Does it work as expected?

@patrickelectric
Copy link
Contributor Author

check now, everything should be fixed

Thanks, have you tested this board with PlatformIO? Does it work as expected?

I did it using openocd, I know that the platformio openocd package does not support it but I'm using the OpenOCD of STM.

Signed-off-by: Patrick José Pereira <[email protected]>
@valeros
Copy link
Member

valeros commented Sep 25, 2024

I did it using openocd, I know that the platformio openocd package does not support it but I'm using the OpenOCD of STM.

I see. Honestly, I’m a bit hesitant to add a new board that we can’t program or debug with our current tools. The latest stable OpenOCD doesn’t support the H5 family, and we don’t use any OpenOCD forks. So, basically, you’ll have to drop st-link and cmsis-dap from the list of upload protocols. That means the board will need to be programmed either through mass storage (which I think the latest ST-Links support) or using an external J-Link debug probe, but that’s going to make the developer experience pretty rough.

@ypearson-bdai
Copy link

its not so bad, just care about building the binary, There are other options to flash other than openocd

@patrickelectric
Copy link
Contributor Author

its not so bad, just care about building the binary, There are other options to flash other than openocd

Yeah, the binary is working fine.

@ypearson-bdai
Copy link

ypearson-bdai commented Oct 2, 2024

its not so bad, just care about building the binary, There are other options to flash other than openocd

Yeah, the binary is working fine.

Yeah this seems to working okay!

@patrickelectric
Copy link
Contributor Author

@valeros I think that is fine as it is, once the tools update the support will be already there. Otherwise, we can just let people waiting.. and that's not good. I prefer to have an option than nothing at all.

@lm324bruno
Copy link

lm324bruno commented Feb 2, 2025

Hi there, any update on when we may expect that change request approved? Many thanks !!!!

@patrickelectric
Copy link
Contributor Author

Ping @valeros

@valeros
Copy link
Member

valeros commented Feb 4, 2025

Hi @patrickelectric, if you insist on merging, then please remove unsupported tools from the debug and upload sections, so only mbed and jlink methods are left (mbed should be the default choice).

@lm324bruno
Copy link

any update? Sorry for bothering, but it seems platformio currently do not support any new nucleo144 boards, wondering if there's a chance to get at least one new board in ...

@patrickelectric
Copy link
Contributor Author

@valeros debug and upload section removed.

@valeros
Copy link
Member

valeros commented Feb 17, 2025

@patrickelectric you shouldn't have removed the entire sections, only unsupported tools.

@patrickelectric
Copy link
Contributor Author

@valeros let me know if it's correct now.

@valeros valeros merged commit e0bb373 into platformio:develop Feb 17, 2025
@valeros
Copy link
Member

valeros commented Feb 17, 2025

Thanks, merged.

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.

missing board Nucleo STM32H563ZI
4 participants