Re: Re: [PATCH] staging: slicoss: replacement of goto statements

2016-06-08 Thread Jaime Arrocha
On Wed, 08 Jun 2016 10:49:47 -0500 Greg KH wrote > On Wed, Jun 08, 2016 at 08:46:06AM -0500, Jaime Arrocha wrote: > > From: Jaime Arrocha > > > > Replaced deprecated goto statements. > > Since when is 'goto' deprecated? >

[PATCH] staging: slicoss: replacement of goto statements

2016-06-08 Thread Jaime Arrocha
From: Jaime Arrocha Replaced deprecated goto statements. Signed-off-by: Jaime Arrocha --- drivers/staging/slicoss/slicoss.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/staging/slicoss/slicoss.c b/drivers/staging/slicoss/slicoss.c index

[PATCH 1/2] staging: slicoss: fix coding style spacing issues

2016-05-29 Thread Jaime Arrocha
From: Jaime Arrocha This patch fixes spacing in variable declarations. This patch series does not depend on each other. Signed-off-by: Jaime Arrocha --- drivers/staging/slicoss/slic.h | 280 - 1 file changed, 140 insertions(+), 140 deletions(-) diff

[PATCH 2/2] staging: slicoss: fix camel case check

2016-05-29 Thread Jaime Arrocha
From: Jaime Arrocha Fixed several checks about camel case use provided by checkpatch.pl. CHECK: Avoid CamelCase Changes verified to work by compiling module. Signed-off-by: Jaime Arrocha --- drivers/staging/slicoss/slic.h| 30 +++ drivers/staging/slicoss/slicoss.c | 50

[PATCH] Staging: android: fix coding style warning

2016-05-22 Thread Jaime Arrocha
From: Jaime Arrocha Fixed checkpatch.pl warning about 'line over 80 characters'. Signed-off-by: Jaime Arrocha --- drivers/staging/android/ion/ion.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android

Re: [PATCH 13/13] Staging: rtl8712: Coding style warning fix for block comment

2015-09-23 Thread Jaime Arrocha
On 09/23/2015 12:31 AM, Sudip Mukherjee wrote: On Wed, Sep 23, 2015 at 02:03:56AM +0530, Punit Vara wrote: This patch is to rtl8712_gp_bitdef.h file that fixes up following warning reported by checkpatch : -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- I

Re: [PATCH] Staging: comedi: Fix coding style issues

2015-09-19 Thread Jaime Arrocha
On 09/19/2015 01:13 PM, punit vara wrote: [PATCH] Staging: comedi: Fix block comment warning [PATCH] Staging: comedi: Coding style warnings fix for block comments ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproj

Re: [PATCH] staging: lustre: lustre: lclient: Removed the else statement

2015-09-17 Thread Jaime Arrocha
On 09/17/2015 11:14 AM, Anjali Menon wrote: Removed the else statement along with some unwanted brackets to fix the following coding style warning detected by checkpatch. WARNING: else is not generally useful after a break or return Signed-off-by: Anjali Menon --- drivers/staging/lustre/lus

[PATCH] staging: gdm724x: Remove test for host endian

2015-05-26 Thread Jaime Arrocha
gdm_endian.c: small changes were done to remove testing for host endianness and in-driver conversion for byte-ordering. The linux/kernel.h functions are used now. gdm_endian.h: removal of code no longer needed with changes in gdm_endian.c. Signed-off-by: Jaime Arrocha --- drivers/staging

Re: [PATCH 1/2] staging: gdm724x: Remove test for host endian

2015-05-26 Thread Jaime Arrocha
On 05/26/2015 10:59 AM, Dan Carpenter wrote: On Tue, May 26, 2015 at 10:29:44AM -0500, Jaime Arrocha wrote: This is the first patch of two. Both patches perform a small clean up done to the section for host endian test. Instead of handling endianness internally, kernel functions were added for

[PATCH 2/2] staging: gdm724x: Remove test for host endian

2015-05-26 Thread Jaime Arrocha
Small code that is no longer needed after first part of the patch is applied. Signed-off-by: Jaime Arrocha --- drivers/staging/gdm724x/gdm_endian.h | 11 --- 1 file changed, 11 deletions(-) diff --git a/drivers/staging/gdm724x/gdm_endian.h b/drivers/staging/gdm724x/gdm_endian.h

[PATCH 1/2] staging: gdm724x: Remove test for host endian

