On Mon, 9 Apr 2012, Arnd Bergmann wrote:
On Friday 06 April 2012, Grant Likely wrote:
On Wed, 28 Mar 2012 08:19:59 +0000, Arnd Bergmann <[email protected]> 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 <[email protected]>
Acked-by: Jason Cooper <[email protected]>
Signed-off-by: Jamie Lentin <[email protected]>
---
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
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss
