Re: [Openocd-development] Radically improving out of the boxperformance for armX

2009-11-26 Thread Nico Coesel
> -Original Message- > From: openocd-development-boun...@lists.berlios.de [mailto:openocd- > development-boun...@lists.berlios.de] On Behalf Of Øyvind Harboe > Sent: vrijdag 27 november 2009 7:59 > To: openocd-development@lists.berlios.de > Subject: [Openocd-development] Radically improving

Re: [Openocd-development] Radically improving out of the box performance for armX

2009-11-26 Thread Øyvind Harboe
On Fri, Nov 27, 2009 at 8:40 AM, David Brownell wrote: > On Thursday 26 November 2009, Ųyvind Harboe wrote: >> How about enabling fast/DCC memory transfers by default? > > This was previously the default on many systems, so not > having it be enabled is a performance regression ... I don't unders

Re: [Openocd-development] Radically improving out of the box performance for armX

2009-11-26 Thread David Brownell
On Thursday 26 November 2009, Øyvind Harboe wrote: > How about enabling fast/DCC memory transfers by default? This was previously the default on many systems, so not having it be enabled is a performance regression ... > We could document a requirement that a correct target > script would disabl

Re: [Openocd-development] [Patch 0/8] more command cleaning

2009-11-26 Thread David Brownell
On Thursday 26 November 2009, Øyvind Harboe wrote: > Does this change the syntax for scripts? And what might have caused "nand list" output to repeat? As if the command got called twice... > nand list #0: NAND 1GiB 3,3V 8-bit (Micron) pagesize: 2048, buswidth: 8, blocksize: 131072, block

[Openocd-development] Radically improving out of the box performance for armX

2009-11-26 Thread Øyvind Harboe
How about enabling fast/DCC memory transfers by default? We could document a requirement that a correct target script would disable fast/DCC memory transfers if necessary. -- Øyvind Harboe US toll free 1-866-980-3434 / International +47 51 63 25 00 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM1

Re: [Openocd-development] [PATCH 5/8] add more stub handlers to testee target

2009-11-26 Thread Øyvind Harboe
What's this??? A pure virtual test target? Does it work with the dummy interface and faux flash driver? Would be neat :-) -- Ø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 programme

Re: [Openocd-development] [Patch 0/8] more command cleaning

2009-11-26 Thread Øyvind Harboe
Does this change the syntax for scripts? -- Ø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

Re: [Openocd-development] [patch 2/3] Cortex-A8: support "reset-assert" event

2009-11-26 Thread Magnus Lundin
David Brownell wrote: > Use the new "reset-assert" event; else SRST; else fail. > Tested on an OMAP3, using the event. > > NOTE: still doesn't handle "reset halt". For some reason > neither VCR nor PRCR seemed effective; they held the value > that was written, but VCR didn't trigger debug entry w

[Openocd-development] [PATCH 7/8] improve jtag_tap_configure

2009-11-26 Thread Zachary T Welch
Splits bulk of the jtag_tap_configure into jtag_tap_configure_event, removing three or four levels of indentation in the process. The resulting code was stylistically improved in other ways, but it should be functionally identical. Signed-off-by: Zachary T Welch --- src/jtag/tcl.c | 165 +++

[Openocd-development] [PATCH 8/8] split jim_newtap_cmd into pieces

2009-11-26 Thread Zachary T Welch
Moves the ID and IR-related option parsing to static helpers, removing two levels of indent. Signed-off-by: Zachary T Welch --- src/jtag/tcl.c | 164 1 files changed, 94 insertions(+), 70 deletions(-) diff --git a/src/jtag/tcl.c b/src/jt

[Openocd-development] [PATCH 6/8] improve jtag_tap_handle_event indentation

2009-11-26 Thread Zachary T Welch
Use 'continue' to reduce identation levels and superfluous logic. Signed-off-by: Zachary T Welch --- src/jtag/tcl.c | 53 ++--- 1 files changed, 30 insertions(+), 23 deletions(-) diff --git a/src/jtag/tcl.c b/src/jtag/tcl.c index 929c784..837196

[Openocd-development] [PATCH 4/8] explode tcl_target_func into many handlers

2009-11-26 Thread Zachary T Welch
Eliminate the monolithic tcl_target_func by registering each of its commands using the new chained command registration mechanism. Also chains the target's commands under the CPU command, though these may not work properly without some further modification. Signed-off-by: Zachary T Welch --- sr

[Openocd-development] [PATCH 3/8] split jim_target into multiple handlers

2009-11-26 Thread Zachary T Welch
The 'target' command group was implemented using its own command dispatching, which can be eliminated by using the new chained command registration mechanism. This patch splits the jim_target() function into individual handlers, which makes them to be visible to the help and usage commands. These

[Openocd-development] [PATCH 5/8] add more stub handlers to testee target

