Zach Welch wrote: > On Sun, 2009-04-19 at 17:54 -0700, Zach Welch wrote: > >> On Sun, 2009-04-19 at 17:18 -0700, Zach Welch wrote: >> >>> Hi all, >>> >>> Since my patch to enable -Werror got in a little faster than I expected, >>> I decided to try and save myself some pain by installing and testing the >>> libftdi and libftd2xx libraries and tested the associated OpenOCD driver >>> builds against each set of configuration options. >>> >>> The attached patch fixes the warnings that appear when --enable-ft2232-* >>> or --enable-presto-* are used. >>> >> Nuts. As soon as this hit the list, I noticed that I should have used >> '%u' and not '%d' in the format specifiers. Please use this version. >> > > Okay... third time is the charm. I neglected to re-test the "other" > library configuration, and it turns out I still needed a couple of casts > to straighten things out between both versions. This version should do > the trick (unless it doesn't... *sigh*). Sorry for the noise. > > Cheers, > > Zach >
-int presto_write(u8 *buf, int size) +int presto_write(u8 *buf, u32 size) Zack, Why are you changing this argument from the machine's natural size to one that is 32 bits wide? Dick _______________________________________________ Openocd-development mailing list [email protected] https://lists.berlios.de/mailman/listinfo/openocd-development
