Re: [PATCH RESEND 1/5] staging: sm750fb: Use memset_io instead of memset

On Wed, Mar 18, 2015 at 06:36:07PM +0530, Sudip Mukherjee wrote:

On Wed, Mar 18, 2015 at 02:25:09PM +0300, Dan Carpenter wrote:

quoted

On Wed, Mar 18, 2015 at 11:12:20AM +0000, Lorenzo Stoakes wrote:

quoted

On 18 March 2015 at 10:50, Vitaly Kuznetsov [off-list ref] wrote:

If it's a white space thing on the same line then it's generally ok to
fix it.  The "one thing per patch" is meant to make patches easier to
review.  If it's a trivial thing and it doesn't make it harder to review
then we are reasonable people.

but Greg K-H has explisitely mentiond not to do so.
I did just that and fixed few whitespace things in the patch to fix the
build failure.

https://lkml.org/lkml/2015/3/10/685

You were making random white space changes and not on the same line.  It
was hard to review because you had to count how many u32 arguments there
were (a million) and really look at it to see what the compile warning
was.  There was no compile warning in the end.  Very annoying.

regards,
dan carpenter