Oliver Neukum 26397b
From b851f7c7b8fd5365e447bb60e1e18eb6de628507 Mon Sep 17 00:00:00 2001
Oliver Neukum 26397b
From: Wesley Cheng <wcheng@codeaurora.org>
Oliver Neukum 26397b
Date: Mon, 18 Oct 2021 12:26:47 -0700
Oliver Neukum 26397b
Subject: [PATCH] usb: dwc3: gadget: Change to dev_dbg() when queuing to
Oliver Neukum 26397b
 inactive gadget/ep
Oliver Neukum 26397b
Git-commit: b851f7c7b8fd5365e447bb60e1e18eb6de628507
Oliver Neukum 26397b
References: git-fixes
Oliver Neukum 26397b
Patch-mainline: v5.16-rc1
Oliver Neukum 26397b
Oliver Neukum 26397b
Since function drivers will still be active until dwc3_disconnect_gadget()
Oliver Neukum 26397b
is called, some applications will continue to queue packets to DWC3
Oliver Neukum 26397b
gadget.  This can lead to a flood of messages regarding failed ep queue,
Oliver Neukum 26397b
as the endpoint is in the process of being disabled.  Change the log
Oliver Neukum 26397b
level to debug, so that it can be enabled when debugging issues.
Oliver Neukum 26397b
Oliver Neukum 26397b
Signed-off-by: Wesley Cheng <wcheng@codeaurora.org>
Oliver Neukum 26397b
Link: https://lore.kernel.org/r/20211018192647.32121-1-wcheng@codeaurora.org
Oliver Neukum 26397b
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Oliver Neukum 26397b
Signed-off-by: Oliver Neukum <oneukum@suse.com>
Oliver Neukum 26397b
---
Oliver Neukum 26397b
 drivers/usb/dwc3/gadget.c | 2 +-
Oliver Neukum 26397b
 1 file changed, 1 insertion(+), 1 deletion(-)
Oliver Neukum 26397b
Oliver Neukum 26397b
diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
Oliver Neukum 26397b
index 4845682a0408..0d32e97f11cd 100644
Oliver Neukum 26397b
--- a/drivers/usb/dwc3/gadget.c
Oliver Neukum 26397b
+++ b/drivers/usb/dwc3/gadget.c
Oliver Neukum 26397b
@@ -1813,7 +1813,7 @@ static int __dwc3_gadget_ep_queue(struct dwc3_ep *dep, struct dwc3_request *req)
Oliver Neukum 26397b
 	struct dwc3		*dwc = dep->dwc;
Oliver Neukum 26397b
 
Oliver Neukum 26397b
 	if (!dep->endpoint.desc || !dwc->pullups_connected || !dwc->connected) {
Oliver Neukum 26397b
-		dev_err(dwc->dev, "%s: can't queue to disabled endpoint\n",
Oliver Neukum 26397b
+		dev_dbg(dwc->dev, "%s: can't queue to disabled endpoint\n",
Oliver Neukum 26397b
 				dep->name);
Oliver Neukum 26397b
 		return -ESHUTDOWN;
Oliver Neukum 26397b
 	}
Oliver Neukum 26397b
-- 
Oliver Neukum 26397b
2.35.3
Oliver Neukum 26397b