staging: most: hdm-usb: stop core from submitting buffers in case of broken pipe
authorChristian Gromm <christian.gromm@microchip.com>
Fri, 19 Aug 2016 09:12:48 +0000 (11:12 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 21 Aug 2016 14:53:56 +0000 (16:53 +0200)
This patch ensures that no more packets are submitted by the core in
case an USB endpoint has reported a broken pipe (-EPIPE).

Signed-off-by: Andrey Shvetsov <andrey.shvetsov@k2l.de>
Signed-off-by: Christian Gromm <christian.gromm@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/most/hdm-usb/hdm_usb.c

index 8f1c09670dfb3df900d1f3c716e9790629e6435e..03cdd0d1283d53c01c31e8219eae6c242c35b871 100644 (file)
@@ -580,6 +580,7 @@ static void hdm_read_completion(struct urb *urb)
                switch (urb->status) {
                case -EPIPE:
                        dev_warn(dev, "Broken IN pipe detected\n");
+                       most_stop_enqueue(&mdev->iface, channel);
                        mbo->status = MBO_E_INVAL;
                        mdev->clear_work[channel].pipe = urb->pipe;
                        schedule_work(&mdev->clear_work[channel].ws);
@@ -936,8 +937,7 @@ static void wq_clear_halt(struct work_struct *wq_obj)
        if (usb_clear_halt(mdev->usb_device, pipe))
                dev_warn(&mdev->usb_device->dev, "Failed to reset endpoint.\n");
 
-       if (mdev->conf[channel].direction & MOST_CH_TX)
-               most_resume_enqueue(&mdev->iface, channel);
+       most_resume_enqueue(&mdev->iface, channel);
        mutex_unlock(&mdev->io_mutex);
 }