[PATCH] mt7601u: process URBs in status EPROTO properly

[PATCH] mt7601u: process URBs in status EPROTO properly

[Zhi Han]

When the usb device being plugged out, before the usb_driver:disconnect
called by e.g workqueue, it is possible that some URBs are still in
processing, and being marked as EPROTO in host controller.

Those URBs should not be scheduled in complete_rx callback function to
get further processing.

Signed-off-by: Zhi Han <z.han@gmx.net>
Acked-by: Jakub Kicinski <kuba@kernel.org>
---
 drivers/net/wireless/mediatek/mt7601u/dma.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/wireless/mediatek/mt7601u/dma.c b/drivers/net/wireless/mediatek/mt7601u/dma.c
index 09f931d4598c..7a9ab787cf3d 100644
--- a/drivers/net/wireless/mediatek/mt7601u/dma.c
+++ b/drivers/net/wireless/mediatek/mt7601u/dma.c
@@ -192,6 +192,7 @@ static void mt7601u_complete_rx(struct urb *urb)
 	case -ECONNRESET:
 	case -ESHUTDOWN:
 	case -ENOENT:
+	case -EPROTO:
 		return;
 	default:
 		dev_err_ratelimited(dev->dev, "rx urb failed: %d\n",

Re: [PATCH] mt7601u: process URBs in status EPROTO properly

[Kalle Valo]

Zhi Han <z.han@gmx.net> wrote:

> When the usb device being plugged out, before the usb_driver:disconnect
> called by e.g workqueue, it is possible that some URBs are still in
> processing, and being marked as EPROTO in host controller.
> 
> Those URBs should not be scheduled in complete_rx callback function to
> get further processing.
> 
> Signed-off-by: Zhi Han <z.han@gmx.net>
> Acked-by: Jakub Kicinski <kuba@kernel.org>

Patch applied to wireless-drivers-next.git, thanks.

0e40dbd56d67 mt7601u: process URBs in status EPROTO properly

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20201217161302.GA12562@E480/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches