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(-)
Someon
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
i
replace comparison "obj" to NULL with "!obj"
Signed-off-by: Trung Thanh Le
---
drivers/staging/android/sw_sync.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/android/sw_sync.c
b/drivers/staging/android/sw_sync.c
index c90838d..29b5c35 100644
--- a/dr
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 annoye
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 -
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: Dani
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
>
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 @@
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
Cc
- 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
+++ b/drivers/stagi
- 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
+++
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 Documentation
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
--- linux-next/drivers/staging/android/timed_gpio.c.orig2014-05-07
00:20:50.429090320 +0100
+++ linux-next/drivers/staging/android/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 Eudyptula
Signed-off-by: John Church
--- linux-next/drivers/staging/android/timed_gpio.c.orig2014-05-07
00:20:50.429090320 +0100
+++ linux-next/drivers/staging/android/timed_gpio.c 2014-05-07
00:22:09.204795052 +0100
@@ -92,7 +92,7 @@ static int timed_gpio_probe(struct platf
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
___
devel mailing list
d
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-b
> 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'
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 insertio
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 whic
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
--- a/drivers/staging/android/uapi/io
21 matches
Mail list logo
