[PATCH] Staging: netlogic: xlr_net.c Fixed code-style warnings

Fixed the following warning generated by checkpatch.pl: WARNING: Missing a blank line after declarations Signed-off-by: Chaitanya Hazarey --- drivers/staging/netlogic/xlr_net.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/netlogic/xlr_net.c b/drivers/staging/netlogic

[PATCH] Staging: netlogic: xlr_net.c Fixed code-style warnings

Fixed the following warning generated by checkpatch.pl: WARNING: Missing a blank line after declarations Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/netlogic/xlr_net.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/netlogic/xlr_net.c b/drivers/staging

[PATCH 3/3] Staging: rtl8192u: r8192U_wx.c Fixed a code-style error about trailing whitespace

Fixed the following error reported by running checkpatch.pl ERROR: trailing whitespace Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u

[PATCH 1/3] Staging: rtl8192u: r8192U_wx.c Fixed code-style warnings

Fixed some of the following warning generated by checkpatch.pl: WARNING: Missing a blank line after declarations Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u

[PATCH 2/3] Staging: rtl8192u: r8192U_wx.c Fixed some more code-style warnings about spaces

Fixed the following warnings issued by checkpatch.pl WARNING: please, no space before tabs Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging

[PATCH 2/3] Staging: rtl8192u: r8192U_wx.c Fixed some more code-style warnings about spaces

Fixed the following warnings issued by checkpatch.pl WARNING: please, no space before tabs Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers

[PATCH 3/3] Staging: rtl8192u: r8192U_wx.c Fixed a code-style error about trailing whitespace

Fixed the following error reported by running checkpatch.pl ERROR: trailing whitespace Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging

[PATCH 1/3] Staging: rtl8192u: r8192U_wx.c Fixed code-style warnings

Fixed some of the following warning generated by checkpatch.pl: WARNING: Missing a blank line after declarations Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH 2/2] Staging: rtl8192u: r8192U_core.c Converted a C99 comment to fix code-style error

Converted a C99 comment to fix the following error: ERROR: trailing statements should be on next line Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c b

