Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755852AbbEVH32 (ORCPT ); Fri, 22 May 2015 03:29:28 -0400 Received: from mail.kmu-office.ch ([178.209.48.109]:56714 "EHLO mail.kmu-office.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751161AbbEVH30 (ORCPT ); Fri, 22 May 2015 03:29:26 -0400 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Date: Fri, 22 May 2015 09:27:38 +0200 From: Stefan Agner To: =?UTF-8?Q?Uwe_Kleine-K=C3=B6nig?= Cc: Joachim Eastwood , Arnd Bergmann , Olof Johansson , Maxime Coquelin , Russell King - ARM Linux , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH soc] ARM: use =?UTF-8?Q?ARM=5FSINGLE=5FARMV=37M=20for?= =?UTF-8?Q?=20ARMv=37-M=20platforms?= In-Reply-To: <20150521190400.GD24769@pengutronix.de> References: <1432161344-1930-1-git-send-email-stefan@agner.ch> <20150521190400.GD24769@pengutronix.de> Message-ID: <1890f941dfa59138442da3e80a6b492a@agner.ch> User-Agent: Roundcube Webmail/1.1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2067 Lines: 47 On 2015-05-21 21:04, Uwe Kleine-König wrote: > On Thu, May 21, 2015 at 07:00:02PM +0200, Joachim Eastwood wrote: >> Hi Stefan, >> >> On 21 May 2015 at 00:35, Stefan Agner wrote: >> > Use the new config symbol ARM_SINGLE_ARMV7M which groups config >> > symbols used by modern ARMv7-M platforms. This allows supporting >> > multiple ARMv7-M platforms in one kernel image. However, a common >> > kernel image requires the combined platforms to share the same >> > main memory layout to be bootable. >> > >> > Acked-by: Uwe Kleine-König >> > Signed-off-by: Stefan Agner >> >> You should have your sign-off on the top and not Uwe's ack. > I don't agree. IMHO the most sensible thing is to sign off (i.e. put > your sob line below) all the things that come from you. So it's > > Signed-off-by: author > Acked-by: someone > Signed-off-by: forwarder > Signed-off-by: maintainer > > if someone already acked before forwarder sent out the mail. If the ack > is between forwarder and maintainer it was the latter who added the Ack. > I'm not sure it's formalized this way, but like that it makes most sense > to me and I seem to recall having read somewhere that the footers are > supposed to tell something about the order of people involved. > > So Stefan did it exactly as i would have done it. Hm, never give much thoughts about that order, its quite development process driven here: I use git format-patch with --signoff. When I send out v1, and get a Ack or Review, I add that to my commit. Hence, when I send v2 of the patch, my sob line ends up below the Ack (that is what happened in that case). If its the last revision, and the patch gets picked up, the forwarder/maintainer will add the Ack and then it would land after my sob... -- Stefan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/