On Fri, Mar 11, 2011 at 11:52:08AM -0800, Dmitry Torokhov wrote: > On Fri, Mar 11, 2011 at 11:42:18AM -0800, Sarah Sharp wrote: > > On Thu, Mar 10, 2011 at 09:52:51PM -0800, Dmitry Torokhov wrote: > > > On Mon, Mar 07, 2011 at 02:11:50PM -0800, Sarah Sharp wrote: > > > > The irq enabling code is going to be refactored into a new function, so > > > > clean up some checkpatch errors before moving it. > > > > > > > > Signed-off-by: Sarah Sharp <sarah.a.sharp@xxxxxxxxxxxxxxx> > > > > > > Looks good to me. > > > > > > Reviewed-by: Dmitry Torokhov <dtor@xxxxxxx> > > > > Your reviewed-by email (dtor@xxxxxxx) is different from the email you > > sent this message from (dmitry.torokhov@xxxxxxxxx). Which is correct? > > > > Both? People use multiple e-mail addresses... You will also see > dtor@xxxxxxxxxx address used by me as well. Just pick the tag as is if > you decide to bother with addng it to the changelog, otherwise feel free > to ignore it. > > I guess I should finally fully migrate to gmail, bt then dtor@xxxxxxx is > much faster to type ;P Ok, no worries. It's just that we're very careful to make sure that the Signed-off-by email addresses match the actual email of the sender. It's a contract according to Documentation/SubmittingPatches (see #12), and we want to make sure the email address is correct. I don't think a Reviewed-by tag is as big deal as a Signed-off-by tag, but I just thought I'd check which email to use. :) Sarah -- 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