Re: [Qemu-devel] [PATCH RFC v19 01/13] target-avr: AVR cores support is added.

[Thomas Huth <huth@tuxfamily.org>]

On 13.06.2017 22:32, Michael Rolnik wrote:
> the last version of my patches were done by Richard. Now I just
> reapplied them to the master. 
> Should I have to remove "signed-off-by Richard" ?

OK, if Richard did change to your patches, you should likely keep the
line. But I think you should remove the "Message-Id" line when sending
patches to the list, since that is normally only used by maintainers
when they finally pick up the patches. Otherwise this looks very
confusing (since the new v19 version has a different Message-Id in the
mail header).

 Thomas


> On Tue, Jun 13, 2017 at 11:09 PM, Thomas Huth <huth@tuxfamily.org
> <mailto:huth@tuxfamily.org>> wrote:
> 
>     On 08.06.2017 20:49, Michael Rolnik wrote:
>     >     1. basic CPU structure
>     >     2. registers
>     >     3. no instructions
>     >     4. saving sreg, rampD, rampX, rampY, rampD, eind in HW representation
>     >
>     > Signed-off-by: Michael Rolnik <mrolnik@gmail.com <mailto:mrolnik@gmail.com>>
>     > Message-Id: <1471522070-77598-2-git-send-email-mrolnik@gmail.com
>     <mailto:1471522070-77598-2-git-send-email-mrolnik@gmail.com>>
>     > Signed-off-by: Richard Henderson <rth@twiddle.net <mailto:rth@twiddle.net>>
> 
>     Why is the final "Signed-off-by" by Richard, if the patch has apparently
>     been sent by you, Michael? Something is wrong here with the order...
> 
>      Thomas