On Tue, Mar 14, 2017 at 06:18:48AM +0800, Greg Kroah-Hartman wrote: > 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? In fact, they look fine to me, now queued up. thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
