From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755387AbcBGW46 (ORCPT ); Sun, 7 Feb 2016 17:56:58 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:59368 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754917AbcBGW45 (ORCPT ); Sun, 7 Feb 2016 17:56:57 -0500 Date: Sun, 7 Feb 2016 14:56:56 -0800 From: Greg KH To: Tapan Prakash T Cc: devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging: android: sync_debug: Fixed braces related coding style issue Message-ID: <20160207225656.GB8463@kroah.com> References: <1453630892-5684-1-git-send-email-tapanprakasht@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1453630892-5684-1-git-send-email-tapanprakasht@gmail.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Jan 24, 2016 at 03:51:32PM +0530, Tapan Prakash T wrote: > This patch fixes checkpatch.pl warning in file sync_debug > WARNING: braces {} are not necessary for single statement blocks > > Signed-off-by: Tapan Prakash T > --- > 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) { > -- > 1.9.1 Someone sent this fix in before you, sorry.