qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v2 09/14] hw/timer: QOM'ify milkymist_sysctl


From: hitmoon
Subject: Re: [Qemu-devel] [PATCH v2 09/14] hw/timer: QOM'ify milkymist_sysctl
Date: Tue, 16 Feb 2016 17:51:13 +0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0



在 2016年02月16日 17:41, Peter Maydell 写道:
On 16 February 2016 at 09:34, hitmoon <address@hidden> wrote:

在 2016年02月16日 02:14, Peter Maydell 写道:
On 27 January 2016 at 02:54, xiaoqiang zhao<address@hidden>  wrote:
* split milkymist_sysctl_init into milkymist_sysctl_info.instance_init
and milkymist_sysctl_realize
I think the "info" in this function name is wrong ?

I can not understand , can you give me more details?
The two functions which you have split the old
milkymist_sysctl_init() into are named "milkymist_sysctl_init()"
and "milkymist_sysctl_realize()". It confused me that you
said "split FUNCTION into STRUCT.FIELDNAME and FUNCTION";
I expected to read "split FUNCTION into FUNCTION and FUNCTION".

If you want you could just say
"Split the old SysBus init function into an instance_init
and a Device realize function."
(at the moment your two bullet points in the commit message
are actually both describing the same thing.)

thanks
-- PMM
I see ;-)




reply via email to

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