diff options
| author | Dmitry Voytik <voytikd@gmail.com> | 2014-09-08 18:16:34 +0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-09-08 13:29:01 -0700 |
| commit | 395262a9e69042b82293f8b6bcd7a48b2fd215de (patch) | |
| tree | 4103a783e58dc93228a465e501c816df18ed15c2 | |
| parent | 0232a42cbe3fe18c2dda833cb8e97253240a9479 (diff) | |
| download | op-kernel-dev-395262a9e69042b82293f8b6bcd7a48b2fd215de.zip op-kernel-dev-395262a9e69042b82293f8b6bcd7a48b2fd215de.tar.gz | |
staging: binder: fix coding style issues
Fix coding style issues:
* put braces in all if-else branches;
* limit the length of changed lines to 80 columns.
checkpatch.pl warning count reduces by 3.
Signed-off-by: Dmitry Voytik <voytikd@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/android/binder.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index 38cfb08..c69c40d 100644 --- a/drivers/staging/android/binder.c +++ b/drivers/staging/android/binder.c @@ -2199,12 +2199,16 @@ retry: struct binder_work *w; struct binder_transaction *t = NULL; - if (!list_empty(&thread->todo)) - w = list_first_entry(&thread->todo, struct binder_work, entry); - else if (!list_empty(&proc->todo) && wait_for_proc_work) - w = list_first_entry(&proc->todo, struct binder_work, entry); - else { - if (ptr - buffer == 4 && !(thread->looper & BINDER_LOOPER_STATE_NEED_RETURN)) /* no data added */ + if (!list_empty(&thread->todo)) { + w = list_first_entry(&thread->todo, struct binder_work, + entry); + } else if (!list_empty(&proc->todo) && wait_for_proc_work) { + w = list_first_entry(&proc->todo, struct binder_work, + entry); + } else { + /* no data added */ + if (ptr - buffer == 4 && + !(thread->looper & BINDER_LOOPER_STATE_NEED_RETURN)) goto retry; break; } |
