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
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
> >
>
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
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
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
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
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
>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
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
-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
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
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
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
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
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
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
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
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?
>
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
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
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
21 matches
Mail list logo