2009-11-26 Thread Zachary T Welch
Prevent everything from crashing when exercising various commands. Signed-off-by: Zachary T Welch --- src/target/testee.c | 24 ++-- 1 files changed, 22 insertions(+), 2 deletions(-) diff --git a/src/target/testee.c b/src/target/testee.c index ca3d56e..f858232 100644 --- a

[Openocd-development] [Patch 0/8] more command cleaning

2009-11-26 Thread Zachary T Welch
Hi all, This series continues to work to simplify OpenOCD's command handling. Herein, you'll find some rather crude and heavy-handed attempts to cleave apart some monolithic top-level command handlers into individual handlers that get registered using the new command chaining mechanism. The code

[Openocd-development] [PATCH 1/8] split jim_jtag_command into multiple handlers

2009-11-26 Thread Zachary T Welch
Explodes the 'jtag' into separate command handlers, which are easier to understand and extend. Makes the code much easier to understand, though further simplifications are possible. This patch tries to minimize the noise when viewed with 'git diff -w'. Gives these commands improved built-in help

[Openocd-development] [PATCH 2/8] begin moving JTAG jim handlers/helpers

2009-11-26 Thread Zachary T Welch
Moves the tertiary jim handlers and required static helpers to the top of tcl.c, defining them in a new registration array that is chained in both the top-level context and under the jtag command. The top-level commands can be removed at some point in the future to reduce clutter. Signed-off-by:

[Openocd-development] [patch 2/3] Cortex-A8: support "reset-assert" event

