From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755094AbbICBBm (ORCPT ); Wed, 2 Sep 2015 21:01:42 -0400 Received: from out4-smtp.messagingengine.com ([66.111.4.28]:56149 "EHLO out4-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751897AbbICBBk (ORCPT ); Wed, 2 Sep 2015 21:01:40 -0400 X-Sasl-enc: 8OUjxUpYCaBurGeesKII5v4lt9iLpte+uPkPJWp5iYwv 1441242099 Date: Wed, 2 Sep 2015 18:01:38 -0700 From: Greg KH To: lysanovsergey@gmail.com Cc: forest@alittletooquiet.net, linux-kernel@vger.kernel.org Subject: Re: [PATCH] drivers:staging:vt6655: Fix coding style issues Message-ID: <20150903010138.GA25982@kroah.com> References: <1440759104-20680-1-git-send-email-lysanovsergey@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1440759104-20680-1-git-send-email-lysanovsergey@gmail.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Aug 28, 2015 at 01:51:44PM +0300, lysanovsergey@gmail.com wrote: > From: Sergey Lysanov > > This patch fixes the following issues reported by checkpatch.pl: > - code indent should use tabs where possible > - space prohibited before that ',' > > Signed-off-by: Sergey Lysanov > --- > drivers/staging/vt6655/rxtx.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c > index 5875d65..64957de 100644 > --- a/drivers/staging/vt6655/rxtx.c > +++ b/drivers/staging/vt6655/rxtx.c > @@ -64,8 +64,8 @@ > /*--------------------- Static Functions --------------------------*/ > > /*--------------------- Static Definitions -------------------------*/ > -#define CRITICAL_PACKET_LEN 256 /* if packet size < 256 -> in-direct send > - packet size >= 256 -> direct send */ > +#define CRITICAL_PACKET_LEN 256 /* if packet size < 256 -> in-direct send > + packet size >= 256 -> direct send */ > > static const unsigned short wTimeStampOff[2][MAX_RATE] = { > {384, 288, 226, 209, 54, 43, 37, 31, 28, 25, 24, 23}, /* Long Preamble */ > @@ -78,7 +78,7 @@ static const unsigned short wFB_Opt0[2][5] = { > }; > static const unsigned short wFB_Opt1[2][5] = { > {RATE_12M, RATE_18M, RATE_24M, RATE_24M, RATE_36M}, /* fallback_rate0 */ > - {RATE_6M , RATE_6M, RATE_12M, RATE_12M, RATE_18M}, /* fallback_rate1 */ > + {RATE_6M, RATE_6M, RATE_12M, RATE_12M, RATE_18M}, /* fallback_rate1 */ The goal was to keep these aligned, which you broke the formatting for :(