[PATCH] staging: exfat: use BIT macro for defining sizes

Fixes checkpatch.pl warning: CHECK: Prefer using the BIT macro Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/exfat/exfat.h b/drivers/staging/exfat/exfat.h index bae180e10609

[PATCH] staging: exfat: cleanup blank line warnings

Fixes checkpatch.pl warnings: CHECK: Please don't use multiple blank lines CHECK: Blank lines aren't necessary after an open brace '{' CHECK: Please use a blank line after function/struct/union/enum declarations Signed-off-by: Valentin Vidic --- drivers/s

[PATCH] staging: exfat: cleanup braces for if/else statements

Fixes checkpatch.pl warnings: CHECK: Unbalanced braces around else statement CHECK: braces {} should be used on all arms of this statement Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat_core.c | 12 ++-- drivers/staging/exfat/exfat_super.c | 16 +--- 2

[PATCH] staging: exfat: fix spelling errors in comments

Fixes checkpatch.pl warnings: CHECK: 'consistancy' may be misspelled - perhaps 'consistency'? CHECK: 'stuct' may be misspelled - perhaps 'struct'? Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat_core.c | 2 +- drivers/staging/exfat/

[PATCH] staging: exfat: cleanup explicit comparisons to NULL

Fixes checkpatch.pl warnings: CHECK: Comparison to NULL could be written "expr" CHECK: Comparison to NULL could be written "!expr" Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat_core.c | 34 - drivers/staging/exfa

[PATCH] staging: exfat: drop local TRUE/FALSE defines

Replace with bool where it makes sense. Also drop unused local variable lossy in fat_find_dir_entry. Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat.h | 3 -- drivers/staging/exfat/exfat_core.c | 81 +++-- drivers/staging/exfat/exfat_nls.c | 2

[PATCH v2] staging: exfat: cleanup explicit comparisons to NULL

Fixes checkpatch.pl warnings: CHECK: Comparison to NULL could be written "expr" CHECK: Comparison to NULL could be written "!expr" Signed-off-by: Valentin Vidic --- v2: fix gcc warning in strsep call drivers/staging/exfat/exfat_core.c | 34 - d

[PATCH 2/2] staging: exfat: cleanup spacing for casts

Fixes checkpatch.pl warnings: CHECK: No space is necessary after a cast Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat_core.c | 88 ++--- drivers/staging/exfat/exfat_super.c | 66 +++--- 2 files changed, 77 insertions(+), 77 deletions

[PATCH 1/2] staging: exfat: cleanup spacing for operators

Fixes checkpatch.pl warnings: CHECK: spaces preferred around that '-' (ctx:VxV) CHECK: spaces preferred around that '+' (ctx:VxV) CHECK: spaces preferred around that '*' (ctx:VxV) CHECK: spaces preferred around that '|' (ctx:VxV) Signed-off-by: Va

[PATCH 2/2 v2] staging: exfat: cleanup casts

Use constants and fix checkpatch.pl warnings: CHECK: No space is necessary after a cast Signed-off-by: Valentin Vidic --- v2: Use constants where possible drivers/staging/exfat/exfat_core.c | 94 ++--- drivers/staging/exfat/exfat_super.c | 66 ++-- 2

[PATCH] staging: exfat: add millisecond support

Drop duplicated date_time_t struct and add millisecond handling for create and modify time. Also drop millisecond field for access time since it is not defined in the the spec. Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat.h | 38 +++--- drivers/staging/exfat/exfat_core.c

[PATCH v3 3/3] staging: exfat: use integer constants

Replace manually generated values with predefined constants. Signed-off-by: Valentin Vidic --- v2: Use constants where possible v3: Split up changes of constants drivers/staging/exfat/exfat_core.c | 18 +- drivers/staging/exfat/exfat_super.c | 8 2 files changed, 13

[PATCH v3 1/3] staging: exfat: cleanup spacing for operators

Fixes checkpatch.pl warnings: CHECK: spaces preferred around that '-' (ctx:VxV) CHECK: spaces preferred around that '+' (ctx:VxV) CHECK: spaces preferred around that '*' (ctx:VxV) CHECK: spaces preferred around that '|' (ctx:VxV) Signed-off-by: Val

[PATCH v3 2/3] staging: exfat: cleanup spacing for casts

Fix checkpatch.pl warnings: CHECK: No space is necessary after a cast Signed-off-by: Valentin Vidic --- v2: Use constants where possible v3: Split up changes of constants drivers/staging/exfat/exfat_core.c | 88 ++--- drivers/staging/exfat/exfat_super.c | 66

[PATCH v2 1/3] staging: exfat: drop duplicate date_time_t struct

Use timestamp_t for everything and cleanup duplicate code. Signed-off-by: Valentin Vidic --- v2: split up into multiple patches drivers/staging/exfat/exfat.h | 35 +++--- drivers/staging/exfat/exfat_super.c | 174 2 files changed, 61 insertions(+), 148

[PATCH v2 2/3] staging: exfat: drop unused field access_time_ms

Not used in the exfat-fuse implementation and spec defines this position should hold the value for CreateUtcOffset. Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat.h | 3 +-- drivers/staging/exfat/exfat_core.c | 1 - 2 files changed, 1 insertion(+), 3 deletions(-) diff --git

[PATCH v2 3/3] staging: exfat: add millisecond support

Use create_time_ms modify_time_ms fields to store the millisecond part of the file timestamp with the precision of 10 ms. Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat_core.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a/drivers/staging

[PATCH v3 3/4] staging: exfat: drop unused field access_time_ms

Spec defines that UtcOffset fields should start in this position instead. Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat.h | 6 -- drivers/staging/exfat/exfat_core.c | 4 +++- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/staging/exfat/exfat.h b

[PATCH v3 2/4] staging: exfat: drop duplicate date_time_t struct

Use timestamp_t for everything and cleanup duplicate code. Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat.h | 35 +++--- drivers/staging/exfat/exfat_super.c | 158 2 files changed, 55 insertions(+), 138 deletions(-) diff --git a/drivers

[PATCH v3 1/4] staging: exfat: drop unused function parameter

sbi parameter not used inside the function so remove it. Also cleanup unused variables generated by this change. Signed-off-by: Valentin Vidic --- v2: split up into multiple patches v3: split up some more fix if statement braces add utc offset fields drivers/staging/exfat/exfat_super.c

[PATCH v3 4/4] staging: exfat: add millisecond support

Use create_time_ms and modify_time_ms fields to store the millisecond part of the file timestamp with the precision of 10 ms. Signed-off-by: Valentin Vidic --- drivers/staging/exfat/exfat_core.c | 28 1 file changed, 28 insertions(+) diff --git a/drivers/staging

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

Fixes checkpatch error for missing spaces. Signed-off-by: Valentin Vidic --- 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 index 155d486f6cb3..ab848c1d55a6 100644

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

Fixes checkpatch warning for whitespace before a quoted newline. Signed-off-by: Valentin Vidic --- 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 index 8dae788972cc..75a0f62d5636

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

Fixes checkpatch error for open/close braces. Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 129 +-- 1 file changed, 41 insertions(+), 88 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c

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

Fixes checkpatch error for missing spaces. Signed-off-by: Valentin Vidic --- 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 32db3b320cbd..9c1787ef14e3 100644 --- a

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

Fixes checkpatch error for whitespace idents. Signed-off-by: Valentin Vidic --- .../Documentation/devicetree/pi433-overlay.dts | 32 +++--- drivers/staging/pi433/pi433_if.c | 8 +++--- 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a

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

Fixes checkpatch warning for error prone comparison. Signed-off-by: Valentin Vidic --- 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 index 1211216bdc2f..b4e6094ad553 100644

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

Fixes checkpatch error for trailing if statements. Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 45 +++- 1 file changed, 31 insertions(+), 14 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433

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

Fixes checkpatch warning for error prone comparison. Signed-off-by: Valentin Vidic --- 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 index 68498c457c49..1211216bdc2f 100644

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

Fixes checkpatch warning for void function return statements. Signed-off-by: Valentin Vidic --- 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 1e50d8f431f0..00cd8193cfd4 100644 --- a

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

Fixes checkpatch warning for 'transfered' and 'devided'. Signed-off-by: Valentin Vidic --- drivers/staging/pi433/Documentation/pi433.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/pi433/Documentation/pi433.txt b/drivers/staging

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

Fixes checkpatch warning for comparing value with NULL. Signed-off-by: Valentin Vidic --- 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 6e6996c68f0f..f32aeb215727

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

Fixes checkpatch warning for multiple blank lines in source. Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 11 --- drivers/staging/pi433/pi433_if.h | 8 drivers/staging/pi433/rf69.c | 2 -- drivers/staging/pi433/rf69.h

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

Fixes checkpatch error for prohibited spaces. Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 34 +- drivers/staging/pi433/rf69.c | 4 ++-- drivers/staging/pi433/rf69.h | 4 ++-- 3 files changed, 21 insertions(+), 21 deletions

[PATCH 02/16] staging: pi433: add space after comma

Fixes checkpatch error for missing spaces. Signed-off-by: Valentin Vidic --- 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 9c1787ef14e3..155d486f6cb3 100644 --- a

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

Fixes checkpatch warning for blank line after function/struct/union/enum declarations. Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index f32aeb215727

[PATCH 08/16] staging: pi433: remove spaces before tabs

Fixes checkpatch warning for hidden spaces before tabs. Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 2 +- drivers/staging/pi433/pi433_if.h | 4 ++-- drivers/staging/pi433/rf69.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging

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

Fixes checkpatch warning for multiple blank lines in source. Signed-off-by: Valentin Vidic --- v2: refresh patch for staging-testing drivers/staging/pi433/pi433_if.c | 11 --- drivers/staging/pi433/pi433_if.h | 8 drivers/staging/pi433/rf69.h | 2

[PATCH 1/8] staging: pi433: collapse else block after return statement

Fixes checkpatch warning: WARNING: else is not generally useful after a break or return Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging

[PATCH 5/8] staging: pi433: fix DATAMODUL_MODULATION_TYPE_OOK value

Reading from the spec, allowed values for modulation scheme after the shift are: 00 FSK 01 OOK 10 - 11 reserved Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69_registers.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433

[PATCH 3/8] staging: pi433: replace unsigned with unsigned int

Fixes checkpatch warning: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Valentin Vidic --- 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/pi

[PATCH 6/8] staging: pi433: use defines for shifting register values

Avoid shifting by magic numbers and use defines instead: SHIFT_DATAMODUL_MODULATION_TYPE SHIFT_LNA_CURRENT_GAIN Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 4 ++-- drivers/staging/pi433/rf69_registers.h | 2 ++ 2 files changed, 4 insertions(+), 2 deletions

[PATCH 4/8] staging: pi433: add parentheses to mask and shift

Fixes checkpatch warning: WARNING: Possible precedence defect with mask then right shift - may need parentheses Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers

[PATCH 8/8] staging: pi433: replace printk calls with dev_dbg

Fixes checkpatch warning: WARNING: printk() should include KERN_ facility level Signed-off-by: Valentin Vidic --- 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

[PATCH 7/8] staging: pi433: avoid logging ENOMEM messages

Fixes checkpatch warning: WARNING: Possible unnecessary 'out of memory' message Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_

[PATCH 2/8] staging: pi433: move var declaration to function level

Fixes checkpatch warning: WARNING: Missing a blank line after declarations Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index

Re: [PATCH 8/8] staging: pi433: replace printk calls with dev_dbg

On Wed, Dec 13, 2017 at 05:52:36PM +0300, Dan Carpenter wrote: > On Wed, Dec 13, 2017 at 03:21:56PM +0100, Valentin Vidic wrote: > > drivers/staging/pi433/pi433_if.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/staging/pi43

[PATCH 2/8 v2] staging: pi433: move var declaration to function level

Fixes checkpatch warning: WARNING: Missing a blank line after declarations Signed-off-by: Valentin Vidic --- v2: use a better variable name drivers/staging/pi433/pi433_if.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers

[PATCH 6/8 v2] staging: pi433: use defines for shifting register values

Avoid shifting by magic numbers and use defines instead. Signed-off-by: Valentin Vidic --- v2: - drop change for SHIFT_DATAMODUL_MODULATION_TYPE - move shifting to the header file drivers/staging/pi433/rf69.c | 16 drivers/staging/pi433/rf69_registers.h | 8

Re: [PATCH 6/8] staging: pi433: use defines for shifting register values

On Wed, Dec 13, 2017 at 05:32:28PM +0200, Marcus Wolf wrote: > Am 13.12.2017 um 16:21 schrieb Valentin Vidic: > > Avoid shifting by magic numbers and use defines instead: > > > >SHIFT_DATAMODUL_MODULATION_TYPE > >SHIFT_LNA_CURRENT_GAIN > >

[PATCH 6/8 v3] staging: pi433: use defines for shifting register values

Avoid shifting by magic numbers and use defines instead. Signed-off-by: Valentin Vidic --- v2: - drop change for SHIFT_DATAMODUL_MODULATION_TYPE - move shifting to the header file v3: - drop auto case - use CURRENT suffix - precompute define values drivers/staging/pi433/rf69.c

[PATCH 2/8 v3] staging: pi433: cleanup local variable declaration

Fix variable naming and checkpatch warning: WARNING: Missing a blank line after declarations Signed-off-by: Valentin Vidic --- v2: use a better variable name v3: keep the variable scope drivers/staging/pi433/pi433_if.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff

[PATCH] staging: pi433: fix CamelCase for dccPercent variables

Fixes checkpatch warning: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 30 +++--- drivers/staging/pi433/rf69.h | 6 +++--- drivers/staging/pi433/rf69_enum.h | 18 +- 3 files changed, 27 insertions

[PATCH 6/8 v4] staging: pi433: remove unused function

As it turns out rf69_get_lna_gain is not used at all. Signed-off-by: Valentin Vidic --- v2: - drop change for SHIFT_DATAMODUL_MODULATION_TYPE - move shifting to the header file v3: - drop auto case - use CURRENT suffix - precompute define values v4: - drop the whole function since it

[PATCH 4/6 v5] staging: pi433: avoid logging ENOMEM messages

Fixes checkpatch warning: WARNING: Possible unnecessary 'out of memory' message Signed-off-by: Valentin Vidic --- v5: resend patchset based on comments drivers/staging/pi433/pi433_if.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/pi433/pi

[PATCH 2/6 v5] staging: pi433: cleanup local variable declaration

Fix variable naming and checkpatch warning: WARNING: Missing a blank line after declarations Signed-off-by: Valentin Vidic --- v5: resend patchset based on comments drivers/staging/pi433/pi433_if.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/staging

[PATCH 5/6 v5] staging: pi433: replace printk calls with dev_dbg

Fixes checkpatch warning: WARNING: printk() should include KERN_ facility level Signed-off-by: Valentin Vidic --- v5: resend patchset based on comments drivers/staging/pi433/pi433_if.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b

[PATCH 3/6 v5] staging: pi433: replace unsigned with unsigned int

Fixes checkpatch warning: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Valentin Vidic --- v5: resend patchset based on comments drivers/staging/pi433/pi433_if.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/stagin

[PATCH 6/6 v5] staging: pi433: remove unused function

