[PATCH 12/16] staging: pi433: rewrite comparison with NULL

2017-12-10 Thread Valentin Vidic
Fixes checkpatch warning for comparing value with NULL. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c

[PATCH 11/16] staging: pi433: remove multiple blank lines

2017-12-10 Thread Valentin Vidic
Fixes checkpatch warning for multiple blank lines in source. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 11 --- drivers/staging/pi433/pi433_if.h | 8 drivers/staging/pi433/rf69.c | 2 -- drivers/s

[PATCH 04/16] staging: pi433: remove space inside parenthesis

2017-12-10 Thread Valentin Vidic
Fixes checkpatch error for prohibited spaces. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 34 +- drivers/staging/pi433/rf69.c | 4 ++-- drivers/staging/pi433/rf69.h | 4 ++-- 3 files chang

[PATCH 14/16] staging: pi433: add blank line after function

2017-12-10 Thread Valentin Vidic
Fixes checkpatch warning for blank line after function/struct/union/enum declarations. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging

[PATCH 13/16] staging: pi433: fix spelling errors in documentation

2017-12-10 Thread Valentin Vidic
Fixes checkpatch warning for 'transfered' and 'devided'. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/Documentation/pi433.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/pi433/Documentation/pi433.txt b/d

[PATCH 01/16] staging: pi433: add space before open parenthesis

2017-12-10 Thread Valentin Vidic
Fixes checkpatch error for missing spaces. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index 32db3b

[PATCH 06/16] staging: pi433: use tabs for code indent

2017-12-10 Thread Valentin Vidic
Fixes checkpatch error for whitespace idents. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- .../Documentation/devicetree/pi433-overlay.dts | 32 +++--- drivers/staging/pi433/pi433_if.c | 8 +++--- 2 files changed, 20 insertions(

[PATCH 16/16] staging: pi433: remove comparison with true

2017-12-10 Thread Valentin Vidic
Fixes checkpatch warning for error prone comparison. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c

[PATCH 07/16] staging: pi433: move trailing statements to next line

2017-12-10 Thread Valentin Vidic
Fixes checkpatch error for trailing if statements. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 45 +++- 1 file changed, 31 insertions(+), 14 deletions(-) diff --git a/drivers/staging/pi433/pi433_

[PATCH 15/16] staging: pi433: remove comparison with false

2017-12-10 Thread Valentin Vidic
Fixes checkpatch warning for error prone comparison. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c

[PATCH 10/16] staging: pi433: remove return from a void function

2017-12-10 Thread Valentin Vidic
Fixes checkpatch warning for void function return statements. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index 1e50d8

[PATCH 03/16] staging: pi433: add space around operators

2017-12-10 Thread Valentin Vidic
Fixes checkpatch error for missing spaces. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c

[PATCH 09/16] staging: pi433: remove spaces before newline

2017-12-10 Thread Valentin Vidic
Fixes checkpatch warning for whitespace before a quoted newline. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/rf69.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c

[PATCH 05/16] staging: pi433: use consistent brace style

2017-12-10 Thread Valentin Vidic
Fixes checkpatch error for open/close braces. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/pi433/pi433_if.c | 129 +-- 1 file changed, 41 insertions(+), 88 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/d

[PATCH] staging: media/atomisp: drop unused qos variable

2017-05-07 Thread Valentin Vidic
Fixes a sparse warning: drivers/staging/media/atomisp/platform/intel-mid/intel_mid_pcihelpers.c:35:5: warning: symbol 'qos' was not declared. Should it be static? Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/media/atomisp/platform/int

[PATCH v2] staging: lustre: cleanup le32 assignment to ldp_flags

2017-05-04 Thread Valentin Vidic
:right side has type int Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- Changes in v2: use a local var to collect flags drivers/staging/lustre/lustre/lmv/lmv_obd.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/lustre/lust

[PATCH] staging: lustre: fix sparse warnings on assignment

2017-04-29 Thread Valentin Vidic
conversions on flag assignment. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/lustre/lustre/lmv/lmv_obd.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/lustre/lustre/lmv/lmv_obd.c b/drivers/staging/lustre/lust

[PATCH v2] staging: lustre: remove void function return

2017-04-26 Thread Valentin Vidic
All statements removed from the end of void functions as reported by checkpatch. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- Changes in v2: updated recipient list and subject drivers/staging/lustre/lustre/lmv/lmv_obd.c | 1 - drivers/staging/lustre/lustre/lmv/lproc_lmv

[PATCH] lustre: remove useless return statements

2017-04-25 Thread Valentin Vidic
All statements removed from the end of void functions as reported by checkpatch. Signed-off-by: Valentin Vidic <valentin.vi...@carnet.hr> --- drivers/staging/lustre/lustre/lmv/lmv_obd.c | 1 - drivers/staging/lustre/lustre/lmv/lproc_lmv.c | 1 - drivers/staging/lustre/lustre/lov/lov

<    1   2