From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH V3 1/5] ARM: kirkwood: Basic support for DNS-320 and DNS-325 Date: Mon, 9 Apr 2012 14:09:04 +0000 Message-ID: <201204091409.04927.arnd@arndb.de> References: <1332885255-8304-1-git-send-email-jm@lentin.co.uk> <201203280820.00139.arnd@arndb.de> <20120406234313.A040E3E127F@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20120406234313.A040E3E127F@localhost> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Grant Likely Cc: Jamie Lentin , devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Jason List-Id: devicetree@vger.kernel.org On Friday 06 April 2012, Grant Likely wrote: > On Wed, 28 Mar 2012 08:19:59 +0000, Arnd Bergmann wrote: > > On Tuesday 27 March 2012, Jamie Lentin wrote: > > > Add support for the DNS-320 and DNS-325. Describe as much as currently possible > > > in the devicetree files, create a board-dnskw.c for everything else. > > > > > > Use IEEE-compliant "okay", rather than "ok" > > > > > > Acked-by: Arnd Bergmann > > > Acked-by: Jason Cooper > > > Signed-off-by: Jamie Lentin > > > --- > > > > One small note about changeset comments: The part about differences from the previous > > version of the same patch normally goes below the '---' line, in front of the diffstat, > > to prevent it from going into the git changelog. Otherwise people will have no idea > > what it's about. > > Counter point on that. I ask submitters to put the changelog before > the --- because I want it in the commit text. It is actually helpful > when trying to determine exactly which iteration of a patch series got > committed. Fair enough, but I think in this case it was written in a confusing way. I don't mind if a list of review comments are included and more text is usally better in the changeset comment, but it would be nice to give some background, either by prefixing it with something "Addressed review comments from xxx:" or changing it to 'This version uses the IEEE-compliant "okay", rather than "ok"'. Arnd