Skip to content

Hex#157

Closed
jonasdn wants to merge 4 commits intostlink-org:masterfrom
jonasdn:hex
Closed

Hex#157
jonasdn wants to merge 4 commits intostlink-org:masterfrom
jonasdn:hex

Conversation

@jonasdn
Copy link
Contributor

@jonasdn jonasdn commented Jun 12, 2013

Hi,

This pull request could be considered as a RFC.

It implements parsing of Intel HEX files in a new file stlink-hex.c.
It adds handling of flashing the parsed hex file to stlink_fwrite_flash in stlink-common.c
and also to the stlink-gui.

It does nothing with the flash/main.c ui. To use it from the command line you just pass the
hex file and specify the base address. Even tho that information is discarded and the stlink_fwrite_flash function uses the start_addr from the hex file.
Could this be improved?

The branch also contains update to the gui, to make it re-read the device memory after
flashing. Should I Submit this patch later on?

Regards
Jonas

@texane
Copy link
Contributor

texane commented Jun 12, 2013

I will have a look asap, and will keep you informed. If you
make any progress in between, please let me know so that
I look at the correct source.

Thanks for helping,

Fabien

2013/6/12 Jonas Danielsson notifications@github.com

Hi,

This pull request could be considered as a RFC.

It implements parsing of Intel HEX files in a new file stlink-hex.c.
It adds handling of flashing the parsed hex file to stlink_fwrite_flash in
stlink-common.c
and also to the stlink-gui.

It does nothing with the flash/main.c ui. To use it from the command line
you just pass the
hex file and specify the base address. Even tho that information is
discarded and the stlink_fwrite_flash function uses the start_addr from the
hex file.
Could this be improved?

The branch also contains update to the gui, to make it re-read the device
memory after
flashing. Should I Submit this patch later on?

Regards

Jonas

You can merge this Pull Request by running

git pull https://github.com/jonasdn/stlink hex

Or view, comment on, or merge it at:

#157
Commit Summary

  • stlink-hex: add implementation of parsing Intel HEX files
  • stlink-gui: add handling of Intel HEX files
  • stlink-common: add Intel HEX file handling of flash write
  • stlink-gui: re-read memory after flashing

File Changes

  • M Makefile.amhttps://github.com/Hex #157/files#diff-0(2)
  • M gui/stlink-gui.chttps://github.com/Hex #157/files#diff-1(157)
  • M src/stlink-common.chttps://github.com/Hex #157/files#diff-2(67)
  • A src/stlink-hex.chttps://github.com/Hex #157/files#diff-3(171)
  • A src/stlink-hex.hhttps://github.com/Hex #157/files#diff-4(22)

Patch Links:

@rusefi
Copy link

rusefi commented May 28, 2014

I wonder what is the status of this :)

@Nightwalker-87
Copy link
Member

Closed in favour of #459.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants