Re: [Openocd-development] [PATCH] transport selection fix

2011-06-24 Thread Jie Zhang
On Sat, Jun 18, 2011 at 4:36 AM, Tomek CEDRO tomek.ce...@gmail.com wrote: On Thu, Jun 16, 2011 at 9:42 PM, Andreas Fritiofson andreas.fritiof...@gmail.com wrote: a ? : b is equivalent to a ? a : b, unless evaluating a has side-effects or if a is volatile, since it's only evaluated once in the

Re: [Openocd-development] [PATCH] transport selection fix

2011-06-18 Thread Tomek CEDRO
On Thu, Jun 16, 2011 at 9:42 PM, Andreas Fritiofson andreas.fritiof...@gmail.com wrote: a ? : b is equivalent to a ? a : b, unless evaluating a has side-effects or if a is volatile, since it's only evaluated once in the former case and twice in the latter. Nice to know, thanks! Is that true

[Openocd-development] [PATCH] transport selection fix

2011-06-16 Thread Tomek CEDRO
This patch fixes transport selection mechanism - if transport is defined it will be selected. Before 'jtag' was always selected no matter what transport in fact was supposed to be used. -- CeDeROM, SQ7MHZ, http://www.tomek.cedro.info

Re: [Openocd-development] [PATCH] transport selection fix

2011-06-16 Thread Tomek CEDRO
On Thu, Jun 16, 2011 at 5:50 PM, Øyvind Harboe oyvind.har...@zylin.com 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, jtag_interface-transports ?

Re: [Openocd-development] [PATCH] transport selection fix

2011-06-16 Thread Øyvind Harboe
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

Re: [Openocd-development] [PATCH] transport selection fix

2011-06-16 Thread Andreas Fritiofson
On Thu, Jun 16, 2011 at 8:02 PM, Tomek CEDRO tomek.ce...@gmail.com wrote: On Thu, Jun 16, 2011 at 5:50 PM, Øyvind Harboe oyvind.har...@zylin.com wrote: I take it is just a bugfix? Yes - it changes: retval = allow_transports(CMD_CTX, jtag_interface-transports ? : jtag_only); into: