[PATCH v3] drivers: staging: lustre: lnet: api-nc.c: Fix Style Warnings

2014-11-17 Thread Thomas Wood
Remove an unecessary return statement in a void function. Remove an unecessary space between a function name and the parentheses. Signed-off-by: Thomas Wood --- Fixed the subject line containing the entire description. Removed an attempt at scissor lines. drivers/staging/lustre/lnet/lnet/api

[PATCH] drivers: staging: lustre: lnet: api-nc.c: Fix Style Warnings

2014-11-17 Thread Thomas Wood
Fixed Subject line containing the entire patch description. >88< Remove an unecessary return statement in a void function. Remove an unecessary space between a function name and the parentheses. Signed-off-by:

[PATCH] drivers: staging: lustre: lnet: api-nc.c: Fix Style Warnings

2014-11-17 Thread Thomas Wood
Fixed Subject line containing the entire patch description. 88 Remove an unecessary return statement in a void function. Remove an unecessary space between a function name and the parentheses. Signed-off-by: Thomas Wood

[PATCH v3] drivers: staging: lustre: lnet: api-nc.c: Fix Style Warnings

2014-11-17 Thread Thomas Wood
Remove an unecessary return statement in a void function. Remove an unecessary space between a function name and the parentheses. Signed-off-by: Thomas Wood tommyandr...@gmail.com --- Fixed the subject line containing the entire description. Removed an attempt at scissor lines. drivers/staging

[PATCH] drivers: staging: lustre: lnet: api-nc.c: Fix Style Warnings Remove an unecessary return statement in a void function. Remove an unecessary space between a function name and the parentheses.

2014-11-16 Thread Thomas Wood
Signed-off-by: Thomas Wood --- I hope this hasn't been duplicated... drivers/staging/lustre/lnet/lnet/api-ni.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lnet/lnet/api-ni.c b/drivers/staging/lustre/lnet/lnet/api-ni.c index 5e6e4e2..faceb95

[PATCH] drivers: staging: lustre: lnet: api-nc.c: Fix Style Warnings Remove an unecessary return statement in a void function. Remove an unecessary space between a function name and the parentheses.

2014-11-16 Thread Thomas Wood
Signed-off-by: Thomas Wood tommyandr...@gmail.com --- I hope this hasn't been duplicated... drivers/staging/lustre/lnet/lnet/api-ni.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lnet/lnet/api-ni.c b/drivers/staging/lustre/lnet/lnet/api-ni.c

