qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v27 06/21] target/rx: CPU definition


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH v27 06/21] target/rx: CPU definition
Date: Sat, 21 Dec 2019 17:55:06 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2

On 12/21/19 5:03 PM, Aleksandar Markovic wrote:


On Saturday, December 21, 2019, Yoshinori Sato <address@hidden <mailto:address@hidden>> wrote:

    Signed-off-by: Yoshinori Sato <address@hidden
    <mailto:address@hidden>>

    Message-Id: <address@hidden
    <mailto:address@hidden>>
    Reviewed-by: Richard Henderson <address@hidden
    <mailto:address@hidden>>
    Message-Id: <address@hidden
    <mailto:address@hidden>>


Hi, Yoshinori,

I noticed you have a lot of line like this

Message-Id: <address@hidden <mailto:address@hidden>>

... sometimes even several in the same message.

May I ask you to delete *all* such lines from commit messages, and resend the series?

I expected this series to be ready for merge, and asked Richard if he could do that to avoid a respin of new version, but apparently Yoshinori missed my comment:
https://www.mail-archive.com/address@hidden/msg664071.html


Also, I don't see  the documentation update for the new target. Please provide dome appropriate content in qemu-doc.texi ( https://github.com/qemu/qemu/blob/master/qemu-doc.texi ), and include it in the next version.

Best regards,

Aleksandar




reply via email to

[Prev in Thread] Current Thread [Next in Thread]