On Wed, Jan 15, 2020 at 12:33:41PM +0200, Felipe Balbi wrote: > > Hi, > > Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> writes: > > On Wed, Jan 15, 2020 at 10:28:13AM +0100, Greg Kroah-Hartman wrote: > >> On Wed, Jan 15, 2020 at 11:14:28AM +0200, Felipe Balbi wrote: > >> > > >> > Hi Greg, > >> > > >> > here's my pull request for v5.6 merge window. If you want anything to be > >> > changed, let me know. > >> > >> Did you send this twice? Any difference? > >> > > > > I did a pull, and get this problem when checking the patches: > > > > Commit: 6a6ae4e8e926 ("usb: gadget: f_ncm: Use atomic_t to track in-flight request") > > Fixes tag: Fixes: 40d133d7f5426 ("usb: gadget: f_ncm: convert to new function interface > > Has these problem(s): > > - Subject has leading but no trailing parentheses > > - Subject has leading but no trailing quotes > > Oh, I missed those. > > > I'll go fix it up, but that will break your tree if you try to merge as > > Thanks for fixing. I'll reset my tree to yours. > > > I will have to rebase :( > > no worries. > > Out of curiosity, what are you using to flag those? Is checkpatch doing > that now? No, I'm using some scripts originally based on code that Stephen uses for linux-next (you would have got the same report from him when he merges in your tree.) I published them on the workflows@vger.k.o mailing list a while back if you are interested in them. thanks, greg k-h