[PATCH v2 3/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Remove unnecessary whitespace.

2014-06-06 Thread Thomas Wood
Remove whitespace before a quoted newline. Remove space between function name and it's parameter list. Signed-off-by: Thomas Wood --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb

[PATCH v2 2/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Add required spaces.

2014-06-06 Thread Thomas Wood
Add spaces around operators, after commas, and between (foo*). Signed-off-by: Thomas Wood --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 48 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c b

[PATCH v2 4/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Remove unnecessary braces.

2014-06-06 Thread Thomas Wood
Remove unnecessary braces from single statement if blocks. Signed-off-by: Thomas Wood --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 21 +++-- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c b/drivers

[PATCH v2 5/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Misc. fixes.

2014-06-06 Thread Thomas Wood
Remove initialization of static value. Invert the empty if statement's test to replace the else statement. Remove parentheses from a return statement. Fix some code indentation. Signed-off-by: Thomas Wood --- I'm pretty sure that the indentation was caused by a mistake by me earlier

[PATCH v2 0/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Fix style errors and warnings.

2014-06-06 Thread Thomas Wood
Changes since v1: * Made single patch into a patch set. * Added better commit messages. Is this better, or do I still have to split up my first patch? >8--8< Remove all style errors from ft1000_debug.c and some warnings. Thomas W

[PATCH v2 1/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Replace spaces with tabs.

2014-06-06 Thread Thomas Wood
Replace the spaces that were used as tabs with actual tabs. Signed-off-by: Thomas Wood --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 974 +++ 1 file changed, 487 insertions(+), 487 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c b

[PATCH v2 4/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Remove unnecessary braces.

2014-06-06 Thread Thomas Wood
Remove unnecessary braces from single statement if blocks. Signed-off-by: Thomas Wood tommyandr...@gmail.com --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 21 +++-- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb

[PATCH v2 5/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Misc. fixes.

2014-06-06 Thread Thomas Wood
Remove initialization of static value. Invert the empty if statement's test to replace the else statement. Remove parentheses from a return statement. Fix some code indentation. Signed-off-by: Thomas Wood tommyandr...@gmail.com --- I'm pretty sure that the indentation was caused by a mistake

[PATCH v2 0/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Fix style errors and warnings.

2014-06-06 Thread Thomas Wood
Changes since v1: * Made single patch into a patch set. * Added better commit messages. Is this better, or do I still have to split up my first patch? 8--8 Remove all style errors from ft1000_debug.c and some warnings. Thomas Wood (5

[PATCH v2 1/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Replace spaces with tabs.

2014-06-06 Thread Thomas Wood
Replace the spaces that were used as tabs with actual tabs. Signed-off-by: Thomas Wood tommyandr...@gmail.com --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 974 +++ 1 file changed, 487 insertions(+), 487 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb

[PATCH v2 3/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Remove unnecessary whitespace.

2014-06-06 Thread Thomas Wood
Remove whitespace before a quoted newline. Remove space between function name and it's parameter list. Signed-off-by: Thomas Wood tommyandr...@gmail.com --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging

[PATCH v2 2/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Add required spaces.

2014-06-06 Thread Thomas Wood
Add spaces around operators, after commas, and between (foo*). Signed-off-by: Thomas Wood tommyandr...@gmail.com --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 48 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb

Re: [PATCH] staging: ft1000: ft1000-usb: ft1000_debug.c: Fix style errors and some warnings.

2014-06-02 Thread Thomas Wood
On 06/02/2014 03:30 AM, Dan Carpenter wrote: > On Sun, Jun 01, 2014 at 07:26:18PM -0700, Thomas Wood wrote: >> Don't use spaces as tabs, and fix other style errors and warnings. >> >> Signed-off-by: Thomas Wood > > Too much in one patch. Break it up into one type

Re: [PATCH] staging: ft1000: ft1000-usb: ft1000_debug.c: Fix style errors and some warnings.

2014-06-02 Thread Thomas Wood
On 06/02/2014 03:30 AM, Dan Carpenter wrote: On Sun, Jun 01, 2014 at 07:26:18PM -0700, Thomas Wood wrote: Don't use spaces as tabs, and fix other style errors and warnings. Signed-off-by: Thomas Wood tommyandr...@gmail.com Too much in one patch. Break it up into one type of fix per patch

[PATCH] staging: ft1000: ft1000-usb: ft1000_debug.c: Fix style errors and some warnings.

2014-06-01 Thread Thomas Wood
Don't use spaces as tabs, and fix other style errors and warnings. Signed-off-by: Thomas Wood --- I think that this is correctly indented now, but I didn't touch the double parentheses on line 322 because I wasn't sure if they were necessary or not. drivers/staging/ft1000/ft1000-usb

[PATCH] staging: ft1000: ft1000-usb: ft1000_debug.c: Fix style errors and some warnings.

2014-06-01 Thread Thomas Wood
Don't use spaces as tabs, and fix other style errors and warnings. Signed-off-by: Thomas Wood tommyandr...@gmail.com --- I think that this is correctly indented now, but I didn't touch the double parentheses on line 322 because I wasn't sure if they were necessary or not. drivers/staging

[PATCH] drivers/w1/w1_int.c: Fix style errors.

2014-05-30 Thread Thomas Wood
Replace spaces at beginning of the string with tabs, and replace foo * bar with foo *bar in a pointer declaration. Signed-off-by: Thomas Wood --- This time using linux-next! drivers/w1/w1_int.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/w1/w1_int.c

[PATCH] staging/goldfish: Clean up trailing whitespace.

2014-05-30 Thread Thomas Wood
When using scripts/checkpatch.pl, there are errors about trailing whitespace. Use scripts/cleanfile to remove these errors. Signed-off-by: Thomas Wood --- drivers/staging/goldfish/goldfish_nand.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging

[PATCH] staging/goldfish: Clean up trailing whitespace.

2014-05-30 Thread Thomas Wood
When using scripts/checkpatch.pl, there are errors about trailing whitespace. Use scripts/cleanfile to remove these errors. Signed-off-by: Thomas Wood tommyandr...@gmail.com --- drivers/staging/goldfish/goldfish_nand.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH] drivers/w1/w1_int.c: Fix style errors.

2014-05-30 Thread Thomas Wood
Replace spaces at beginning of the string with tabs, and replace foo * bar with foo *bar in a pointer declaration. Signed-off-by: Thomas Wood tommyandr...@gmail.com --- This time using linux-next! drivers/w1/w1_int.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git