Marcus Overhagen <marcus.overhagen <at> gmail.com> writes: > > Increasing the timeout when polling for card status to 100ms > as used at other places in this driver fixes SD card detection. > > Also use correct interval when doing the interrupt transfer, > this fixes the "xhci_queue_intr_tx: 74 callbacks suppressed" > spamming to syslog that was occuring when this driver is used. > > Signed-off-by: Marcus Overhagen <marcus.overhagen <at> gmail.com> > --- > drivers/staging/rts5139/rts51x_transport.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/rts5139/rts51x_transport.c b/drivers/staging/rts5139/rts51x_transport.c > index 89e4d80..c172f4a 100644 > --- a/drivers/staging/rts5139/rts51x_transport.c > +++ b/drivers/staging/rts5139/rts51x_transport.c > <at> <at> -635,12 +635,12 <at> <at> int rts51x_get_epc_status(struct rts51x_chip *chip, u16 *status) > ep = chip->usb->pusb_dev->ep_in[usb_pipeendpoint(pipe)]; > > /* fill and submit the URB */ > - /* We set interval to 1 here, so the polling interval is controlled > - * by our polling thread */ > + /* Set interval to 10 here to match the endpoint descriptor, > + * the polling interval is controlled by the polling thread */ > usb_fill_int_urb(chip->usb->intr_urb, chip->usb->pusb_dev, pipe, > - status, 2, urb_done_completion, &urb_done, 1); > + status, 2, urb_done_completion, &urb_done, 10); > > - result = rts51x_msg_common(chip, chip->usb->intr_urb, 50); > + result = rts51x_msg_common(chip, chip->usb->intr_urb, 100); > > return interpret_urb_result(chip, pipe, 2, result, > chip->usb->intr_urb->actual_length);
Hi Marcus, I'm trying to apply this patch to 3.8.0-18 kernel tree but looks like it doesn't want to take (Ubuntu 13.04 stock) I don't suppose you have a patch for this version?, not quite sure if reading the patch file properly but assuming I get changes in what is the correct way to just compile this module and insert into running config? Thanks Simon -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
