[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH] hw/core/or-irq: Mark the device
From: |
Alistair Francis |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH] hw/core/or-irq: Mark the device with cannot_instantiate_with_device_add_yet |
Date: |
Fri, 27 Jan 2017 08:40:56 -0800 |
On Fri, Jan 27, 2017 at 6:51 AM, Thomas Huth <address@hidden> wrote:
> The "or-irq" device is just used internally. It's strange to
> see this device showing up in the "-device ?" help text. Let's mark it
> with cannot_instantiate_with_device_add_yet to hide it from the users.
I agree that it is strange to be showing up to users, but is this
really the best way to do this?
I thought we were trying to get rid of
cannot_instantiate_with_device_add_yet. Would it maybe be better to
have an internal filed that sets this device as internal?
Thanks,
Alistair
>
> Signed-off-by: Thomas Huth <address@hidden>
> ---
> hw/core/or-irq.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/hw/core/or-irq.c b/hw/core/or-irq.c
> index 1ac090d..2487fa3 100644
> --- a/hw/core/or-irq.c
> +++ b/hw/core/or-irq.c
> @@ -89,6 +89,9 @@ static void or_irq_class_init(ObjectClass *klass, void
> *data)
> dc->props = or_irq_properties;
> dc->realize = or_irq_realize;
> dc->vmsd = &vmstate_or_irq;
> +
> + /* It's just used internally, and should not be exposed to users */
> + dc->cannot_instantiate_with_device_add_yet = true;
> }
>
> static const TypeInfo or_irq_type_info = {
> --
> 1.8.3.1
>
>