[PATCH 4/4] drivers: android: binder: Fix compiler warning

2013-03-11 Thread Mirsal Ennaime
Fix an instance of -Wdeclaration-after-statement Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index 18a83e2..c833b53 100644 --- a

[PATCH 2/4] drivers: android: binder: Fix code style

2013-03-11 Thread Mirsal Ennaime
* Use tabs * Remove a couple of "80-columns" checkpatch warnings * Separate code paths with empty lines for readability Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c | 22 +- 1 file changed, 17 insertions(+), 5 deletions(-) diff --git

[PATCH 1/4] drivers: android: binder: Move the node release code to a separate function

2013-03-11 Thread Mirsal Ennaime
in 80 columns. Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c | 80 +++--- 1 file changed, 49 insertions(+), 31 deletions(-) diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index 24456a0..11b3f7b 100644 --- a

[PATCH 3/4] drivers: android: binder: Remove excessive indentation

2013-03-11 Thread Mirsal Ennaime
Remove one level of indentation from the binder proc page release code by using slightly different control semantics. This is a cosmetic patch which removes checkpatch "80-columns" warnings Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c | 24 +

[PATCH 0/4] Cosmetic changes to the android binder proc release code

2013-03-11 Thread Mirsal Ennaime
Hello, These are cleanup patches related to the binder_deferred_release function in the android binder staging driver which improve readability while removing checkpatch and compiler warnings. drivers/staging/android/binder.c | 121 +++ 1 file changed, 7

[PATCH v2 1/3] drivers: android: binder: Move the node release code to a separate function

2013-03-11 Thread Mirsal Ennaime
columns. Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c | 76 +++--- 1 file changed, 46 insertions(+), 30 deletions(-) diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index 24456a0..43f823d 100644 --- a/drivers

[PATCH v2 2/3] drivers: android: binder: Fix code style

2013-03-11 Thread Mirsal Ennaime
* Use tabs * Remove a few "80-columns" checkpatch warnings * Separate code paths with empty lines for readability Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c | 42 +- 1 file changed, 28 insertions(+), 14 deletions(-) di

[PATCH v2 3/3] drivers: android: binder: Remove excessive indentation

2013-03-11 Thread Mirsal Ennaime
Remove one level of indentation from the binder proc page release code by using slightly different control semantics. This is a cosmetic patch which removes checkpatch "80-columns" warnings Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c | 25 ++

[PATCH v2 0/3] Cosmetic changes to the android binder proc release code

2013-03-11 Thread Mirsal Ennaime
Hello, These are cleanup patches related to the binder_deferred_release function in the android binder staging driver which improve readability while removing checkpatch and compiler warnings. drivers/staging/android/binder.c | 137 +--- 1 file changed, 85 insertions(+),

[PATCH v3 2/4] drivers: android: binder: Fix code style in binder_deferred_release

2013-03-12 Thread Mirsal Ennaime
* Use tabs where applicable * Remove a few "80-columns" checkpatch warnings * Separate code paths with empty lines for readability Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c | 32 ++-- 1 file changed, 22 insertions(+), 10

[PATCH v3 1/4] drivers: android: binder: Move the node release code to a separate function

2013-03-12 Thread Mirsal Ennaime
columns. Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c | 76 +++--- 1 file changed, 46 insertions(+), 30 deletions(-) diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index 24456a0..9180a5b 100644 --- a/drivers

[PATCH v3 4/4] drivers: android: binder: Use __func__ in debug messages

2013-03-12 Thread Mirsal Ennaime
Debug messages sent in binder_deferred_release begin with "binder_release:" which is a bit misleading as binder_release is not directly part of the call stack. Use __func__ instead for debug messages in binder_deferred_release. Signed-off-by: Mirsal Ennaime --- drivers/staging/androi

[PATCH v3 3/4] drivers: android: binder: Remove excessive indentation

2013-03-12 Thread Mirsal Ennaime
Remove one level of indentation from the binder proc page release code by using slightly different control semantics. Signed-off-by: Mirsal Ennaime --- drivers/staging/android/binder.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/drivers/staging

[PATCH v3 0/4] Cosmetic changes to the android binder proc release code

2013-03-12 Thread Mirsal Ennaime
Hello, These are cleanup patches related to the binder_deferred_release function in the android binder staging driver which improve readability while removing checkpatch and compiler warnings. drivers/staging/android/binder.c | 138 +--- 1 file changed, 84 insertions(+),