|
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 thisMessage-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
[Prev in Thread] | Current Thread | [Next in Thread] |