On Sat, 22 Jan 2022 20:15:57 +0200 Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > On Sat, Jan 22, 2022 at 7:23 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > > On Tue, 18 Jan 2022 13:25:04 -0800 > > Gwendal Grignou <gwendal@xxxxxxxxxxxx> wrote: > > ... > > > > Fixes: f75095753 ("iio:proximity:sx9360: Add sx9360 support") > > This is part of the tag block so should appear. > > > > fixes tag should be here. > > > Signed-off-by: Jongpil Jung <jongpil19.jung@xxxxxxxxxxx> > > > Signed-off-by: Gwendal Grignou <gwendal@xxxxxxxxxxxx> > > ...and the submitter's SoB must be last (according to the documentation). > Hi Andy, If it's a handling chain, rather that about co development etc then I'd expect it to be in this order to indicate that Gwendal was on the route to upstream. "Any further SoBs (Signed-off-by:'s) following the author's SoB are from people handling and transporting the patch, but were not involved in its development. SoB chains should reflect the **real** route a patch took as it was propagated to the maintainers and ultimately to Linus, with the first SoB entry signalling primary authorship of a single author." Jonathan