On Fri, Apr 5, 2024 at 6:06 AM Michael Ellerman <m...@ellerman.id.au> wrote: > Andy Shevchenko <andy.shevche...@gmail.com> writes: > > On Thu, Apr 4, 2024 at 2:57 AM Finn Thain <fth...@linux-m68k.org> wrote: > >> On Thu, 4 Apr 2024, Andy Shevchenko wrote: > > > >> > > Cc: Benjamin Herrenschmidt <b...@kernel.crashing.org> > >> > > Cc: Michael Ellerman <m...@ellerman.id.au> > >> > > Cc: Nicholas Piggin <npig...@gmail.com> > >> > > Cc: Christophe Leroy <christophe.le...@csgroup.eu> > >> > > Cc: "Aneesh Kumar K.V" <aneesh.ku...@kernel.org> > >> > > Cc: "Naveen N. Rao" <naveen.n....@linux.ibm.com> > >> > > Cc: linux-m...@lists.linux-m68k.org > >> > > >> > Second, please move these Cc to be after the '---' line > >> > >> I thought they were placed above the line for audit (and signing) > >> purposes. > > > > I didn't get this, sorry. > > > >> There are thousands of Cc lines in the mainline commit messages > >> since v6.8. > > > > Having thousands of mistaken cases does not prove it's a good thing to > > follow. I answered Jiri why it's better the way I suggested. > > > >> > > Link: https://github.com/vivier/qemu-m68k/issues/44 > >> > > Link: https://lore.kernel.org/all/1078874617.9746.36.camel@gaston/ > >> > > >> > Missed Fixes tag? > >> > >> Would this be ok: Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2") > >> I have to ask because some reviewers do not like to see a Fixes tag cite > >> that commit. > > > > Yes, or you even may dig into the history.git from history group (see > > git.kernel.org) for the real first patch that brought it. > > > >> > > Signed-off-by: Finn Thain <fth...@linux-m68k.org> > >> > > --- > >> > (here is a good location for Cc:) > >> > >> Documentation/process/submitting-patches.rst indicats that it should be > >> above the "---" separator together with Acked-by etc. Has this convention > >> changed recently? > > The docs don't really say where to put the Cc: tags, although they are > mentioned along with other tags which clearly are intended to go above > the separator.
He-h... Documentation needs constant updates too, for one reason or another. This is normal process and in particular Cc (rather long) lists needs to be reconsidered. > > I see, I will prepare a patch to discuss this aspect. > > FYI there was a discussion about this several years ago, where at least > some maintainers agreed that Cc: tags don't add much value and are > better placed below the --- separator. Thanks, I'll definitely read this. But I'm 100% sure the environment aspect and mobile device screen sizes were not discussed there. > Thread starts here: > https://lore.kernel.org/all/87y31eov1l....@concordia.ellerman.id.au/ -- With Best Regards, Andy Shevchenko
