Re: [Openocd-development] Beagleboard extra clocks in reset

2009-10-05 Thread David Brownell
[ RESEND -- got a bounce from the Berlios list outage ] On Sunday 06 September 2009, Øyvind Harboe wrote: > > Public ICEpick docs are incomplete, but the ones I found > > were not clear on whether the state mattered ... maybe the > > issue is that it *does* matter somewhat, and clocking in > > JTA

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-06 Thread Øyvind Harboe
So what happens when you modify the code to use runidle instead of jtag_add_extraclocks()? If that works, then it's a good indication that we should move to implement events here and that the event sites have been identified correctly. -- Øyvind Harboe Embedded software and hardware consulting

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-06 Thread Øyvind Harboe
On Sun, Sep 6, 2009 at 7:06 PM, David Brownell wrote: > On Sunday 06 September 2009, Dirk Behme wrote: >> This was the first start of OpenOCD after cold reset of the board. > > Yours acted more politely than mine... > > >> Yes, with some further re-starts of OpenOCD without board reset, it >> behav

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-06 Thread David Brownell
On Sunday 06 September 2009, Dirk Behme wrote: > This was the first start of OpenOCD after cold reset of the board. Yours acted more politely than mine... > Yes, with some further re-starts of OpenOCD without board reset, it > behaves different. I can't say if this happens with your tck100.pat

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-06 Thread Øyvind Harboe
On Sat, Sep 5, 2009 at 3:40 PM, Holger Freyther wrote: > On Friday 04 September 2009 21:39:08 Øyvind Harboe wrote: >> > Shouldn't it do that when TRST is released instead of when >> > it's asserted? >> >> Fixed in take #2. >> >> Testing anyone? > > Yes, can you tell me how the expected result looks

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-06 Thread Dirk Behme
David Brownell wrote: > On Saturday 05 September 2009, David Brownell wrote: >> ... see the appended patch, which >> is yours plus (a) "quilt" removed spurious end-of-line >> whitespace, and (b) the icepic.cfg update is there > > Doesn't behave quite right ... I got ... RCLK - adaptive extra clo

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-06 Thread David Brownell
On Saturday 05 September 2009, David Brownell wrote: > ... see the appended patch, which > is yours plus (a) "quilt" removed spurious end-of-line > whitespace, and (b) the icepic.cfg update is there Doesn't behave quite right ... Open On-Chip Debugger 0.3.0-in-development (2009-09-05-19:21) svn:e

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread Dirk Behme
Holger Freyther wrote: > On Friday 04 September 2009 21:39:08 Øyvind Harboe wrote: >>> Shouldn't it do that when TRST is released instead of when >>> it's asserted? >> Fixed in take #2. >> >> Testing anyone? > > Yes, can you tell me how the expected result looks like? It's my understanding that O

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread Dirk Behme
David Brownell wrote: > On Saturday 05 September 2009, Øyvind Harboe wrote: >> Add "jtag_extra_clocks 100" to your config file > > to icepick.cfg in fact ... see the appended patch, which > is yours plus (a) "quilt" removed spurious end-of-line > whitespace, and (b) the icepic.cfg update is there

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread Holger Freyther
On Friday 04 September 2009 21:39:08 Øyvind Harboe wrote: > > Shouldn't it do that when TRST is released instead of when > > it's asserted? > > Fixed in take #2. > > Testing anyone? Yes, can you tell me how the expected result looks like? Debugging the bootloader should just continue to work? ho

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread David Brownell
On Saturday 05 September 2009, Øyvind Harboe wrote: > Take #3 of this patch should probably define a post-tap-reset > event where beagleboard config file could do whatever it > needed... Such an event should IMO be a separate patch... ___ Openocd-develo

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread David Brownell
On Saturday 05 September 2009, Øyvind Harboe wrote: > Add "jtag_extra_clocks 100" to your config file to icepick.cfg in fact ... see the appended patch, which is yours plus (a) "quilt" removed spurious end-of-line whitespace, and (b) the icepic.cfg update is there Not tested. - Dave --- src/fl

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread Dirk Behme
Øyvind Harboe wrote: > Before I commit the change, could you also test a non-beagleboard > target for regressions? Unfortunately, I have only BeagleBoard where I can attach Flyswatter :( Anybody else? > I won't be able to run tests until next week, but I can test that > it builds + commit the ch

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread Øyvind Harboe
Before I commit the change, could you also test a non-beagleboard target for regressions? I won't be able to run tests until next week, but I can test that it builds + commit the change before then. I think it makes sense to commit this change as is and not go for take #3 w/the events. If we need

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread Dirk Behme
Øyvind Harboe wrote: On Sat, Sep 5, 2009 at 9:32 AM, Dirk Behme wrote: Øyvind Harboe wrote: Did you specify the extraclocks via the command? Uuups ;) Where should I put the extraclocks command with witch exact syntax? Add "jtag_extra_clocks 100" to your config file Ahh :) ... RCLK - adap

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread Øyvind Harboe
On Sat, Sep 5, 2009 at 9:32 AM, Dirk Behme wrote: > Øyvind Harboe wrote: >> >> Did you specify the extraclocks via the command? > > Uuups ;)  Where should I put the extraclocks command with witch exact > syntax? Add "jtag_extra_clocks 100" to your config file -- Øyvind Harboe Embedded softwar

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread Dirk Behme
Øyvind Harboe wrote: > Did you specify the extraclocks via the command? Uuups ;) Where should I put the extraclocks command with witch exact syntax? > Take #3 of this patch should probably define a post-tap-reset > event where beagleboard config file could do whatever it > needed... This would

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-05 Thread Øyvind Harboe
Did you specify the extraclocks via the command? Take #3 of this patch should probably define a post-tap-reset event where beagleboard config file could do whatever it needed... -- Øyvind Harboe Embedded software and hardware consulting services http://www.zylin.com

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-04 Thread Dirk Behme
Øyvind Harboe wrote: >> Shouldn't it do that when TRST is released instead of when >> it's asserted? > > Fixed in take #2. > > Testing anyone? > > a) the patch should do no harm to other targets > b) it should fix the extraclocks problem with Beagleboard... Hmm, I'm not really happy with this p

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-04 Thread Øyvind Harboe
> Shouldn't it do that when TRST is released instead of when > it's asserted? Fixed in take #2. Testing anyone? a) the patch should do no harm to other targets b) it should fix the extraclocks problem with Beagleboard... -- Øyvind Harboe Embedded software and hardware consulting services htt

Re: [Openocd-development] Beagleboard extra clocks in reset

2009-09-04 Thread David Brownell
On Friday 04 September 2009, Øyvind Harboe wrote: > I've written up a patch(it compiles, but I haven't run it), which > adds a command to add extra clocks in TAP_RESET. Shouldn't it do that when TRST is released instead of when it's asserted? If for no other reason than to be consistent about clo

[Openocd-development] Beagleboard extra clocks in reset

2009-09-04 Thread Øyvind Harboe
I've written up a patch(it compiles, but I haven't run it), which adds a command to add extra clocks in TAP_RESET. It also makes TAP_RESET a forbidden end state in jtag_add_xxx_scan(). I'm not planning to dive into Cortex A8 or Beagleboard stuff, but perhaps this can be a useful start for someone