This patch fixes following checkpatch.pl issue: ERROR: space required before the open parenthesis '(' ERROR: trailing whitespace ERROR: space required after that ',' ERROR: "foo * bar" should be "foo *bar" Signed-off-by: Atul Kumar Pant <atulpant.linux@xxxxxxxxx> --- drivers/usb/class/cdc-acm.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c index 11da5fb284d0..638c9cdaa041 100644 --- a/drivers/usb/class/cdc-acm.c +++ b/drivers/usb/class/cdc-acm.c @@ -203,7 +203,7 @@ static int acm_wb_is_avail(struct acm *acm) n = ACM_NW; spin_lock_irqsave(&acm->write_lock, flags); for (i = 0; i < ACM_NW; i++) - if(acm->wb[i].use) + if (acm->wb[i].use) n--; spin_unlock_irqrestore(&acm->write_lock, flags); return n; @@ -1006,7 +1006,7 @@ static int wait_serial_change(struct acm *acm, unsigned long arg) } } while (!rv); - + return rv; } @@ -1257,7 +1257,7 @@ static int acm_probe(struct usb_interface *intf, if (control_interface == data_interface) { /* some broken devices designed for windows work this way */ - dev_warn(&intf->dev,"Control and data interfaces are not separated!\n"); + dev_warn(&intf->dev, "Control and data interfaces are not separated!\n"); combined_interfaces = 1; /* a popular other OS doesn't use it */ quirks |= NO_CAP_LINE; @@ -1428,7 +1428,7 @@ static int acm_probe(struct usb_interface *intf, goto err_free_write_urbs; if (h.usb_cdc_country_functional_desc) { /* export the country data */ - struct usb_cdc_country_functional_desc * cfd = + struct usb_cdc_country_functional_desc *cfd = h.usb_cdc_country_functional_desc; acm->country_codes = kmalloc(cfd->bLength - 4, GFP_KERNEL); -- 2.25.1