Re: [Openocd-development] New development version of OpenOCD available at last!
On Wed, Jun 15, 2011 at 5:33 PM, Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com wrote: On 23:04 Tue 07 Jun , Ųyvind Harboe wrote: Nobody has stepped up to do the work of being a release manager. If no-one do it I'll Much appreciated! You can read up on the scripts to produce releases in openocd and post your plan to the list. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] Implementation of a remote bitbang jtag driver
Very cool! I think I'd like to see error propagation on *all* socket calls, but otherwise I don't have any objections. Would you be willing to add such error propagation? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] New development version of OpenOCD available at last!
The swd is some time away from being merged with master, so I wouldn't want to see it in the next release of openocd, unless the release drags out. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] Implementation of a remote bitbang jtag driver
On Wed, Jun 15, 2011 at 10:51 PM, Uhler, Richard ruh...@qualcomm.com wrote: I don't understand. Can you clarify what you mean by error propagation and which socket calls do or don't have it? If it' s just a matter of changing some code in the remote_bitbang driver I would be happy to add this error propagation. e.g. putc() can fail... if you unplug the wire Where is the TCP/IP server btw? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] New development version of OpenOCD available at last!
On Wed, Jun 15, 2011 at 10:55 PM, Tomek CEDRO tomek.ce...@gmail.com wrote: On Wed, Jun 15, 2011 at 5:34 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: The swd is some time away from being merged with master, so I wouldn't want to see it in the next release of openocd, unless the release drags out. I would suggest 0.4.1 for the current code freeze, as there should be no issues. 0.5.0 for working swd..? :-) Nah. 0.4.1 implies that it is a variant of 0.4. What we have now is *completely* different from 0.4, so 0.5.0 is better -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] New development version of OpenOCD available at last!
On Wed, Jun 15, 2011 at 11:16 PM, Tomek CEDRO tomek.ce...@gmail.com wrote: On Wed, Jun 15, 2011 at 8:57 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: 0.4.1 implies that it is a variant of 0.4. What we have now is *completely* different from 0.4, so 0.5.0 is better Yeah, isn't current HEAD somehow development/bugfix of 0.4.0? Are there any totally new features/design change since 0.4.0? If no, then it could be 0.4.1, which should be simple and accurate solution to have new release and tag for some master/head point in time..? That should do the job :-) There are close to a 1000 commits since 0.4, I say bump it to 0.5. :-) -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Openocd release known issues
I think we should stick to distributing source packages only. For the source package, we can include the version of Jim Tcl we tested with. If someone wants something else, they can start with cloning the openocd repository and checking out the release tag? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Active maintainers
On Tue, Jun 14, 2011 at 8:35 AM, Xiaofan Chen xiaof...@gmail.com wrote: On Tue, Jun 14, 2011 at 1:44 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: Looking at the list, this somehow reminds me of those heated discussions about GPL and D2XX. https://lists.berlios.de/pipermail/openocd-development/2009-June/thread.html Last one by Zach Welsh. http://lists.berlios.de/pipermail/openocd-development/2009-July/009518.html Both side of course have their points. Personally I am more with the opinion of adding the D2xx linking exception which Dominic, Rick, Spencer, Magnus Lundin and others agreed. On the other hand, Øyvind Harboe, Zach Welsh, David Brownell and others are more with the pure GPL side. In the end, it is not possible to relicense OpenOCD to add D2xx linking exception. It is impossible to relicense OpenOCD as it would require all copyright holders to agree to linking + exception, which they don't. Yes that is the outcome and I respect the decision. We are all not lawyers and it may well be true that alternative ways proposed at that time are not possible in the end. Still I felt that Zach's tone alienated others. But again that is just my personal opinion and that was why I said I was kind of glad that he is gone. But you'll take his work any day. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Active maintainers
Not a lot of people can match Zach's coding abilities. The OpenOCD community is extremely fortunate to have had his influence and code. Especially the longer terms effects of his foresight, code quality and presence. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Active maintainers
On Tue, Jun 14, 2011 at 8:59 AM, Xiaofan Chen xiaof...@gmail.com wrote: On Tue, Jun 14, 2011 at 2:52 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: Not a lot of people can match Zach's coding abilities. The OpenOCD community is extremely fortunate to have had his influence and code. Especially the longer terms effects of his foresight, code quality and presence. That is probably true. On the other hand, I would prefer Magnus Lundin, Dominic Rath and Rick Altherr to stay than having Zach. Again, this is just my personal opinion. Perhaps you should study the logs for the quantity of contributions. I would have loved nothing better if they all were able to contribute time to OpenOCD. I'm deeply grateful for the time that they all could find for the OpenOCD community. On the other hand, I think the lost of David Brownell is the greatest lost. If not, probably 0.5 is already out. David Brownell was pretty clear on that he would not be the release manager next time. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] breakpoint action
Perhaps GDB offers such a scripting capability? Otherwise search for target events in the manual or in the openocd code. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] General SWD Support in OpenOCD
Hi Tomek, it's great that your focusing on functionality. Cracking the technical problems and making a patch series that's right for OpenOCD are two hard problems. Perhaps better attack one at the time? One hard thing about SWD is to crack the technical problems, the other hard thing is to create a series of patches where as many as possible of the list are able to follow what was done to OpenOCD. This is crucial as otherwise you would be the only one able to modify the code, at which point the value of your patches would drop sharply. Your contributions to OpenOCD can never be greater than that of one man, but if you enable the rest of the community to work on SWD then you have reached another level entirely. There is an optimal size of git patches where the maximum number of people are able to follow what was done to the code. That's the size we should aim for. Generally this means small patches with one change per commit. Note that with interactive git rebase you can create the history after you have the solution. At this point, I think we're ready to accept the first commit, which would do the following: -creation of src/transport directory for transport layer -creation of src/interface directory for interface layer -moving transport.{c,h} from src/jtag into src/transport and updating the headers and Makefile.am You can then rebase your work on top openocd master and your next commit would be much smaller. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] transport: move files over to transport folder
Just to clarify: this patch does nothing, it just moves files around. It makes subsequent swd patches less noisy. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
[Openocd-development] [PATCH] Moved git submodule routine before autotools invocation as they may depend on submodules. Bootstrap will first checkout modules then run autotools
From: CeDeROM cede...@tlen.pl --- bootstrap | 27 ++- 1 files changed, 10 insertions(+), 17 deletions(-) diff --git a/bootstrap b/bootstrap index 0965245..7062fc0 100755 --- a/bootstrap +++ b/bootstrap @@ -1,9 +1,14 @@ - HEAD -#!/bin/sh -=== #!/bin/sh -e -#fetch necessary submodules before runing autotools +#Prepare necessary submodules before runing autotools +if [ $1 = nosubmodule ]; then +SKIP_SUBMODULE=1 +elif [ -n $1 ]; then +echo $0: Illegal argument $1 +echo USAGE: $0 [nosubmodule] +exit 1 +fi + if [ -n $SKIP_SUBMODULE ]; then echo Skipping submodule setup else @@ -12,11 +17,7 @@ else git submodule update fi -# Run the autotools bootstrap sequence to create the configure script - -# Abort execution on error -set -e - +#Check what libtool version to use if which libtoolize /dev/null; then libtoolize=libtoolize elif which glibtoolize /dev/null; then @@ -26,14 +27,6 @@ else exit 1 fi -if [ $1 = nosubmodule ]; then -SKIP_SUBMODULE=1 -elif [ -n $1 ]; then -echo $0: Illegal argument $1 -echo USAGE: $0 [nosubmodule] -exit 1 -fi - # bootstrap the autotools ( set -x -- 1.7.4.1 ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
[Openocd-development] [PATCH] Moved git submodule routine before autotools invocation as they may depend on submodules. Bootstrap will first checkout modules then run autotools
From: CeDeROM cede...@tlen.pl --- bootstrap | 27 ++- 1 files changed, 10 insertions(+), 17 deletions(-) diff --git a/bootstrap b/bootstrap index 0965245..7062fc0 100755 --- a/bootstrap +++ b/bootstrap @@ -1,9 +1,14 @@ - HEAD -#!/bin/sh -=== #!/bin/sh -e -#fetch necessary submodules before runing autotools +#Prepare necessary submodules before runing autotools +if [ $1 = nosubmodule ]; then +SKIP_SUBMODULE=1 +elif [ -n $1 ]; then +echo $0: Illegal argument $1 +echo USAGE: $0 [nosubmodule] +exit 1 +fi + if [ -n $SKIP_SUBMODULE ]; then echo Skipping submodule setup else @@ -12,11 +17,7 @@ else git submodule update fi -# Run the autotools bootstrap sequence to create the configure script - -# Abort execution on error -set -e - +#Check what libtool version to use if which libtoolize /dev/null; then libtoolize=libtoolize elif which glibtoolize /dev/null; then @@ -26,14 +27,6 @@ else exit 1 fi -if [ $1 = nosubmodule ]; then -SKIP_SUBMODULE=1 -elif [ -n $1 ]; then -echo $0: Illegal argument $1 -echo USAGE: $0 [nosubmodule] -exit 1 -fi - # bootstrap the autotools ( set -x -- 1.7.4.1 ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
[Openocd-development] [PATCH] transport: move files over to transport folder
as we introduce swd and jtag as two transports, we want to start up with a new transport folder to organize the code a bit. --- configure.in |1 + src/Makefile.am |2 + src/jtag/Makefile.am |2 - src/jtag/adapter.c|2 +- src/jtag/core.c |2 +- src/jtag/drivers/ft2232.c |2 +- src/jtag/transport.c | 370 - src/jtag/transport.h | 80 -- src/openocd.c |2 +- src/target/adi_v5_swd.c |2 +- src/transport/Makefile.am | 11 ++ src/transport/transport.c | 370 + src/transport/transport.h | 86 +++ 13 files changed, 475 insertions(+), 457 deletions(-) delete mode 100644 src/jtag/transport.c delete mode 100644 src/jtag/transport.h create mode 100644 src/transport/Makefile.am create mode 100644 src/transport/transport.c create mode 100644 src/transport/transport.h diff --git a/configure.in b/configure.in index 8315a18..cb62c85 100644 --- a/configure.in +++ b/configure.in @@ -1167,6 +1167,7 @@ AC_OUTPUT(dnl src/helper/Makefile dnl src/jtag/Makefile dnl src/jtag/drivers/Makefile dnl +src/transport/Makefile dnl src/xsvf/Makefile dnl src/svf/Makefile dnl src/target/Makefile dnl diff --git a/src/Makefile.am b/src/Makefile.am index 3d7b05f..60d1189 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -4,6 +4,7 @@ SUBDIRS = \ jtag \ helper \ target \ + transport \ flash \ svf \ xsvf \ @@ -98,6 +99,7 @@ libopenocd_la_LIBADD = \ $(top_builddir)/src/svf/libsvf.la \ $(top_builddir)/src/pld/libpld.la \ $(top_builddir)/src/jtag/libjtag.la \ + $(top_builddir)/src/transport/libtransport.la \ $(top_builddir)/src/flash/libflash.la \ $(top_builddir)/src/target/libtarget.la \ $(top_builddir)/src/server/libserver.la \ diff --git a/src/jtag/Makefile.am b/src/jtag/Makefile.am index fa964a3..2717dc5 100644 --- a/src/jtag/Makefile.am +++ b/src/jtag/Makefile.am @@ -56,7 +56,6 @@ libjtag_la_SOURCES = \ interface.c \ interfaces.c \ tcl.c \ - transport.c \ $(DRIVERFILES) noinst_HEADERS = \ @@ -66,7 +65,6 @@ noinst_HEADERS = \ interfaces.h \ minidriver.h \ jtag.h \ - transport.h \ minidriver/minidriver_imp.h \ minidummy/jtag_minidriver.h diff --git a/src/jtag/adapter.c b/src/jtag/adapter.c index e16f756..94843d7 100644 --- a/src/jtag/adapter.c +++ b/src/jtag/adapter.c @@ -35,7 +35,7 @@ #include minidriver.h #include interface.h #include interfaces.h -#include transport.h +#include transport/transport.h #ifdef HAVE_STRINGS_H #include strings.h diff --git a/src/jtag/core.c b/src/jtag/core.c index 22d0b8c..b26701e 100644 --- a/src/jtag/core.c +++ b/src/jtag/core.c @@ -33,7 +33,7 @@ #include jtag.h #include interface.h -#include transport.h +#include transport/transport.h #ifdef HAVE_STRINGS_H #include strings.h diff --git a/src/jtag/drivers/ft2232.c b/src/jtag/drivers/ft2232.c index bde1700..38ead56 100644 --- a/src/jtag/drivers/ft2232.c +++ b/src/jtag/drivers/ft2232.c @@ -81,7 +81,7 @@ /* project specific includes */ #include jtag/interface.h -#include jtag/transport.h +#include transport/transport.h #include helper/time_support.h #if IS_CYGWIN == 1 diff --git a/src/jtag/transport.c b/src/jtag/transport.c deleted file mode 100644 index 6022880..000 --- a/src/jtag/transport.c +++ /dev/null @@ -1,370 +0,0 @@ -/* - * Copyright (c) 2010 by David Brownell - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; either version 2 of the License, or - * (at your option) any later version. - - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software Foundation, - * Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA - */ - -#ifdef HAVE_CONFIG_H -#include config.h -#endif - -/** @file - * Infrastructure for specifying and managing the transport protocol - * used in a given debug or programming session. - * - * Examples of debug-capable transports are JTAG or SWD. - * Additionally, JTAG supports boundary scan testing. - * - * Examples of programming-capable transports include SPI or UART; - * those are used (often mediated by a ROM bootloader) for ISP style - * programming, to perform an initial load of code into flash, or - * sometimes into SRAM. Target code could use variant options to - * decide how to use such protocols. For
Re: [Openocd-development] [PATCH] Moved git submodule routine before autotools invocation as they may depend on submodules. Bootstrap will first checkout modules then run autotools
ignore this patch. I sent it out by accident. On Mon, Jun 13, 2011 at 4:01 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: From: CeDeROM cede...@tlen.pl --- bootstrap | 27 ++- 1 files changed, 10 insertions(+), 17 deletions(-) diff --git a/bootstrap b/bootstrap index 0965245..7062fc0 100755 --- a/bootstrap +++ b/bootstrap @@ -1,9 +1,14 @@ - HEAD -#!/bin/sh -=== #!/bin/sh -e -#fetch necessary submodules before runing autotools +#Prepare necessary submodules before runing autotools +if [ $1 = nosubmodule ]; then + SKIP_SUBMODULE=1 +elif [ -n $1 ]; then + echo $0: Illegal argument $1 + echo USAGE: $0 [nosubmodule] + exit 1 +fi + if [ -n $SKIP_SUBMODULE ]; then echo Skipping submodule setup else @@ -12,11 +17,7 @@ else git submodule update fi -# Run the autotools bootstrap sequence to create the configure script - -# Abort execution on error -set -e - +#Check what libtool version to use if which libtoolize /dev/null; then libtoolize=libtoolize elif which glibtoolize /dev/null; then @@ -26,14 +27,6 @@ else exit 1 fi -if [ $1 = nosubmodule ]; then - SKIP_SUBMODULE=1 -elif [ -n $1 ]; then - echo $0: Illegal argument $1 - echo USAGE: $0 [nosubmodule] - exit 1 -fi - # bootstrap the autotools ( set -x -- 1.7.4.1 ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
[Openocd-development] [PATCH] transport: move files over to transport folder
as we introduce swd and jtag as two transports, we want to start up with a new transport folder to organize the code a bit. --- configure.in|1 + src/Makefile.am |2 ++ src/jtag/Makefile.am|2 -- src/jtag/adapter.c |2 +- src/jtag/core.c |2 +- src/jtag/drivers/ft2232.c |2 +- src/openocd.c |2 +- src/target/adi_v5_swd.c |2 +- src/transport/Makefile.am | 11 +++ src/{jtag = transport}/transport.c |2 +- src/{jtag = transport}/transport.h |6 ++ 11 files changed, 26 insertions(+), 8 deletions(-) create mode 100644 src/transport/Makefile.am rename src/{jtag = transport}/transport.c (99%) rename src/{jtag = transport}/transport.h (95%) diff --git a/configure.in b/configure.in index 8315a18..cb62c85 100644 --- a/configure.in +++ b/configure.in @@ -1167,6 +1167,7 @@ AC_OUTPUT(dnl src/helper/Makefile dnl src/jtag/Makefile dnl src/jtag/drivers/Makefile dnl +src/transport/Makefile dnl src/xsvf/Makefile dnl src/svf/Makefile dnl src/target/Makefile dnl diff --git a/src/Makefile.am b/src/Makefile.am index 3d7b05f..60d1189 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -4,6 +4,7 @@ SUBDIRS = \ jtag \ helper \ target \ + transport \ flash \ svf \ xsvf \ @@ -98,6 +99,7 @@ libopenocd_la_LIBADD = \ $(top_builddir)/src/svf/libsvf.la \ $(top_builddir)/src/pld/libpld.la \ $(top_builddir)/src/jtag/libjtag.la \ + $(top_builddir)/src/transport/libtransport.la \ $(top_builddir)/src/flash/libflash.la \ $(top_builddir)/src/target/libtarget.la \ $(top_builddir)/src/server/libserver.la \ diff --git a/src/jtag/Makefile.am b/src/jtag/Makefile.am index fa964a3..2717dc5 100644 --- a/src/jtag/Makefile.am +++ b/src/jtag/Makefile.am @@ -56,7 +56,6 @@ libjtag_la_SOURCES = \ interface.c \ interfaces.c \ tcl.c \ - transport.c \ $(DRIVERFILES) noinst_HEADERS = \ @@ -66,7 +65,6 @@ noinst_HEADERS = \ interfaces.h \ minidriver.h \ jtag.h \ - transport.h \ minidriver/minidriver_imp.h \ minidummy/jtag_minidriver.h diff --git a/src/jtag/adapter.c b/src/jtag/adapter.c index e16f756..94843d7 100644 --- a/src/jtag/adapter.c +++ b/src/jtag/adapter.c @@ -35,7 +35,7 @@ #include minidriver.h #include interface.h #include interfaces.h -#include transport.h +#include transport/transport.h #ifdef HAVE_STRINGS_H #include strings.h diff --git a/src/jtag/core.c b/src/jtag/core.c index 22d0b8c..b26701e 100644 --- a/src/jtag/core.c +++ b/src/jtag/core.c @@ -33,7 +33,7 @@ #include jtag.h #include interface.h -#include transport.h +#include transport/transport.h #ifdef HAVE_STRINGS_H #include strings.h diff --git a/src/jtag/drivers/ft2232.c b/src/jtag/drivers/ft2232.c index bde1700..38ead56 100644 --- a/src/jtag/drivers/ft2232.c +++ b/src/jtag/drivers/ft2232.c @@ -81,7 +81,7 @@ /* project specific includes */ #include jtag/interface.h -#include jtag/transport.h +#include transport/transport.h #include helper/time_support.h #if IS_CYGWIN == 1 diff --git a/src/openocd.c b/src/openocd.c index a5002d1..57676f5 100644 --- a/src/openocd.c +++ b/src/openocd.c @@ -31,7 +31,7 @@ #include openocd.h #include jtag/driver.h #include jtag/jtag.h -#include jtag/transport.h +#include transport/transport.h #include helper/ioutil.h #include helper/util.h #include helper/configuration.h diff --git a/src/target/adi_v5_swd.c b/src/target/adi_v5_swd.c index 39f3857..5a3570d 100644 --- a/src/target/adi_v5_swd.c +++ b/src/target/adi_v5_swd.c @@ -50,7 +50,7 @@ #include arm_adi_v5.h #include helper/time_support.h -#include jtag/transport.h +#include transport/transport.h #include jtag/interface.h #include jtag/swd.h diff --git a/src/transport/Makefile.am b/src/transport/Makefile.am new file mode 100644 index 000..7c6224a --- /dev/null +++ b/src/transport/Makefile.am @@ -0,0 +1,11 @@ +include $(top_srcdir)/common.mk + +#METASOURCES = AUTO +noinst_LTLIBRARIES = libtransport.la +libtransport_la_SOURCES = \ + transport.c + +noinst_HEADERS = \ + transport.h + +MAINTAINERCLEANFILES = $(srcdir)/Makefile.in diff --git a/src/jtag/transport.c b/src/transport/transport.c similarity index 99% rename from src/jtag/transport.c rename to src/transport/transport.c index 6022880..b5e4b90 100644 --- a/src/jtag/transport.c +++ b/src/transport/transport.c @@ -44,7 +44,7 @@ #include helper/log.h -#include transport.h +#include transport/transport.h extern struct command_context *global_cmd_ctx; diff --git a/src/jtag/transport.h b/src/transport/transport.h similarity index 95% rename from src/jtag/transport.h rename to src/transport/transport.h index d2d38ec..6ece39e 100644 --- a/src/jtag/transport.h +++
[Openocd-development] [PATCH] transport: move files over to transport folder
as we introduce swd and jtag as two transports, we want to start up with a new transport folder to organize the code a bit. --- configure.in|1 + src/Makefile.am |2 ++ src/jtag/Makefile.am|2 -- src/jtag/adapter.c |2 +- src/jtag/core.c |2 +- src/jtag/drivers/ft2232.c |2 +- src/openocd.c |2 +- src/target/adi_v5_swd.c |2 +- src/transport/Makefile.am | 11 +++ src/{jtag = transport}/transport.c |2 +- src/{jtag = transport}/transport.h |6 ++ 11 files changed, 26 insertions(+), 8 deletions(-) create mode 100644 src/transport/Makefile.am rename src/{jtag = transport}/transport.c (99%) rename src/{jtag = transport}/transport.h (95%) diff --git a/configure.in b/configure.in index 8315a18..cb62c85 100644 --- a/configure.in +++ b/configure.in @@ -1167,6 +1167,7 @@ AC_OUTPUT(dnl src/helper/Makefile dnl src/jtag/Makefile dnl src/jtag/drivers/Makefile dnl +src/transport/Makefile dnl src/xsvf/Makefile dnl src/svf/Makefile dnl src/target/Makefile dnl diff --git a/src/Makefile.am b/src/Makefile.am index 3d7b05f..60d1189 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -4,6 +4,7 @@ SUBDIRS = \ jtag \ helper \ target \ + transport \ flash \ svf \ xsvf \ @@ -98,6 +99,7 @@ libopenocd_la_LIBADD = \ $(top_builddir)/src/svf/libsvf.la \ $(top_builddir)/src/pld/libpld.la \ $(top_builddir)/src/jtag/libjtag.la \ + $(top_builddir)/src/transport/libtransport.la \ $(top_builddir)/src/flash/libflash.la \ $(top_builddir)/src/target/libtarget.la \ $(top_builddir)/src/server/libserver.la \ diff --git a/src/jtag/Makefile.am b/src/jtag/Makefile.am index fa964a3..2717dc5 100644 --- a/src/jtag/Makefile.am +++ b/src/jtag/Makefile.am @@ -56,7 +56,6 @@ libjtag_la_SOURCES = \ interface.c \ interfaces.c \ tcl.c \ - transport.c \ $(DRIVERFILES) noinst_HEADERS = \ @@ -66,7 +65,6 @@ noinst_HEADERS = \ interfaces.h \ minidriver.h \ jtag.h \ - transport.h \ minidriver/minidriver_imp.h \ minidummy/jtag_minidriver.h diff --git a/src/jtag/adapter.c b/src/jtag/adapter.c index e16f756..94843d7 100644 --- a/src/jtag/adapter.c +++ b/src/jtag/adapter.c @@ -35,7 +35,7 @@ #include minidriver.h #include interface.h #include interfaces.h -#include transport.h +#include transport/transport.h #ifdef HAVE_STRINGS_H #include strings.h diff --git a/src/jtag/core.c b/src/jtag/core.c index 22d0b8c..b26701e 100644 --- a/src/jtag/core.c +++ b/src/jtag/core.c @@ -33,7 +33,7 @@ #include jtag.h #include interface.h -#include transport.h +#include transport/transport.h #ifdef HAVE_STRINGS_H #include strings.h diff --git a/src/jtag/drivers/ft2232.c b/src/jtag/drivers/ft2232.c index bde1700..38ead56 100644 --- a/src/jtag/drivers/ft2232.c +++ b/src/jtag/drivers/ft2232.c @@ -81,7 +81,7 @@ /* project specific includes */ #include jtag/interface.h -#include jtag/transport.h +#include transport/transport.h #include helper/time_support.h #if IS_CYGWIN == 1 diff --git a/src/openocd.c b/src/openocd.c index a5002d1..57676f5 100644 --- a/src/openocd.c +++ b/src/openocd.c @@ -31,7 +31,7 @@ #include openocd.h #include jtag/driver.h #include jtag/jtag.h -#include jtag/transport.h +#include transport/transport.h #include helper/ioutil.h #include helper/util.h #include helper/configuration.h diff --git a/src/target/adi_v5_swd.c b/src/target/adi_v5_swd.c index 39f3857..5a3570d 100644 --- a/src/target/adi_v5_swd.c +++ b/src/target/adi_v5_swd.c @@ -50,7 +50,7 @@ #include arm_adi_v5.h #include helper/time_support.h -#include jtag/transport.h +#include transport/transport.h #include jtag/interface.h #include jtag/swd.h diff --git a/src/transport/Makefile.am b/src/transport/Makefile.am new file mode 100644 index 000..7c6224a --- /dev/null +++ b/src/transport/Makefile.am @@ -0,0 +1,11 @@ +include $(top_srcdir)/common.mk + +#METASOURCES = AUTO +noinst_LTLIBRARIES = libtransport.la +libtransport_la_SOURCES = \ + transport.c + +noinst_HEADERS = \ + transport.h + +MAINTAINERCLEANFILES = $(srcdir)/Makefile.in diff --git a/src/jtag/transport.c b/src/transport/transport.c similarity index 99% rename from src/jtag/transport.c rename to src/transport/transport.c index 6022880..b5e4b90 100644 --- a/src/jtag/transport.c +++ b/src/transport/transport.c @@ -44,7 +44,7 @@ #include helper/log.h -#include transport.h +#include transport/transport.h extern struct command_context *global_cmd_ctx; diff --git a/src/jtag/transport.h b/src/transport/transport.h similarity index 95% rename from src/jtag/transport.h rename to src/transport/transport.h index d2d38ec..6ece39e 100644 --- a/src/jtag/transport.h +++
Re: [Openocd-development] [PATCH] transport: move files over to transport folder
On Mon, Jun 13, 2011 at 7:32 PM, Tomek CEDRO tomek.ce...@gmail.com wrote: Hey Oyvind! Thank you for support in this matter! Just please add those patches to new branch v0.5 as they bring general code reorganization and are not part of 0.4.x :-) I'll update master branch. The way git works we don't cut development branches, we just keep working on master branch. We may cut a 0.5 branch from some previous time in master history later on. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] transport: move files over to transport folder
On Mon, Jun 13, 2011 at 10:10 PM, Tomek CEDRO tomek.ce...@gmail.com wrote: On Mon, Jun 13, 2011 at 7:53 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: We never will merge 0.5.0 release branch into the master branch, ideally it would receive zero, but realistically might receive a few commits. Possibly reverting commits in master branch? Yeah now I know how things work. Just wanted to freeze 0.4.1 (?) somehow but its not gona be released so there is no such need (as it can be done anytime anywas) - we have better motivation for faster 0.5.0 release :-) And the more harmless commits(like the one I just did) that we can get in before, the easier it will be to cherry pick commits between the two branches... -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Active maintainers
On Tue, Jun 14, 2011 at 4:38 AM, Xiaofan Chen xiaof...@gmail.com wrote: On Sun, Jun 12, 2011 at 5:16 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: I'll update the berlios and sourceforge to reflect the active maintainers. We would of course be delighted if anyone below decides to take a more active role again and re-enabling them is quickly done. Lately Spencer Oliver and I have been the de-facto maintainers of OpenOCD. These should be removed from Berlios as they are inactive: duane (Duane Ellis) vpalatin (Vincent Palatin) mlu (Magnus Lundin) drath (Dominic Rath) bodylove (Carsten Schlote) kc8apf (Rick Altherr) zwelch (Zach Welch) These should be removed from sourceforge, no longer active: duaneellis duaneellis mlusf Magnus Lundin npitre Nicolas Pitre zwelch Zach Welch Looking at the list, this somehow reminds me of those heated discussions about GPL and D2XX. https://lists.berlios.de/pipermail/openocd-development/2009-June/thread.html Last one by Zach Welsh. http://lists.berlios.de/pipermail/openocd-development/2009-July/009518.html Both side of course have their points. Personally I am more with the opinion of adding the D2xx linking exception which Dominic, Rick, Spencer, Magnus Lundin and others agreed. On the other hand, Øyvind Harboe, Zach Welsh, David Brownell and others are more with the pure GPL side. In the end, it is not possible to relicense OpenOCD to add D2xx linking exception. It is impossible to relicense OpenOCD as it would require all copyright holders to agree to linking + exception, which they don't. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Fwd: [Jim-devel] New release of Jim Tcl (0.71) soon
Hi Steve, I intend to upgrade OpenOCD to the latest release once it is available. Thanks for your efforts! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
[Openocd-development] Active maintainers
I'll update the berlios and sourceforge to reflect the active maintainers. We would of course be delighted if anyone below decides to take a more active role again and re-enabling them is quickly done. Lately Spencer Oliver and I have been the de-facto maintainers of OpenOCD. These should be removed from Berlios as they are inactive: duane (Duane Ellis) vpalatin (Vincent Palatin) mlu (Magnus Lundin) drath (Dominic Rath) bodylove (Carsten Schlote) kc8apf (Rick Altherr) zwelch (Zach Welch) These should be removed from sourceforge, no longer active: duaneellis duaneellis mlusf Magnus Lundin npitre Nicolas Pitre zwelch Zach Welch -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH 1/2] dsp5680xx - flash, read speed, crc, cleanup
Merged your entire master branch. http://repo.or.cz/w/openocd.git/blob/HEAD:/HACKING should be updated with some tips on how to write commit comments: topic: short comment blank line longer comments over several lines... You may want to create your own topic branch in that repository and perhaps even delete the master branch: git checkout -b mytopic git push rodrigo :master = deletes master branch from your repository once your topic branch has been merged with master branch, delete it from your repository. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Adding M0 support
Hi Kevin, could you submit a patch that renames cortex-m3.c = cortex-m.c and then a patch with your Cortex M0 work on top of cortex-m.c? Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] General SWD Support in OpenOCD
Hi Tomek, for others to be able to review your work, it is essential that it is a sequence of small logical changes. This will also create a much greater degree of continuity in the official master branch, which is very important to the maintainers. We also want to reorder your commits, so that we can get all changes that are non-swd related pushed to the master repository before we push any swd specific changes. For instance, we could probably move things into the transport folder first and then you can rebase your branch on top of the master branch. Are you familiar with git remote add/rm? You can have both the official and your git repository commits in your local repository. You probably want to delete all branches in your git repository, except for the swd branch and then continue to rebase your swd branch and forcing a push of the updated swd branch as we move manage to take out small parts of your work in commits that we push to the master branch. Can you summarise what kind of changes in your branch that has nothing to do with swd as such and could probably be out as separate commits and committed to the master branch already? Moving files into the transport folder seems like a good candidate. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] General SWD Support in OpenOCD
Hi Tomek, thank you so much for being willing to do the work of rebasing, splitting into patches, etc. This saves maintainers a lot of work and opens up for everybody being able to pitch in and make suggestions for improvements to your work. I plan to look at the SWD stuff, but it will not be in the immediate future, but it looks like you're picking up some interest here on the list now anyway... -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] New development version of OpenOCD available at last!
On Sat, Jun 11, 2011 at 9:31 AM, Freddie Chopin freddie_cho...@op.pl wrote: On 2011-06-10 21:57, Tomek CEDRO wrote: Freddie, why dont you become a release engineer? The work is now done for 0.4.0 I guess its only a matter of snapping the source tree and making a build as you do everyday :-) I really don't know the ways of release manager (; Oyvind says you should be qualified and I don't consider myself to be that. If you start by creating a list of open issues that we should fix and then posting that list to the mailing list and incorporating feedback, then you're half a release engineer already. Here the trick is to try to represent the interests of the community rather than your own. Also we have to make a decision of what we're not going to fix for the release. At some point a certain commit in the git master branch would be the point of a release and you declare so on the list, we tag it and were done. Once we have the tag, there are scripts in git that you'd have to look into that creates source release packages. I am not keen on OpenOCD project providing and support binaries, I want to pass that buck to package managers. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH 1/2] dsp5680xx - flash, read speed, crc, cleanup
You are the only maintainer of this target that I know of on the list, so unless I hear protests I'll commit it after giving it a quick review. Here is the web page to view your git repository btw: http://repo.or.cz/w/openocd/dsp568013.git -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Peter Stuge is now an OpenOCD maintainer
Seems like a good opportunity for someone in the community to step up to the plate and take over the project if they want to. Has someone obviously qualified stepped up and offered to help on libusb as a maintainer? Has Peter refused such an offer? Has someone qualified stepped up and offered to do the work of being a release engineer? Has someone offered to pay Peter for the time he spends on libusb so he can address these issues? Is there anything in the libusb license that stops someone from forking the project? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Peter Stuge is now an OpenOCD maintainer
Well, Peter has certainly refused to step down, despite everybody else on the list pretty much voicing their support to the idea that this is the right thing for him to do, if he doesn't have the time/doesn't want to be involved. Ah. This is the tricky bit. Has someone *qualified* offered to take on this work? If someone unqualified offered to take over as maintainer, then Peter would do the community a disservice if he stepped down. That someone is vocal on the list and supported by the hordes, doesn't make him qualified. This is where Peter must exercise his judgment. It is a lot of unpaid work and grief to run such a project. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Peter Stuge is now an OpenOCD maintainer
On Fri, Jun 10, 2011 at 2:32 PM, Pete Batard pbat...@gmail.com wrote: On 2011.06.10 13:24, Øyvind Harboe wrote: Well, Peter has certainly refused to step down, despite everybody else on the list pretty much voicing their support to the idea that this is the right thing for him to do, if he doesn't have the time/doesn't want to be involved. Ah. This is the tricky bit. Has someone *qualified* offered to take on this work? The tricky part I believe is that someone *qualified* (Not me. I'm pretty sure I could do a better job than what Peter is doing right now, but I have expressed on the list many times that I didn't want it) would offer to take on this work, but will only do so if Peter effectively steps down. I see. Peter won't be alone on this project. He will be one of the maintainers, perhaps that will work better for everybody? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Peter Stuge is now an OpenOCD maintainer
But he doesn't have the time for that. He also puts too much emphasis on things like git/trac which are important but should not trumpet the proper going of a project. The OpenOCD project needs someone like him to set the bar for higher patch quality. This improves quality of OpenOCD and reduces work for the maintainers. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] New development version of OpenOCD available at last!
Please - cut a simple release now - nothing fancy, one or two RC, two weeks for feedback and voila. I want someone qualified to step up to do the work of a release manager or I'm happy with current situation. Figure out what outstanding problems there are, work with the list to get them sorted out, etc. We can cut a 0.5 branch at some commit in the past if the master branch has gone too far ahead by the time someone steps up. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Adding M0 support
What would it take to convert cortex-m3.c to cortex-m.c and avoid code duplication like we had for cortex-a8/9.c? Is there a lot of code duplication here? I haven't looked. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] ft2232: fix memory leak in ft2232_large_scan()
On Thu, Jun 9, 2011 at 8:37 AM, Peter Stuge pe...@stuge.se wrote: Andreas Fritiofson wrote: Strange thing with this function is that the allocated buffer doesn't seem to be used for anything. The data read into it doesn't go anywhere. Maybe the entire function is flawed, or is the data really supposed to be discarded? If noone can answer this then I would vote for removing the buffer completely instead of doing allocation which isn't used anyway. Definitely! I'm holding off on this patch in favor of a patch that deletes cruft instead of decorating it :-) -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] ft2232: fix memory leak in ft2232_large_scan()
Definitely! I'm holding off on this patch in favor of a patch that deletes cruft instead of decorating it :-) I have a soft spot for patches that delete code that nobody understands -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Outstanding patches = Fix: Correctly exit function: ft2232_init when an error occurred
On Thu, Jun 9, 2011 at 7:29 AM, Peter Stuge pe...@stuge.se wrote: Øyvind Harboe wrote: Nit, the construct below is a little bit fancy. I'd prefer splitting it over multiple lines to make it more accessible to the casual reader. Or just use goto, please, which is clear, simple, well understood, and elegant all at once. :) I think goto is OK sometimes. When it is used to unwind the stack frame and free up resources, in lieu of exceptions and resource tracking it can clean up the code significantly. The patch uses code duplication and if statements inside the cleanup fn(if (buffer) free(buffer)), which can be avoided using exception handling and resource tracking. In the case of this patch, it looks like the if (ft2232_buffer) free(ft2232_buffer) is a case of conservative programming. I'd much rather have an assert in this case than to have broken code auto-repair and be harder to get right. Note that I'm not holding off committing the patch for the above, as maintainer I'm interested in progress as well as more philosophical coding style discussions. :-) If someone does come forth with an even more cleaner patch, then I'm OK with that. This is now growing into a case-study of how to formulate a patch more than a fix. The openocd community has much to learn about how to formulate patches. As long as there is progress, I don't mind holding off the patch. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH 1/5] ft2232: Clean up infinite loop condition in ft2232_init()
Can I commit? Surely this one is straightforward!!!??? :-) -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Adding M0 support
Hi Kevin, from the maintainers I can say that we're very much interested in such contributions. By posting it to the list, we hopefully also can get others to pitch in! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
[Openocd-development] Peter Stuge is now an OpenOCD maintainer
I'm very pleased to announce Peter Stuge as an OpenOCD maintainer. The community is very fortunate that he is able and willing to donate a little bit of time in commenting and committing some of the patches. His work so far, both in terms of actual code and patches and contributions in mentoring the list speaks for itself. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] New functionalities in Interface API: interface_signal, bitbang
Now I just have to learn how to rebase my fork with openocd git master to produce patches ;-) I am so pleased to hear that you are willing to make the effort to learn how to do these git operations. That you are willing and able to do this part of the maintenance reduces the burden on the maintainers. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Outstanding patches = Fix: Correctly exit function: ft2232_init when an error occurred
To me this patch looks good. Any objections? Nit, the construct below is a little bit fancy. I'd prefer splitting it over multiple lines to make it more accessible to the casual reader. In terms of splitting this patch up, it might be possible to move the ft2232_quit() fn first, but the patch is already pretty small, so no objection from me. + return ft2232_quit(), ERROR_JTAG_INIT_FAILED; -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Configuration file for stm32f2xxx
Please provide a git patch. That way I don't have to piece together the story from the email thread. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Configuration file for stm32f2xxx
Do an interactive rebase and squash 1+3 together. git rebase --help git rebase -i origin/master = move last commit up below first commit and change pick to s (for squash). git format-patch origin/master -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH 2/2] Fix unused variable warnings (errors) detected with GCC 4.7.0 - dubious fixes
It builds with normal build options, but fails with -O3: CFLAGS=-O3 ./configure --enable-dummy make clean make -s = libtool: compile: gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../.. -I../../src -I../../src -DPKGDATADIR=\/usr/local/share/openocd\ -I../../jimtcl -I../../jimtcl -O3 -Wall -Wstrict-prototypes -Wformat-security -Wshadow -Wextra -Wno-unused-parameter -Wbad-function-cast -Wcast-align -Wredundant-decls -Werror -MT core.lo -MD -MP -MF .deps/core.Tpo -c core.c -o core.o cc1: warnings being treated as errors core.c: In function ‘jtag_get_speed_readable’: core.c:1675:6: error: ‘jtag_speed_var’ may be used uninitialized in this function -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH 2/2] Fix unused variable warnings (errors) detected with GCC 4.7.0 - dubious fixes
Strike that. I pushed a fix for that -O3 warning and that -O3 warning wasn't related to your patch. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] General SWD Support in OpenOCD
Hi Tomek, thanks for working on SWD for OpenOCD. This is obviously a big change and I hope that everybody on the list understands that even if this was ready as-is it we would have to allow plenty of time for everybody to review this code before it goes into the master branch. I hope that we can find a sequence of small enough patches that most people can get their head around each one of the changes with the time they have available to review the changes and follow OpenOCD. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] General SWD Support in OpenOCD
On Tue, Jun 7, 2011 at 7:05 PM, Tomek CEDRO tomek.ce...@gmail.com wrote: On Tue, Jun 7, 2011 at 5:02 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: First thing I tried was to rebase your master branch on top of origin/master and that didn't work... Uhm, this is based on a version from March or April... and there are also some unnecessary changes that should be filtered out I guess. Could you rebase it on top of master branch and force a push? Sure - that wont destroy anything at my side? You can safely try it locally. You'll have to learn about about rebasing before this is done... -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] New development version of OpenOCD available at last!
Nobody has stepped up to do the work of being a release manager. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Configuration file for stm32f2xxx
Merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] use one configuration for all JTAGkey devices.
Any objections? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] use one configuration for all JTAGkey devices.
On Mon, Jun 6, 2011 at 9:24 AM, Yegor Yefremov yegorsli...@googlemail.com wrote: Any objections? Another option were to remove all jtagkey configuration files and leave the jtagkey.cfg only. The description is different... -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Configuration file for stm32f2xxx
Any objections? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Configuration file for stm32f2xxx
To create a patch: git add tcl/tcl/target/stm32f2xxx.cfg git commit = write something like: stm32f2xxx: config file yyy git format-patch HEAD^ Remove the below, it's just copy and paste junk. # For more information about the configuration files, take a look at: # openocd.texi -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Configuration file for stm32f2xxx
On Mon, Jun 6, 2011 at 11:40 AM, Andreas Fritiofson andreas.fritiof...@gmail.com wrote: On Mon, Jun 6, 2011 at 11:23 AM, Øyvind Harboe oyvind.har...@zylin.com wrote: To create a patch: git add tcl/tcl/target/stm32f2xxx.cfg Another minor thing I noticed now, why three x in the file name? And in the flash driver too it seems? Shouldn't it be stm32f2xx? (also, to avoid confusion, I'll just point out that the above git snippet has tcl/ duplicated in the path) No reason really, when I implemented it, I thought there were multiple versions of this chip. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Outstanding patches = Fix: Correctly exit function: ft2232_init when an error occurred
Is this patch ready to be committed? Any objections? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips: add nor flash write from memory block
Merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
[Openocd-development] Adding gdbserver as a target
I've been pondering adding a new target that would use a gdbserver as a backend. Here it would be possible to use e.g. OpenOCD's thread support against commercial GDB servers that add support for targets that OpenOCD does not (yet) support. This would of course open up for commercial JTAG vendors to create closed source target support, yet take advantage of OpenOCD. Some may have strong opinions and feelings about this, but I don't particularly care one way or the other, I'm just happy to see OpenOCD used in more places. I would like to see open source implementations of target support, but if the silicon vendors have too many layers of NDA, then I don't mind that obscurium.com has to forego open source target support for their MCU. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH 2/2] Fix unused variable warnings (errors) detected with GCC 4.7.0 - dubious fixes
usbprog.c is clearly broken, I don't think this warning should be fixed as that would be shuffling a bug under the carpet. I've pushed most of the other fixes, but you'll need to run through two versions of GCC or try -O3 to get the residual warnings I think. Try: CFLAGS=-O3 -g ./configure make clean make -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips: add nor flash write from memory block
Ref. your todo: shouldn't that be sorted out before we merge it? How does this affect the arm target? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] dsp5680xx - verification after flashing
For next time, please change the first line of the comment in the patch to: dsp560xxx: fix stuff... Merged both patches. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH 2/2] Fix unused variable warnings (errors) detected with GCC 4.7.0 - dubious fixes
Hi Freddie, I just wanted to merge those changes that were ready to go in as that was all I have time for this morning. Some of your changes broke compiling on gcc 4.4.3(generated more warnings), so I didn't want to put them in there yet. I think gcc 4.4.3 discovered more unused code once you removed some of that unused code. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Adding gdbserver as a target
I definitely want to see a gdb server target. I don't think a gdb server openocd interface makes any sense. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH 2/2] Fix unused variable warnings (errors) detected with GCC 4.7.0 - dubious fixes
Ok, I'll wait then. Maybe I'll investigate on my own - just 4 files so maybe I'll find something. To create the residual commit: git fetch origin git rebase origin/master = your commits will now contain only the residual changes. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Adding gdbserver as a target
On Sat, Jun 4, 2011 at 10:54 AM, Tomek CEDRO tomek.ce...@gmail.com wrote: On Sat, Jun 4, 2011 at 8:31 AM, Øyvind Harboe oyvind.har...@zylin.com wrote: I definitely want to see a gdb server target. I don't think a gdb server openocd interface makes any sense. From current organization: -target is a device/cpu -target needs physical connection using interface No reason why target should require this in the future. I'd also like to see a dummy target for testing purposes that requires no interface. For a gdb server target, no interface would be required. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Adding gdbserver as a target
On Sat, Jun 4, 2011 at 11:33 AM, Tomek CEDRO tomek.ce...@gmail.com wrote: On Sat, Jun 4, 2011 at 9:26 AM, Øyvind Harboe oyvind.har...@zylin.com wrote: No reason why target should require this in the future. I'd also like to see a dummy target for testing purposes that requires no interface. Hmm, such dummy target could become pretty good universal framework for abstracting many other things :-) Yup. Perhaps a good first step? It wouldn't require any special hardware or software, which is a Good Thing when cooperating... -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Use DCC on ARM to implement console
On Fri, Jun 3, 2011 at 12:03 PM, Tom Schouten t...@zwizwa.be wrote: Hi list, I was wondering if anyone has tried before to use the DCC on ARM to implement a bidirectional console over JTAG, eliminating the need for a serial console. This is supported today, if not documented satisfactorily. What did you find in the manual on this topic? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Stepping backwards using ETB
On Fri, Jun 3, 2011 at 8:08 PM, Drasko DRASKOVIC drasko.drasko...@gmail.com wrote: On Fri, Jun 3, 2011 at 3:45 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: Anyone given any thought to adding support to the GDB server to stepping backwards? http://www.gnu.org/software/gdb/news/reversible.html Hi Øyvind, Where do you see that work should be done - on GDB server side or OpenOCD ? Or some work on GDB as well ? The GDB server in OpenOCD could add support for these new packets. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] Fix unused variable warnings (errors) detected with GCC 4.7.0
Very cool! Perhaps you could split this patch into several patches with the first patch to go in are with the most obvious fixes? E.g. I was less sure about etb.c, but it might be fine... -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] New configuration for uptech2410 board.
Merged. Thanks! BTW: I think in the file of target/samsung_s3c2410.cfg, the line 19 should not be 0x, the correct one is: set _CPUTAPID 0x0032409d Patch? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH 1/2] dsp5680xx - code cleanup
I'll let these cool off for a day or so and then merge if I hear nothing further. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] Fix unused variable warnings (errors) detected with GCC 4.7.0
On Thu, Jun 2, 2011 at 10:11 PM, Freddie Chopin freddie_cho...@op.plwrote: On 2011-06-02 08:12, Øyvind Harboe wrote: Perhaps you could split this patch into several patches with the first patch to go in are with the most obvious fixes? E.g. I was less sure about etb.c, but it might be fine... Any hints on the method to split such commited change into several commits? GIT is still not my strongest side (; wip = work-in-progress, new branch below git checkout -b wip git reset HEAD^ git add foo.c git commit git add bar.c git commit git rebase origin/master gir format-patch origin/master = generate patches since origin/master 4\/3!! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] TCL Memory tests
My main concern about such a test would be that it doesn't run very quickly on the target. A great tool to have before moving on to running code on the target though! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] bootstrap: patch proposal
Would you mind formulating a patch? It would be greatly appreciated. Thanks! 2011/6/1 Tormod Volden lists.tor...@gmail.com: On Wed, Jun 1, 2011 at 7:22 AM, Øyvind Harboe wrote: Merged very slightly different version that prints out error message upon invalid argument. From the commit: +if [ $1 == nosubmodule ]; then Note that if $1 is empty, some (most) shells will see just [ == nosubmodule ] and fail. So always quote variables that possibly can be empty, like [ $1 = nosubmodule ]. I quoted also the string here, but just for consistency and good measures. Note also that == is a bashism. In sh scripts, use = for comparison. Tormod -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] bootstrap: patch proposal
How about this? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer From f81b65e167dfe137d946d0add26d74b3e9ad8fce Mon Sep 17 00:00:00 2001 From: Tormod Volden lists.tor...@gmail.com Date: Wed, 1 Jun 2011 17:03:05 +0200 Subject: [PATCH] bootstrap: make script more robust Suggestions copied and pasted from the mailing list. Any mistakes are the comitter's. --- bootstrap |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/bootstrap b/bootstrap index 39ece23..fc92610 100755 --- a/bootstrap +++ b/bootstrap @@ -27,7 +27,7 @@ automake --gnu --add-missing --copy # current source snapshots (working from GIT, or some source snapshot, etc) # otherwise the documentation will fail to build due to missing version.texi -if [ $1 == nosubmodule ]; then +if [ $1 = nosubmodule ]; then echo Skipping submodule setup elif [ -z $1 ]; then echo Setting up submodules -- 1.7.4.1 ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Outstanding patches = Fix: Correctly exit function: ft2232_init when an error occurred
Hi Tomek, I haven't read over this patch. Do you think it is ready or does it need work? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH v2] bootstrap: Various code improvements
Merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] bootstrap: patch proposal
I see the bug in the code, but I don't understand why the list hasn't complained and you're running into this problem. What environment are you using? bash? dash? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips target
Perhaps some cleanup of ancient history is called for? -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips target
Any objections? I haven't looked over it, but I see there has been some discussion on this topic. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] bootstrap: patch proposal
Merged very slightly different version that prints out error message upon invalid argument. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips target
Merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] SMDK6410 is not target, move file to board
Merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips target
Merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] Board definition for mini6410/tiny6410 (ARM1176)
Merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips target
Endian host bugfix merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips target
On Sat, May 28, 2011 at 2:31 PM, Drasko DRASKOVIC drasko.drasko...@gmail.com wrote: On Sat, May 28, 2011 at 2:27 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: Endian host bugfix merged. What's the impact ? Will mdw now inverse bytes for BE targets ? I checked quickly last night on my BE target, mdw was showing good order of bytes - exactly what I wrote in the memory... The impact is that the code will behave the *same* on big and small endian hosts. Whether that's broken or working is another matter entirely... -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] Fixed values for Samung NAND chips
Merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips target
On Fri, May 27, 2011 at 1:27 PM, Mahr, Stefan stefan.m...@sphairon.com wrote: My patch fixes an endianness problem when OpenOCD is build for running on a big endian host. For little endian host everything works as usual. Ack. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] mips target
Looking at this patch, it's clearly wrong to cast a pointer to a host 32 bit integer to a pointer to a sequence of bytes to shift out. I don't know about the rest of the issues offhand, but I see Drask was going to have a look at it. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] OpenOCD for STM32F2xx microcontroller
There is more than one stm32 flash driver, use the other one -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] OpenOCD for STM32F2xx microcontroller
Sorry but I only know stm32x... could you pls give a name to the other one? cd src find . | grep stm32 -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Untested or slightly tested code
On Wed, May 25, 2011 at 1:33 PM, j. m. norris jmnor...@ieee.org wrote: Would it make sense for those individuals who are submitting code that is untested or slightly tested to include a short write-up on their setup? This write-up would allow those with a little free time on their hands or for someone who wants to learn about the development involved with OpenOCD to possibly help test and not spend a lot of time getting the setup right. Absolutely. And if someone has patches with documentation when testing beta features, please post that as well! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Untested or slightly tested code
On Wed, May 25, 2011 at 7:43 PM, Rodrigo Rosa rodrigorosa...@gmail.com wrote: should the short write-up go in the email with the corresponding patch? I think splitting out the docs in a separate patch is just fine. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] [PATCH] support for non cfi spansion flash
Merged. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Provisional eCos RTOS support
Merged. Thanks! -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] OMAP3530 / beagleboard supper status
I know sw breakpoints w/cortex-a cpus need work to deal with cache and mmu handling. Multi-core cortex a CPUs w/hardware and software breakpoints, especially when only halting a single CPU shoud be very interesting. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
Re: [Openocd-development] Provisional eCos RTOS support
Any objections to merging? It's a big step in the right direction and hopefully this will encourage more people to pitch in. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development