After some further thought, I think your approach is fine.

Before I would merge it, I would ask a few things:

- write openocd.texi documentation including example. Unless
other users than you can make effective use of this, then there
is no point in merging this patch with OpenOCD.

- try to clean up the implementation so it works across file format
types. I think the "offset" should operate on addresses rather than
byte offset in file across chunks. Try to stick with OpenOCD whitespace
conventions.

Along with the patch, please post the results of a few tests run.

At least one test on binary, at least one test on a non .bin file (e.g. elf)
and show it works across elf sections w/gaps.

-- 
Øyvind Harboe
US toll free 1-866-980-3434 / International +47 51 63 25 00
http://www.zylin.com/zy1000.html
ARM7 ARM9 ARM11 XScale Cortex
JTAG debugger and flash programmer
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to