[PATCH V3 1/5] ARM: kirkwood: Basic support for DNS-320 and DNS-325

On Mon, 9 Apr 2012, Arnd Bergmann wrote:

On Friday 06 April 2012, Grant Likely wrote:

quoted

On Wed, 28 Mar 2012 08:19:59 +0000, Arnd Bergmann [off-list ref] wrote:

quoted

On Tuesday 27 March 2012, Jamie Lentin wrote:

quoted

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 <arnd@arndb.de>
Acked-by: Jason Cooper <redacted>
Signed-off-by: Jamie Lentin <redacted>
---

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"'.

Re-reading, it really isn't obvious I meant "Use IEEE-compliant..." as a 
changelog of sorts. Maybe this would have been a better commit message?

---------

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.

Changes since last submission (V2):-

* Use IEEE-compliant "okay", rather than "ok" (adressing comments from 
Scott Wood)

---------

	Arnd

-- 
Jamie Lentin