Messages in this thread Patch in this message | | | From | Atul Kumar Pant <> | Subject | [PATCH v1] kernel: usb: Adds space, removes trailing whitespace and fixes pointer declaration. | Date | Wed, 16 Aug 2023 02:18:41 +0530 |
| |
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@gmail.com> --- 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
| |