[PATCH 8/8] staging:rtl8821ae: fix ERROR: (foo*) should be (foo *)

2014-02-06 Thread Gokulnath A
This patch fixes the following checkpatch.pl issues in rtl8821ae/pci.c file ERROR: (foo*) should be (foo *) Signed-off-by: Gokulnath A --- drivers/staging/rtl8821ae/pci.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers

[PATCH 6/8] staging:rtl8821ae: Fix code indentation errors in pci.c

2014-02-06 Thread Gokulnath A
fixed all the code indentation errors found by checkpatch.pl script Signed-off-by: Gokulnath A --- drivers/staging/rtl8821ae/pci.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers/staging/rtl8821ae/pci.c index

[PATCH 7/8] staging:rtl8821ae: fix ERROR: spaces required around that '&&' in pci.c

2014-02-06 Thread Gokulnath A
Fixed the error found by checkpatch.pl related to spacing requirement in the && condition. Signed-off-by: Gokulnath A --- drivers/staging/rtl8821ae/pci.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers/staging/rtl8821

[PATCH 3/8] staging:rtl8821ae fix ERROR: space required after that in pci.c

2014-02-06 Thread Gokulnath A
fixed all the spacing related issues found by checkpatch.pl script. Signed-off-by: Gokulnath A --- drivers/staging/rtl8821ae/pci.c | 54 - 1 file changed, 27 insertions(+), 27 deletions(-) diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers/staging

[PATCH 0/8] staging:rtl8821ae:pci.c code cleanup

2014-02-06 Thread Gokulnath A
The following series of patches are used to Fix all the errors found by checkpatch.pl script in the pci.c. Few warnings are still present in the patches submitted. That can be taken care in the next set of patches. Gokulnath A (8): staging:rtl8821ae:pci.c trailing whitespace cleanup

[PATCH 5/8] staging:rtl8821ae fix ERROR: space prohibited after that

2014-02-06 Thread Gokulnath A
Fixed all the errors related to space prohibition found by checkpatch.pl script Signed-off-by: Gokulnath A --- drivers/staging/rtl8821ae/pci.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers/staging

[PATCH 4/8] staging:rtl8821ae fix ERROR: space required before that

2014-02-06 Thread Gokulnath A
Fixed all the errors related to spacing found by checkpatch.pl script Signed-off-by: Gokulnath A --- drivers/staging/rtl8821ae/pci.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers/staging/rtl8821ae/pci.c index

[PATCH 2/8] staging:rtl8821ae remove C99 comments in pci.c

2014-02-06 Thread Gokulnath A
Removed all the C99 comments found by checkpatch.pl script Signed-off-by: Gokulnath A --- drivers/staging/rtl8821ae/pci.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers/staging/rtl8821ae/pci.c index 618a3cb..a1bf11b

[PATCH 1/8] staging:rtl8821ae: trailing whitespace cleanup in pci.c

2014-02-06 Thread Gokulnath A
Fixed all the trailing whitespace errors found by checkpatch.pl script Signed-off-by: Gokulnath A --- drivers/staging/rtl8821ae/pci.c | 250 1 file changed, 125 insertions(+), 125 deletions(-) diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers

Re: [PATCH v2 2/4] staging: bcm: line over 80 characters in InterfaceIdleMode.c

2013-12-21 Thread Gokulnath A
Hi Dan, On Sat, Dec 21, 2013 at 2:00 PM, Dan Carpenter wrote: > On Fri, Dec 20, 2013 at 08:54:30PM +0530, Gokulnath A wrote: >> From: Gokulnath Avanashilingam > > No need. This should come from your email. > >> >> Fixed all the line over 80 characters warning fo

Re: [PATCHv2 1/3] silicom: fix whitespace issues in bypass.c

2013-12-20 Thread Gokulnath A
On Fri, Dec 20, 2013 at 10:31 PM, Greg Kroah-Hartman wrote: > On Fri, Dec 20, 2013 at 08:36:48PM +0530, Gokulnath A wrote: >> Hello Michael, >> >> On Fri, Dec 20, 2013 at 7:21 PM, Michael Hoefler >> wrote: >> > This patch addresses several prob

[PATCH v2 1/4] staging:bcm: Remove Developer Debug prints in InterfaceIdleMode.c

2013-12-20 Thread Gokulnath A
From: Gokulnath Avanashilingam Removed the developer debug prints BCM_DEBUG_PRINT() as per the TODO list, also removed braces for the if-statement to match coding style Signed-off-by: Gokulnath Avanashilingam --- drivers/staging/bcm/InterfaceIdleMode.c | 57 ++--- 1

[PATCH v2 4/4] staging: bcm: fixed warning about no and prohibited space in InterfaceIdleMode.c.

2013-12-20 Thread Gokulnath A
From: Gokulnath Avanashilingam This patch fixes all the warning related to spacing issues found by checkpatch.pl script in InterfaceIdleMode.c Signed-off-by: Gokulnath Avanashilingam --- drivers/staging/bcm/InterfaceIdleMode.c | 24 1 file changed, 12 insertions(+), 12

[PATCH v2 2/4] staging: bcm: line over 80 characters in InterfaceIdleMode.c

2013-12-20 Thread Gokulnath A
From: Gokulnath Avanashilingam Fixed all the line over 80 characters warning found by checkpatch.pl script. Signed-off-by: Gokulnath Avanashilingam --- drivers/staging/bcm/InterfaceIdleMode.c | 153 +--- 1 file changed, 99 insertions(+), 54 deletions(-) diff --git

[PATCH v2 3/4] staging: bcm: using time_after and time_before in InterfaceIdleMode.c

2013-12-20 Thread Gokulnath A
From: Gokulnath Avanashilingam Used the time_after and time_before insted of comparing the jiffies directly.This will fix the warnings and errors found by the checkpatch.pl script. Signed-off-by: Gokulnath Avanashilingam --- drivers/staging/bcm/InterfaceIdleMode.c | 6 +++--- 1 file changed, 3

[PATCH v2 0/4] staging: bcm: Fixing coding style in InterfaceIdleMode.c

2013-12-20 Thread Gokulnath A
This patch series fix all the warning and errors found by checkpatch.pl script, Additionally removed the developer debug BCM_DEBUG_PRINT() prints as per TODO list. Gokulnath Avanashilingam (4): staging:bcm: Remove Developer Debug prints in InterfaceIdleMode.c staging: bcm: line over 80 charact

Re: [PATCHv2 1/3] silicom: fix whitespace issues in bypass.c

2013-12-20 Thread Gokulnath A
Hello Michael, On Fri, Dec 20, 2013 at 7:21 PM, Michael Hoefler wrote: > This patch addresses several problems in bypass.c found by checkpatch. > Furthermore it removes/adds some empty lines to make the code more readable. > > The following problems are fixed: > - line over 80 characters > - sp

[PATCH v3 2/2] staging: bcm: line over 80 characters in InterfaceDld.c

2013-12-18 Thread Gokulnath A
Fixed all the line over 80 characters warning found by checkpatch.pl script. Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceDld.c | 70 ++ 1 file changed, 49 insertions(+), 21 deletions(-) diff --git a/drivers/staging/bcm/InterfaceDld.c b

[PATCH v3 0/2] staging: bcm: Fixing coding style in InterfaceDld.c

2013-12-18 Thread Gokulnath A
As per comments from Greg KH and Dan Carpenter updated and split the patch series. Gokulnath A (2): staging:bcm: Removed developer debug prints in InterfaceDld.c staging: bcm: line over 80 characters in InterfaceDld.c drivers/staging/bcm/InterfaceDld.c | 140

[PATCH v3 1/2] staging:bcm: Removed developer debug prints in InterfaceDld.c

2013-12-18 Thread Gokulnath A
Removed the developer debug prints BCM_DEBUG_PRINT() as per the TODO list, also removed braces for the if-statement to match coding style Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceDld.c | 70 -- 1 file changed, 14 insertions(+), 56 deletions

