On Thu, 2017-03-30 at 01:48 -0700, Nicholas A. Bellinger wrote: > Just for future reference, the flow of these tags should reflect the > history of the patch. Eg: > > Reviewed-by: First reviewer <foo@xxxxxxx> > Tested-by: First tester <foo2@xxxxxxxx> > Reviewed-by: Second reviewer <foo3@xxxxxxxx> > Signed-off-by: Patch Author <you@xxxxxxxxxxxxxx> > > and then once the subsystem maintainer merges it into his tree, they add > their own: > > Signed-off-by: Subsystem Maintainer <superturboarray@xxxxxxxxxxxxxxxx> Hi Nic, I agree that these tags should reflect the history of the patch. I think that means that the patch author should be mentioned first, Reviewed-by / Tested-by tags next and the subsystem maintainer sign-off last. At least, that's how most other maintainers do it. Bart.