As it turns out rf69_get_lna_gain is not used at all. Signed-off-by: Valentin Vidic --- v5: resend patchset based on comments drivers/staging/pi433/rf69.c | 18 -- drivers/staging/pi433/rf69.h | 1 - 2 files changed, 19 deletions(-) diff --git a/drivers/staging/pi433/rf69.c b

[PATCH 1/6 v5] staging: pi433: collapse else block after return statement

Fixes checkpatch warning: WARNING: else is not generally useful after a break or return Signed-off-by: Valentin Vidic --- v5: resend patchset based on comments drivers/staging/pi433/pi433_if.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging

[PATCH] staging: pi433: fix CamelCase for fifo variables

Fixes checkpatch warnings: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/Documentation/pi433

[PATCH v2] staging: pi433: replace simple switch statements

Use const array to map switch cases to resulting values. Signed-off-by: Valentin Vidic --- v2: use correct type for const arrays drivers/staging/pi433/rf69.c | 233 ++- 1 file changed, 93 insertions(+), 140 deletions(-) diff --git a/drivers/staging/pi433/rf69.c

[PATCH v3] staging: pi433: replace simple switch statements

Use const array to map switch cases to resulting values. Signed-off-by: Valentin Vidic --- v2: use correct type for const arrays v3: add missing static keyword for af_map drivers/staging/pi433/rf69.c | 233 ++- 1 file changed, 93 insertions(+), 140 deletions

[PATCH] staging: pi433: add SPDX-License-Identifier tag

Use GPL-2.0+ based on the license text in each of the files. Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 1 + drivers/staging/pi433/pi433_if.h | 3 ++- drivers/staging/pi433/rf69.c | 1 + drivers/staging/pi433/rf69.h | 3 ++- drivers

[PATCH] staging: pi433: cleanup comments in rf69.h

Fixes checkpatch warning: WARNING: line over 80 characters Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.h | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/staging/pi433/rf69.h b/drivers/staging/pi433/rf69.h index c131ffbdc2db..d80c41966d39

Re: [PATCH] staging: pi433: add descriptions for mutex locks

On Sun, Apr 08, 2018 at 04:15:30PM +0200, Marcus Wolf wrote: > The hardware of Pi433 is working with every Raspberry Pi (on zero, you > need to solder the GPIO-pins) and with several other fruits like banana > pi. The only thing is, that you need different versions of the driver, > according to the

Re: [PATCH] staging: pi433: add descriptions for mutex locks

On Sun, Apr 08, 2018 at 05:22:46PM +0200, Marcus Wolf wrote: > Regarding your patch, I did not understand, why you did not remove > the mutex_lock in pi433_write. Wasn't it the goal to remove it? Is it possible for more than one userspace program to open the pi433 device and send messages? In tha

[PATCH] staging: pi433: use help in Kconfig description

Fixes checkpatch warning: WARNING: prefer 'help' over '---help---' for new help texts Signed-off-by: Valentin Vidic --- drivers/staging/pi433/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433/Kconfig b/drivers/staging

Re: [PATCH] staging: pi433: add descriptions for mutex locks

On Thu, Apr 19, 2018 at 11:25:19AM +0200, Marcus Wolf wrote: > let me know, what you like to have. For sure with just one station and > no other 433MHz equipment, options for testing are quite limited. I can get Rpi3 and with two shields test 433MHz communication between Rpi2 and Rpi3. -- Valent

[PATCH v2] staging: pi433: cleanup tx_fifo locking

not require locking after kfifo_reset is removed. Signed-off-by: Valentin Vidic --- v2: print a warning if partial fifo write happens drivers/staging/pi433/pi433_if.c | 23 ++- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b

Re: [PATCH v2] staging: pi433: add mutex fixing concurrency issues.

