[PATCH 2/3] staging: lustre: Add blank line after variable declaration

2015-08-11 Thread Swee Hua Law
Add blank line after variable declaration Signed-off-by: Swee Hua Law --- drivers/staging/lustre/lustre/llite/lloop.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/lustre/lustre/llite/lloop.c b/drivers/staging/lustre/lustre/llite/lloop.c index b643f11..de5eaa0 100644

[PATCH 1/3] staging: lustre: Do not init global to NULL

2015-08-11 Thread Swee Hua Law
Remove "= NULL" in global variable Signed-off-by: Swee Hua Law --- drivers/staging/lustre/lustre/llite/lloop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/lloop.c b/drivers/staging/lustre/lustre/llite/lloop.c index d61423

[PATCH 3/3] staging: lustre: Remove the space before \n

2015-08-11 Thread Swee Hua Law
Remove the extra space character right before \n in the string Signed-off-by: Swee Hua Law --- drivers/staging/lustre/lustre/llite/lloop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/lloop.c b/drivers/staging/lustre/lustre/llite

[PATCH 0/3] staging: lustre: fix checkpatch problems in llite/lloop.c

2015-08-11 Thread Swee Hua Law
Fix 3 checkpatch problems in drivers/staging/lustre/lustre/llite/lloop.c [PATCH 1/3] staging: lustre: Do not init global to NULL [PATCH 2/3] staging: lustre: Add blank line after variable declaration [PATCH 3/3] staging: lustre: Remove the space before \n ___

[PATCH V2 4/4] staging: lustre: checkpatch: argument alignment for readability

2015-08-10 Thread Swee Hua Law
Fix checkpatch problem: move last argument of the hlist..() back to same line Signed-off-by: Swee Hua Law --- drivers/staging/lustre/lustre/llite/remote_perm.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/remote_perm.c b/drivers

[PATCH V2 3/4] staging: lustre: checkpatch: move */ block comment to next line

2015-08-10 Thread Swee Hua Law
Fix checkpatch problem: move */ from end of line to new line Signed-off-by: Swee Hua Law --- drivers/staging/lustre/lustre/llite/remote_perm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/remote_perm.c b/drivers/staging/lustre/lustre

[PATCH V2 2/4] staging: lustre: checkpatch: symbol == NULL should be !symbol

2015-08-10 Thread Swee Hua Law
Fix checkpatch problem: change == NULL comparison to !symbol Signed-off-by: Swee Hua Law --- drivers/staging/lustre/lustre/llite/remote_perm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/remote_perm.c b/drivers/staging/lustre

[PATCH V2 1/4] staging: lustre: checkpatch: do not init global to NULL

2015-08-10 Thread Swee Hua Law
Fix checkpatch problem: remove NULL assignment fro global variable Signed-off-by: Swee Hua Law --- drivers/staging/lustre/lustre/llite/remote_perm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/remote_perm.c b/drivers/staging

[PATCH V2 0/4] staging: lustre: split checkpatch fix for remote_perm.c into series

2015-08-10 Thread Swee Hua Law
Split patch into series. Fix various coding style issue in remote_perm.c: [PATCH V2 1/4] staging: lustre: checkpatch: do not init global to NULL [PATCH V2 2/4] staging: lustre: checkpatch: symbol == NULL should be !symbol [PATCH V2 3/4] staging: lustre: checkpatch: move */ block comment to next li

[PATCH] Add space after ,

2015-08-09 Thread Swee Hua Law
add space after , to fix coding style issue Signed-off-by: Swee Hua Law --- drivers/staging/rtl8723au/core/rtw_sreset.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723au/core/rtw_sreset.c b/drivers/staging/rtl8723au/core/rtw_sreset.c index 29a29d9

[PATCH] Fix various coding style problem

2015-08-09 Thread Swee Hua Law
1) do not initialise globals to NULL 2) improve readability of hlist_for_each_entry_safe() 3) move */ to a separate line 4) change symbol == NULL to !symbol Signed-off-by: Swee Hua Law --- drivers/staging/lustre/lustre/llite/remote_perm.c | 17 - 1 file changed, 8 insertions