nd added comments
to it.
Module builded successfully with sparse without warnings.
Signed-off-by: Surendra Patil
---
drivers/staging/ozwpan/ozpd.c| 28 ++--
drivers/staging/ozwpan/ozpd.h| 5 +
drivers/staging/ozwpan/ozproto.c | 15 +--
drivers
d-off-by: Surendra Patil
---
drivers/staging/ozwpan/ozpd.c| 29 +++--
drivers/staging/ozwpan/ozproto.c | 9 -
drivers/staging/ozwpan/ozproto.h | 2 --
3 files changed, 15 insertions(+), 25 deletions(-)
diff --git a/drivers/staging/ozwpan/ozpd.c b/drivers
_releases().
Signed-off-by: Surendra Patil
---
drivers/staging/ozwpan/ozproto.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/ozwpan/ozproto.c b/drivers/staging/ozwpan/ozproto.c
index e7138ed..b47cec1 100644
--- a/drivers/staging/ozwpan/ozproto.c
+++ b/drivers/stag
checkpatch displays below errors for bypasslib/bp_ioctl.h file
ERROR: Macros with complex values should be enclosed in parenthesis
Hence added parenthesis for macros with complex values.
Signed-off-by: Surendra Patil
---
drivers/staging/silicom/bypasslib/bp_ioctl.h | 4 ++--
1 file
else so made them static.
Signed-off-by: Surendra Patil
---
drivers/staging/silicom/bp_mod.h| 2 --
drivers/staging/silicom/bpctl_mod.c | 20 ++--
2 files changed, 10 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/silicom/bp_mod.h b/drivers/staging/silicom
: externs should be avoided in .c files
bpctl_mod.c:125: WARNING: externs should be avoided in .c files
bp_ioctl.h:54: ERROR: Macros with complex values should be enclosed in
parenthesis
bp_ioctl.h:159: ERROR: Macros with complex values should be enclosed in
parenthesis
Signed-off-by: Surendra Patil
Removed bp_proc_create() declaration from bp_mod.h,
already declared static in bpctl_mod.c in first patch
Signed-off-by: Surendra Patil
---
drivers/staging/silicom/bp_mod.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/silicom/bp_mod.h b/drivers/staging/silicom/bp_mod.h
: WARNING: Unnecessary parentheses - maybe == should be = ?
Signed-off-by: Surendra Patil
---
drivers/staging/silicom/bpctl_mod.c | 28 ++--
drivers/staging/silicom/bypasslib/bp_ioctl.h | 4 ++--
2 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/drivers
Fixed few coding style errors in header files
base.h:58: WARNING: please, no space before tabs
base.h:81: ERROR: "(foo*)" should be "(foo *)"
pci.h:338: ERROR: "foo * bar" should be "foo *bar"
pci.h:151: ERROR: do not use C99 // comments
Signed-off-
tabs where possible
wifi.h:2303: ERROR: space prohibited after that open parenthesis '('
wifi.h:2408: ERROR: spaces required around that '=' (ctx:VxV)
Signed-off-by: Surendra Patil
---
drivers/staging/rtl8821ae/wifi.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
tabs where possible
wifi.h:2303: ERROR: space prohibited after that open parenthesis '('
wifi.h:2408: ERROR: spaces required around that '=' (ctx:VxV)
Signed-off-by: Surendra Patil
---
drivers/staging/rtl8821ae/wifi.h | 450 ---
1 file c
and will let it be because it doesn't seem to be an
issue in practice, but please fix it. The obvious fix would seem to
change the size of "2" to be "MAX_RF_PATH", but I'll abstain from
doing those kinds of changes in the merge when it doesn't seem to
affect the buil
Changed array size from "bb_swing_idx_ofdm[2]" to
"bb_swing_idx_ofdm[MAX_RF_PATH]"
Reported-By: Linus Torvalds
Signed-off-by: Surendra Patil
---
drivers/staging/rtl8821ae/wifi.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl882
Changed array size from "bb_swing_idx_ofdm[2]" to
"bb_swing_idx_ofdm[MAX_RF_PATH]"
as discussed on thread - https://lkml.org/lkml/2014/2/1/75
Signed-off-by: Surendra Patil
---
drivers/staging/rtl8821ae/wifi.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --g
Fixed Below coding style errors -
octeon-hcd.h:146: ERROR: spaces prohibited around that ':' (ctx:WxW)
octeon-hcd.h:147: ERROR: spaces prohibited around that ':' (ctx:WxW)
total: 243 errors, 0 warnings, 1819 lines checked - fixed all errors
Signed-off-by: Surendra Patil
hcd.c:3228: ERROR: return is not a function, parentheses are not required
Signed-off-by: Surendra Patil
---
drivers/staging/octeon-usb/octeon-hcd.c | 31 +++
1 file changed, 15 insertions(+), 16 deletions(-)
diff --git a/drivers/staging/octeon-usb/octeon-hcd.c
b/drive
Fixed a coding style issue,removed space after the function pointer name
Signed-off-by: Surendra Patil
---
drivers/staging/comedi/comedidev.h | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/drivers/staging/comedi/comedidev.h
b/drivers
17 matches
Mail list logo