All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v6 1/7] powerpc: Extract EPAPR_MAGIC constants into processor.h
Date: Tue, 30 Oct 2012 11:16:36 +0100	[thread overview]
Message-ID: <508FA904.4070402@denx.de> (raw)
In-Reply-To: <20121030100424.0C81A2001B0@gemini.denx.de>

Hi Wolfgang,

On 10/30/2012 11:04 AM, Wolfgang Denk wrote:
>> By extracting these defines into a header, they can be re-used by other
>> C sources as well. This will be done by the SPL framework OS boot
>> support.
>>
>> Signed-off-by: Stefan Roese <sr@denx.de>
>> ---
>> Changes in v6:
>> - Fix compile warning: release.S:354:0: warning: "EPAPR_MAGIC" redefined
> 
> Please re-read
> http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions
> 
> You are supposed to provide a _history_ of changes here, but instead
> you describe only the latest change.  This is not how it's suppoosed
> to be done.

As you know this patch is part of a patch-series. And this is the first
time that this patch has a change. So this summary covers the complete
history for this patch.

> Also, your submission includes neither any "In-reply-to:" nor any
> "References:" header, i. e. there is no way to match it to any
> previous mail thread.

Yes, I should have done this. Sorry.

>  This is very bad.  How are we supposed to know
> what you are actually talking about, or where we would find any of the
> previous patches or the other 6 patches of this series?

In this version of the patch series, I only made this small change to
this patch 1/7. I wanted to spare the list a resending of the complete
patchset for such a small change.

So what is the recommended way to do this? Is it really
recommended/required to repost the complete patch series upon a small
change in only one patch? No problem, I can do this. patman makes it
very easy. :)

Should I repost the complete series again?

Thanks,
Stefan

  reply	other threads:[~2012-10-30 10:16 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-30  9:45 [U-Boot] [PATCH v6 1/7] powerpc: Extract EPAPR_MAGIC constants into processor.h Stefan Roese
2012-10-30 10:04 ` Wolfgang Denk
2012-10-30 10:16   ` Stefan Roese [this message]
2012-10-30 11:05     ` Wolfgang Denk
2012-10-30 13:33       ` Stefan Roese
2012-10-30 15:43         ` Simon Glass
2012-10-30 16:44         ` Tom Rini
2012-11-07 22:43 ` Anatolij Gustschin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=508FA904.4070402@denx.de \
    --to=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.