Re: [PATCH RESEND] staging: android: sync_debug.c: remove unnecessary braces

From: Greg KH
Date: Sun Feb 07 2016 - 17:56:48 EST


On Tue, Jan 19, 2016 at 12:29:41PM +0100, Robin Krahl wrote:
> Remove unnecessary braces {} around a single statement within a for loop.
> Indicated by checkpatch.
>
> Signed-off-by: Robin Krahl <robin.krahl@xxxxxxxxx>
> ---
> drivers/staging/android/sync_debug.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/staging/android/sync_debug.c b/drivers/staging/android/sync_debug.c
> index f45d13c..02a1649 100644
> --- a/drivers/staging/android/sync_debug.c
> +++ b/drivers/staging/android/sync_debug.c
> @@ -158,9 +158,8 @@ static void sync_print_fence(struct seq_file *s, struct sync_fence *fence)
> seq_printf(s, "[%p] %s: %s\n", fence, fence->name,
> sync_status_str(atomic_read(&fence->status)));
>
> - for (i = 0; i < fence->num_fences; ++i) {
> + for (i = 0; i < fence->num_fences; ++i)
> sync_print_pt(s, fence->cbs[i].sync_pt, true);
> - }
>
> spin_lock_irqsave(&fence->wq.lock, flags);
> list_for_each_entry(pos, &fence->wq.task_list, task_list) {
> --
> 2.5.0
>


Someone sent a fixed up patch before this one :(