On Tue, Mar 14, 2017 at 09:10:18AM +1100, Tobin C. Harding wrote: > On Mon, Mar 13, 2017 at 06:54:06PM +1100, Tobin C. Harding wrote: > > Simple patch set. Clears 29 checkpatch errors/warnings. > > > > Patch 01 does various fixes, all whitespace only changes. > > > > Is this ok putting multiple checkpatch types in one patch if they are > > all whitespace? Please advise, I would like to make review as easy as > > possible. > > > > Patch 02 adds parentheses to arithmetic macro definitions (24 > > identical cases). > > > > Tobin C. Harding (2): > > staging: ks7010: fix checkpatch whitespace warns > > staging: ks7010: add parentheses to complex macro > > > > drivers/staging/ks7010/ks_wlan_ioctl.h | 68 +++++++++++++++++----------------- > > 1 file changed, 34 insertions(+), 34 deletions(-) > > > > -- > > 2.7.4 > > > > Drop this patch set please. I see Matthew Giassa started work on this > file before me. I don't see those patches still in my queue. Is there anything wrong with this series that would prevent it from being acceptable? Multiple people working on the same file isn't a reason for one to stop... thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
