From: Oliver Neukum Date: Mon, 15 Aug 2016 09:17:55 +0000 (+0200) Subject: cdc-wdm: cleanup debug messages X-Git-Tag: v4.14-rc1~2393^2~135 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=13a88bf5e0fa886394d7148b9eec1f58d14608b5;p=platform%2Fkernel%2Flinux-rpi.git cdc-wdm: cleanup debug messages Dynamic debugging will already add the function (and the line number) to a debug message if one requests that. It makes no sense to add them unconditionally in a driver. Signed-off-by: Oliver Neukum Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/usb/class/cdc-wdm.c b/drivers/usb/class/cdc-wdm.c index 56d3ab8..bf4bb58 100644 --- a/drivers/usb/class/cdc-wdm.c +++ b/drivers/usb/class/cdc-wdm.c @@ -314,8 +314,7 @@ static void wdm_int_callback(struct urb *urb) && !test_bit(WDM_DISCONNECTING, &desc->flags) && !test_bit(WDM_SUSPENDING, &desc->flags)) { rv = usb_submit_urb(desc->response, GFP_ATOMIC); - dev_dbg(&desc->intf->dev, "%s: usb_submit_urb %d", - __func__, rv); + dev_dbg(&desc->intf->dev, "submit response URB %d", rv); } spin_unlock(&desc->iuspin); if (rv < 0) { @@ -574,7 +573,7 @@ retry: } if (!desc->reslength) { /* zero length read */ - dev_dbg(&desc->intf->dev, "%s: zero length - clearing WDM_READ\n", __func__); + dev_dbg(&desc->intf->dev, "zero length - clearing WDM_READ"); clear_bit(WDM_READ, &desc->flags); rv = service_outstanding_interrupt(desc); spin_unlock_irq(&desc->iuspin); @@ -1081,7 +1080,7 @@ static void wdm_disconnect(struct usb_interface *intf) if (!desc->count) cleanup(desc); else - dev_dbg(&intf->dev, "%s: %d open files - postponing cleanup\n", __func__, desc->count); + dev_dbg(&intf->dev, "%d open files - postponing cleanup\n", desc->count); mutex_unlock(&wdm_mutex); }