[PATCH 1/2] Staging: rtl8192u: r8192U_core.c Fixed open brace { placement

Fixed open brace placement to address the following code-style errors: ERROR: that open brace { should be on the previous line Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_core.c | 69 -- 1 file changed, 23 insertions(+), 46 deletions

[PATCH 1/2] Staging: rtl8192u: r8192U_core.c Fixed open brace { placement

Fixed open brace placement to address the following code-style errors: ERROR: that open brace { should be on the previous line Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_core.c | 69 -- 1 file changed, 23 insertions(+), 46

[PATCH 2/2] Staging: rtl8192u: r8192U_core.c Converted a C99 comment to fix code-style error

Converted a C99 comment to fix the following error: ERROR: trailing statements should be on next line Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c

[PATCH 2/3] Staging: rtl8192u: r8192U_core.c Rearranged statements to fix code-style errors

Rearranged statements around if and switch statements to address the following error: ERROR: trailing statements should be on next line Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_core.c | 553 - 1 file changed, 402 insertions(+), 151

[PATCH 3/3] Staging: rtl8192u: r8192U_core.c Converted some C99 comments to fix code-style errors

Converted some C99 warnings to fix the following error: ERROR: trailing statements should be on next line Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_core.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/drivers

[PATCH 1/3] Staging: rtl8192u: r8192U_core.c Added a space after ','

Added spaces after ',' to fix the following warning: ERROR: space required after that ',' (ctx:VxV) Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_core.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/staging/rtl8192u

[PATCH 1/3] Staging: rtl8192u: r8192U_core.c Added a space after ','

Added spaces after ',' to fix the following warning: ERROR: space required after that ',' (ctx:VxV) Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_core.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/staging

[PATCH 3/3] Staging: rtl8192u: r8192U_core.c Converted some C99 comments to fix code-style errors

Converted some C99 warnings to fix the following error: ERROR: trailing statements should be on next line Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_core.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git

[PATCH 2/3] Staging: rtl8192u: r8192U_core.c Rearranged statements to fix code-style errors

Rearranged statements around if and switch statements to address the following error: ERROR: trailing statements should be on next line Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_core.c | 553 - 1 file changed, 402 insertions

Re: [PATCH 2/4] Staging: rtl8192u: r8192U_wx.c Removed all C99 comments

On Tue, Jun 3, 2014 at 2:42 PM, Greg KH wrote: > So keep working on things, > these aren't lost, I'll get to them in a few weeks. > Ok will do, Chaitanya -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More

Re: [PATCH 2/4] Staging: rtl8192u: r8192U_wx.c Removed all C99 comments

On Tue, Jun 3, 2014 at 11:36 AM, Dan Carpenter wrote: > It always feels much better after the dead code is gone. So patches are still good ? No change required for my side, right ? I just need to wait for Greg's needle to come to this point and pick them up. Best, Chaitanya -- To unsubscribe

Re: [PATCH 2/4] Staging: rtl8192u: r8192U_wx.c Removed all C99 comments

On Tue, Jun 3, 2014 at 11:36 AM, Dan Carpenter dan.carpen...@oracle.com wrote: It always feels much better after the dead code is gone. So patches are still good ? No change required for my side, right ? I just need to wait for Greg's needle to come to this point and pick them up. Best,

Re: [PATCH 2/4] Staging: rtl8192u: r8192U_wx.c Removed all C99 comments

On Tue, Jun 3, 2014 at 2:42 PM, Greg KH gre...@linuxfoundation.org wrote: So keep working on things, these aren't lost, I'll get to them in a few weeks. Ok will do, Chaitanya -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

[PATCH 2/4] Staging: rtl8192u: r8192U_wx.c Removed all C99 comments

To address the error - ERROR: do not use C99 // comments Removed all C99 comments. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 130 +-- 1 file changed, 65 insertions(+), 65 deletions(-) diff --git a/drivers/staging/rtl8192u

[PATCH 1/4] Staging: rtl8192u: r8192U_wx.c removed some dead code

Removed dead code, commented out printks and DMESG. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 675a12d..7dc2a96 100644

[PATCH 4/4] Staging: rtl8192u: r8192U_wx.c Removed some more commented out dead code

Removed dead code from the file. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 09b1e81..ceb06d8

[PATCH 3/4] Staging: rtl8192u: r8192U_wx.c Added a single space for code-style issue

Added a space around '|' to address: ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging

[PATCH 3/4] Staging: rtl8192u: r8192U_wx.c Added a single space for code-style issue

Added a space around '|' to address: ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers

[PATCH 4/4] Staging: rtl8192u: r8192U_wx.c Removed some more commented out dead code

Removed dead code from the file. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 09b1e81

[PATCH 2/4] Staging: rtl8192u: r8192U_wx.c Removed all C99 comments

To address the error - ERROR: do not use C99 // comments Removed all C99 comments. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 130 +-- 1 file changed, 65 insertions(+), 65 deletions(-) diff --git a/drivers/staging

[PATCH 1/4] Staging: rtl8192u: r8192U_wx.c removed some dead code

Removed dead code, commented out printks and DMESG. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 675a12d

Re: [PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

Ok will be quite for till the last ones get accepted. Sorry for the bother ! Chaitanya On Thu, May 29, 2014 at 2:41 PM, Dan Carpenter wrote: > On Thu, May 29, 2014 at 02:00:51PM -0700, Greg KH wrote: >> On Thu, May 29, 2014 at 01:43:01PM -0700, Chaitanya Hazarey wrote: >> >

[PATCH 1/5] Staging: rtl8192u: r8192U_wx.c Fixed one misplaced brace

Fixed a misplaced brace in the r8192_wx_set_scan_type function. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index

[PATCH 3/5] Staging: rtl8192u: r8192U_wx.c Removed all C99 comments

To address the error - ERROR: do not use C99 // comments Removed all C99 comments. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 130 +-- 1 file changed, 65 insertions(+), 65 deletions(-) diff --git a/drivers/staging/rtl8192u

[PATCH 4/5] Staging: rtl8192u: r8192U_wx.c Added a single space for code-style issue

Added a space around '|' to address: ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging

[PATCH 5/5] Staging: rtl8192u: r8192U_wx.c Removed commented dead code

Removed dead code from the file. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 09b1e81..ceb06d8

[PATCH 2/5] Staging: rtl8192u: r8192U_wx.c removed commented dead code

Removed dead code, commented out printks and DMESG. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 675a12d..7dc2a96 100644

Re: [PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

I have 1/5 through to 5/5. Will resend rightaway with slightly changed subjects. Chaitanya On Thu, May 29, 2014 at 1:43 PM, Greg KH wrote: > On Wed, May 28, 2014 at 04:23:47PM -0700, Chaitanya Hazarey wrote: >> Fixed a misplaced brace in a function >> >> Signed-off-

[PATCH 1/5] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

Fixed a misplaced brace in the r8192_wx_set_scan_type function. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index

[PATCH 3/5] Staging: rtl8192u: r8192U_wx.c Removed C99 comments

To address the error - ERROR: do not use C99 // comments Removed all C99 comments. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 130 +-- 1 file changed, 65 insertions(+), 65 deletions(-) diff --git a/drivers/staging/rtl8192u

[PATCH 2/5] Staging: rtl8192u: r8192U_wx.c removed dead code

Removed dead code, commented out printks and DMESG. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 675a12d..7dc2a96 100644

[PATCH 4/5] Staging: rtl8192u: r8192U_wx.c Added a space for code-style issue

Added a space around '|' to address: ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging

[PATCH 5/5] Staging: rtl8192u: r8192U_wx.c Removed commented out dead code

Removed dead code from the file. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 09b1e81..ceb06d8

[PATCH 5/5] Staging: rtl8192u: r8192U_wx.c Removed commented out dead code

Removed dead code from the file. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 09b1e81

[PATCH 2/5] Staging: rtl8192u: r8192U_wx.c removed dead code

Removed dead code, commented out printks and DMESG. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 675a12d

[PATCH 4/5] Staging: rtl8192u: r8192U_wx.c Added a space for code-style issue

Added a space around '|' to address: ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers

[PATCH 3/5] Staging: rtl8192u: r8192U_wx.c Removed C99 comments

To address the error - ERROR: do not use C99 // comments Removed all C99 comments. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 130 +-- 1 file changed, 65 insertions(+), 65 deletions(-) diff --git a/drivers/staging

[PATCH 1/5] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

Fixed a misplaced brace in the r8192_wx_set_scan_type function. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c

Re: [PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

I have 1/5 through to 5/5. Will resend rightaway with slightly changed subjects. Chaitanya On Thu, May 29, 2014 at 1:43 PM, Greg KH gre...@linuxfoundation.org wrote: On Wed, May 28, 2014 at 04:23:47PM -0700, Chaitanya Hazarey wrote: Fixed a misplaced brace in a function Signed-off

[PATCH 5/5] Staging: rtl8192u: r8192U_wx.c Removed commented dead code

Removed dead code from the file. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 09b1e81

[PATCH 2/5] Staging: rtl8192u: r8192U_wx.c removed commented dead code

Removed dead code, commented out printks and DMESG. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 675a12d

[PATCH 4/5] Staging: rtl8192u: r8192U_wx.c Added a single space for code-style issue

Added a space around '|' to address: ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers

[PATCH 1/5] Staging: rtl8192u: r8192U_wx.c Fixed one misplaced brace

Fixed a misplaced brace in the r8192_wx_set_scan_type function. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c

[PATCH 3/5] Staging: rtl8192u: r8192U_wx.c Removed all C99 comments

To address the error - ERROR: do not use C99 // comments Removed all C99 comments. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 130 +-- 1 file changed, 65 insertions(+), 65 deletions(-) diff --git a/drivers/staging

Re: [PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

Ok will be quite for till the last ones get accepted. Sorry for the bother ! Chaitanya On Thu, May 29, 2014 at 2:41 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Thu, May 29, 2014 at 02:00:51PM -0700, Greg KH wrote: On Thu, May 29, 2014 at 01:43:01PM -0700, Chaitanya Hazarey wrote: I

[PATCH 2/5] Staging: rtl8192u: r8192U_wx.c removed dead code

Removed dead code, commented out printks and DMESG. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 675a12d..7dc2a96 100644

[PATCH 1/5] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

Fixed a misplaced brace in the r8192_wx_set_scan_type function. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index

[PATCH 3/5] Staging: rtl8192u: r8192U_wx.c Removed C99 comments

To address the error - ERROR: do not use C99 // comments Removed all C99 comments. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 130 +-- 1 file changed, 65 insertions(+), 65 deletions(-) diff --git a/drivers/staging/rtl8192u

[PATCH 4/5] Staging: rtl8192u: r8192U_wx.c Added a space for code-style issue

Added a space around '|' to address: ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging

[PATCH 5/5] Staging: rtl8192u: r8192U_wx.c Removed commented out dead code

Removed dead code from the file. Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 09b1e81..f2c970e 100644 --- a/drivers

Re: [PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Various Code cleanups and code-style additions

ok will send a new one out soon. Thanks for looking into this and sorry about the repetition. Chaitanya On Wed, May 28, 2014 at 4:36 PM, Greg KH wrote: > On Wed, May 28, 2014 at 04:23:48PM -0700, Chaitanya Hazarey wrote: >> Removed dead code >> Deleted stray newlines &g

Re: [PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed misplaced brace

Please ignore this patch, there seems something wrong with it, will resend with a new subject. Thanks, Chaitanya On Wed, May 28, 2014 at 4:17 PM, Chaitanya Hazarey wrote: > Fixed a misplaced brace in a function > > Signed-off-by: Chaitanya Hazarey > --- > drivers/staging/rtl81

[PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

Fixed a misplaced brace in a function Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 1af7c5d..675a12d 100644

[PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Various Code cleanups and code-style additions

Removed dead code Deleted stray newlines Added newlines for better code readability Addressed the following errors: ERROR: do not use C99 // comments ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 155

Re: [PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Code cleanup and code-style additions

Please ignore this patch, there seems something wrong with it, will resend with a new subject. Thanks, Chaitanya On Wed, May 28, 2014 at 4:17 PM, Chaitanya Hazarey wrote: > Removed dead code > Deleted stray newlines > Added newlines for better code readability > > Addresse

[PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Code cleanup and code-style additions

Removed dead code Deleted stray newlines Added newlines for better code readability Addressed the following errors: ERROR: do not use C99 // comments ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 155

[PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed misplaced brace

Fixed a misplaced brace in a function Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 1af7c5d..675a12d 100644

[PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Added {} braces and newlines

be on the previous line Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 84 1 file changed, 46 insertions(+), 38 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 1ba03ad..1af7c5d 100644

[PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Added missing spaces

r that open parenthesis '(' Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8192U_wx.c | 146 +-- 1 file changed, 73 insertions(+), 73 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index dab71

[PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed misplaced brace

Fixed a misplaced brace in a function Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 1af7c5d..675a12d

[PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Code cleanup and code-style additions

Removed dead code Deleted stray newlines Added newlines for better code readability Addressed the following errors: ERROR: do not use C99 // comments ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 155

Re: [PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Code cleanup and code-style additions

Please ignore this patch, there seems something wrong with it, will resend with a new subject. Thanks, Chaitanya On Wed, May 28, 2014 at 4:17 PM, Chaitanya Hazarey c...@24.io wrote: Removed dead code Deleted stray newlines Added newlines for better code readability Addressed the following

[PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

Fixed a misplaced brace in a function Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 1af7c5d..675a12d

[PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Various Code cleanups and code-style additions

Removed dead code Deleted stray newlines Added newlines for better code readability Addressed the following errors: ERROR: do not use C99 // comments ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 155

Re: [PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Fixed misplaced brace

Please ignore this patch, there seems something wrong with it, will resend with a new subject. Thanks, Chaitanya On Wed, May 28, 2014 at 4:17 PM, Chaitanya Hazarey c...@24.io wrote: Fixed a misplaced brace in a function Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging

Re: [PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Various Code cleanups and code-style additions

ok will send a new one out soon. Thanks for looking into this and sorry about the repetition. Chaitanya On Wed, May 28, 2014 at 4:36 PM, Greg KH gre...@linuxfoundation.org wrote: On Wed, May 28, 2014 at 04:23:48PM -0700, Chaitanya Hazarey wrote: Removed dead code Deleted stray newlines

[PATCH 5/5] Staging: rtl8192u: r8192U_wx.c Removed commented out dead code

Removed dead code from the file. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 09b1e81..f2c970e 100644

[PATCH 1/5] Staging: rtl8192u: r8192U_wx.c Fixed a misplaced brace

Fixed a misplaced brace in the r8192_wx_set_scan_type function. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c

[PATCH 3/5] Staging: rtl8192u: r8192U_wx.c Removed C99 comments

To address the error - ERROR: do not use C99 // comments Removed all C99 comments. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 130 +-- 1 file changed, 65 insertions(+), 65 deletions(-) diff --git a/drivers/staging

[PATCH 4/5] Staging: rtl8192u: r8192U_wx.c Added a space for code-style issue

Added a space around '|' to address: ERROR: need consistent spacing around '|' (ctx:VxW) Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers

[PATCH 2/5] Staging: rtl8192u: r8192U_wx.c removed dead code

Removed dead code, commented out printks and DMESG. Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 675a12d

[PATCH 1/2] Staging: rtl8192u: r8192U_wx.c Added missing spaces

parenthesis '(' Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 146 +-- 1 file changed, 73 insertions(+), 73 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index dab716f

[PATCH 2/2] Staging: rtl8192u: r8192U_wx.c Added {} braces and newlines

be on the previous line Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8192U_wx.c | 84 1 file changed, 46 insertions(+), 38 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_wx.c b/drivers/staging/rtl8192u/r8192U_wx.c index 1ba03ad

[PATCH] Staging: rtl8192u: r8180_93cx6.c Added missing spaces

'(' ERROR: spaces required around that '<' (ctx:VxV) ERROR: spaces required around that '=' (ctx:VxV) ERROR: spaces required around that '==' (ctx:VxV) Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8180_93cx6.c | 46 +- 1 file changed, 23 inserti

Re: [PATCH 1/6] Staging: rtl8192u: r8180_93cx6.c & r8192U_wx.c fix checkpatch.pl errors and warnings

Well, it is what it is, will send the patch the right way. Best, Chaitanya On Tue, May 27, 2014 at 1:39 PM, Dan Carpenter wrote: > On Tue, May 27, 2014 at 01:09:45PM -0700, Chaitanya Hazarey wrote: >> Fixed the following: >> >> ERROR: do not use C99 // comments >>

Re: [PATCH] Staging: rtl8192u: r8180_93cx6.c & r8192U_wx.c fix checkpatch.pl errors and warnings

at 11:56 PM, DaeSeok Youn wrote: > Hi, > > 2014-05-27 14:43 GMT+09:00 Chaitanya Hazarey : >> Fixed the following: >> >> ERROR: do not use C99 // comments >> ERROR: else should follow close brace '}' >> ERROR: need consistent spacing around '*' (ctx:WxV) >&

[PATCH 2/6] Staging: rtl8192u: r8180_93cx6.c & r8192U_wx.c fix checkpatch.pl errors and warnings

ion WARNING: Missing a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/

[PATCH 4/6] Staging: rtl8192u: r8180_93cx6.c & r8192U_wx.c fix checkpatch.pl errors and warnings

ion WARNING: Missing a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/

[PATCH 3/6] Staging: rtl8192u: r8180_93cx6.c & r8192U_wx.c fix checkpatch.pl errors and warnings

ion WARNING: Missing a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/

[PATCH 5/6] Staging: rtl8192u: r8180_93cx6.c & r8192U_wx.c fix checkpatch.pl errors and warnings

ion WARNING: Missing a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/

[PATCH 6/6] Staging: rtl8192u: r8180_93cx6.c & r8192U_wx.c fix checkpatch.pl errors and warnings

ion WARNING: Missing a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/

[PATCH 1/6] Staging: rtl8192u: r8180_93cx6.c & r8192U_wx.c fix checkpatch.pl errors and warnings

VxV) ERROR: space required after that close brace '}' ERROR: spaces required around that '==' (ctx:VxV) ERROR: space required before the open parenthesis '(' Signed-off-by: Chaitanya Hazarey --- drivers/staging/rtl8192u/r8180_93cx6.c | 58 1 file changed, 29 inserti

Re: [PATCH] Staging: rtl8192u: r8180_93cx6.c & r8192U_wx.c fix checkpatch.pl errors and warnings

Hey there, Thanks for looking into this, Will be sending out the modified version soon. Thanks, Chaitanya On Mon, May 26, 2014 at 11:56 PM, DaeSeok Youn wrote: > Hi, > > 2014-05-27 14:43 GMT+09:00 Chaitanya Hazarey : >> Fixed the following: >> >> ERROR: do not u

Re: [PATCH] Staging: rtl8192u: r8180_93cx6.c r8192U_wx.c fix checkpatch.pl errors and warnings

Hey there, Thanks for looking into this, Will be sending out the modified version soon. Thanks, Chaitanya On Mon, May 26, 2014 at 11:56 PM, DaeSeok Youn daeseok.y...@gmail.com wrote: Hi, 2014-05-27 14:43 GMT+09:00 Chaitanya Hazarey c...@24.io: Fixed the following: ERROR: do not use C99

[PATCH 1/6] Staging: rtl8192u: r8180_93cx6.c r8192U_wx.c fix checkpatch.pl errors and warnings

) ERROR: space required after that close brace '}' ERROR: spaces required around that '==' (ctx:VxV) ERROR: space required before the open parenthesis '(' Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u/r8180_93cx6.c | 58 1 file changed, 29

[PATCH 6/6] Staging: rtl8192u: r8180_93cx6.c r8192U_wx.c fix checkpatch.pl errors and warnings

a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u

[PATCH 5/6] Staging: rtl8192u: r8180_93cx6.c r8192U_wx.c fix checkpatch.pl errors and warnings

a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u

[PATCH 3/6] Staging: rtl8192u: r8180_93cx6.c r8192U_wx.c fix checkpatch.pl errors and warnings

a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u

[PATCH 4/6] Staging: rtl8192u: r8180_93cx6.c r8192U_wx.c fix checkpatch.pl errors and warnings

a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u

[PATCH 2/6] Staging: rtl8192u: r8180_93cx6.c r8192U_wx.c fix checkpatch.pl errors and warnings

a blank line after declarations WARNING: please, no spaces at the start of a line WARNING: suspect code indent for conditional statements (16, 16) WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaitanya Hazarey c...@24.io --- drivers/staging/rtl8192u

Re: [PATCH] Staging: rtl8192u: r8180_93cx6.c r8192U_wx.c fix checkpatch.pl errors and warnings

daeseok.y...@gmail.com wrote: Hi, 2014-05-27 14:43 GMT+09:00 Chaitanya Hazarey c...@24.io: Fixed the following: ERROR: do not use C99 // comments ERROR: else should follow close brace '}' ERROR: need consistent spacing around '*' (ctx:WxV) ERROR: need consistent spacing around '|' (ctx:VxW

  1   2   >