2015-05-26 Thread Jaime Arrocha
-by: Jaime Arrocha --- drivers/staging/gdm724x/gdm_endian.c | 52 +++--- 1 file changed, 23 insertions(+), 29 deletions(-) diff --git a/drivers/staging/gdm724x/gdm_endian.c b/drivers/staging/gdm724x/gdm_endian.c index f6cc90a..609a433 100644 --- a/drivers/staging

Re: Re: [PATCH] staging: gdm724: adding kernel endianness header

2015-05-09 Thread Jaime Arrocha
On Sat, 09 May 2015 05:59:40 -0500 Dan Carpenter wrote > On Fri, May 08, 2015 at 06:29:24PM -0500, Jaime Arrocha wrote: > > >From TODO list: remove test for host endian > > Included header to gather information about host endianness. > > Please

[PATCH] staging: gdm724: adding kernel endianness header

2015-05-08 Thread Jaime Arrocha
>From TODO list: remove test for host endian Included header to gather information about host endianness. Please let me know if the code addition requires corrections to meet standards. Signed-off-by: Jaime Arrocha --- drivers/staging/gdm724x/gdm_endian.c | 22 ++ 1 f

Re: [PATCH] staging: comedi: coding style identation error fix

2015-05-07 Thread Jaime Arrocha
On 05/07/2015 08:21 AM, Sudip Mukherjee wrote: On Thu, May 07, 2015 at 08:01:53AM -0500, Jaime Arrocha wrote: On 05/07/2015 12:34 AM, Sudip Mukherjee wrote: On Wed, May 06, 2015 at 05:13:41PM -0500, Jaime Arrocha wrote: Errors found by checkpatch.pl. ERROR: code indent should use tabs

Re:[PATCH] staging: comedi: drivers: das16m1.c: checkpatch error fix

2015-05-07 Thread Jaime Arrocha
On 05/07/2015 09:27 AM, Jaime Arrocha wrote: -- Forwarded message -- From: Jaime Arrocha Date: Wed, May 6, 2015 at 4:08 PM Subject: [PATCH] staging: comedi: drivers: das16m1.c: checkpatch error fix To: g...@kroah.com Cc: abbo...@mev.co.uk, hswee...@visionengravers.com, de

Re: [PATCH] staging: comedi: coding style identation error fix

2015-05-07 Thread Jaime Arrocha
On 05/07/2015 12:34 AM, Sudip Mukherjee wrote: On Wed, May 06, 2015 at 05:13:41PM -0500, Jaime Arrocha wrote: Errors found by checkpatch.pl. ERROR: code indent should use tabs where possible /drivers/staging/comedi/drivers/das16m1.c:49 /drivers/staging/comedi/drivers/das16m1.c:50 Signed-off

[PATCH] staging: comedi: coding style identation error fix

2015-05-06 Thread Jaime Arrocha
Errors found by checkpatch.pl. ERROR: code indent should use tabs where possible /drivers/staging/comedi/drivers/das16m1.c:49 /drivers/staging/comedi/drivers/das16m1.c:50 Signed-off-by: Jaime Arrocha --- drivers/staging/comedi/drivers/das16m1.c |4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH] staging: comedi: drivers: das16m1.c: checkpatch error fix

2015-05-06 Thread Jaime Arrocha
Errors found by checkpatch.pl. ERROR: code indent should use tabs where possible /drivers/staging/comedi/drivers/das16m1.c:49 /drivers/staging/comedi/drivers/das16m1.c:50 Signed-off-by: Jaime Arrocha --- drivers/staging/comedi/drivers/das16m1.c |4 ++-- 1 file changed, 2 insertions(+), 2

Re: [PATCH] staging: gdm72xx: enclose complex define statement

2015-04-21 Thread Jaime Arrocha
On Tue, 21 Apr 2015 07:40:15 + Greg KH wrote > On Mon, Apr 20, 2015 at 10:11:51PM -0500, Jaime Arrocha wrote: > > This patch fixes the warning found by checkpatch.pl: > > ERROR: Macros with complex values should be enclosed in parentheses > > >

[PATCH] staging: gdm72xx: enclose complex define statement

2015-04-20 Thread Jaime Arrocha
This patch fixes the warning found by checkpatch.pl: ERROR: Macros with complex values should be enclosed in parentheses Signed-off-by: Jaime Arrocha --- drivers/staging/gdm72xx/usb_ids.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/gdm72xx