Return-path: Received: from mail-qk0-f194.google.com ([209.85.220.194]:36012 "EHLO mail-qk0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933207AbdLRMbO (ORCPT ); Mon, 18 Dec 2017 07:31:14 -0500 MIME-Version: 1.0 In-Reply-To: <1513556924.31581.51.camel@perches.com> References: <1513556924.31581.51.camel@perches.com> From: Andy Shevchenko Date: Mon, 18 Dec 2017 14:31:12 +0200 Message-ID: (sfid-20171218_133240_408233_6DA36803) Subject: Re: [trivial PATCH] treewide: Align function definition open/close braces To: Joe Perches Cc: Jiri Kosina , Linus Torvalds , "linux-kernel@vger.kernel.org" , "linux-acpi@vger.kernel.org" , amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Linux Media Mailing List , MPT-FusionLinux.pdl@broadcom.com, linux-scsi , netdev , "open list:TI WILINK WIRELES..." , acpi4asus-user , Platform Driver , linux-rtc@vger.kernel.org, Linux FS Devel , ocfs2-devel@oss.oracle.com, linux-xfs@vger.kernel.org, linux-audit@redhat.com, ALSA Development Mailing List , linuxppc-dev Content-Type: text/plain; charset="UTF-8" Sender: linux-wireless-owner@vger.kernel.org List-ID: On Mon, Dec 18, 2017 at 2:28 AM, Joe Perches wrote: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work > properly for these modified functions. > > Miscellanea: > > o Remove extra trailing ; and blank line from xfs_agf_verify > > Signed-off-by: Joe Perches > drivers/platform/x86/eeepc-laptop.c | 2 +- > diff --git a/drivers/platform/x86/eeepc-laptop.c b/drivers/platform/x86/eeepc-laptop.c > index 5a681962899c..4c38904a8a32 100644 > --- a/drivers/platform/x86/eeepc-laptop.c > +++ b/drivers/platform/x86/eeepc-laptop.c > @@ -492,7 +492,7 @@ static void eeepc_platform_exit(struct eeepc_laptop *eeepc) > * potentially bad time, such as a timer interrupt. > */ > static void tpd_led_update(struct work_struct *work) > - { > +{ > struct eeepc_laptop *eeepc; > > eeepc = container_of(work, struct eeepc_laptop, tpd_led_work); > diff --git a/drivers/rtc/rtc-ab-b5ze-s3.c b/drivers/rtc/rtc-ab-b5ze-s3.c > index a319bf1e49de..ef5c16dfabfa 100644 Acked-by: Andy Shevchenko for PDx86 changes. -- With Best Regards, Andy Shevchenko