]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
USB: serial: fix regression in Visor/Palm OS module for kernels >= 2.6.24
authorBrad Sawatzky <brad+kernel@swatter.net>
Wed, 26 Mar 2008 02:32:43 +0000 (22:32 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 2 Apr 2008 22:06:09 +0000 (15:06 -0700)
Fixes a bug/inconsistency revealed by the additional sanity checking in
   commit 063a2da8f01806906f7d7b1a1424b9afddebc443
introduced in the original 2.6.24 branch.

The Handspring Visor / PalmOS 4 device structure defines .num_bulk_out=2
but the usb-serial probe returns num_bulk_out=3, triggering the check in
the above commit and forcing a bail out when the device (a Garmin iQue in
my case) attempts to connect.  The patch bumps the expected number of
endpoints to 3.

FWIW, this patch will probably solve the following kernel bug report for
Treo users (identical symptoms, different model PalmOS units):
  <http://bugzilla.kernel.org/show_bug.cgi?id=10118>

Signed-off-by: Brad Sawatzky <brad+kernel@swatter.net>
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/serial/visor.c

index 22b3f78a388ce0d40486ede8c98f9af41051dc49..c2b01f7c3197ad0f74ebac9f14b21d71c00d04f5 100644 (file)
@@ -191,7 +191,7 @@ static struct usb_serial_driver handspring_device = {
        .id_table =             id_table,
        .num_interrupt_in =     NUM_DONT_CARE,
        .num_bulk_in =          2,
-       .num_bulk_out =         2,
+       .num_bulk_out =         NUM_DONT_CARE,
        .num_ports =            2,
        .open =                 visor_open,
        .close =                visor_close,