[Patch v2 2/3] staging: vt6656: Replace embedded function name with __func__ in rf.c

2017-04-02 Thread Daniel Cashman
From: Dan Cashman Change embedded function name in vnt_rf_set_txpower with %s format with __func__ argument to make it consistent with other part of if-else and kernel coding style standards as reported by checkpatch. Signed-off-by: Daniel Cashman --- drivers/staging/vt6656/rf.c | 2 +- 1

[Patch v2 3/3] staging: vt6656: Split arguments to avoid 80-char violation in rf.c

2017-04-02 Thread Daniel Cashman
From: Dan Cashman Wrap arguments of call to vnt_control_out() to avoid exceeding 80 character limit, but maintain alignment. Signed-off-by: Daniel Cashman --- drivers/staging/vt6656/rf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/rf.c b

[Patch v2 1/3] staging: vt6656: convert spaces to tabs for rf.c

2017-04-02 Thread Daniel Cashman
From: Dan Cashman Address checkpatch errors encountered in rf.c by removing use of spaces and replacing with properly aligned tabs. Signed-off-by: Daniel Cashman --- drivers/staging/vt6656/rf.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging

[Patch v2 0/3] staging: vt6656: Fix checkpatch style violations.

2017-04-02 Thread Daniel Cashman
ion name in a debug statement and an exceeded 80-char line, respectively. Signed-off-by: Daniel Cashman Dan Cashman (3): staging: vt6656: convert spaces to tabs for rf.c staging: vt6656: Replace embedded function name with __func__ in rf.c staging: vt6656: Split arguments to avoid 80-c

[PATCH] staging: vt6656: Fix spaces, char limit and embedded func name in print.

2017-04-01 Thread Daniel Cashman
From: Dan Cashman Address errors and warning found in rf.c by checkpatch kernel style script. Specifically, change spaces to tabs, split function arguments across a new line to avoid 80 character limit, and remove use of embedded function name in a dev_dbg() call. Signed-off-by: Daniel Cashman

Re: [PATCH v2] staging: media: atomisp: Fix style. remove space before ',' and convert to tabs.

2017-04-01 Thread Daniel Cashman
My apologies; I did not review the TODO. Consider this withdrawn. Dan On 03/29/2017 10:30 AM, Alan Cox wrote: > On Wed, 2017-03-29 at 09:57 -0700, Daniel Cashman wrote: >> From: Dan Cashman >> >> Signed-off-by: Dan Cashman > > > As the TODO asks - please no

[PATCH v2] staging: media: atomisp: Fix style. remove space before ', ' and convert to tabs.

2017-03-29 Thread Daniel Cashman
From: Dan Cashman Signed-off-by: Dan Cashman --- drivers/staging/media/atomisp/i2c/ap1302.c | 4 ++-- drivers/staging/media/atomisp/i2c/gc0310.c | 2 +- drivers/staging/media/atomisp/i2c/gc2235.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/media/atomis

[PATCH] Remove atomisp/i2c style errors.

2017-03-28 Thread Daniel Cashman
From: Dan Cashman Remove two ' , ' issues and change spaces to tabs found by poking around in drivers/staging/. Warnings left untouched. Test: Run checkpatch script in drivers/staging/media/atomisp/i2c before and after change. Errors go from 3 to 0. Signed-off-by: Dan Cashman --- drivers/sta