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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

patrickelectric
Copy link

Fix #730

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

ping @valeros

boards/nucleo_h563zi.json Outdated Show resolved Hide resolved
boards/nucleo_h563zi.json Outdated Show resolved Hide resolved
boards/nucleo_h563zi.json Outdated Show resolved Hide resolved
@valeros
Copy link
Member

valeros commented Sep 23, 2024

Hi @patrickelectric, please review my comments above.

boards/nucleo_h563zi.json Outdated Show resolved Hide resolved
@patrickelectric
Copy link
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
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
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!

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
3 participants