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

GUI failed to parse .hex file #713

Closed
6 tasks done
FoxyHawk opened this issue Jun 10, 2018 · 3 comments · Fixed by #718
Closed
6 tasks done

GUI failed to parse .hex file #713

FoxyHawk opened this issue Jun 10, 2018 · 3 comments · Fixed by #718

Comments

@FoxyHawk
Copy link

FoxyHawk commented Jun 10, 2018

  • Programmer/board type: Stlink/v2-onboard
  • Programmer firmware version: V2.J29.M18
  • Operating system: Arch Linux (Manjaro)
  • Stlink tools version and/or git commit hash: v1.5.0-2 (lastest Manjaro repo)
  • Stlink commandline tool name: stlink-gui
  • Target chip (and optional board): STM32F767ZI

Reading of .hex files in stlink-gui differ from Windows Official ST-link Utility.
It seems to failed to interpret the first line of the file as an address offset.

:020000040800F2
:1000000000000820290300082F030008330300081C
:100010007D03000881030008C903000800000000F8
:1000200000000000000000000000000011040008B3
:10003000150400080000000019040008150D000850
:100040002D0400082D0400082D0400082D040008CC
......

Based on https://en.wikipedia.org/wiki/Intel_HEX, the first line must be read as start at 0x08000000.
On Arch Linux, the file become unusable (but the programming and writing are fine btw) and the result on the card is unpredictable.
rep_arch_read
rep_windows_card_read

On Windows the file is correctly parsed and the result is what I expect.
rep_windows_read
rep_windows_card_write

The .hex file in .txt (only a basic blink LED for my card):
test2.txt

@xor-gate
Copy link
Member

@dev26th I would like you to have a look at this issue because you have integrated intel hex reading some time ago in #459.

@dev26th
Copy link
Contributor

dev26th commented Jun 24, 2018

Sure, it's just a small piece of code

@xor-gate
Copy link
Member

Thanks for reporting and resolving!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants