lkml.org 
[lkml]   [2006]   [May]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: usb-serial ipaq kernel problem
    On Mon, May 29, 2006 at 02:11:10PM -0300, Luiz Fernando N. Capitulino wrote:
    >
    > Frank, could you try this one please?
    >
    > I have no sure whether this makes sense, but every USB-Serial driver
    > I know exits in the write URB callback if the URB got an error.

    It looks sane to me at least.
    The machine is now running with this patch (and my ipaq_open patch, see
    http://www.ussg.iu.edu/hypermail/linux/kernel/0605.2/1901.html).
    If the problem is still there, it should occur within 24 hours in our
    usage mode (25 ipaqs rebooting every 15 minutes to provide lots of
    connects and disconnects). I'll let you know the results.

    Frank

    >
    > -------------------
    >
    > usbserial: ipaq: Don't handle URB on errors.
    >
    > ipaq_write_bulk_callback() should exit if the URB got an error, otherwise
    > we can get weird problems.
    >
    > Signed-off-by: Luiz Fernando N. Capitulino <lcapitulino@mandriva.com.br>
    >
    > ---
    >
    > drivers/usb/serial/ipaq.c | 1 +
    > 1 files changed, 1 insertions(+), 0 deletions(-)
    >
    > 6ad106187344769a4722f9e6d6f265529844d568
    > diff --git a/drivers/usb/serial/ipaq.c b/drivers/usb/serial/ipaq.c
    > index 9a5c979..d263a5e 100644
    > --- a/drivers/usb/serial/ipaq.c
    > +++ b/drivers/usb/serial/ipaq.c
    > @@ -855,6 +855,7 @@ static void ipaq_write_bulk_callback(str
    >
    > if (urb->status) {
    > dbg("%s - nonzero write bulk status received: %d", __FUNCTION__, urb->status);
    > + return;
    > }
    >
    > spin_lock_irqsave(&write_list_lock, flags);
    > --
    > 1.3.2
    >
    >
    >
    > --
    > Luiz Fernando N. Capitulino

    --
    Frank Gevaerts frank.gevaerts@fks.be
    fks bvba - Formal and Knowledge Systems http://www.fks.be/
    Stationsstraat 108 Tel: ++32-(0)11-21 49 11
    B-3570 ALKEN Fax: ++32-(0)11-22 04 19
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-05-29 21:47    [W:0.036 / U:31.548 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site