[PATCH] staging: rtl8188eu: remove return at end of void function call

2016-03-18 Thread Nicholas Sim
Remove unnecessary return statement from last line of void function call Signed-off-by: Nicholas Sim --- drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c b/drivers/staging/rtl8188eu/core

[PATCH] staging: rtl8188eu: remove return at end of void function call

2016-03-23 Thread Nicholas Sim
Remove unnecessary return statements from last lines of void function call (several) Signed-off-by: Nicholas Sim --- drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 17 - 1 file changed, 17 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c b/drivers

[PATCH] staging: xgifb: ensure braces on all arms of if stmt

2016-03-23 Thread Nicholas Sim
Added braces on else arm of if statement where if arm already has braces as suggested for clarity in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/xgifb/vb_setmode.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/xgifb

[PATCH] staging: xgifb: remove extra braces from if stmt (single branch)

2016-03-23 Thread Nicholas Sim
Remove braces from one branch of if statement where both branches only have a single line of code, as suggested in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/xgifb/vb_setmode.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH] staging: rtl8192u: remove blank lines before braces (closing)

2016-04-05 Thread Nicholas Sim
Remove unneeded blank lines occuring before closing braces Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 35 -- 1 file changed, 35 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u

[PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-05 Thread Nicholas Sim
Remove unneeded blank lines appearing after opening braces as suggested by checkpatch.pl Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u

[PATCH] staging: rtl8192u: add blank line after declarations

2016-04-05 Thread Nicholas Sim
Add a blank line after function/struct/union/enum declarations for readability, as suggested in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/staging/rtl8192u

[PATCH] staging: rtl8192u: rewrite NULL comparison for pointers

2016-04-05 Thread Nicholas Sim
When testing pointers, it is not necessary to explicitly compare to NULL. Rewrite if condition as (!ptr) or (ptr) as suggested in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH] staging: rts5208: ensure braces on all arms of if stmt

2016-04-05 Thread Nicholas Sim
Added braces on if arm of if statement where else arm already needs braces as suggested for clarity in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/rts5208/ms.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rts5208/ms.c b

[PATCH] staging: wlan-ng: rewrite NULL comparison

2016-04-05 Thread Nicholas Sim
It is not necessary to compare explicitly to NULL. Rewrite if condition as (!dev) or (dev) as suggested in Documentation/CodingStyle Signed-off-by: Nicholas Sim --- drivers/staging/wlan-ng/hfa384x_usb.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging

[PATCH] staging: rts5208: ensure braces on all arms of if stmt

2016-04-05 Thread Nicholas Sim
Added braces on if arm of if statement where else arm already needs braces as suggested for clarity in Documentation/CodingStyle (several) Signed-off-by: Nicholas Sim --- drivers/staging/rts5208/sd.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers