drivers/usb/serial/usb-serial.c in 2.5.48 fails to compile with the
following error:

drivers/usb/serial/usb-serial.c:842: dereferencing pointer to incompletetype

Is the following patch correct?

This patch puts all the code to interpret the result code from an URB into
a single place, instead of copying it everywhere throughout transport.c

This patch changes freecom.c and isd200.c to use the new data-moving logic
instead of the old data-moving logic.  This allows for code consolidation
and better error-handling.

This patch changes the data buffer type from char* to void*, and fixes some
problems with debug prints and comments.

fixed a bug if CONFIG_VIDEO_PROC_FS was not enabled.
removed unneeded #ifdefs
removed bool nonsense.

Included in this patch:

- (From John Tyner) Move allocation of memory out of send_control_msg. With
the allocation moved to open, control messages are less expensive since
they don't allocate and free memory every time.
- (From John Tyner) Change the behaviour of send_control_msg to return 0 on
success instead of the number of bytes transferred.
- Clean up of a couple down_interruptible() calls that weren't checking for
failure
- Rewrite of proc fs entries to use one file per value instead of parsing
in the kernel

Sorry about sending the screwed up patches the first time.  Here's the
fix for the two missing "{".

This patch makes the futex code check utime only when waiting.
This makes it possible to do futex wakes without clearing the register
containing the utime parameter.

The code also becomes cleaner.

This patch moves retval = -ENOMEM out of the vma loop and after the
fail_nomem label.

The fail label is added and is used when retval is already set.

Make sure the non-boot CPU's aren't taking interrupts before they are ready..

This cleans up the dcache code to always use the proper dcache functions
(d_unhashed and __d_drop) instead of accessing the dentry lists
directly.

In other words: use "d_unhashed(dentry)" instead of doing a manual
"list_empty(&dentry->d_hash)" test.  And use "__d_drop(dentry)" instead
of doing "list_del_init(&dentry->d_hash)" by hand.

This will help the dcache-rcu patches.

into home.transmeta.com:/home/torvalds/v2.5/linux

Also some cleanups wrt struct member initialization style.