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

board/arm/stm32h747i_disco: add support of flash with openocd #24682

Merged
merged 2 commits into from
May 6, 2020

Conversation

ABOSTM
Copy link
Collaborator

@ABOSTM ABOSTM commented Apr 24, 2020

Allow to flash either Cortex M4 or M7 with OpenOCD
(depending on which Board/Core has been compiled)
Command: west flash

Warning: Dependency with recent OpenOCD patch:
Windows: https://gnutoolchains.com/arm-eabi/openocd/
version 20200408
Linux : http://openocd.zylin.com/
SHA1: 0a804222da63c5f849efa23b019a59e2dea76842

Edit:
OpenOCD dependancy: zephyrproject-rtos/openocd#27

Note: It is necessary to first update ST-Link firmware (see #24671)

Known Restrictions:

Sometimes flashing doesn't work and it is necessary to erase the flash to be able make it work again.
Debugging Cortex M7 works, but it doesn't work with Cortex M4.

Allow to flash either Cortex M4 or M7 with OpenOCD
(depending on which Board/Core has been compiled)
Command: west flash

Warning: Dependency with recent OpenOCD patch:
Windows: https://gnutoolchains.com/arm-eabi/openocd/
         version 20200408
Linux : http://openocd.zylin.com/
        SHA1: 0a804222da63c5f849efa23b019a59e2dea76842

Signed-off-by: Alexandre Bourdiol <[email protected]>
@erwango erwango added DNM This PR should not be merged (Do Not Merge) area: Flashing platform: STM32 ST Micro STM32 labels Apr 28, 2020
@ABOSTM ABOSTM marked this pull request as ready for review April 29, 2020 17:17
@ABOSTM ABOSTM requested a review from erwango as a code owner April 29, 2020 17:17
Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update doc regarding openocd compatibility, potential issues that could be faced and how to overcome.
Otherwise LGTM

@ABOSTM ABOSTM requested a review from nashif as a code owner April 30, 2020 11:53
@ABOSTM
Copy link
Collaborator Author

ABOSTM commented Apr 30, 2020

Done

@ABOSTM ABOSTM force-pushed the STM32H747_OPENOCD branch 2 times, most recently from d2a9f04 to 411a98b Compare April 30, 2020 13:05
Add documentation for OpenOCD usage and restrictions

Signed-off-by: Alexandre Bourdiol <[email protected]>
@erwango erwango removed the DNM This PR should not be merged (Do Not Merge) label Apr 30, 2020
@erwango
Copy link
Member

erwango commented May 4, 2020

Failing tests: qemu_x86_64:tests/kernel/mem_protect/syscalls/kernel.memory_protection.syscalls / tests/kernel/mem_protect/syscalls/kernel.memory_protection.syscalls

@erwango erwango requested a review from galak May 4, 2020 12:18
@erwango erwango closed this May 5, 2020
@erwango erwango reopened this May 5, 2020
@erwango
Copy link
Member

erwango commented May 5, 2020

Trying to get CI passed

@erwango erwango added this to the v2.3.0 milestone May 5, 2020
@galak galak merged commit 18f2129 into zephyrproject-rtos:master May 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants