HEADSUP new usb code coming in.

Rink Springer rink at FreeBSD.org
Wed Aug 20 20:06:38 UTC 2008


Hi Alfred,

On Tue, Aug 19, 2008 at 12:44:13PM -0700, Alfred Perlstein wrote:
> http://mu.org/~bright/usb2/usb4bsd.diff.gz

Whenever I try this patch, my build dies with:

cc -c -O2 -pipe -fno-strict-aliasing  -std=c99  -Wall -Wredundant-decls
-Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes
-Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign
-fformat-extensions -nostdinc  -I. -I../../.. -I../../../contrib/altq
-D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common
-finline-limit=8000 --param inline-unit-growth=100 --param
large-function-growth=1000  -mno-align-long-strings
-mpreferred-stack-boundary=2  -mno-mmx -mno-3dnow -mno-sse -mno-sse2
-mno-sse3 -ffreestanding -fstack-protector -Werror
../../../dev/usb2/core/usb2_generic.c
cc1: warnings being treated as errors
../../../dev/usb2/core/usb2_generic.c: In function 'ugen_ioctl':
../../../dev/usb2/core/usb2_generic.c:1409: warning: 'ep_index' may be
used uninitialized in this function
../../../dev/usb2/core/usb2_generic.c:1409: note: 'ep_index' was
declared here
*** Error code 1

I'm using a small patch to get it to compile:

-	uint8_t ep_index;
+	uint8_t ep_index = 0;

However, I am unsure if this is correct.

Futhermore, 'device usb2_bluetooth' gives a lot of compile errors. Am I
using the right patch here?

Regards,

-- 
Rink P.W. Springer                                - http://rink.nu
"Anyway boys, this is America. Just because you get more votes doesn't
 mean you win." - Fox Mulder


More information about the freebsd-usb mailing list