Re: [PATCH] Staging: android: fix coding style warning

On Sun, May 22, 2016 at 10:52:15PM -0500, Jaime Arrocha wrote: > 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

Re: [PATCH] Staging: android: fix coding style warning

On Sun, May 22, 2016 at 10:52:15PM -0500, Jaime Arrocha wrote: > 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(-)

[PATCH] Staging: android: fix coding style warning

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

[PATCH] Staging: android: fix coding style warning

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/ion/ion.c

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wednesday 01 July 2015 05:59 PM, Frans Klaver wrote: > On Wed, Jul 1, 2015 at 2:22 PM, Sohny Thomas wrote: >> >> >> On Wednesday 01 July 2015 05:37 PM, Frans Klaver wrote: >>> On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas wrote: - removed non-existant issue from TODO file >>> >>>

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wed, Jul 1, 2015 at 2:22 PM, Sohny Thomas wrote: > > > On Wednesday 01 July 2015 05:37 PM, Frans Klaver wrote: >> On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas wrote: >>> - removed non-existant issue from TODO file >> >> s,existant,existent, > Thanks missed that >> >>> kuid_t or uid_t not

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wednesday 01 July 2015 05:37 PM, Frans Klaver wrote: > On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas wrote: >> - removed non-existant issue from TODO file > > s,existant,existent, Thanks missed that > >> kuid_t or uid_t not present in staging/android >> - fixed 80 char limit exceeding line

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wed, Jul 01, 2015 at 02:07:05PM +0200, Frans Klaver wrote: > On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas wrote: > > - removed non-existant issue from TODO file > > s,existant,existent, > > > kuid_t or uid_t not present in staging/android > > - fixed 80 char limit exceeding line > > - a

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas wrote: > - removed non-existant issue from TODO file s,existant,existent, > kuid_t or uid_t not present in staging/android > - fixed 80 char limit exceeding line > - a newline after decelartions as per checkpatch.pl > - fixed an unnecessary typedef

[PATCH] Staging: android: fix coding style and TODO file

- removed non-existant issue from TODO file kuid_t or uid_t not present in staging/android - fixed 80 char limit exceeding line - a newline after decelartions as per checkpatch.pl - fixed an unnecessary typedef as reported by checkpatch.pl --- drivers/staging/android/TODO | 7

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wed, Jul 1, 2015 at 2:22 PM, Sohny Thomas sohnytho...@zoho.com wrote: On Wednesday 01 July 2015 05:37 PM, Frans Klaver wrote: On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas sohnytho...@zoho.com wrote: - removed non-existant issue from TODO file s,existant,existent, Thanks missed that

[PATCH] Staging: android: fix coding style and TODO file

- removed non-existant issue from TODO file kuid_t or uid_t not present in staging/android - fixed 80 char limit exceeding line - a newline after decelartions as per checkpatch.pl - fixed an unnecessary typedef as reported by checkpatch.pl --- drivers/staging/android/TODO | 7

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas sohnytho...@zoho.com wrote: - removed non-existant issue from TODO file s,existant,existent, kuid_t or uid_t not present in staging/android - fixed 80 char limit exceeding line - a newline after decelartions as per checkpatch.pl - fixed an

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wednesday 01 July 2015 05:37 PM, Frans Klaver wrote: On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas sohnytho...@zoho.com wrote: - removed non-existant issue from TODO file s,existant,existent, Thanks missed that kuid_t or uid_t not present in staging/android - fixed 80 char limit

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wed, Jul 01, 2015 at 02:07:05PM +0200, Frans Klaver wrote: On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas sohnytho...@zoho.com wrote: - removed non-existant issue from TODO file s,existant,existent, kuid_t or uid_t not present in staging/android - fixed 80 char limit exceeding line

Re: [PATCH] Staging: android: fix coding style and TODO file

