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? thanks, greg k-h -- 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
