Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> writes: > On Fri, Feb 10, 2012 at 09:44:08AM +0100, Bjørn Mork wrote: >> Add a flag to tell wdm_read/wdm_write that a reset is in progress, >> and wake any blocking read/write before taking the mutexes. This >> allows the device to reset without waiting for blocking IO to >> finish. >> >> Signed-off-by: Bjørn Mork <bjorn@xxxxxxx> >> Acked-by: Oliver Neukum <oneukum@xxxxxxx> >> --- >> Hello Greg, >> >> I just received an ack in private mail from Oliver for this >> patch. Please apply to usb-next. > > I would, but it fails to apply: > > patching file drivers/usb/class/cdc-wdm.c > Hunk #2 succeeded at 340 (offset -1 lines). > Hunk #3 succeeded at 423 (offset -1 lines). > Hunk #4 FAILED at 868. > Hunk #5 succeeded at 865 (offset -14 lines). > Hunk #6 succeeded at 883 with fuzz 2 (offset -14 lines). > 1 out of 6 hunks FAILED -- saving rejects to file drivers/usb/class/cdc-wdm.c.rej > > What went wrong? I suspect I have missed something important somewhere. I must admit that the patch was actually against linux-next. I just assumed that any patch against this driver which was there would be in your usb-next. Big mistake. I see now that there are some patches in your usb-linus branch which are not in your usb-next branch: bjorn@nemi:/usr/local/src/git/linux$ git log --oneline v3.2..usb/usb-linus -- drivers/usb/class/cdc-wdm.c 15699e6 USB: cdc-wdm: Avoid hanging on interface with no USB_CDC_DMM_TYPE 655e247 USB: cdc-wdm: better allocate a buffer that is at least as big as we tell the USB core 62aaf24 USB: cdc-wdm: call wake_up_all to allow driver to shutdown on device removal e8537bd USB: cdc-wdm: use two mutexes to allow simultaneous read and write c428b70 USB: cdc-wdm: updating desc->length must be protected by spin_lock 65db430 USB: convert drivers/usb/* to use module_usb_driver() bjorn@nemi:/usr/local/src/git/linux$ git log --oneline v3.2..usb/usb-next -- drivers/usb/class/cdc-wdm.c fec67b4 usb: cdc-wdm: Add device-id for Huawei 3G/LTE modems 820c629 USB: cdc-wdm: avoid printing odd-looking "cdc-wdm-176" names 7e3054a USB: cdc-wdm: Avoid hanging on interface with no USB_CDC_DMM_TYPE 8143a89 USB: cdc-wdm: kill the now unnecessary bMaxPacketSize0 field and udev variable 8457d99 USB: cdc-wdm: no need to use usb_alloc_coherent cafbe85 USB: cdc-wdm: better allocate a buffer that is at least as big as we tell the USB core 19b85b3 USB: cdc-wdm: no need to fill the in request URB every time it's submitted 65db430 USB: convert drivers/usb/* to use module_usb_driver() The main reason this patch failed was because the "USB: cdc-wdm: use two mutexes to allow simultaneous read and write" is missing. I am more than happy to fix up the mess I've caused, but am afraid I'll need a hint or two on how exactly... Bjørn -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
