On May 24, 2012, at 18:24 , Alexander Osipenko wrote:

> At least one problem have been caught: latest FW does not select JTAG
> transport by default.

congratulations, Alexander!

> I've added EMU_CMD_SELECT_IF command to the jlink_init().
> At least initial setup steps are passed now, my cores is detected normally.

great, I confirm this, with the initialisation added I was able to program the 
flash, to run a debug session, with stepping, run, halt, etc.

I think the problem was fixed.

I will inform Segger about this new finding. they should fix their inits, but 
this is no longer urgent, OpenOCD will run even with existing firmware.

> The second patch is on top of previous, "unsupported versions"

I think the first patch should be updated, the code can be preserved, but the 
two entries should be removed, with the second patch those versions are no 
longer to be marked as unsupported.


I have two more observations:

- in the jlink_check_supported() function, the pointer definition triggers a 
warning, which turns into an error with the current build settings.

instead of:

        const char **p = unsupported_versions;

I suggest:

        const char * const *p = unsupported_versions;


- there is a small error in a comment:

instead of:

        * Some versions of Segger's software does not select JTAG interface by 
default.

I suggest:

        * Some versions of Segger's software do not select the JTAG interface 
by default.


regards,

Liviu


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to