From d1b2a11d7ad1e0f915213cad1fe78f22a285dc63 Mon Sep 17 00:00:00 2001 From: Malcolm Priestley Date: Thu, 27 Feb 2014 23:06:15 +0000 Subject: [PATCH] staging: vt6656: s_nsBulkOutIoCompleteWrite Remove variable status. Remove status and move debug messages to switch statement. Remove duplicate second debug message. Signed-off-by: Malcolm Priestley Signed-off-by: Greg Kroah-Hartman --- drivers/staging/vt6656/usbpipe.c | 24 +++++------------------- 1 file changed, 5 insertions(+), 19 deletions(-) diff --git a/drivers/staging/vt6656/usbpipe.c b/drivers/staging/vt6656/usbpipe.c index 57efb7f3587d..a4a4bf24aad6 100644 --- a/drivers/staging/vt6656/usbpipe.c +++ b/drivers/staging/vt6656/usbpipe.c @@ -584,38 +584,29 @@ static void s_nsBulkOutIoCompleteWrite(struct urb *urb) struct vnt_usb_send_context *context = urb->context; struct vnt_private *priv = context->pDevice; u8 context_type = context->type; - int status; DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"---->s_nsBulkOutIoCompleteWrite\n"); switch (urb->status) { case 0: - case -ETIMEDOUT: + DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO + "Write %d bytes\n", context->uBufLen); break; case -ECONNRESET: case -ENOENT: case -ESHUTDOWN: context->bBoolInUse = false; return; + case -ETIMEDOUT: default: + DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO + "BULK Out failed %d\n", urb->status); break; } if (!netif_device_present(priv->dev)) return; - - status = urb->status; - - if (status == STATUS_SUCCESS) { - DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO - "Write %d bytes\n", context->uBufLen); - } else { - DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO - "BULK Out failed %d\n", status); - } - - if (CONTEXT_DATA_PACKET == context_type) { if (context->pPacket != NULL) { dev_kfree_skb_irq(context->pPacket); @@ -625,11 +616,6 @@ static void s_nsBulkOutIoCompleteWrite(struct urb *urb) } priv->dev->trans_start = jiffies; - - if (status != STATUS_SUCCESS) { - DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO - "Send USB error! [%08xh]\n", status); - } } if (priv->bLinkPass == true) { -- 2.34.1