[PATCH] staging: android: ion: Remove unnused function ion_handle_buffer

Remove the function ion_handle_buffer since it is not used anywhere. Signed-off-by: Johanna Abrahamsson --- drivers/staging/android/ion/ion.c | 5 - drivers/staging/android/ion/ion_priv.h | 2 -- 2 files changed, 7 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers

[PATCH] staging: android: ion: Remove ret variable in ion_handle_put_nolock

It is not necessary to save the return value of kref_put since it is directly returned. Signed-off-by: Johanna Abrahamsson --- drivers/staging/android/ion/ion.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android

[PATCH v2 2/2] staging: android: ion: Remove valid_handle variable in ion_free_nolock

It is not neccessary to save the value of ion_handle_validate since it is only used once. Signed-off-by: Johanna Abrahamsson --- drivers/staging/android/ion/ion.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging

[PATCH v2 0/2] staging: android: ion: cleanup of the ion_free_nolock function

ee as per request from Laura Abbott Johanna Abrahamsson (2): staging: android: ion: Do not BUG on handle client mismatch staging: android: ion: Remove valid_handle variable in ion_free_nolock drivers/staging/android/ion/ion.c | 8 +--- 1 file changed, 1 insertion(+), 7 dele

[PATCH v2 1/2] staging: android: ion: Do not BUG on handle client mismatch

The ion_free_nolock() function should not BUG on a handle client mismatch. Signed-off-by: Johanna Abrahamsson --- drivers/staging/android/ion/ion.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index 291d5b8..575407d

Re: [PATCH 1/2] staging: android: ion: Do not BUG on handle client mismatch

g and leaking memory is preferrable to crashing the kernel. Best Regards, Johanna Abrahamsson ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH 2/2] staging: android: ion: Remove valid_handle variable in ion_free_nolock

It is not neccessary to save the value of ion_handle_validate since it is only used once. Signed-off-by: Johanna Abrahamsson --- drivers/staging/android/ion/ion.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging

[PATCH 1/2] staging: android: ion: Do not BUG on handle client mismatch

The ion_free() and ion_free_nolock() functions should not BUG on a handle client mismatch. It is not necessary to explicitly check for that mismatch when ion_handle_validate is used. Signed-off-by: Johanna Abrahamsson --- drivers/staging/android/ion/ion.c | 4 1 file changed, 4 deletions

[PATCH 0/2] staging: android: ion: cleanup of the ion_free functions

This is a small cleanup of the ion_free() and ion_free_nolock() methods. They are in a patchset because they depend lightly on each other, as in that patch 2 won't apply due to changes in the context. Since this is my first patchset please let me know if I am doing it wrong. Jo

[PATCH] staging: android: ion: Fix alignment

Alignment should match open parenthesis as per checkpatch.pl. Signed-off-by: Johanna Abrahamsson --- drivers/staging/android/ion/ion.c | 42 +++ 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers

[PATCH] staging: android: sw_sync: checkpatch fixes

This patch fixes the checkpatch.pl check: CHECK: Alignment should match open parenthesis Signed-off-by: Johanna Abrahamsson --- 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