From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756119Ab0ECTLg (ORCPT ); Mon, 3 May 2010 15:11:36 -0400 Received: from kroah.org ([198.145.64.141]:49331 "EHLO coco.kroah.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755115Ab0ECTLe (ORCPT ); Mon, 3 May 2010 15:11:34 -0400 Date: Mon, 3 May 2010 11:45:33 -0700 From: Greg KH To: Lars Lindley Cc: Pekka Enberg , greg KH , Pavel Machek , devel@driverdev.osuosl.org, kernel list Subject: Re: [PATCH] staging: winbond: localpara.h whitespace and indentation fixes. Message-ID: <20100503184533.GA10778@kroah.com> References: <1268727716-20422-1-git-send-email-lindley@coyote.org> <4BA25348.1080308@cs.helsinki.fi> <4BA2B0AB.4070607@coyote.org> <20100428225010.GC24604@kroah.com> <4BDD3900.1000705@coyote.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4BDD3900.1000705@coyote.org> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, May 02, 2010 at 10:34:08AM +0200, Lars Lindley wrote: > On 2010-04-29 00:50, Greg KH wrote: > > On Fri, Mar 19, 2010 at 12:00:59AM +0100, Lars Lindley wrote: > >> On 2010-03-18 17:22, Pekka Enberg wrote: > >>> Lars Lindley wrote: > >>>> +/* refer Bitmap2RateValue table */ > >>>> +#define LOCAL_ALL_SUPPORTED_RATES_BITMAP 0x130c1a66 /* > >>>> + * the bitmap value > >>>> + * of all the H/W > >>>> + * supported rates: > >>>> + * 1, 2, 5.5, 11, > >>>> + * 6, 9, 12, 18, > >>>> + * 24, 36, 48, 54 > >>>> + */ > >>> > >>> The comments look funny. You might want to move them above the #defines. > >>> > >>>> + u8 region; /* real region setting of the device */ > >>>> + u8 Reserved_1[2]; > >>>> + > >>>> +/* power-save variables */ > >>> > >>> I think this wants to be moved one indentation level to the right. > >>> > >> Ok here's a patch that applies over the old one to fix this.. > >> Hope it looks all right.. :) > > > > Can you merge the two of them together and resend? > > > > thanks, > > > > greg k-h > > > > I have somehow messed up my git tree so I can't get rebase to work but > here's the diff for localpara.h and that combines the two patches.. > Hope it works for you Greg. Yes it did, but it lacked a change log entry showing what the patch did, as well as a signed-off-by: line allowing me to apply the patch :( Care to resend it again with that required information? thanks, greg k-h