On Thu, Jun 07, 2018 at 08:45:03AM -0400, Hugo Lefeuvre wrote: > After discussing this issue on the kernel newbies mailing list[0] we > came to the conclusion that it is very unlikely that pi433_release and > pi433_ioctl would ever run concurrently in this case. This is also > true for read/write.

Re: rf69_set_deviation in rf69.c (pi433 driver)

On Thu, Jun 21, 2018 at 10:03:45PM -0400, Hugo Lefeuvre wrote: > I'll prepare a patch addressing both issues. However I don't own test devices > so it would be really great if you could test it ! I have two pi433 devices now so I should be able to tests things, just let me know... -- Valentin __

[PATCH] staging: pi433: replace simple switch statements

Use const array to map switch cases to resulting values. Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 233 ++- 1 file changed, 93 insertions(+), 140 deletions(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c index

[PATCH] lustre: remove useless return statements

All statements removed from the end of void functions as reported by checkpatch. Signed-off-by: Valentin Vidic --- drivers/staging/lustre/lustre/lmv/lmv_obd.c | 1 - drivers/staging/lustre/lustre/lmv/lproc_lmv.c | 1 - drivers/staging/lustre/lustre/lov/lov_obd.c | 1 - 3 files changed, 3

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

All statements removed from the end of void functions as reported by checkpatch. Signed-off-by: Valentin Vidic --- Changes in v2: updated recipient list and subject drivers/staging/lustre/lustre/lmv/lmv_obd.c | 1 - drivers/staging/lustre/lustre/lmv/lproc_lmv.c | 1 - drivers/staging/lustre

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

conversions on flag assignment. Signed-off-by: Valentin Vidic --- 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/lustre/lmv/lmv_obd.c index 09b469243d73

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

:right side has type int Signed-off-by: Valentin Vidic --- 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/lustre/lmv/lmv_obd.c b/drivers/staging

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

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 --- drivers/staging/media/atomisp/platform/intel-mid/intel_mid_pcihelpers.c | 1 - 1 file

[PATCH] staging: pi433: fix CamelCase for maxMinus variables

Fixes checkpatch warnings: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/Documentation/pi433.txt | 10 +- drivers/staging/pi433/rf69.c

[PATCH] staging: pi433: fix CamelCase for payload identifiers

Fixes checkpatch warnings: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 2 +- drivers/staging/pi433/rf69.c | 6 +++--- drivers/staging/pi433/rf69.h | 2 +- drivers/staging/pi433/rf69_enum.h | 2 +- 4 files

[PATCH] staging: pi433: replace shifting with BIT macro

Fixes checkpatch warnings: CHECK: Prefer using the BIT macro Signed-off-by: Valentin Vidic --- 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 index 960249c600a5

[PATCH] staging: pi433: remove unnecessary parentheses

Fixes checkpatch warnings: CHECK: Unnecessary parentheses around 'mantisse != mantisse16' CHECK: Unnecessary parentheses around 'mantisse != mantisse20' CHECK: Unnecessary parentheses around 'mantisse != mantisse24' Signed-off-by: Valentin Vidic --- dr

[PATCH] staging: pi433: align function parameters with open parenthesis

Fixes checkpatch warnings: CHECK: Alignment should match open parenthesis --- drivers/staging/pi433/pi433_if.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index f56425ffbe90..edcd7e798f99 1006

[PATCH v2] staging: pi433: align function parameters with open parenthesis

Fixes checkpatch warnings: CHECK: Alignment should match open parenthesis Signed-off-by: Valentin Vidic --- v2: add missing Signed-off-by drivers/staging/pi433/pi433_if.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b

Re: [PATCH] staging: pi433: Cleanup codestyle, indent statements after case labels

On Sun, Dec 24, 2017 at 02:42:57PM +0100, Marcus Wolf wrote: > > int rf69_set_dagc(struct spi_device *spi, enum dagc dagc) > > { > > switch (dagc) { > > - case normalMode: return rf69_write_reg(spi, > > REG_TESTDAGC, DAGC_NORMAL); > > - case improve:re

[PATCH 1/8] staging: pi433: fix CamelCase for syncValues

Fixes checkpatch warnings: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 18 +- drivers/staging/pi433/rf69.h | 2 +- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging

[PATCH 2/8] staging: pi433: fix CamelCase for powerLevel

Fixes checkpatch warnings: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 8 drivers/staging/pi433/rf69.h | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c

[PATCH 3/8] staging: pi433: fix CamelCase for antennaImpedance

Fixes checkpatch warnings: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/Documentation/pi433.txt | 2 +- drivers/staging/pi433/pi433_if.h | 2 +- drivers/staging/pi433/rf69.c | 4 ++-- drivers/staging/pi433/rf69.h

[PATCH 4/8] staging: pi433: fix CamelCase for Ohm identifiers

Fixes checkpatch warnings: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/Documentation/pi433.txt | 4 ++-- drivers/staging/pi433/pi433_if.c | 2 +- drivers/staging/pi433/rf69.c | 4 ++-- drivers

[PATCH 5/8] staging: pi433: fix CamelCase for currentValue

Local variable storing the value for modulation register so replace with modulation_reg. Fixes checkpatch warnings: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers

[PATCH 7/8] staging: pi433: fix CamelCase for regValue

Local variable storing the new value for dio register so replace with dio_value. Update regaddr to dio_addr to match. Fixes checkpatch warnings: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 24 1 file changed, 12

[PATCH 6/8] staging: pi433: fix CamelCase for newValue

Local variable storing the new value for bandwidth register so replace with bandwidth. Fixes checkpatch warnings: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a

[PATCH 8/8] staging: pi433: fix CamelCase for DIONumber

Fixes checkpatch warnings: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 4 ++-- drivers/staging/pi433/rf69.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c index

[PATCH] staging: pi433: fix CamelCase for txStartCondition

Fixes checkpatch warning: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.h | 2 +- drivers/staging/pi433/rf69.c | 4 ++-- drivers/staging/pi433/rf69.h | 2 +- drivers/staging/pi433/rf69_enum.h | 2 +- 4 files changed, 5 insertions(+), 5

[PATCH] staging: pi433: fix CamelCase for thresholdDecrement

Fixes checkpatch warning: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/Documentation/pi433.txt | 2 +- drivers/staging/pi433/pi433_if.h | 2 +- drivers/staging/pi433/rf69.c | 4 ++-- drivers/staging/pi433/rf69.h

[PATCH] staging: pi433: fix CamelCase for addressFiltering

Fixes checkpatch warning: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 4 ++-- drivers/staging/pi433/pi433_if.h | 2 +- drivers/staging/pi433/rf69.c | 4 ++-- drivers/staging/pi433/rf69.h | 2 +- drivers/staging/pi433/rf69_enum.h

[PATCH] staging: pi433: fix CamelCase for address_filtering enum

Fixes checkpatch warnings: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/Documentation/pi433.txt | 12 ++-- drivers/staging/pi433/pi433_if.c | 6 +++--- drivers/staging/pi433/rf69.c

[PATCH] staging: pi433: fix CamelCase for packetFormat enum

Fixes checkpatch warnings: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 8 drivers/staging/pi433/rf69.c | 8 drivers/staging/pi433/rf69.h | 2 +- drivers

[PATCH] staging: pi433: fix CamelCase for Address variables

Fixes checkpatch warnings: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/rf69.c | 8 drivers/staging/pi433/rf69.h | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/pi433/rf69.c b

[PATCH] staging: pi433: fix CamelCase for afterSyncInterrupt

Fixes checkpatch warning: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic --- drivers/staging/pi433/pi433_if.c | 2 +- drivers/staging/pi433/rf69.c | 2 +- drivers/staging/pi433/rf69_enum.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging

  1   2   >