[PATCH] Staging: comedi: s626: Remove unnecessary cast on void pointer

2016-01-20 Thread Lucas Tanure
The conversion from void pointer to any other pointer type is guaranteed by the C programming language. Signed-off-by: Lucas Tanure --- drivers/staging/comedi/drivers/s626.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/comedi/drivers/s626.c b

[PATCH] Staging: rtl8712: Remove unnecessary cast on void pointer

2016-01-19 Thread Lucas Tanure
The conversion from void pointer to any other pointer type is guaranteed by the C programming language. The semantic patch used to find this: @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T *)x)->f | - (T *) e ) Signed-off-by: Lucas Tan

Re: Eudyptula Challenge (Task 10)

2014-07-28 Thread Lucas Tanure
Hi Christoph, What kind of things that you think a newbie can fix ? Thanks -- Lucas Tanure +55 (19) 988176559 On Mon, Jul 28, 2014 at 9:53 AM, Christoph Hellwig wrote: > Can you folks please stop this challenge Bullshit? More checkpatch > fixes is not something we'll need at all.

[PATCH] staging: android: Clean up else statement from sync_fence_poll()

2014-07-14 Thread Lucas Tanure
Kernel coding style. Remove useless else statement after return. Signed-off-by: Lucas Tanure --- drivers/staging/android/sync.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c index c9a0c2c..e7b2e02 100644

Re: [PATCH v4] staging: android: Clean up else statement from binder_send_failed_reply

2014-07-14 Thread Lucas Tanure
Hi, This patch got accepted ? Thanks -- Lucas Tanure +55 (19) 988176559 On Sun, Jul 13, 2014 at 9:31 PM, Lucas Tanure wrote: > Kernel coding style. Remove useless else statement after return. > Changes from v1 and v2: Fix warning for mixed declarations and code. > Declaration o

Re: [PATCH v3] staging: android: Clean up else statement from binder_send_failed_reply

2014-07-13 Thread Lucas Tanure
Ok. I will do it. 4th could be the right one =p. Thanks for your time Cheers -- Lucas Tanure +55 (19) 988176559 On Sun, Jul 13, 2014 at 9:13 PM, Greg Kroah-Hartman wrote: > On Sun, Jul 13, 2014 at 08:25:13PM -0300, Lucas Tanure wrote: >> Kernel coding style. Remove useless else

[PATCH v4] staging: android: Clean up else statement from binder_send_failed_reply

2014-07-13 Thread Lucas Tanure
Kernel coding style. Remove useless else statement after return. Changes from v1 and v2: Fix warning for mixed declarations and code. Declaration of "struct binder_transaction *next" made outside of while. Changes from v3: Removed initialization to NULL for next variable. Signed-off

[PATCH v3] staging: android: Clean up else statement from binder_send_failed_reply

2014-07-13 Thread Lucas Tanure
Kernel coding style. Remove useless else statement after return. Changes from v1 and v2: Fix warning for mixed declarations and code. Declaration of "struct binder_transaction *next" made outside of while, and initialized with NULL. Signed-off-by: Lucas Tanure --- drivers/stagi

Re: [RESEND PATCH] staging: android: Clean up binder_send_failed_reply

2014-07-13 Thread Lucas Tanure
Sorry, I didn't know. I will fix. Thanks -- Lucas Tanure +55 (19) 988176559 On Sun, Jul 13, 2014 at 4:07 PM, Greg Kroah-Hartman wrote: > On Sun, Jul 13, 2014 at 11:49:29AM -0300, Lucas Tanure wrote: >> Kernel coding style. Remove useless else statement after return. >> >

Re: [RESEND PATCH] staging: android: Clean up binder_send_failed_reply

2014-07-13 Thread Lucas Tanure
I missed on trailing whitespace in first patch. Didn't saw a ERROR: trailing whitespace in the first patch. Sorry, Thanks -- Lucas Tanure +55 (19) 988176559 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverprojec

[RESEND PATCH] staging: android: Clean up binder_send_failed_reply

2014-07-13 Thread Lucas Tanure
Kernel coding style. Remove useless else statement after return. Signed-off-by: Lucas Tanure --- drivers/staging/android/binder.c | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c

[PATCH] staging: android: Clean up binder_send_failed_reply

2014-07-13 Thread Lucas Tanure
Kernel coding style. Remove useless else statement after return. Signed-off-by: Lucas Tanure --- drivers/staging/android/binder.c | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c

[RESEND PATCH] staging: android: Clean up binder_buffer_size()

2014-07-12 Thread Lucas Tanure
Kernel coding style. Remove useless else statement after return. Signed-off-by: Lucas Tanure --- drivers/staging/android/binder.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index cfe4bc8..1a0966b

[PATCH] Remove useless else after a return statement

2014-07-12 Thread Lucas Tanure
Run Checkpatch and fixed the first warning. Signed-off-by: Lucas Tanure --- drivers/staging/android/binder.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index cfe4bc8..1a0966b 100644 --- a/drivers