From d620c46a93dcd06344722ff5cd9df0761a513980 Mon Sep 17 00:00:00 2001 From: Ben Hutchings Date: Sat, 27 Oct 2018 06:08:51 +0100 Subject: [PATCH] staging: vt6656: Fix misleading indentation Fix the compiler warnings: drivers/staging/vt6656/dpc.c:712:5: warning: this 'if' clause does not guard... drivers/staging/vt6656/main_usb.c:1101:7: warning: this 'if' clause does not guard... by reducing indentation of the following statements in RXbBulkInProcessData() and reformatting the kstrstr() function to kernel coding style. Both functions have been removed in a later version, so there is no corresponding upstream commit. Signed-off-by: Ben Hutchings --- drivers/staging/vt6656/dpc.c | 4 ++-- drivers/staging/vt6656/main_usb.c | 21 +++++++++++---------- 2 files changed, 13 insertions(+), 12 deletions(-) diff --git a/drivers/staging/vt6656/dpc.c b/drivers/staging/vt6656/dpc.c index c0ec5b37aa7c..1cb8fc3e2e0d 100644 --- a/drivers/staging/vt6656/dpc.c +++ b/drivers/staging/vt6656/dpc.c @@ -712,8 +712,8 @@ int RXbBulkInProcessData(struct vnt_private *pDevice, struct vnt_rcb *pRCB, if (FrameSize < 12) return false; - skb->data += cbHeaderOffset; - skb->tail += cbHeaderOffset; + skb->data += cbHeaderOffset; + skb->tail += cbHeaderOffset; skb_put(skb, FrameSize); skb->protocol=eth_type_trans(skb, skb->dev); skb->ip_summed=CHECKSUM_NONE; diff --git a/drivers/staging/vt6656/main_usb.c b/drivers/staging/vt6656/main_usb.c index e18071f121c5..fe57114b5d23 100644 --- a/drivers/staging/vt6656/main_usb.c +++ b/drivers/staging/vt6656/main_usb.c @@ -1092,17 +1092,18 @@ static int device_xmit(struct sk_buff *skb, struct net_device *dev) /* find out the start position of str2 from str1 */ static unsigned char *kstrstr(const unsigned char *str1, - const unsigned char *str2) { - int str1_len = strlen(str1); - int str2_len = strlen(str2); + const unsigned char *str2) +{ + int str1_len = strlen(str1); + int str2_len = strlen(str2); - while (str1_len >= str2_len) { - str1_len--; - if(memcmp(str1,str2,str2_len)==0) - return (unsigned char *) str1; - str1++; - } - return NULL; + while (str1_len >= str2_len) { + str1_len--; + if (memcmp(str1, str2, str2_len) == 0) + return (unsigned char *)str1; + str1++; + } + return NULL; } static int Config_FileGetParameter(unsigned char *string,