lkml.org 
[lkml]   [2015]   [Jan]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v3 2/4] staging: rtl8192u: fix whitespace
On Wed, Dec 31, 2014 at 09:40:13PM -0800, Joe Perches wrote:
> On Thu, 2015-01-01 at 10:17 +0530, Sudip Mukherjee wrote:
> > On Wed, Dec 31, 2014 at 09:33:43PM +0000, Lorenzo Stoakes wrote:
> > > This patch fixes warnings/errors raised by checkpatch.pl relating to whitespace
> > > in r8192U_dm.c and additionally it removes inconsistent whitespace
> > > throughout.
> > >
> > > Additionally some 0x00... padding has been added for alignment to the
> > > edca_setting_* variables.
> >
> > i may be wrong, but looks like you are doing many different things in the patch, not just whitespace errors.
> > marked a few, please see inline -
>
> Don't get carried away with patch type separation
> and please remember to trim your replies to just
> the areas you comment on.
sorry for not trimming that. I noticed the big size while sending, but at that time the damage has already been done.
but is it ok if we mention one thing in the commit log and do other things also? shouldn't we atleast mention that in the commit?

sorry again.
sudip
>
>


\
 
 \ /
  Last update: 2015-01-01 07:41    [W:0.048 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site