Remove the unnecessary else branch after return statement as suggested by
checkpatch.

Signed-off-by: Mrinal Pandey <mrinal...@gmail.com>
---
 drivers/android/binder.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/android/binder.c b/drivers/android/binder.c
index f50c5f182bb5..5fdf982ec83b 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,
-- 
2.25.1

Attachment: signature.asc
Description: PGP signature

Reply via email to