hasdata does not need to be initialized to zero. It will be assigned a value in the following judgment conditions. Remove the repeated ';' from code. Signed-off-by: Li kunyu <kunyu@xxxxxxxxxxxx> --- drivers/net/usb/catc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/usb/catc.c b/drivers/net/usb/catc.c index e7fe9c0f63a9..268c32521691 100644 --- a/drivers/net/usb/catc.c +++ b/drivers/net/usb/catc.c @@ -280,7 +280,7 @@ static void catc_irq_done(struct urb *urb) struct catc *catc = urb->context; u8 *data = urb->transfer_buffer; int status = urb->status; - unsigned int hasdata = 0, linksts = LinkNoChange; + unsigned int hasdata, linksts = LinkNoChange; int res; if (!catc->is_f5u011) { @@ -335,7 +335,7 @@ static void catc_irq_done(struct urb *urb) } } resubmit: - res = usb_submit_urb (urb, GFP_ATOMIC); + res = usb_submit_urb(urb, GFP_ATOMIC); if (res) dev_err(&catc->usbdev->dev, "can't resubmit intr, %s-%s, status %d\n", @@ -781,7 +781,7 @@ static int catc_probe(struct usb_interface *intf, const struct usb_device_id *id intf->altsetting->desc.bInterfaceNumber, 1)) { dev_err(dev, "Can't set altsetting 1.\n"); ret = -EIO; - goto fail_mem;; + goto fail_mem; } netdev = alloc_etherdev(sizeof(struct catc)); -- 2.18.2