[PATCH 02/20] staging: rtl8192u: r8192U_core: fix consistent spacing code style error

Raphaël Beamonte raphael.beamonte at gmail.com
Tue Aug 18 06:02:49 UTC 2015


2015-08-18 1:44 GMT-04:00 Sudip Mukherjee <sudipm.mukherjee at gmail.com>:
> I think that will be better, since you are sending the patch for
> consistent spacing.

Actually, going through the bunch of patches, it seems that the
whitespace neatening one is taking care of that. That one patch was
mostly about taking care of the checkpatch "consistent spacing"
warning.
Two solutions here:
(1) I can merge both patches and make a general whitespace neatening
one, taking care of that too.
(2) I can clarify the commit messages of each patches to explain the
aim of each.
Of course, in (1) too I'd take care of explaining the whole point of
the commit. What do you think would be the best approach at this
point?

> And some of your patches are not having any commit message. Greg will
> not accept such patches. So while you are sending take care of that
> also.

Ok!

Thanks again!
Raphaël


More information about the devel mailing list