> On Tuesday, July 19, 2011 4:15 PM, Simon Barner [bar...@gmx.de] wrote: > > Hi Jonas, > > > I can see one change that the referred commit introduced > > to the ARM-JTAG-EW driver. It is that a speed setting in > > the init script now actually is executed. Any such setting > > was previously silently ignored. > > Do you mean the following in jtag/core.c? > > + retval = jtag->speed(jtag_speed_var); > + if (retval != ERROR_OK) > + return retval;
Yes that's the code. The initial speed setting was previously done inside the init function of the drivers. The commit removed this code duplication by moving the intial speed setting to core.c. However armjtagew_init of arm-jtag-ew.c was for some reason missing the speed setting. > > > > The log shows that it is the speed setting that result in > > the crash. > > > > Do you know if a speed setting after init worked before > > the commit? > > If not, please try to execute adapter_khz 1000 at the telnet > > prompt with the OpenOCD build of the version before this > > commit. > > I just tested it with the last version from before the commit, and this > also results in the same error. You can find the log as an attachment to > the ticket. This shows that my commit only revealed an existing bug in the ARM-JTAG-EW speed setting function. The bug was introduced in an earlier version or has been there from the beginning. Do a bisect to find out. I do not own an ARM-JTAG-EW interface, so this is all support I can give. Best regards Jonas Hörberg _______________________________________________ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
