On Fri, Nov 19, 2010 at 01:06:24PM +0800, Yuan, Hang wrote: > It might be because we use Gerrit internally. Will check it. > > Regards, > Henry > > > -----Original Message----- > > From: Greg KH [mailto:greg@xxxxxxxxx] > > Sent: Friday, November 19, 2010 1:05 PM > > To: Luo, Yifei > > Cc: linux-usb; Yuan, Hang > > Subject: Re: [PATCH v2] usb: gadget: langwell_udc: add usb test mode support > > > > On Fri, Nov 19, 2010 at 01:00:46PM +0800, Luo Andy wrote: > > > On Fri, Nov 19, 2010 at 04:24:39AM +0800, Greg KH wrote: > > > > On Thu, Nov 18, 2010 at 06:33:29PM +0800, Luo Andy wrote: > > > > > On Thu, Nov 18, 2010 at 05:28:16AM +0800, Greg KH wrote: > > > > > > On Tue, Nov 16, 2010 at 01:24:07PM +0800, andy.luo wrote: > > > > > > > This patch adds test mode support for Langwell gadget driver. > > > > > > > > > > > > > > Change-Id: Icaf402c0afd497265c53a81af6f691fb81181ed4 > > > > > > > > > > > > That is not a valid field for a kernel patch, please remove it. Also, > > > > > > where in the world did it come from? > > > > > > > > > > oops, will pay more attention about it. > > > > > > > > You didn't answer the question :( > > > > > > Do you mean by adding patch author like this "Form: Yuan Han > > <yuan.hang@xxxxxxxxx>"? > > > > No, the only question I asked above, "where in the world did that > > Change-Id field come from?" > > > > I've been seeing it a lot recently, all from people who are brand new to > > submitting kernel patches, and I want to know what is adding it and how > > it is adding it so we can keep it from happening. > > > > So again, how did that get added? It is gererated by gerrit hook script of ".git/hooks/commit-msg". Thanks, Andy -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
