Patch awaiting review at http://openocd.zylin.com/247 . Next time, please
push it directly to gerrit yourself. Read HACKING for instructions.
Thanks!
___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailm
I just want to add that there is a tcl hook that gets called when gdb
connects. Depending on how you use openocd/gdb, that one can be more
convenient for doing a reset init than with monitor commands from within
gdb.
As has been said already, openocd (and gdb) should do as little as possible
to th
On Thu, Nov 3, 2011 at 1:43 AM, Tomek CEDRO wrote:
> Hello Andreas :-)
>
> On Thu, Nov 3, 2011 at 12:36 AM, Andreas Fritiofson
> wrote:
> > This won't even compile. You pass a pointer-to-int, but swd_bus_read_ack
> > expects a pointer-to-pointer-to-char.
>
On Thu, Nov 3, 2011 at 12:41 AM, Tomek CEDRO wrote:
> Hey,
>
> I have some problems with pointers and need some support plz ;-)
>
I'm not sure I understand what the problem is, but I can give some general
hints.
int swd_bus_read_ack(swd_ctx_t *swdctx, swd_operation_t operation, char
> **ack){
>
On Tue, Nov 1, 2011 at 11:16 PM, Tomek CEDRO wrote:
> Hey does this code is related anyhow to arm_adi_v5 or this is totally
> alternative solution?
>
>
Yes, it *uses* arm_adi_v5.
/Andreas
___
Openocd-development mailing list
Openocd-development@lists.b
On Sat, Oct 29, 2011 at 10:56 AM, wrote:
> This is an automated email from Gerrit.
>
> ?yvind Harboe (oyvindhar...@gmail.com) just uploaded a new patch set to
> Gerrit, which you can find at http://openocd.zylin.com/134
>
> -- gerrit
>
> commit 620ba7e6cd57c951fafa0f1ffab2db102fe2a60f
> Author: Ø
On Wed, Oct 19, 2011 at 2:23 PM, freddie_chopin wrote:
> W dniu 2011-10-19 14:14:11 użytkownik Akos Vandra
> napisał:
> > #lpcfixchecksum takes only binary files, so
> > #make a binary file from the elf, and fix the checksum.
> > arm-eabi-objcopy -O binary $FILE tmp.bin
> > lpcfixchecksum tmp.bin
Hi!
On Thu, Oct 20, 2011 at 12:47 AM, Evan Hunter wrote:
> You are right – Looking at it again, the “qP” code looks like I got halfway
> through implementing it, then found that I should be using qThreadExtraInfo
> instead.
>
> Please feel free to remove it. The only reason I haven’t already is
Sorry for bringing this old thread up (no pun intended).
This code got a few minutes of my attention after I browsed through the
clang static analysis report someone posted recently. rtos.c was one of the
first in the list, and while the bug report probably was a false positive, I
noticed some ser
On Sun, Oct 9, 2011 at 11:28 PM, Akos Vandra wrote:
> P.S. Is it just me, or the default reply address is the sender
> himself, and not the list?
>
The list server doesn't set Reply-To to the list address, for some very good
reason which I have forgotten now. Easiest fix is to set reply-to-all a
On Sun, Oct 9, 2011 at 10:43 PM, Peter Stuge wrote:
> Øyvind Harboe wrote:
> > On Sun, Oct 9, 2011 at 3:40 AM, Peter Stuge wrote:
> > > Transfer rate: 3 KB/sec, 13848 bytes/write.
> >
> > USB roundtrips?
>
> Ḯ'll have a look with usbmon soon.
>
U+0618 LATIN CAPITAL LETTER I WITH DIRT ON TOP? I
On Sun, Oct 9, 2011 at 12:37 PM, Simon Barner wrote:
> Dear list,
>
> I have already posted the attached script some weeks ago.
>
> It enables the PLL of the STM32F107 and thus configures
> the MCU to run at 72 MHz which allows for higher JTAG speeds.
>
> Since this should be useful for everybody
On Wed, Aug 31, 2011 at 1:21 AM, Andreas Fritiofson <
andreas.fritiof...@gmail.com> wrote:
>
>
> On Tue, Aug 16, 2011 at 8:40 AM, Simon Barner wrote:
>
>> On 16.08.2011 01:25, Andreas Fritiofson wrote:
>> >
>> >
>> > On Wed, Aug 10, 201
On Sat, Oct 8, 2011 at 6:12 PM, Luca BRUNO wrote:
> Luca Bruno scrisse:
>
> > This patch fix udev rules to work with any listed tty-based adaptor.
>
> Speaking of udev, another related patch is currently sitting in the
> bug tracker at
> http://sourceforge.net/apps/trac/openocd/ticket/39
>
> Woul
On Fri, Oct 7, 2011 at 12:03 AM, Andreas Fritiofson <
andreas.fritiof...@gmail.com> wrote:
>
>
> On Thu, Sep 22, 2011 at 9:15 PM, Simon Barner wrote:
>
>> Dear list,
>>
>> I finally found some time to rebase my patches and to split
>> them into sma
On Fri, Oct 7, 2011 at 2:37 PM, Attila Kinali wrote:
> On Thu, 6 Oct 2011 23:07:44 +0200
> Andreas Fritiofson wrote:
>
> > It obviously hangs during init, which could mean just about anything.
> It's
> > impossible to tell without a debug log (-d3).
>
> I s
On Fri, Oct 7, 2011 at 7:34 AM, Dmitry E. Oboukhov wrote:
>
> >>> I cant upload an image into internal SRAM (AT91SAM9).
> >>> it can really write only 20 bytes.
> >>> load_image into internal flash area works fine.
>
> >> But why it doesn't fail if it writes the other regions?
> >> It crashes onl
On Mon, Sep 19, 2011 at 4:30 PM, Matt Reimer wrote:
> The code in xscale_receive() that tries to skip invalid reads (i.e.
> reads that don't have the DBG_SR[0] 'valid' bit set) seems to be
> wrong, as it only looks at the first word's valid flag rather than
> each word's own valid flag. Am I read
Hi,
On Sat, Sep 24, 2011 at 2:00 PM, Dmitry E. Oboukhov wrote:
> > Dmitry E. Oboukhov wrote:
> >> I cant upload an image into internal SRAM (AT91SAM9).
> >> it can really write only 20 bytes.
> >> load_image into internal flash area works fine.
>
> But why it doesn't fail if it writes the other r
On Thu, Sep 22, 2011 at 9:15 PM, Simon Barner wrote:
> Dear list,
>
> I finally found some time to rebase my patches and to split
> them into smaller pieces (as suggested by Ųyvind Harboe).
>
> Please refer to the commit messages for the purpose
> of the individual patches.
>
> I would really app
On Thu, Oct 6, 2011 at 11:01 PM, Øyvind Harboe wrote:
> I find gerrit intriguing as a way of managing patches.
>
> Can I have a show of hands of contributors for/against/don't care/don't
> know?
>
Haven't heard of it until now, but sure, a tool like that could be useful.
What would the workflow l
On Thu, Oct 6, 2011 at 11:14 PM, Tomek CEDRO wrote:
> On Thu, Oct 6, 2011 at 9:07 PM, Andreas Fritiofson
> wrote:
> > That log doesn't show that OpenOCD hangs after accessing flash. The
> "Flash
> > bank access DONE" part is wrong, the flash bank has on
Hi,
On Wed, Oct 5, 2011 at 3:15 PM, Attila Kinali wrote:
> Moin,
>
> Has anyone got a Atmel SAM3N (N, not U) working with OpenOCD?
>
> SAM3U works with OpenOCD 0.5.0, but not with SAM3N.
> The device is properly detected, but when accessing the flash, OpenOCD
> hangs indefinitely.
>
> See the at
Hi,
On Sun, Oct 2, 2011 at 7:34 PM, Karl Kurbjun wrote:
> This additional output was helpful when debugging a FTDI device with a
> broken
> EEPROM. I was expecting to connect to a cable with a particular
> description,
> but the device was not found. This error message would helps quickly
> id
It's merged. Thanks!
/Andreas
___
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development
On Sat, Aug 27, 2011 at 12:43 AM, Evan Hunter wrote:
> Hi All,
>
> ** **
>
> I've noticed that if my processor gets reset whilst debugging a program,
> GDB knows nothing about it. I would have thought GDB should stop, in a
> similar way as hitting a breakpoint.
>
> ** **
>
> Is there curr
On Tue, Aug 16, 2011 at 8:40 AM, Simon Barner wrote:
> On 16.08.2011 01:25, Andreas Fritiofson wrote:
> >
> >
> > On Wed, Aug 10, 2011 at 12:01 AM, Spencer Oliver > <mailto:s...@spen-soft.co.uk>> wrote:
> >
> > On 09/08/2011 22:15, Øyvind
Did we decide whether patches should be posted to the list, in addition to
the pull request, to ease reviews? Personally I much rather check the
patches via a web interface, if one is available for the repo. On the other
hand, having the patches in the list archive could be good (if they're not
imm
On Tue, Aug 30, 2011 at 9:25 PM, Eric Wetzel wrote:
> On the discussion of style, I mentioned that the Linux coding standard
> doesn't really like typedefs. Here is an article from Greg K-H that
> partially addresses the subject:
> http://www.linuxjournal.com/article/5780?page=0,2
>
> I don't rea
On Wed, Aug 31, 2011 at 12:27 AM, Martin Schmölzer <
martin.schmoel...@student.tuwien.ac.at> wrote:
> Hi,
>
> On Tue, 2011-08-30 at 15:25 -0400, Eric Wetzel wrote:
> > I don't think I have a way to verify the changes I made to
> > src/flash/ocl/at91sam7x (from the second patch) or the changes I wo
On Fri, Aug 26, 2011 at 8:51 PM, Sam Jansen wrote:
> **
> On 26/08/11 19:18, Andreas Fritiofson wrote:
>
> On Fri, Aug 26, 2011 at 7:10 PM, Sam Jansen wrote:
>
> At this stage I got the libftdi driver giving the same error message as
>> the libftd2xx driver: trying to
On Fri, Aug 26, 2011 at 7:10 PM, Sam Jansen wrote:
> **
> Hello,
>
> In attempting to use openocd with an Amontec JTAG key to program a Xilinx
> FPGA, I encountered a series of issues I though others might be interested
> in. I don't have sensible proposed solutions for any of the problems I
> en
On Mon, Aug 22, 2011 at 9:10 PM, Jean-Christophe PLAGNIOL-VILLARD <
plagn...@jcrosoft.com> wrote:
> On 19:35 Tue 09 Aug , Jean-Christophe PLAGNIOL-VILLARD wrote:
> > tested in buildroot
> any comments?
>
Spelling error:
> > + LDFLAGS="$LFFLAGS $LIBUSB_LDFLAGS"
>
I'm pretty sure I've seen s
On Tue, Aug 16, 2011 at 4:19 PM, Jie Zhang wrote:
> On Mon, Aug 15, 2011 at 6:21 PM, Andreas Fritiofson
> wrote:
> > > +if test -f $srcdir/guess-rev.sh ; then
> > Great, but you should probably check if it's executable instead of just a
> > regular file.
>
On Wed, Aug 10, 2011 at 12:01 AM, Spencer Oliver wrote:
> On 09/08/2011 22:15, Øyvind Harboe wrote:
>
>> Any objections?
>>
>>
> I would like to give this a test-run tomorrow.
>
Did you (or anyone else for that matter) get a chance to test it?
Any thoughts on the target patch? Specifically the s
On Mon, Aug 15, 2011 at 5:03 PM, Jie Zhang wrote:
> On Mon, Aug 15, 2011 at 9:35 AM, Spencer Oliver
> wrote:
> > On 15 August 2011 14:22, Eric Wetzel wrote:
> >> if test $cross_compiling = no; then
> >> # guess-rev.sh only exists in the repository, not in the released
> archives
> >> AC_CHECK
Thank you all for the encouraging comments!
I've been an OpenOCD user and followed the project several years now and
watched it getting better and better. I'm excited to join the maintainer
crew, taking a closer part in that positive progress.
I'll start by getting acquainted with the procedures
On Wed, Aug 10, 2011 at 12:01 AM, Spencer Oliver wrote:
> On 09/08/2011 22:15, Øyvind Harboe wrote:
>
>> Any objections?
>>
>>
> I would like to give this a test-run tomorrow.
>
> One observation - other targets that do not yet support the new functions
> will output a LOG_ERROR to the user. Maybe
The following changes since commit 194e3c5bc5e0fbb7d41cfcbe913c4264782cdd5b:
Rodrigo L. Rosa (1):
fix tapenabler return code
are available in the git repository at:
http://repo.or.cz/r/openocd/andreasf.git rlink
Andreas Fritiofson (7):
rlink: fix indentation errors
rlink
The following changes since commit 194e3c5bc5e0fbb7d41cfcbe913c4264782cdd5b:
Rodrigo L. Rosa (1):
fix tapenabler return code
are available in the git repository at:
http://repo.or.cz/r/openocd/andreasf.git async_algo
Andreas Fritiofson (5):
target: add async algorithm entries
On Fri, Aug 5, 2011 at 11:26 AM, Spencer Oliver wrote:
>
> Release tags are annotated, but not rc tags
>
>
Oh, but they are, or am I completely oblivious of git tags (quite possible)?
$ git checkout v0.4.0-rc2~2
$ git describe
v0.4.0-rc1-193-g747a607
http://repo.or.cz/w/openocd.git/tag/cd8ad2e96
On Fri, Aug 5, 2011 at 11:02 AM, Spencer Oliver wrote:
> On 5 August 2011 09:58, Andreas Fritiofson
> wrote:
> >
> >
> > On Fri, Aug 5, 2011 at 8:56 AM, Øyvind Harboe
> > wrote:
> >>
> >> When I run git describe now I get v0.4.0-973-g0d7a948 rathe
On Fri, Aug 5, 2011 at 8:56 AM, Øyvind Harboe wrote:
> When I run git describe now I get v0.4.0-973-g0d7a948 rather than
> a v0.5.0-rc2-.
>
> Is that intentional?
>
> I think it's nice that we stick to v0.4.0- until v0.5.0- goes out
> of the door.
>
> I have no particular opinion, exce
Repost, it didn't make it to the list.
On Thu, Aug 4, 2011 at 8:39 PM, Andreas Fritiofson <
andreas.fritiof...@gmail.com> wrote:
> On Sat, Jul 30, 2011 at 6:44 PM, Øyvind Harboe wrote:
>
>> Hi Andreas,
>>
>> could you maintain these patches in an official
On Wed, Aug 3, 2011 at 5:48 PM, Spencer Oliver wrote:
> On 3 August 2011 15:00, Andreas Fritiofson
> wrote:
> > If the pipe method work, fine. However, looking through the git2cl code
> it
> > apparently assumes git log is run with LC_ALL=C and that wouldn't be th
On Thu, Aug 4, 2011 at 5:12 PM, Simon Barner wrote:
> I applied the patches 1/5 - 5/5 against my fixed ARM-JTAG-EW driver
> and yield now (STM32F107, JTAG @ 6 MHz). Also, debugging the target
> seems to work decently.
>
> flash erase_address 0x0800 0x4
> erased address 0x0800 (length
On Wed, Aug 3, 2011 at 3:32 PM, Tomek CEDRO wrote:
> Hello Jean, please provide full featured RC package (generated by
> "make distcheck", containing configure scripts) to test it on FreeBSD,
> as for now bootstrap fails, so the status is no-go.
> Best regards,
> Tomek
>
>
Yes, we haven't had a c
On Wed, Aug 3, 2011 at 12:28 PM, Luca Bruno wrote:
> Andreas Fritiofson scrisse:
>
> > > > make dist should use git2cl to genereate ChangeLog from git
> > > > history, populating the placeholder file in released tarball.
>
> > > Still not working
On Sat, Jul 30, 2011 at 7:10 PM, Tomek CEDRO wrote:
> Hello! :-) Trying to bootstrap on FreeBSD 8.2-RELEASE fails with the
> 0.5.0-rc2 as given at:
>
>
> http://openocd.git.sourceforge.net/git/gitweb.cgi?p=openocd/openocd;a=snapshot;h=d4cd6f032015552f00bf4b5a90f25f5f958e9d9e;sf=tgz
>
> This shoul
On Sat, Jul 30, 2011 at 4:26 AM, Mahr, Stefan wrote:
> Hi Drasko.
>
> > Drasko DRASKOVIC (1):
> > mips32 : Fixed memory byte access
>
> Your patch fixes the broken byte access, but not the big endian host issue.
> Since both problems are tied together, I would prefer a more common
> solution.
On Fri, Jul 29, 2011 at 6:24 PM, Jie Zhang wrote:
> On Fri, Jun 24, 2011 at 3:14 PM, Jie Zhang wrote:
> > On Mon, Jun 20, 2011 at 6:50 AM, Steve Bennett
> wrote:
> >> The default is -Werror, so warnings become errors
> >> and stop the build.
> >> Might be better to simply #define FT_STATUS inst
On Fri, Jul 29, 2011 at 5:38 PM, Spencer Oliver wrote:
> On 29 July 2011 15:43, Andreas Fritiofson
> wrote:
> >
> > That one won't help for all scripts out there that are currently sourcing
> > target/stm32.cfg.
> > /Andreas
>
> Good point - how about
&g
On Fri, Jul 29, 2011 at 4:09 PM, Spencer Oliver wrote:
> On 29 July 2011 14:54, Andreas Fritiofson
> wrote:
> >
> > On Thu, Jul 28, 2011 at 1:52 PM, Spencer Oliver
> > wrote:
> >>
> >> delete mode 100644 tcl/target/stm32.cfg
> >> delete mode
On Thu, Jul 28, 2011 at 1:52 PM, Spencer Oliver wrote:
> delete mode 100644 tcl/target/stm32.cfg
> delete mode 100644 tcl/target/stm32f2xxx.cfg
>
For compatibility with existing user scripts, it would be best to keep these
files during the 0.5.x period. Just make them print a deprecation warnin
On Wed, Jul 27, 2011 at 12:22 AM, Steve Franks wrote:
> On Tue, Jul 26, 2011 at 3:06 PM, Andreas Fritiofson
> wrote:
> > 0.4.0 is rather old, you may be seeing a bug that has already been fixed.
> > Grab the latest version from the repo and give it a try. I don't think
On Tue, Jul 26, 2011 at 11:25 PM, Steve Franks wrote:
> Every year or so I try openocd, and still no joy. Actually, seems the
> script and config stuff has all finally matured/stabilized, so I'll
> assume I can make it work w/o too much pain, after I fix this hang:
>
> I have a vanilla lpc2138 on
On Fri, Jul 8, 2011 at 1:19 AM, Matthew Lai wrote:
> Hello!
>
> I'm trying to embed a FT2232D based programmer into my board with a STM32
> (Cortex-M3 MCU).
>
> I want the programmer to be compatible with jtagkey, so I looked at
> schematics of compaible designs.
>
> I noticed that while the JTAG
On Fri, Jul 22, 2011 at 2:55 PM, Peter Horn wrote:
> Am 22.07.2011 13:36, schrieb Andreas Fritiofson:
>
>
>> I don't see how this error could be caused by my patches. If I'm not
>> mistaken, none of them have run at this point. It fails during init, and
>
On Fri, Jul 22, 2011 at 4:57 PM, Freddie Chopin wrote:
> On 2011-07-21 23:02, Andreas Fritiofson wrote:
>
>> However I can't explain Freddie's reproduced failures. Maybe post the
>> hex files (matching, slightly different and very different) and I can
>> see
On Fri, Jul 22, 2011 at 11:59 AM, Peter Horn wrote:
>
>
>>Patches 1/7 to 4/7 work ok, the problem is caused by patch 5/7
>>
>>
>>
>>On Windows using the embedded RLink, OpenOCD:
>>
>>
>>Open On-Chip Debugger 0.5.0-dev-snapshot (2011-07-21-07:10)
>>Licensed under GNU GPL v2
>>F
On Fri, Jul 22, 2011 at 12:29 AM, Peter Horn wrote:
> Am 19.07.2011 22:43, schrieb Peter Horn:
>
>>
>> > Last time I've used OpenOCD with RLink stepping was awfully slow,
> > about 3 seconds or more per step. This was some weeks ago and on
> > Windows / Mingw. On Linux, stepping is considerably f
On Thu, Jul 21, 2011 at 10:44 PM, Magnus S. wrote:
> Freddie Chopin writes:
>
> >
> > I can partially confirm...
> >
> > Tested with 0.5.0-rc1
> >
> > Comparing with "right image" works fine:
> >
> >
> > Comparing with "almost right image" (compilation date inside has
> > changed) works fine too
On Mon, Jul 18, 2011 at 3:46 PM, Spencer Oliver wrote:
> On 17 July 2011 10:18, Luca Bruno wrote:
> > make dist should use git2cl to genereate ChangeLog from git history,
> > populating the placeholder file in released tarball.
> >
>
> Still not working for srcdir != builddir
>
> make[1]: Enterin
On Mon, Jul 18, 2011 at 4:59 PM, Jean-Claude Repetto wrote:
> Le 18/07/2011 16:25, Andreas Fritiofson a écrit :
>
>
>> Yes it's best to put these on hold for 0.5.0. It needs some testing on
>> the standalone RLink or other variants, I currently only have access to
&
On Mon, Jul 18, 2011 at 2:33 PM, Tomek CEDRO wrote:
> Hello Andreas!
>
> Do you know what is the difference between JTAG (on Stm32Primer) and
> SWD (on Stm32Primer2) RLink interface? Would they run on existing
> driver? Do you have some kind of manual or technical specification?
Other than the
On Mon, Jul 18, 2011 at 4:03 PM, Spencer Oliver wrote:
> On 18 July 2011 11:13, Andreas Fritiofson
> wrote:
> > This patch set fixes some general problems in the RLink interface driver.
> >
> > Most importantly it fixes a performance bug that have been causin
accesses (really inefficient) and algorithm uploads caused the flash
programming to be latency bound in many cases. Now it should scale better
with increased throughput.
Signed-off-by: Andreas Fritiofson
---
contrib/loaders/flash/stm32x.S | 71 --
src/flash/nor/stm32x.c | 210
Access the different flash banks' registers using a bank specific register
base and a register specific offset. This is equivalent but feels more
natural.
Some accesses were discovered that maybe should not be hard coded to bank0
registers. Add a note about that.
Signed-off-by: An
Signed-off-by: Andreas Fritiofson
---
src/target/cortex_m3.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/src/target/cortex_m3.c b/src/target/cortex_m3.c
index 98a775c..a2f8b78 100644
--- a/src/target/cortex_m3.c
+++ b/src/target/cortex_m3.c
@@ -2329,6 +2329,8
Split armv7m_run_algorithm into two pieces and use them to reimplement it.
The arch_info parameter is used to keep context between the two calls, so
both calls must refer to the same armv7m_algorithm struct. Ugly but works
for a proof-of-concept.
Signed-off-by: Andreas Fritiofson
---
src/target
algorithm is flashing the current.
Signed-off-by: Andreas Fritiofson
---
src/target/target.c | 75 ++
src/target/target.h | 22 +
src/target/target_type.h |2 +
3 files changed, 99 insertions(+), 0 deletions(-)
diff --git a/src
This patch set enables targets to implement asynchronous algorithms that
can be run in the background while OpenOCD performs other tasks. It also
implements async algorithm support for ARMv7-M and enables it for the
Cortex-M3 target. The final two patches rewrites the stm32x flash driver
to use an
Add a helper function for running the queue if it would overflow otherwise.
Use it to simplify the queue fill level checks and optimize in a few cases
that would previously run the queue prematurely.
Signed-off-by: Andreas Fritiofson
---
src/jtag/drivers/rlink.c | 78
__FILE__ and __LINE__ are already printed using the log macros.
Signed-off-by: Andreas Fritiofson
---
src/jtag/drivers/rlink.c | 15 +--
1 files changed, 5 insertions(+), 10 deletions(-)
diff --git a/src/jtag/drivers/rlink.c b/src/jtag/drivers/rlink.c
index 46882a5..c501b56
After correcting the reply size counter, it should be safe to rely on it
for the number of bytes expected in the USB read, instead of reading the
endpoint maximum. This doesn't make things go any faster but it's nicer and
removes the local buffer.
Signed-off-by: Andreas Fritiofson
---
After the reply_index handling is fixed, there's no need to special case
the out scan.
Signed-off-by: Andreas Fritiofson
---
src/jtag/drivers/rlink.c | 28 +---
1 files changed, 9 insertions(+), 19 deletions(-)
diff --git a/src/jtag/drivers/rlink.c b/src/jtag/dr
dtc_queue.reply_index was wrongly being increased during out scans, causing
the queue to be sent before the out buffer was full. This patch increases
raw upload speed by 50% or so.
Signed-off-by: Andreas Fritiofson
---
src/jtag/drivers/rlink.c | 16 +---
1 files changed, 9
Remove unnecessary block scopes to reduce indentation level.
Signed-off-by: Andreas Fritiofson
---
src/jtag/drivers/rlink.c | 203 +++---
1 files changed, 100 insertions(+), 103 deletions(-)
diff --git a/src/jtag/drivers/rlink.c b/src/jtag/drivers
Indentation was inconsistent and some lines not indented at all. Quickfix
using Eclipse's auto-indentation.
Signed-off-by: Andreas Fritiofson
---
src/jtag/drivers/rlink.c | 726 +++---
1 files changed, 363 insertions(+), 363 deletions(-)
diff --
This patch set fixes some general problems in the RLink interface driver.
Most importantly it fixes a performance bug that have been causing decreased
throughput.
Speed test on a STM32 Primer (STM32F103 platform with built in RLink) with the
following openocd.cfg
---
source [find interface/rlin
On Fri, Jul 8, 2011 at 1:26 PM, Drasko DRASKOVIC wrote:
> On Fri, Jul 8, 2011 at 1:19 PM, Andreas Fritiofson
> wrote:
> > I looked briefly at the memory read functions in mips32_dmaacc.c and
> > mips32_pracc.c and it looks like the type usage is a bit confused. The
> >
On Fri, Jul 8, 2011 at 2:05 PM, Mahr, Stefan wrote:
> > Where are those functions defined and how do they know what the target
> endianness is?
>
> They doesn't know the target endianness, but host endianness.
>
>
You can't convert between target and host endianness if you don't know both.
I'm ge
On Fri, Jul 8, 2011 at 2:05 PM, Mahr, Stefan wrote:
> > Where are those functions defined and how do they know what the target
> endianness is?
>
> They doesn't know the target endianness, but host endianness.
>
>
> > It sounds a little strange to do the swapping at this low level.
>
> You need sw
On Fri, Jul 8, 2011 at 1:38 PM, Mahr, Stefan wrote:
> > are you sure about this ?
> >
> > It seems to me that buffer[i] is directly filled by target, and I see
> > no reason that it is in the host endianess...
>
> Hi Drasko,
>
> Yes I'm sure. I tested it on my big endian host platform.
>
> I do no
On Fri, Jul 8, 2011 at 1:17 PM, Drasko DRASKOVIC wrote:
> I am just wandering, would :
> t32 = *(uint32_t*)((void *)&buffer[i]);
> quite the compiler ;)
>
Yes probably, but it would still crash on an architecture that doesn't
support unaligned accesses.
/Andreas
On Fri, Jul 8, 2011 at 12:52 PM, Øyvind Harboe wrote:
> What puzzles me is that there is no warning on x86, even if I the
> -Wcast-align option
> is there
>
>
> -Wcast-align
>Warn whenever a pointer is cast such that the required alignment
> of the target is increased. For example, wa
On Fri, Jul 1, 2011 at 12:47 PM, Spencer Oliver wrote:
> On 1 July 2011 00:41, Andrew Leech wrote:
>
> One other point, do you know whether you can get openocd to add it's own
> > program directory to the search path for the source [find ...] lines?
> It'd
> > be great to only need the openocd.cf
On Wed, Jul 6, 2011 at 11:35 PM, Øyvind Harboe wrote:
> On Wed, Jul 6, 2011 at 11:28 PM, Mahr, Stefan
> wrote:
> "mips32_pracc_read_mem" casts uint32 to void, so we need to cast it
> back to uint32. I found no suitable macro in actual sources.
> >>>
> >>> Hmm then I think we ought t
On Wed, Jun 29, 2011 at 9:17 PM, Rodrigo Rosa wrote:
> On Wed, Jun 29, 2011 at 12:44 AM, Øyvind Harboe
> wrote:
> > This is disturbing. Why switch to global variables?
> >
> > +static uint32_t data_read_dummy;
> > static int jtag_data_write(struct target * target, uint32_t instr,int
> > num_bits
On Tue, Jun 28, 2011 at 9:19 AM, Domien Nowicki wrote:
> **
> 1. The USB Blaster clone does not use the original FT245 chip, and so it
> tries to emulate its behavior.
> As it turns out, the API call FT_GetLatencyTimer is not properly emulated
> by the clone, and this makes OpenOCD abort. In real
Resending, Cc to the list got lost somewhere...
On Sat, Jun 18, 2011 at 8:27 PM, Andreas Fritiofson <
andreas.fritiof...@gmail.com> wrote:
>
>
> On Sat, Jun 18, 2011 at 6:38 PM, Peter Horn wrote:
>
>> Hi Andreas
>>
>> Am 18.06.2011 17:32, schrieb Andreas F
Great, but does it really work? There has been discussions about this on
this list in the past. I think the conclusion was that there's no way to
robustly know GDB's intentions based on the remote commands. When you do a
source-level step in GDB, it may send a step command to OpenOCD. Then we can
d
On Thu, Jun 16, 2011 at 8:02 PM, Tomek CEDRO wrote:
> On Thu, Jun 16, 2011 at 5:50 PM, Øyvind Harboe
> wrote:
> > I take it is just a bugfix?
>
> Yes - it changes:
>
> retval = allow_transports(CMD_CTX, jtag_interface->transports ? :
> jtag_only);
>
> into:
>
> retval = allow_transports(CMD_CTX,
On Fri, Jun 10, 2011 at 2:36 AM, Tomek CEDRO wrote:
> Uhm, from what I understand LAYOUT is only used for initialization of
> specific device based on FT*232 device, maybe other minor stuff, all
> work is done in fact by queue_flush() and it is common for all devices
> as it simply translates que
On Fri, Jun 10, 2011 at 2:18 AM, Tomek CEDRO wrote:
> On Thu, Jun 9, 2011 at 11:46 PM, Andreas Fritiofson
> wrote:
> > Setting the FTDI pins to Hi-Z is not necessarily the same as setting the
> > pins in the JTAG connector to Hi-Z. There is arbitrary dongle-specific
> >
Hi all!
The recent activity surrounding the ft2232 driver makes it obvious that
there are lots of bugfixes and structural improvements that need to be done.
Unfortunately, trying to understand the code or even just browsing around in
it is complicated by the fact that its cluttered with functions
On Fri, Jun 10, 2011 at 1:40 AM, Tomek CEDRO wrote:
> Hello Andreas :-)
>
> On Thu, Jun 9, 2011 at 11:30 PM, Andreas Fritiofson
> wrote:
> >> I have created simple quit() that sets all port pins high and as input
> >> (mpsse command 0x80 0x82) but that di
On Thu, Jun 9, 2011 at 10:17 PM, Tomek CEDRO wrote:
> Hello Laurent! :-)
>
> On Thu, Jun 9, 2011 at 6:21 AM, Laurent Gauch
> wrote:
> > But before to close the handle we should still be able :
> > -1- to tristate signal High-Z via all the out enable signals used in the
> > specific layout, BUT B
On Fri, Jun 10, 2011 at 12:26 AM, Peter Stuge wrote:
> Andreas Fritiofson wrote:
> > > + retval = ft2232_initone(ft2232_vid[i], ft2232_pid[i],
> more,
> > > &try_more);
> > >
> >
> > This is a good start, but what's with the
This together with patch 5/5 is equivalent to my patch, only with gotos.
Both patches are perfectly fine with me, although I prefer getting rid of
the forward declaration as well.
/Andreas
___
Openocd-development mailing list
Openocd-development@lists.be
1 - 100 of 191 matches
Mail list logo
