[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190522124652.GB22148@kroah.com>
Date: Wed, 22 May 2019 14:46:52 +0200
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: Geordan Neukum <gneukum1@...il.com>
Cc: devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org,
Matt Sickler <Matt.Sickler@...tronics.com>,
Dan Carpenter <dan.carpenter@...cle.com>
Subject: Re: [PATCH 3/6] staging: kpc2000: kpc_i2c: newline fixups to meet
linux style guide
On Wed, May 22, 2019 at 12:13:59PM +0000, Geordan Neukum wrote:
> The linux coding style document states:
>
> 1) That braces should not be used where a single single statement
> will do. Therefore all instances of single block statements
> wrapped in braces that do not meet the qualifications of any
> of the exceptions to the rule should be fixed up.
>
> 2) That the declaration of variables local to a given function
> should be immediately followed by a blank newline. Therefore,
> the single instance of this in kpc2000_i2c.c should be fixed
> up.
This really should be 2 different patches, but given that this file is
so messy, I'll take it for now :)
thanks,
greg k-h
Powered by blists - more mailing lists