Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752547AbbFZSMc (ORCPT ); Fri, 26 Jun 2015 14:12:32 -0400 Received: from mail-pa0-f42.google.com ([209.85.220.42]:34971 "EHLO mail-pa0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751829AbbFZSMY (ORCPT ); Fri, 26 Jun 2015 14:12:24 -0400 Date: Fri, 26 Jun 2015 23:42:16 +0530 From: Sunny Kumar To: Alan Stern Cc: mdharm-usb@one-eyed-alien.net, gregkh@linuxfoundation.org, linux-usb@vger.kernel.org, usb-storage@lists.one-eyed-alien.net, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/1] usb: storage : Remove c99 style commenting Message-ID: <20150626181216.GA4775@sunny-DevBox> References: <1435299263-15919-1-git-send-email-sunny.kumar.roy@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2669 Lines: 66 On Fri, Jun 26, 2015 at 10:08:42AM -0400, Alan Stern wrote: > On Fri, 26 Jun 2015, Sunny Kumar wrote: > > > This patch fixes checkpatch.pl warning c99 style commenting. > > > > Signed-off-by: Sunny Kumar > > --- > > drivers/usb/storage/sddr55.c | 16 ++++++++-------- > > 1 file changed, 8 insertions(+), 8 deletions(-) > > > > diff --git a/drivers/usb/storage/sddr55.c b/drivers/usb/storage/sddr55.c > > index aacedef..54d0a59 100644 > > --- a/drivers/usb/storage/sddr55.c > > +++ b/drivers/usb/storage/sddr55.c > > @@ -209,10 +209,10 @@ static int sddr55_read_data(struct us_data *us, > > unsigned int len, offset; > > struct scatterlist *sg; > > > > - // Since we only read in one block at a time, we have to create > > - // a bounce buffer and move the data a piece at a time between the > > - // bounce buffer and the actual transfer buffer. > > - > > + /* Since we only read in one block at a time, we have to create > > + * a bounce buffer and move the data a piece at a time between the > > + * bounce buffer and the actual transfer buffer. > > + */ > > len = min((unsigned int) sectors, (unsigned int) info->blocksize >> > > info->smallpageshift) * PAGESIZE; > > buffer = kmalloc(len, GFP_NOIO); > > @@ -336,10 +336,10 @@ static int sddr55_write_data(struct us_data *us, > > return USB_STOR_TRANSPORT_FAILED; > > } > > > > - // Since we only write one block at a time, we have to create > > - // a bounce buffer and move the data a piece at a time between the > > - // bounce buffer and the actual transfer buffer. > > - > > + /* Since we only write one block at a time, we have to create > > + * a bounce buffer and move the data a piece at a time between the > > + * bounce buffer and the actual transfer buffer. > > + */ > > len = min((unsigned int) sectors, (unsigned int) info->blocksize >> > > info->smallpageshift) * PAGESIZE; > > buffer = kmalloc(len, GFP_NOIO); > > Why did you fix just these two sites? There are lots of other places > in usb-storage that use C99-style comments: > > $ cd drivers/usb/storage > $ egrep '[^:]//' *.[ch] | wc > 177 1635 9562 > > (The [^:] is to avoid matching things like "http://", and as a result > this misses the four places where a // comment starts at the beginning > of a line.) > > Why not fix all of them? These were C99 muliline comments so thought of fixing .. > > Alan Stern > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/