On Fri, 2014-03-07 at 17:41 +0300, Dan Carpenter wrote: > On Fri, Mar 07, 2014 at 08:18:44AM -0600, Romer, Benjamin M wrote: > > > > On Thu, 2014-03-06 at 12:01 -0800, gregkh@xxxxxxxxxxxxxxxxxxx wrote: > > > On Thu, Mar 06, 2014 at 10:01:04AM -0600, Romer, Benjamin M wrote: > > > > This patch updates the MAINTAINERS file to add the maintainer for the > > > > Unisys s-Par driver set. > > > > > > > > Signed-off-by: Ken Cox <jkc@xxxxxxxxxx> > > > > signed-off-by: Ben Romer <sparmaintainer@xxxxxxxxxx> > > > > > > Please capitalize things properly. > > > > > > And your From: address does not match your "Signed-off-by:" line. Why > > > are you trying to use an "alias" as a maintainer or a sign-off? We want > > > a "real" address / name please, especially as it's obvious you have one :) > > > > > > thanks, > > > > > > greg k-h > > > > The email address I listed is a group address - I'll change the contents of the file > > so they more accurately describe how we've set things up. > > The file is fine (I think). > > Signed-off-by is like signing a legal document. You should sign on your > own behalf and not for a group of people. And if the Signed-off-by > doesn't match your From then how do we know it's really you? > > regards, > dan carpenter > That's what I mean though, the file isn't quite accurate. sparmaintainer@xxxxxxxxxx is a group email address for which I'm the primary contact, but there are other developers and managers with access to it. I'll list myself and one other person by email address for handling patches, and put the group address as the mailing list entry. Also, the file should say "Supported" instead of "Maintained", so that wasn't quite right either. :) _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel