Removed checkpatch.pl error
ERROR: Macros with complex values should be enclosed in parentheses
+#define MFIE_STRING(x) case MFIE_TYPE_ ##x: return #x
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8192e/rtllib_rx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
Removed checkpatch.pl error
ERROR: space prohibited before that ',' (ctx:WxE)
+433, 433, 477} } ,
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 4 ++--
drivers/staging/rtl8192e/rtllib_rx.c | 4 ++--
drivers/staging/rtl8192e/rtllib_sof
This patch removed indentation with a tab error
ERROR: code indent should use tabs where possible
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/hal/rf.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/hal/rf.c
b/drivers/staging
Removed checkpatch.pl below warning
WARNING: Unnecessary parentheses - maybe == should be = ?
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c
b
Removed checkpatch.pl error
ERROR: space prohibited before that ',' (ctx:WxW)
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c | 4 ++--
drivers/staging/rtl8188eu/hal/odm.c | 8
drivers/staging/rtl8188eu/hal/p
Removed checkpatch.pl error
ERROR: code indent should use tabs where possible
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/hal/rf.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/rf.c
b/drivers/staging/rtl8188eu/hal/rf.c
Removed checkpatch.pl below warning
WARNING: Missing a blank line after declarations
Signed-off-by: Pushpendra Singh
---
.../staging/rtl8188eu/hal/Hal8188ERateAdaptive.c | 1 +
drivers/staging/rtl8188eu/hal/hal_intf.c | 2 ++
drivers/staging/rtl8188eu/hal/odm.c| 3
Removed checkpatch.pl below warning
WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/hal/bb_cfg.c| 15 +++
drivers/staging/rtl8188eu/hal/fw.c| 3 +--
drivers/staging/rtl8188eu
Removed checkpatch.pl below warning
WARNING: please, no spaces at the start of a line
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c
b/drivers
remove below checkpatch.pl warning
WARNING: please, no space before tabs
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/core/rtw_xmit.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_xmit.c
b/drivers/staging
remove checckpatch.pl warning
WARNINGROR: switch and case should be at the same indent
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/core/rtw_xmit.c | 32 +--
1 files changed, 17 insertions(+), 17 deletions(-)
diff --git a/drivers/staging/rtl8188eu
remove checkpatch.pl warning
WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/core/rtw_efuse.c| 3 +--
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 14 +-
drivers/staging/rtl8188eu/core
Removed following checkpatch.pl warning from mutiple files
WARNING: Missing a blank line after declarations
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/core/rtw_debug.c | 3 ++-
drivers/staging/rtl8188eu/core/rtw_efuse.c | 12
drivers/staging/rtl8188eu
remove checkpatch.pl warning
WARNING: Unnecessary parentheses - maybe == should be = ?
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/core/rtw_ioctl_set.c | 4 ++--
drivers/staging/rtl8188eu/core/rtw_mlme.c | 2 +-
drivers/staging/rtl8188eu/core/rtw_pwrctrl.c | 2
Removed unnecessary space before that ','
ERROR: space prohibited before that ',' (ctx:WxE)
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/core/rtw_ieee80211.c | 8 +++---
drivers/staging/rtl8188eu/core/rtw_mlme.c | 18 ++--
drivers/st
remove checkpatch.pl warning
WARNING: please, no spaces at the start of a line
Signed-off-by: Pushpendra Singh
---
drivers/staging/rtl8188eu/core/rtw_led.c | 2 +-
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 2 +-
drivers/staging/rtl8188eu/core/rtw_recv.c | 6 +-
drivers
their is no statements for if condition, but else have so instead
of using else condition ,use a single condition statement if(!qtype)
Signed-off-by: Puhspendra Singh
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/driver
Remove checkpatch.pl warning
WARNING: Possible unnecessary 'out of memory' message
+ if (skb == NULL) {
+ pr_debug("No Network buffers available\n");
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-usb/ft1000_hw.c | 1 -
1 file changed,
Removed Checkpatch.pl warning
WARNING: braces {} are not necessary for single statement blocks
+ if (dev->app_info[i].fileobject == NULL) {
+ break;
+ }
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c
Remove checkpatch.pl warning
WARNING: braces {} are not necessary for any arm of this statement
+ if (tempword & 0x1f) {
[...]
+ } else {
[...]
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c
Remove checkpatch.pl warning
WARNING: Unnecessary parentheses - maybe == should be = ?
+ if ((dev->app_info[i].fileobject == NULL))
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
d
Removed Checkpatch.pl warning
WARNING: Missing a blank line after declarations
+ struct ft1000_info *ft_info;
+ ft_info = netdev_priv(dev);
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Remove checkpatch.pl warning
WARNING: Unnecessary parentheses
+ if ((tempword != *ppseudohdr)) {
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ft1000/ft1000
WARNING: Possible unnecessary 'out of memory' message
+ if (skb == NULL) {
+ pr_debug("No Network buffers available\n");
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-usb/ft1000_hw.c | 1 -
1 file changed, 1 deletion(-)
diff --g
WARNING: Possible unnecessary 'out of memory' message
+ if (skb == NULL) {
+ pr_debug("No Network buffers available\n");
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 1 -
1 file changed, 1 deletion(-)
diff --g
Remove checkpatch.pl warning
WARNING: Unnecessary parentheses - maybe == should be = ?
+ if ((dev->app_info[i].fileobject == NULL))
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
d
their is no statements for if condition, but else have so instead
of using else condition ,use a single condition statement if(!qtype)
Signed-off-by: Puhspendra Singh
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/driver
Removed Checkpatch.pl warning
WARNING: Missing a blank line after declarations
+ struct ft1000_info *ft_info;
+ ft_info = netdev_priv(dev);
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Removed checkpatch.pl warnings
WARNING: braces {} are not necessary for single statement blocks
+ if (tempword == 0xfefe) {
+ break;
+ }
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c
Removed Checkpatch.pl warning
WARNING: braces {} are not necessary for single statement blocks
+ if ((dev->app_info[i].fileobject == NULL)) {
+ break;
+ }
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c
WARNING: Unnecessary parentheses
+ if ((tempword != *ppseudohdr)) {
Signed-off-by: Pushpendra Singh
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c
b/drivers
31 matches
Mail list logo