On Sat, Apr 19, 2014 at 3:59 PM, Paul Fertser <[email protected]> wrote:
> On Sat, Apr 19, 2014 at 01:49:48PM +0800, Xiaofan Chen wrote:
>> Another minor problem, I think the following "./configure --help" output is
>> not really correct.
>>   --enable-openjtag_ftdi  Enable building support for the OpenJTAG Programmer
>>                           with ftdi driver
>>
>> I believe it should be
>>   --enable-openjtag_libftdi  Enable building support for the OpenJTAG 
>> Programmer
>>                           with libftdi driver
>>
>> The reason is that "--enable-openjtag_libftdi" actually works
>
> Hm, looking through the configure.ac I can't see how it can be so,
> --enable-openjtag_libftdi just can't work, and --enable-openjtag_ftdi
> seems to be entirely correct.
>

You are of course right again.

On the other hand, ---enable-openjtag_libftdi does not generate an invalid
option warning, which is strange.

And the name --enable-openjtag_ftdi is not consistent with other configuration
name and it is not really using "ftdi" driver but rather "libftdi" driver.

If you look at the generated config.h file, you will know what I mean.
I think this configuration name should be changed, not necessary
for the 0.8 release, but probably post release.

/* 1 if you want the OpenJTAG drvier. */
#define BUILD_OPENJTAG 1

/* 1 if you want the OpenJTAG driver with FTD2XX driver. */
#define BUILD_OPENJTAG_FTD2XX 0

/* 1 if you want to build OpenJTAG with FTDI driver. */
#define BUILD_OPENJTAG_LIBFTDI 1

-- 
Xiaofan

------------------------------------------------------------------------------
Learn Graph Databases - Download FREE O'Reilly Book
"Graph Databases" is the definitive new guide to graph databases and their
applications. Written by three acclaimed leaders in the field,
this first edition is now available. Download your free book today!
http://p.sf.net/sfu/NeoTech
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to