On Wednesday 01 July 2015 05:59 PM, Frans Klaver wrote: On Wed, Jul 1, 2015 at 2:22 PM, Sohny Thomas sohnytho...@zoho.com wrote: On Wednesday 01 July 2015 05:37 PM, Frans Klaver wrote: On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas sohnytho...@zoho.com wrote: - removed non-existant issue

Re: [PATCH] staging: android: fix coding style error (v2)

Hi Dan, Thanks for your valuable input. It's appreciated. On Wed, 14 Jan 2015 12:28:44 +0300 Dan Carpenter wrote: > > lots of Cc:'s [..] > > This block is not needed. This is a whitespace patch not something > controversial where we will need to look at who was CC'd so that we > can get

Re: [PATCH] staging: android: fix coding style error (v2)

Put the v2 in "[PATCH v2]" so that it gets removed and not stored in the permanent changelog. On Wed, Jan 14, 2015 at 10:10:36AM +0100, Patrick Boettcher wrote: > Simple style fix - 80 char limit was exceeded. > > This is second version of the patch. Thanks Joe Perches. Put this line under the

[PATCH] staging: android: fix coding style error (v2)

Simple style fix - 80 char limit was exceeded. This is second version of the patch. Thanks Joe Perches. Context: eudyptula challenge (http://eudyptula-challenge.org/) Signed-off-by: Patrick Boettcher Cc: Greg Kroah-Hartman Cc: Arnd Bergmann Cc: John Stultz Cc: Peter Senna Tschudin Cc:

Re: [PATCH] staging: android: fix coding style error

On Wed, 2015-01-14 at 09:32 +0100, Arnd Bergmann wrote: > On Wednesday 14 January 2015 09:20:42 Patrick Boettcher wrote: > > diff --git a/drivers/staging/android/sync_debug.c > > b/drivers/staging/android/sync_debug.c [] > > @@ -96,7 +96,8 @@ static void sync_print_pt(struct seq_file *s, struct

Re: [PATCH] staging: android: fix coding style error

On Wednesday 14 January 2015 09:20:42 Patrick Boettcher wrote: > diff --git a/drivers/staging/android/sync_debug.c > b/drivers/staging/android/sync_debug.c > index 1532a86..9a2aaf8 100644 > --- a/drivers/staging/android/sync_debug.c > +++ b/drivers/staging/android/sync_debug.c > @@ -96,7 +96,8 @@

[PATCH] staging: android: fix coding style error

Simple style fix - 80 char limit was exceeded. Context: eudyptula challenge (http://eudyptula-challenge.org/) Signed-off-by: Patrick Boettcher Cc: Greg Kroah-Hartman Cc: Arnd Bergmann Cc: John Stultz Cc: Peter Senna Tschudin Cc: Daniel Vetter Cc: Maarten Lankhorst Cc: Tapasweni Pathak

[PATCH] staging: android: fix coding style error

Simple style fix - 80 char limit was exceeded. Context: eudyptula challenge (http://eudyptula-challenge.org/) Signed-off-by: Patrick Boettcher patrick.boettc...@posteo.de Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Arnd Bergmann a...@arndb.de Cc: John Stultz john.stu...@linaro.org Cc:

Re: [PATCH] staging: android: fix coding style error

On Wednesday 14 January 2015 09:20:42 Patrick Boettcher wrote: diff --git a/drivers/staging/android/sync_debug.c b/drivers/staging/android/sync_debug.c index 1532a86..9a2aaf8 100644 --- a/drivers/staging/android/sync_debug.c +++ b/drivers/staging/android/sync_debug.c @@ -96,7 +96,8 @@

Re: [PATCH] staging: android: fix coding style error

On Wed, 2015-01-14 at 09:32 +0100, Arnd Bergmann wrote: On Wednesday 14 January 2015 09:20:42 Patrick Boettcher wrote: diff --git a/drivers/staging/android/sync_debug.c b/drivers/staging/android/sync_debug.c [] @@ -96,7 +96,8 @@ static void sync_print_pt(struct seq_file *s, struct

[PATCH] staging: android: fix coding style error (v2)

Simple style fix - 80 char limit was exceeded. This is second version of the patch. Thanks Joe Perches. Context: eudyptula challenge (http://eudyptula-challenge.org/) Signed-off-by: Patrick Boettcher patrick.boettc...@posteo.de Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Arnd

Re: [PATCH] staging: android: fix coding style error (v2)

Put the v2 in [PATCH v2] so that it gets removed and not stored in the permanent changelog. On Wed, Jan 14, 2015 at 10:10:36AM +0100, Patrick Boettcher wrote: Simple style fix - 80 char limit was exceeded. This is second version of the patch. Thanks Joe Perches. Put this line under the --

Re: [PATCH] staging: android: fix coding style error (v2)

Hi Dan, Thanks for your valuable input. It's appreciated. On Wed, 14 Jan 2015 12:28:44 +0300 Dan Carpenter dan.carpen...@oracle.com wrote: lots of Cc:'s [..] This block is not needed. This is a whitespace patch not something controversial where we will need to look at who was CC'd so that

[PATCH] Staging: android: fix coding style

- add line after declaration Signed-off-by: Sorin Facaoaru --- drivers/staging/android/sync.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c index e7b2e02..0d37495 100644 --- a/drivers/staging/android/sync.c +++

[PATCH] Staging: android: fix coding style

- add line after declaration Signed-off-by: Sorin Facaoaru --- drivers/staging/android/sw_sync.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/android/sw_sync.c b/drivers/staging/android/sw_sync.c index a76db3f..863d4b1 100644 --- a/drivers/staging/android/sw_sync.c +++

[PATCH] Staging: android: fix coding style

- add line after declaration Signed-off-by: Sorin Facaoaru w...@sorin.cc --- drivers/staging/android/sync.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c index e7b2e02..0d37495 100644 --- a/drivers/staging/android/sync.c +++

[PATCH] Staging: android: fix coding style

- add line after declaration Signed-off-by: Sorin Facaoaru w...@sorin.cc --- drivers/staging/android/sw_sync.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/android/sw_sync.c b/drivers/staging/android/sw_sync.c index a76db3f..863d4b1 100644 ---

Re: [PATCH] Staging: android: fix coding style issue in timed_gpio.c

On Wed, May 07, 2014 at 07:23:33AM +0100, John Church wrote: > This patch fixes a coding style issue. It modifies a single line > that was over 80 characters long. > > Thanks, > John > Signed-off-by: John Church This patch isn't sent in the correct way. Use git send-email or read

Re: [PATCH] Staging: android: fix coding style issue in timed_gpio.c

On Wed, May 07, 2014 at 07:23:33AM +0100, John Church wrote: This patch fixes a coding style issue. It modifies a single line that was over 80 characters long. Thanks, John Signed-off-by: John Church j.chu...@ymail.com This patch isn't sent in the correct way. Use git send-email or read

Re: [PATCH] Staging: android: fix coding style issue for Eudyptula Challenge in timed_gpio.c

On Wed, May 07, 2014 at 12:44:31AM +0100, John Church wrote: > > Signed-off-by: John Church What coding style issue? You need to be specific in your body of your email, and put a blank line after the signed-off-by and before the patch, otherwise the tools can complain. And drop the

Re: [PATCH] Staging: android: fix coding style issue for Eudyptula Challenge in timed_gpio.c

On Wed, May 07, 2014 at 12:44:31AM +0100, John Church wrote: Signed-off-by: John Church j.chu...@ymail.com What coding style issue? You need to be specific in your body of your email, and put a blank line after the signed-off-by and before the patch, otherwise the tools can complain. And

Re: [PATCH] staging: android: fix coding style

On Thu, Mar 13, 2014 at 10:44:55AM +0100, mo...@2600fr.org wrote: > From: mourn > > Signed-off-by: mourn As said in the thread for your other thread, this patch can't be accepted unless you use your real name, sorry. greg k-h -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH] staging: android: fix coding style

On Thu, Mar 13, 2014 at 03:13:04PM +0100, mourn wrote: > > > The changes look good, however you should be a bit more specific about > > what the > > fixes are, and that you used checkpatch.pl to find them (I assume). > > > > Yes, you are right. I can edit the commit log. > > > Also,

Re: [PATCH] staging: android: fix coding style

On Thu, Mar 13, 2014 at 03:13:04PM +0100, mourn wrote: The changes look good, however you should be a bit more specific about what the fixes are, and that you used checkpatch.pl to find them (I assume). Yes, you are right. I can edit the commit log. Also, Signed-off-by: should

Re: [PATCH] staging: android: fix coding style

On Thu, Mar 13, 2014 at 10:44:55AM +0100, mo...@2600fr.org wrote: From: mourn mo...@2600fr.org Signed-off-by: mourn mo...@2600fr.org As said in the thread for your other thread, this patch can't be accepted unless you use your real name, sorry. greg k-h -- To unsubscribe from this list: send

Re: [PATCH] staging: android: fix coding style

> The changes look good, however you should be a bit more specific about > what the > fixes are, and that you used checkpatch.pl to find them (I assume). > Yes, you are right. I can edit the commit log. > Also, Signed-off-by: should state your full name, which I'm also assuming > isn't >

Re: [PATCH] staging: android: fix coding style

I'll try that again... On Thu, Mar 13, 2014 at 01:50:47PM +, Mark Einon wrote: > On Thu, Mar 13, 2014 at 10:44:55AM +0100, mo...@2600fr.org wrote: > > From: mourn > > > > Signed-off-by: mourn > > --- > > drivers/staging/android/uapi/ion.h | 12 ++-- > > 1 file changed, 6

Re: [PATCH] staging: android: fix coding style

On Thu, Mar 13, 2014 at 10:44:55AM +0100, mo...@2600fr.org wrote: > From: mourn > > Signed-off-by: mourn > --- > drivers/staging/android/uapi/ion.h | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) The changes look good, however you should be a bit more specific about

[PATCH] staging: android: fix coding style

From: mourn Signed-off-by: mourn --- drivers/staging/android/uapi/ion.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/android/uapi/ion.h b/drivers/staging/android/uapi/ion.h index f09e7c1..6aa4956 100644 ---

[PATCH] staging: android: fix coding style

From: mourn mo...@2600fr.org Signed-off-by: mourn mo...@2600fr.org --- drivers/staging/android/uapi/ion.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/android/uapi/ion.h b/drivers/staging/android/uapi/ion.h index f09e7c1..6aa4956 100644 ---

Re: [PATCH] staging: android: fix coding style

On Thu, Mar 13, 2014 at 10:44:55AM +0100, mo...@2600fr.org wrote: From: mourn mo...@2600fr.org Signed-off-by: mourn mo...@2600fr.org --- drivers/staging/android/uapi/ion.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) The changes look good, however you should be a

Re: [PATCH] staging: android: fix coding style

I'll try that again... On Thu, Mar 13, 2014 at 01:50:47PM +, Mark Einon wrote: On Thu, Mar 13, 2014 at 10:44:55AM +0100, mo...@2600fr.org wrote: From: mourn mo...@2600fr.org Signed-off-by: mourn mo...@2600fr.org --- drivers/staging/android/uapi/ion.h | 12 ++-- 1 file

Re: [PATCH] staging: android: fix coding style

The changes look good, however you should be a bit more specific about what the fixes are, and that you used checkpatch.pl to find them (I assume). Yes, you are right. I can edit the commit log. Also, Signed-off-by: should state your full name, which I'm also assuming isn't 'mourn'.