From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756405AbbGTUJl (ORCPT ); Mon, 20 Jul 2015 16:09:41 -0400 Received: from mail-wg0-f53.google.com ([74.125.82.53]:36487 "EHLO mail-wg0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754061AbbGTUJj (ORCPT ); Mon, 20 Jul 2015 16:09:39 -0400 Date: Mon, 20 Jul 2015 22:09:15 +0200 From: Mario Bambagini To: Joe Perches Cc: mario.bambagini@gmail.com, marek.belisko@gmail.com, gregkh@linuxfoundation.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging: fix to match 80-char rule Message-ID: <20150720200903.GB5270@lynx> References: <1437331311-16973-1-git-send-email-mario.bambagini@gmail.com> <1437334081.24289.21.camel@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1437334081.24289.21.camel@perches.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Jul 19, 2015 at 12:28:01PM -0700, Joe Perches wrote: > On Sun, 2015-07-19 at 20:41 +0200, Mario Bambagini wrote: > > Fixed comment line to match the 80-char rules. > > The line contained 81 characters. > > The file is fully compliant with the code style, now. > > > > Signed-off-by: Mario Bambagini > > --- > > drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c > > index e5cc5be..931e45e 100644 > > --- a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c > > +++ b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c > > @@ -24,7 +24,7 @@ > > License along with this program; if not, write to the > > Free Software Foundation, Inc., 59 Temple Place - > > Suite 330, Boston, MA 02111-1307, USA. > > - -----------------------------------------------------------------------------*/ > > + ----------------------------------------------------------------------------*/ > > Perhaps it'd be nicer to use the normal comment block style > > /* > * > * > */ > > Something like: > --- > drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c | 56 ++++++++++++------------ > 1 file changed, 29 insertions(+), 27 deletions(-) > > diff --git a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c > index e5cc5be..fc7d19f 100644 > --- a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c > +++ b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c > @@ -1,30 +1,32 @@ > -/*--------------------------------------------------------------------------- > - FT1000 driver for Flarion Flash OFDM NIC Device > - > - Copyright (C) 1999 David A. Hinds. All Rights Reserved. > - Copyright (C) 2002 Flarion Technologies, All rights reserved. > - Copyright (C) 2006 Patrik Ostrihon, All rights reserved. > - Copyright (C) 2006 ProWeb Consulting, a.s, All rights reserved. > - > - The initial developer of the original code is David A. Hinds > - . Portions created by David A. Hinds. > - > - This file was modified to support the Flarion Flash OFDM NIC Device > - by Wai Chan (w.chan@flarion.com). > - > - Port for kernel 2.6 created by Patrik Ostrihon (patrik.ostrihon@pwc.sk) > - > - This program is free software; you can redistribute it and/or modify it > - under the terms of the GNU General Public License as published by the Free > - Software Foundation; either version 2 of the License, or (at your option) any > - later version. This program is distributed in the hope that it will be useful, > - but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY > - or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for > - more details. You should have received a copy of the GNU General Public > - License along with this program; if not, write to the > - Free Software Foundation, Inc., 59 Temple Place - > - Suite 330, Boston, MA 02111-1307, USA. > - -----------------------------------------------------------------------------*/ > +/* > + * FT1000 driver for Flarion Flash OFDM NIC Device > + * > + * Copyright (C) 1999 David A. Hinds. All Rights Reserved. > + * Copyright (C) 2002 Flarion Technologies, All rights reserved. > + * Copyright (C) 2006 Patrik Ostrihon, All rights reserved. > + * Copyright (C) 2006 ProWeb Consulting, a.s, All rights reserved. > + * > + * The initial developer of the original code is David A. Hinds > + * . Portions created by David A. Hinds. > + * > + * This file was modified to support the Flarion Flash OFDM NIC Device > + * by Wai Chan (w.chan@flarion.com). > + * > + * Port for kernel 2.6 created by Patrik Ostrihon (patrik.ostrihon@pwc.sk) > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; either version 2 of the License, or (at your > + * option) any later version. > + * > + * This program is distributed in the hope it will be useful, but WITHOUT > + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or > + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for > + * more details. > + * > + * You should have received a copy of the GNU General Public License along with > + * this program. If not, see . > + */ > > #include > #include > > I agree, the normal comment is far better.