public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 2/4] patman: Add meaningful statements instead of blah blah
@ 2012-04-25 15:45 Vikram Narayanan
  2012-04-26 18:01 ` Simon Glass
  0 siblings, 1 reply; 2+ messages in thread
From: Vikram Narayanan @ 2012-04-25 15:45 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Vikram Narayanan <vikram186@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
---
 tools/patman/README |    9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/tools/patman/README b/tools/patman/README
index d9820ab..d98f081 100644
--- a/tools/patman/README
+++ b/tools/patman/README
@@ -132,16 +132,15 @@ Series-prefix: prefix
 
 Cover-letter:
 This is the patch set title
-blah blah
-more blah blah
+This patch set fixes the errors when CONFIG_xxx is enabled.
+Tested on xyz board.
 END
 	Sets the cover letter contents for the series. The first line
 	will become the subject of the cover letter
 
 Series-notes:
-blah blah
-blah blah
-more blah blah
+Sorry that I couldn't find time to reply for the comments posted on the v1 of my
+patch. This patch superseeds v1. Please comment.
 END
 	Sets some notes for the patch series, which you don't want in
 	the commit messages, but do want to send, The notes are joined
-- 
1.7.4.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [U-Boot] [PATCH 2/4] patman: Add meaningful statements instead of blah blah
  2012-04-25 15:45 [U-Boot] [PATCH 2/4] patman: Add meaningful statements instead of blah blah Vikram Narayanan
@ 2012-04-26 18:01 ` Simon Glass
  0 siblings, 0 replies; 2+ messages in thread
From: Simon Glass @ 2012-04-26 18:01 UTC (permalink / raw)
  To: u-boot

Hi Vikram,

On Thu, Apr 26, 2012 at 3:45 AM, Vikram Narayanan <vikram186@gmail.com>wrote:

> Signed-off-by: Vikram Narayanan <vikram186@gmail.com>
> Cc: Simon Glass <sjg@chromium.org>
>

It would be better if you added a commit message for each of these patches,
even a brief one. Otherwise:

Acked-by: Simon Glass <sjg@chromium.org>


> ---
>  tools/patman/README |    9 ++++-----
>  1 files changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/tools/patman/README b/tools/patman/README
> index d9820ab..d98f081 100644
> --- a/tools/patman/README
> +++ b/tools/patman/README
> @@ -132,16 +132,15 @@ Series-prefix: prefix
>
>  Cover-letter:
>  This is the patch set title
> -blah blah
> -more blah blah
> +This patch set fixes the errors when CONFIG_xxx is enabled.
> +Tested on xyz board.
>  END
>        Sets the cover letter contents for the series. The first line
>        will become the subject of the cover letter
>
>  Series-notes:
> -blah blah
> -blah blah
> -more blah blah
> +Sorry that I couldn't find time to reply for the comments posted on the
> v1 of my
> +patch. This patch superseeds v1. Please comment.
>  END
>        Sets some notes for the patch series, which you don't want in
>        the commit messages, but do want to send, The notes are joined
> --
> 1.7.4.1
>
>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-04-26 18:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-04-25 15:45 [U-Boot] [PATCH 2/4] patman: Add meaningful statements instead of blah blah Vikram Narayanan
2012-04-26 18:01 ` Simon Glass

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox