* [PATCH] drivers/usb/core: devio.c: Removed various errors and warnings generated by checkpatch.pl
@ 2015-04-11 7:53 Chase Metzger
2015-04-11 8:52 ` Hans de Goede
2015-04-11 17:20 ` Joe Perches
0 siblings, 2 replies; 4+ messages in thread
From: Chase Metzger @ 2015-04-11 7:53 UTC (permalink / raw)
To: gregkh; +Cc: stern, hdegoede, oneukum, linux-usb, linux-kernel, Chase Metzger
Fixed several errors and warnings.
Lines 29, 103, 1319, 1906 and 2408: removed unnecessary spaces or added appropriate spaces.
Lines 1040 and 1591: changed dev_printk(KERN_DEBUG, ...) to dev_dbg(...).
Signed-off-by: Chase Metzger <chasemetzger15@gmail.com>
---
drivers/usb/core/devio.c | 15 +++++++--------
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c
index 1163553..62c0be6 100644
--- a/drivers/usb/core/devio.c
+++ b/drivers/usb/core/devio.c
@@ -29,7 +29,7 @@
* 22.12.1999 0.1 Initial release (split from proc_usb.c)
* 04.01.2000 0.2 Turned into its own filesystem
* 30.09.2005 0.3 Fix user-triggerable oops in async URB delivery
- * (CAN-2005-3055)
+ * (CAN-2005-3055)
*/
/*****************************************************************************/
@@ -103,7 +103,7 @@ MODULE_PARM_DESC(usbfs_snoop, "true to log all usbfs traffic");
#define snoop(dev, format, arg...) \
do { \
if (usbfs_snoop) \
- dev_info(dev , format , ## arg); \
+ dev_info(dev, format, ## arg); \
} while (0)
enum snoop_when {
@@ -1040,7 +1040,7 @@ static int proc_control(struct usb_dev_state *ps, void __user *arg)
snoop_urb(dev, NULL, pipe, max(i, 0), min(i, 0), COMPLETE, NULL, 0);
}
if (i < 0 && i != -EPIPE) {
- dev_printk(KERN_DEBUG, &dev->dev, "usbfs: USBDEVFS_CONTROL "
+ dev_dbg(&dev->dev, "usbfs: USBDEVFS_CONTROL "
"failed cmd %s rqt %u rq %u len %u ret %d\n",
current->comm, ctrl.bRequestType, ctrl.bRequest,
ctrl.wLength, i);
@@ -1319,7 +1319,7 @@ static int proc_do_submiturb(struct usb_dev_state *ps, struct usbdevfs_urb *uurb
is_in = (uurb->endpoint & USB_ENDPOINT_DIR_MASK) != 0;
u = 0;
- switch(uurb->type) {
+ switch (uurb->type) {
case USBDEVFS_URB_TYPE_CONTROL:
if (!usb_endpoint_xfer_control(&ep->desc))
return -EINVAL;
@@ -1591,8 +1591,7 @@ static int proc_do_submiturb(struct usb_dev_state *ps, struct usbdevfs_urb *uurb
}
if (ret) {
- dev_printk(KERN_DEBUG, &ps->dev->dev,
- "usbfs: usb_submit_urb returned %d\n", ret);
+ dev_dbg(&ps->dev->dev, "usbfs: usb_submit_urb returned %d\n", ret);
snoop_urb(ps->dev, as->userurb, as->urb->pipe,
0, ret, COMPLETE, NULL, 0);
async_removepending(as);
@@ -1906,7 +1905,7 @@ static int proc_releaseinterface(struct usb_dev_state *ps, void __user *arg)
return -EFAULT;
if ((ret = releaseintf(ps, ifnum)) < 0)
return ret;
- destroy_async_on_interface (ps, ifnum);
+ destroy_async_on_interface(ps, ifnum);
return 0;
}
@@ -2408,7 +2407,7 @@ static int usbdev_notify(struct notifier_block *self,
}
static struct notifier_block usbdev_nb = {
- .notifier_call = usbdev_notify,
+ .notifier_call = usbdev_notify,
};
static struct cdev usb_device_cdev;
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] drivers/usb/core: devio.c: Removed various errors and warnings generated by checkpatch.pl
2015-04-11 7:53 [PATCH] drivers/usb/core: devio.c: Removed various errors and warnings generated by checkpatch.pl Chase Metzger
@ 2015-04-11 8:52 ` Hans de Goede
2015-04-11 17:20 ` Joe Perches
1 sibling, 0 replies; 4+ messages in thread
From: Hans de Goede @ 2015-04-11 8:52 UTC (permalink / raw)
To: Chase Metzger, gregkh; +Cc: stern, oneukum, linux-usb, linux-kernel
Hi,
On 04/11/2015 09:53 AM, Chase Metzger wrote:
> Fixed several errors and warnings.
>
> Lines 29, 103, 1319, 1906 and 2408: removed unnecessary spaces or added appropriate spaces.
> Lines 1040 and 1591: changed dev_printk(KERN_DEBUG, ...) to dev_dbg(...).
>
> Signed-off-by: Chase Metzger <chasemetzger15@gmail.com>
Looks good to me:
Acked-by: Hans de Goede <hdegoede@redhat.com>
Regards,
Hans
> ---
> drivers/usb/core/devio.c | 15 +++++++--------
> 1 file changed, 7 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c
> index 1163553..62c0be6 100644
> --- a/drivers/usb/core/devio.c
> +++ b/drivers/usb/core/devio.c
> @@ -29,7 +29,7 @@
> * 22.12.1999 0.1 Initial release (split from proc_usb.c)
> * 04.01.2000 0.2 Turned into its own filesystem
> * 30.09.2005 0.3 Fix user-triggerable oops in async URB delivery
> - * (CAN-2005-3055)
> + * (CAN-2005-3055)
> */
>
> /*****************************************************************************/
> @@ -103,7 +103,7 @@ MODULE_PARM_DESC(usbfs_snoop, "true to log all usbfs traffic");
> #define snoop(dev, format, arg...) \
> do { \
> if (usbfs_snoop) \
> - dev_info(dev , format , ## arg); \
> + dev_info(dev, format, ## arg); \
> } while (0)
>
> enum snoop_when {
> @@ -1040,7 +1040,7 @@ static int proc_control(struct usb_dev_state *ps, void __user *arg)
> snoop_urb(dev, NULL, pipe, max(i, 0), min(i, 0), COMPLETE, NULL, 0);
> }
> if (i < 0 && i != -EPIPE) {
> - dev_printk(KERN_DEBUG, &dev->dev, "usbfs: USBDEVFS_CONTROL "
> + dev_dbg(&dev->dev, "usbfs: USBDEVFS_CONTROL "
> "failed cmd %s rqt %u rq %u len %u ret %d\n",
> current->comm, ctrl.bRequestType, ctrl.bRequest,
> ctrl.wLength, i);
> @@ -1319,7 +1319,7 @@ static int proc_do_submiturb(struct usb_dev_state *ps, struct usbdevfs_urb *uurb
> is_in = (uurb->endpoint & USB_ENDPOINT_DIR_MASK) != 0;
>
> u = 0;
> - switch(uurb->type) {
> + switch (uurb->type) {
> case USBDEVFS_URB_TYPE_CONTROL:
> if (!usb_endpoint_xfer_control(&ep->desc))
> return -EINVAL;
> @@ -1591,8 +1591,7 @@ static int proc_do_submiturb(struct usb_dev_state *ps, struct usbdevfs_urb *uurb
> }
>
> if (ret) {
> - dev_printk(KERN_DEBUG, &ps->dev->dev,
> - "usbfs: usb_submit_urb returned %d\n", ret);
> + dev_dbg(&ps->dev->dev, "usbfs: usb_submit_urb returned %d\n", ret);
> snoop_urb(ps->dev, as->userurb, as->urb->pipe,
> 0, ret, COMPLETE, NULL, 0);
> async_removepending(as);
> @@ -1906,7 +1905,7 @@ static int proc_releaseinterface(struct usb_dev_state *ps, void __user *arg)
> return -EFAULT;
> if ((ret = releaseintf(ps, ifnum)) < 0)
> return ret;
> - destroy_async_on_interface (ps, ifnum);
> + destroy_async_on_interface(ps, ifnum);
> return 0;
> }
>
> @@ -2408,7 +2407,7 @@ static int usbdev_notify(struct notifier_block *self,
> }
>
> static struct notifier_block usbdev_nb = {
> - .notifier_call = usbdev_notify,
> + .notifier_call = usbdev_notify,
> };
>
> static struct cdev usb_device_cdev;
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drivers/usb/core: devio.c: Removed various errors and warnings generated by checkpatch.pl
2015-04-11 7:53 [PATCH] drivers/usb/core: devio.c: Removed various errors and warnings generated by checkpatch.pl Chase Metzger
2015-04-11 8:52 ` Hans de Goede
@ 2015-04-11 17:20 ` Joe Perches
2015-04-28 10:56 ` Greg KH
1 sibling, 1 reply; 4+ messages in thread
From: Joe Perches @ 2015-04-11 17:20 UTC (permalink / raw)
To: Chase Metzger; +Cc: gregkh, stern, hdegoede, oneukum, linux-usb, linux-kernel
On Sat, 2015-04-11 at 00:53 -0700, Chase Metzger wrote:
> Fixed several errors and warnings.
[]
> Lines 1040 and 1591: changed dev_printk(KERN_DEBUG, ...) to dev_dbg(...).
These changes hav the possibly undesired effect of
removing these messages completely when DEBUG is not
#defined or when CONFIG_DYNAMIC_DEBUG is not enabled
and these messages are not specifically enabled.
> diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c
[]
> @@ -1040,7 +1040,7 @@ static int proc_control(struct usb_dev_state *ps, void __user *arg)
> snoop_urb(dev, NULL, pipe, max(i, 0), min(i, 0), COMPLETE, NULL, 0);
> }
> if (i < 0 && i != -EPIPE) {
> - dev_printk(KERN_DEBUG, &dev->dev, "usbfs: USBDEVFS_CONTROL "
> + dev_dbg(&dev->dev, "usbfs: USBDEVFS_CONTROL "
> "failed cmd %s rqt %u rq %u len %u ret %d\n",
> current->comm, ctrl.bRequestType, ctrl.bRequest,
> ctrl.wLength, i);
[]
> @@ -1591,8 +1591,7 @@ static int proc_do_submiturb(struct usb_dev_state *ps, struct usbdevfs_urb *uurb
> }
>
> if (ret) {
> - dev_printk(KERN_DEBUG, &ps->dev->dev,
> - "usbfs: usb_submit_urb returned %d\n", ret);
> + dev_dbg(&ps->dev->dev, "usbfs: usb_submit_urb returned %d\n", ret);
> snoop_urb(ps->dev, as->userurb, as->urb->pipe,
> 0, ret, COMPLETE, NULL, 0);
> async_removepending(as);
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drivers/usb/core: devio.c: Removed various errors and warnings generated by checkpatch.pl
2015-04-11 17:20 ` Joe Perches
@ 2015-04-28 10:56 ` Greg KH
0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2015-04-28 10:56 UTC (permalink / raw)
To: Joe Perches
Cc: Chase Metzger, stern, hdegoede, oneukum, linux-usb, linux-kernel
On Sat, Apr 11, 2015 at 10:20:38AM -0700, Joe Perches wrote:
> On Sat, 2015-04-11 at 00:53 -0700, Chase Metzger wrote:
> > Fixed several errors and warnings.
> []
> > Lines 1040 and 1591: changed dev_printk(KERN_DEBUG, ...) to dev_dbg(...).
>
> These changes hav the possibly undesired effect of
> removing these messages completely when DEBUG is not
> #defined or when CONFIG_DYNAMIC_DEBUG is not enabled
> and these messages are not specifically enabled.
That's ok, we can live with that :)
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-04-28 10:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-11 7:53 [PATCH] drivers/usb/core: devio.c: Removed various errors and warnings generated by checkpatch.pl Chase Metzger
2015-04-11 8:52 ` Hans de Goede
2015-04-11 17:20 ` Joe Perches
2015-04-28 10:56 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).