* [PATCH 0/3] Staging: comedi: Checkpatch cleanups in ni_mio_common.c
@ 2014-01-11 4:07 Chase Southwood
2014-01-11 5:49 ` Chase Southwood
0 siblings, 1 reply; 3+ messages in thread
From: Chase Southwood @ 2014-01-11 4:07 UTC (permalink / raw)
To: linux-kernel@vger.kernel.org
This patch series fixes several warnings reported by checkpatch.pl in ni_mio_common.c of the comedi driver.
Among the issues fixed:
*Many unnecessary braces have been removed.
*Improper indentation has been corrected.
*Extra whitespace before semicolons has been removed.
*Extra whitespace after function pointer name has been removed.
Several checkpatch warnings still remain (mainly 80 character+ line lengths), but no new warnings have been introduced, and no functionality changes have been made.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/3] Staging: comedi: Checkpatch cleanups in ni_mio_common.c
2014-01-11 4:07 Chase Southwood
@ 2014-01-11 5:49 ` Chase Southwood
0 siblings, 0 replies; 3+ messages in thread
From: Chase Southwood @ 2014-01-11 5:49 UTC (permalink / raw)
To: linux-kernel
Oops, left out some of my cover letter. Here is the amended patchset
cover letter.
On Fri, Jan 10, 2014 at 10:07 PM, Chase Southwood
<chase.southwood@yahoo.com> wrote:
>
> This patch series fixes several warnings reported by checkpatch.pl in ni_mio_common.c of the comedi driver.
>
> Among the issues fixed:
> *Many unnecessary braces have been removed.
> *Improper indentation has been corrected.
> *Extra whitespace before semicolons has been removed.
> *Extra whitespace after function pointer name has been removed.
>
> Several checkpatch warnings still remain (mainly 80 character+ line lengths), but no new warnings have been introduced, and no functionality changes have been made.
>
Chase Southwood (3):
Staging: comedi: fix numerous brace coding style issues in ni_mio_common.c.
Staging: comedi: fix indentation coding style issue in ni_mio_common.c.
Staging: comedi: fix extra whitespace style issues in ni_mio_common.c.
drivers/staging/comedi/drivers/ni_mio_common.c | 138 +++++++++----------------
1 file changed, 50 insertions(+), 88 deletions(-)
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/3] Staging: comedi: Checkpatch cleanups in ni_mio_common.c
[not found] <1389412583.62399.YahooMailNeo@web164002.mail.gq1.yahoo.com>
@ 2014-01-13 10:07 ` Ian Abbott
0 siblings, 0 replies; 3+ messages in thread
From: Ian Abbott @ 2014-01-13 10:07 UTC (permalink / raw)
To: Chase Southwood, gregkh@linuxfoundation.org
Cc: Ian Abbott, hsweeten@visionengravers.com,
devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
On 2014-01-11 03:56, Chase Southwood wrote:
> This patch series fixes several warnings reported by checkpatch.pl in
> ni_mio_common.c of the comedi driver.
>
> Among the issues fixed:
> *Many unnecessary braces have been removed.
> *Improper indentation has been corrected.
> *Extra whitespace before semicolons has been removed.
> *Extra whitespace after function pointer name has been removed.
>
> Several checkpatch warnings still remain (mainly 80 character+ line
> lengths), but no new warnings have been introduced, and no functionality
> changes have been made.
All three patches are nice and harmless.
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
--
-=( Ian Abbott @ MEV Ltd. E-mail: <abbotti@mev.co.uk> )=-
-=( Tel: +44 (0)161 477 1898 FAX: +44 (0)161 718 3587 )=-
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-01-13 10:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1389412583.62399.YahooMailNeo@web164002.mail.gq1.yahoo.com>
2014-01-13 10:07 ` [PATCH 0/3] Staging: comedi: Checkpatch cleanups in ni_mio_common.c Ian Abbott
2014-01-11 4:07 Chase Southwood
2014-01-11 5:49 ` Chase Southwood
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox