On Sat, Jan 22, 2022 at 11:08 AM Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
>
> 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.
Done: The patch is at
https://lore.kernel.org/all/20220122213444.745152-1-gwendal@xxxxxxxxxxxx/.
Given the title change, it is not automatically connected to this
patch. I should have added a --in-reply-to argument to my git
send-mail command, will do better next time.
Gwendal.
> >
> >
> > > 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
