aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/android
diff options
context:
space:
mode:
authorMrinal Pandey <mrinalmni@gmail.com>2020-07-24 18:43:48 +0530
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2020-07-29 17:05:38 +0200
commit72b93c79dbbe261371abb1bf3cf7302cfe31e8d9 (patch)
tree100a998de9750026c648304dad43b4772944cbde /drivers/android
parentdrivers: android: Fix a variable declaration coding style issue (diff)
downloadlinux-dev-72b93c79dbbe261371abb1bf3cf7302cfe31e8d9.tar.xz
linux-dev-72b93c79dbbe261371abb1bf3cf7302cfe31e8d9.zip
drivers: android: Remove the use of else after return
Remove the unnecessary else branch after return statement as suggested by checkpatch. Signed-off-by: Mrinal Pandey <mrinalmni@gmail.com> Link: https://lore.kernel.org/r/20200724131348.haz4ocxcferdcsgn@mrinalpandey Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/android')
-rw-r--r--drivers/android/binder.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/android/binder.c b/drivers/android/binder.c
index 5b310eea9e52..2123655ebb7c 100644
--- a/drivers/android/binder.c
+++ b/drivers/android/binder.c
@@ -1969,9 +1969,8 @@ static void binder_send_failed_reply(struct binder_transaction *t,
binder_thread_dec_tmpref(target_thread);
binder_free_transaction(t);
return;
- } else {
- __release(&target_thread->proc->inner_lock);
}
+ __release(&target_thread->proc->inner_lock);
next = t->from_parent;
binder_debug(BINDER_DEBUG_FAILED_TRANSACTION,