[PATCH v1] kernel: usb: Adds space, removes trailing whitespace and fixes pointer declaration.

From: Atul Kumar Pant
Date: Tue Aug 15 2023 - 16:50:18 EST


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