[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] hw/core/register: Mark the de
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] hw/core/register: Mark the device with cannot_instantiate_with_device_add_yet |
Date: |
Fri, 03 Feb 2017 10:13:14 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Thomas Huth <address@hidden> writes:
> The "qemu,register" device needs to be wired up in source code, there
> is no way the user can add this device with the "-device" parameter or
> the "device_add" monitor command yet.
Well, the user can (I tried), but it won't do squat then.
> Signed-off-by: Thomas Huth <address@hidden>
> ---
> v2: Changed patch description and comment
>
> hw/core/register.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/hw/core/register.c b/hw/core/register.c
> index 4bfbc50..1416691 100644
> --- a/hw/core/register.c
> +++ b/hw/core/register.c
> @@ -274,9 +274,18 @@ void register_finalize_block(RegisterInfoArray *r_array)
> g_free(r_array);
> }
>
> +static void register_class_init(ObjectClass *oc, void *data)
> +{
> + DeviceClass *dc = DEVICE_CLASS(oc);
> +
> + /* Needs to be wired up in source code, see register_init_block32() */
Bear with me... Please phrase this exactly like the similar comments
elsewhere:
/* Reason: needs to be wired up by register_init_block32() to work */
> + dc->cannot_instantiate_with_device_add_yet = true;
> +}
> +
> static const TypeInfo register_info = {
> .name = TYPE_REGISTER,
> .parent = TYPE_DEVICE,
> + .class_init = register_class_init,
> };
>
> static void register_register_types(void)
With comment and commit message touched up:
Reviewed-by: Markus Armbruster <address@hidden>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] hw/core/register: Mark the device with cannot_instantiate_with_device_add_yet,
Markus Armbruster <=