[PATCH 0/2] staging: ks7010: fix checkpatch ks_wlan_ioctl.h

Greg Kroah-Hartman gregkh at linuxfoundation.org
Mon Mar 13 22:18:48 UTC 2017


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


More information about the devel mailing list