2009-11-26 Thread David Brownell
Use the new "reset-assert" event; else SRST; else fail. Tested on an OMAP3, using the event. NOTE: still doesn't handle "reset halt". For some reason neither VCR nor PRCR seemed effective; they held the value that was written, but VCR didn't trigger debug entry when the reset vector fired (maybe

[Openocd-development] [patch 3/3] omap3530.cfg: use new "reset-assert" event

2009-11-26 Thread David Brownell
Replaces previous "reset-assert-pre" workaround. --- tcl/target/omap3530.cfg |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/tcl/target/omap3530.cfg +++ b/tcl/target/omap3530.cfg @@ -65,10 +65,10 @@ proc omap3_dbginit {target} { jtag_rclk 1000 $_TARGETNAME configure -event "r

[Openocd-development] [patch 1/3] target: groundwork for "reset-assert" event

2009-11-26 Thread David Brownell
This defines a "reset-assert" event and a supporting utility routine, and documents both how targets should implement it and how config scripts should use it. Core-specific updates are needed to make this work. --- This is necessary for supporting reset on boards without SRST. I'm sending around t

Re: [Openocd-development] write_phys_memory and read_phys_memory

2009-11-26 Thread David Brownell
> > > > Error: type 'xscale' is missing write_phys_memory > > > > Error: type 'xscale' is missing read_phys_memory > > In this case a minimal correct implementation of those two > methods would be to return ERROR_FAIL.  Obviously, something > that does the relevant hard work would be better. Comm

Re: [Openocd-development] [PATCH] pull request: Typos and documentation fixes

2009-11-26 Thread David Brownell
On Thursday 26 November 2009, Zach Welch wrote: > For the future, commit messages should have a short subject line and at > least one line of description. Agreed with the usual exception: there are minor patches where the subject line suffices, and there's no point in trying to create a second s

Re: [Openocd-development] [PATCH] pull request: Typos and documentation fixes

2009-11-26 Thread Zach Welch
On Thu, 2009-11-26 at 18:12 +0100, Uwe Hermann wrote: > Hi, > > please pull from the "doc-fixes" branch from my OpenOCD repo: > > Clone URL: > > git clone git://gitorious.org/openocd/openocd.git > > > The branch has a bunch of minor typos and documentation fixes. Pulled, rebased against the

Re: [Openocd-development] write_phys_memory and read_phys_memory

2009-11-26 Thread David Brownell
On Thursday 26 November 2009, Zach Welch wrote: > On Thu, 2009-11-26 at 14:05 +0100, Øyvind Harboe wrote: > > Hi Andrey, > > > > > Error: type 'xscale' is missing write_phys_memory > > > Error: type 'xscale' is missing read_phys_memory > > > > Ignore these errors. They are only reminders to the d

[Openocd-development] [PATCH] pull request: Typos and documentation fixes

2009-11-26 Thread Uwe Hermann
Hi, please pull from the "doc-fixes" branch from my OpenOCD repo: Clone URL: git clone git://gitorious.org/openocd/openocd.git The branch has a bunch of minor typos and documentation fixes. The "git format-patch" output is attached for easier review. Thanks, Uwe. -- http://www.hermann-uw

Re: [Openocd-development] 'erase_sector' and 'protect' flash commands

2009-11-26 Thread Zach Welch
Pushed, with another minor-but-related help text fix. Thanks! --Z On Wed, 2009-11-25 at 08:36 -0500, Eric Wetzel wrote: > Hello, > > 'flash erase_sector' and 'flash protect' have not been working for me > recently. Passing them the correct number of arguments (3 and 4, > respectively) returns t

Re: [Openocd-development] write_phys_memory and read_phys_memory

2009-11-26 Thread Zach Welch
On Thu, 2009-11-26 at 14:05 +0100, Øyvind Harboe wrote: > Hi Andrey, > > > Error: type 'xscale' is missing write_phys_memory > > Error: type 'xscale' is missing read_phys_memory > > Ignore these errors. They are only reminders to the developers > to add this support to the xscale target. I saw t

Re: [Openocd-development] How to set includ path with "libftdi" in configure?

2009-11-26 Thread Albert ARIBAUD
loody a écrit : > Hi: > thanks for your kind help ^^ > it works find right now. > 2009/11/26 Dean Glazeski : >> You appear to mixing the two FTDI libraries in that configure command. If >> you want to use the FTD2XX libraries, you need to use --enable-ft2232_ftd2xx >> instead of --enable-ft2232_li

Re: [Openocd-development] write_phys_memory and read_phys_memory

2009-11-26 Thread Andrey Khurri
Thank you for referring to this. It solved the problem. Regarding the previous problem "couldn't open blob-im2" I also solved it: I needed to run both 'openocd' and 'telnet localhost ' from the same directory (my mistake). Cheers, Andrey Øyvind Harboe wrote: > Have you tried: > > https://l

Re: [Openocd-development] write_phys_memory and read_phys_memory

2009-11-26 Thread Øyvind Harboe
Have you tried: https://lists.berlios.de/pipermail/openocd-development/2009-November/012907.html -- Ø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 _

Re: [Openocd-development] How to set includ path with "libftdi" in configure?

2009-11-26 Thread loody
Hi: thanks for your kind help ^^ it works find right now. 2009/11/26 Dean Glazeski : > You appear to mixing the two FTDI libraries in that configure command.  If > you want to use the FTD2XX libraries, you need to use --enable-ft2232_ftd2xx > instead of --enable-ft2232_libftdi.  If you want to use

Re: [Openocd-development] write_phys_memory and read_phys_memory

2009-11-26 Thread Andrey Khurri
Hi Øyvind, Thanks for your previous message. Do you have any idea about the nature of the following "runtime" error (I get it when I try to protect a memory part for blob)? > reset halt JTAG tap: imote2.cpu tap/device found: 0x79265013 (mfg: 0x009, part: 0x9265, ver: 0x7) target state: halted

Re: [Openocd-development] write_phys_memory and read_phys_memory

2009-11-26 Thread Øyvind Harboe
Hi Andrey, > Error: type 'xscale' is missing write_phys_memory > Error: type 'xscale' is missing read_phys_memory Ignore these errors. They are only reminders to the developers to add this support to the xscale target. Try writing up a fresh bug report using these guidelines: http://openocd.ber

Re: [Openocd-development] [PATCH] Fix LIBFTDI path issue

2009-11-26 Thread Albert ARIBAUD
David Brownell a écrit : > On Wednesday 25 November 2009, Albert ARIBAUD wrote: >>> http://packages.debian.org/squeeze/amd64/libftdi-dev/filelist >>> >>> Maybe they treat this /usr/include/ftdi thing as a bug and >>> work around it ... :) >> Hmm... Checking the versions, Debian is talking about 0.

[Openocd-development] write_phys_memory and read_phys_memory

2009-11-26 Thread Andrey Khurri
Hi, I am sorry if this the wrong place for this kind of help request. I am using Openocd (latest version from git) to program my imote2 sensor via an Amontec JTAG-tiny. To configure and build openocd I use the following commands: ./configure --prefix=/usr/local/lib/ --enable-ft2232_libftdi CFL

Re: [Openocd-development] [PATCH] Fix LIBFTDI path issue

2009-11-26 Thread David Brownell
On Wednesday 25 November 2009, Albert ARIBAUD wrote: > > >  http://packages.debian.org/squeeze/amd64/libftdi-dev/filelist > > > > Maybe they treat this /usr/include/ftdi thing as a bug and > > work around it ... :) > > Hmm... Checking the versions, Debian is talking about 0.16. I'm using > the

Re: [Openocd-development] [PATCH] Fix LIBFTDI path issue

2009-11-26 Thread Albert ARIBAUD
Albert ARIBAUD a écrit : > David Brownell a écrit : >> On Wednesday 25 November 2009, Albert ARIBAUD wrote: >>> Dean Glazeski a écrit : This patch shouldn't be necessary. I have the libftdi version working fine with current head. I think this might be an issue with mixing librari