[PATCH V2] staging:bcm: Code cleanup for InterfaceDld.c

2013-12-17 Thread Gokulnath A
As per the TODO, 1. Removed the developer debug BCM_DEBUG_PRINT() messages. 2. Fixed all the errors found by checkpatch.pl script. 3. Fixed all the line over 80 characters warning found by checkpatch.pl script. Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceDld.c | 122

Re: [PATCH 1/8] staging:bcm:InterfaceDld: fix checkpatch error

2013-12-14 Thread Gokulnath A
On Sat, Dec 14, 2013 at 4:41 AM, Dan Carpenter wrote: > On Sat, Dec 14, 2013 at 01:05:29AM +0530, Gokulnath A wrote: >> fixed all the errors found by checkpatch.pl >> > > This is not very useful. The subjects of this patch series could be > improved as well. i will take

[PATCH 3/8] staging:bcm:InterfaceInit: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceInit.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/bcm/InterfaceInit.c b/drivers/staging/bcm/InterfaceInit.c index 3acdb58

[PATCH 7/8] staging:bcm:hostmibs: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/hostmibs.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/bcm/hostmibs.c b/drivers/staging/bcm/hostmibs.c index f55300d..38acfe3 100644 --- a/drivers

[PATCH 8/8] staging:bcm:Qos: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/Qos.c | 255 +- 1 file changed, 95 insertions(+), 160 deletions(-) diff --git a/drivers/staging/bcm/Qos.c b/drivers/staging/bcm/Qos.c index 1609a2b

[PATCH 4/8] staging:bcm:InterfaceIsr: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceIsr.c | 107 - 1 file changed, 45 insertions(+), 62 deletions(-) diff --git a/drivers/staging/bcm/InterfaceIsr.c b/drivers/staging/bcm/InterfaceIsr.c

[PATCH 5/8] staging:bcm:InterfaceRx: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceRx.c | 154 +++--- 1 file changed, 62 insertions(+), 92 deletions(-) diff --git a/drivers/staging/bcm/InterfaceRx.c b/drivers/staging/bcm/InterfaceRx.c index

[PATCH 1/8] staging:bcm:InterfaceDld: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceDld.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/bcm/InterfaceDld.c b/drivers/staging/bcm/InterfaceDld.c index 463bdee..a8d0063 100644 --- a/drivers

[PATCH 6/8] staging:bcm:InterfaceTx: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceTx.c | 123 -- 1 file changed, 51 insertions(+), 72 deletions(-) diff --git a/drivers/staging/bcm/InterfaceTx.c b/drivers/staging/bcm/InterfaceTx.c index

[PATCH 2/8] staging:bcm:InterfaceIdleMode: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceIdleMode.c | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/drivers/staging/bcm/InterfaceIdleMode.c b/drivers/staging/bcm/InterfaceIdleMode.c index

[PATCH] staging:ktap:ktap.c: fix coding style

2013-10-27 Thread Gokulnath A
fixed the errors and warnings found by checkpatch.pl script. Signed-off-by: Gokulnath A --- drivers/staging/ktap/interpreter/ktap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ktap/interpreter/ktap.c b/drivers/staging/ktap/interpreter/ktap.c index

[PATCH] staging:ktap:ktap.c: fix coding style

2013-10-27 Thread Gokulnath A
fixed the errors and warnings found by checkpatch.pl script. Signed-off-by: Gokulnath A --- drivers/staging/ktap/interpreter/ktap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ktap/interpreter/ktap.c b/drivers/staging/ktap/interpreter/ktap.c index

[PATCH] staging:ktap:loader.c fix coding style

2013-10-27 Thread Gokulnath A
fixed the errors and warnings generated by checkpatch.pl script. Signed-off-by: Gokulnath A --- drivers/staging/ktap/interpreter/loader.c | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/drivers/staging/ktap/interpreter/loader.c b/drivers/staging