diff mbox series

[4/4] usb: dwc3: gadget: remove unnecessary dev_info()

Message ID 20180820103001.6453-4-felipe.balbi@linux.intel.com (mailing list archive)
State New, archived
Headers show
Series [1/4] usb: dwc3: gadget: return errors from __dwc3_gadget_start_isoc() | expand

Commit Message

Felipe Balbi Aug. 20, 2018, 10:30 a.m. UTC
Running out of requests on isochronous endpoints is part of normal
operation. We don't really need to know about it every time it
happens.

Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
---
 drivers/usb/dwc3/gadget.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Laurent Pinchart Aug. 20, 2018, 11:26 a.m. UTC | #1
Hi Felipe,

Thank you for the patch.

On Monday, 20 August 2018 13:30:01 EEST Felipe Balbi wrote:
> Running out of requests on isochronous endpoints is part of normal
> operation. We don't really need to know about it every time it
> happens.
> 
> Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  drivers/usb/dwc3/gadget.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
> index 0bac9b02f28b..8d3e0f2cde8b 100644
> --- a/drivers/usb/dwc3/gadget.c
> +++ b/drivers/usb/dwc3/gadget.c
> @@ -1261,8 +1261,6 @@ static int __dwc3_gadget_start_isoc(struct dwc3_ep
> *dep) u16 frame_number;
> 
>  	if (list_empty(&dep->pending_list)) {
> -		dev_info(dep->dwc->dev, "%s: ran out of requests\n",
> -				dep->name);
>  		dep->flags |= DWC3_EP_PENDING_REQUEST;
>  		return -EAGAIN;
>  	}
diff mbox series

Patch

diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
index 0bac9b02f28b..8d3e0f2cde8b 100644
--- a/drivers/usb/dwc3/gadget.c
+++ b/drivers/usb/dwc3/gadget.c
@@ -1261,8 +1261,6 @@  static int __dwc3_gadget_start_isoc(struct dwc3_ep *dep)
 	u16 frame_number;
 
 	if (list_empty(&dep->pending_list)) {
-		dev_info(dep->dwc->dev, "%s: ran out of requests\n",
-				dep->name);
 		dep->flags |= DWC3_EP_PENDING_REQUEST;
 		return -EAGAIN;
 	}