From: "Matthias Weißer" <weisserm@arcor.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Do not copy to same address
Date: Tue, 12 Apr 2011 09:49:07 +0200 [thread overview]
Message-ID: <4DA403F3.2090304@arcor.de> (raw)
In-Reply-To: <4DA3FEF9.5030805@aribaud.net>
Am 12.04.2011 09:27, schrieb Albert ARIBAUD:
> Hi Matthias,
>
> Le 12/04/2011 09:13, Matthias Wei?er a ?crit :
>> Am 12.04.2011 09:06, schrieb Albert ARIBAUD:
>>> Hi Matthias,
>>>
>>> Le 12/04/2011 08:58, Matthias Weisser a ?crit :
>>>> In some cases (e.g. bootm with a elf payload) there is a in place
>>>> copy of
>>>> data to the same address. Catching this saves some ms while booting.
>>>>
>>>> Signed-off-by: Matthias Weisser<weisserm@arcor.de>
>>>> ---
>>>> lib/string.c | 9 +++++++++
>>>> 1 files changed, 9 insertions(+), 0 deletions(-)
>>>
>>> I believe that is a V2 patch, right? Please tag it as V2 in the subject
>>> line, and add patch history below the commit message delimiter ('---' ).
>>
>> No, it is a replacement for http://patchwork.ozlabs.org/patch/79447/
>> which picks up a suggestion from Wolfgang.
>
> So it is a V3, not V2, but still you must tag it so that readers who see
> the previous patch can relate it to the 'replacement' -- yes, even if
> the files touched by V2 are different.
Well, as the patch is only slightly related to my original one I thought
it is better to start a new patch as I had to change the subject also.
The only relation between them would be the reference in the mail
header. Maybe Wolfgang can bring some light into this situation.
What would be the right way to post this patch? And what would be the
right way to post a patch doing exactly the same to the ARM optimized
version of memcpy?
Sorry for all that administrative questions.
Regards,
Matthias
next prev parent reply other threads:[~2011-04-12 7:49 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-12 6:58 [U-Boot] [PATCH] Do not copy to same address Matthias Weisser
2011-04-12 7:05 ` Mike Frysinger
2011-04-12 7:16 ` Matthias Weißer
2011-04-12 7:06 ` Albert ARIBAUD
2011-04-12 7:13 ` Matthias Weißer
2011-04-12 7:27 ` Albert ARIBAUD
2011-04-12 7:49 ` Matthias Weißer [this message]
2011-04-14 6:28 ` Albert ARIBAUD
2011-05-23 9:03 ` [U-Boot] [PATCH V2] memcpy/memmove: " Matthias Weisser
2011-05-23 21:07 ` Alexander Holler
2011-05-23 21:55 ` Wolfgang Denk
2011-05-23 22:12 ` Alexander Holler
2011-05-23 22:22 ` Wolfgang Denk
2011-05-23 22:38 ` Alexander Holler
2011-05-24 3:47 ` Mike Frysinger
2011-05-24 13:03 ` Alexander Holler
2011-05-24 19:37 ` Scott Wood
2011-05-24 20:39 ` Wolfgang Denk
2011-06-14 6:18 ` Matthias Weißer
2011-06-16 16:04 ` Matthias Weisser
2011-06-30 6:31 ` Matthias Weißer
2011-07-25 22:28 ` Wolfgang Denk
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=4DA403F3.2090304@arcor.de \
--to=weisserm@arcor.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox