-
-
Notifications
You must be signed in to change notification settings - Fork 320
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
Conversation
Signed-off-by: Patrick José Pereira <[email protected]>
ping @valeros |
Hi @patrickelectric, please review my comments above. |
@valeros 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]>
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 |
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! |
@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. |
Hi there, any update on when we may expect that change request approved? Many thanks !!!! |
Ping @valeros |
Hi @patrickelectric, if you insist on merging, then please remove unsupported tools from the |
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 ... |
@valeros debug and upload section removed. |
@patrickelectric you shouldn't have removed the entire sections, only unsupported tools. |
Signed-off-by: Patrick José Pereira <[email protected]>
@valeros let me know if it's correct now. |
Thanks, merged. |
Fix #730