DAPLink icon indicating copy to clipboard operation
DAPLink copied to clipboard

STM32H743XX HIC

Open gaborcsapo opened this issue 3 years ago • 3 comments

Hi, please find below the changes for the STM32H743XX HIC that we've been working on. There is also an extension of of the usual DAPLink code in the "source/hic_hal/stm32/stm32h743ii/extended_features" folder which are used by the UDB (Universal Debug Board) stm32h743ii HIC. The additional features include:

  • A new USB CDC endpoint to flash and get logs from a second target.
  • Another new USB CDC endpoint to interact with the HIC. Users can send easy shell commands to toggle pins, or to get any info from the HIC.
  • Additional DAPLink debugging functionality: persistent fault info, breakpoints on asserts when debugger is connected to debug UDB, watchdog, backtraces
  • Target power measurement

Let me know how you'd like to review these changes. We touched the common code only to fix bugs and to change USB CDC stack to support 3 endpoints. If needed, I can give a demo of the UDB features.

gaborcsapo avatar Jun 24 '22 08:06 gaborcsapo

Hi @gaborcsapo, this pull-request needs to simplified. One of the first step would be to rebase the branch to remove all duplicated commits (due merge from main/develop or cherry-picked). Second step would be to re-group and possibly squash commits into different categories:

  • Changes that modify existing code / other HIC
  • Adding stm32h743ii HIC (generic part)
  • Adding UDB board source (source/hic_hal/stm32/stm32h743ii/extended_features should probably be source/board/udb) and tests.

mathias-arm avatar Jun 28 '22 03:06 mathias-arm

@mathias-arm, finished with cleanup. I hope this is what you meant.

gaborcsapo avatar Jun 30 '22 09:06 gaborcsapo

Hey, any updates on the review?

gaborcsapo avatar Sep 01 '22 11:09 gaborcsapo