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 <f...@bar.com> > Tested-by: First tester <f...@bar2.com> > Reviewed-by: Second reviewer <f...@bar3.com> > Signed-off-by: Patch Author <y...@yourdomain.com> > > and then once the subsystem maintainer merges it into his tree, they add > their own: > > Signed-off-by: Subsystem Maintainer <superturboar...@linux-domain.org>
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.
