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
> 
> 
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to