[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] vmmouse: replace PROP_PTR property with gpi
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] vmmouse: replace PROP_PTR property with gpio pin to i8042 |
Date: |
Tue, 07 Feb 2012 13:39:26 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.0.92 (gnu/linux) |
Paolo Bonzini <address@hidden> wrote:
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/pc.c | 7 +++----
> hw/pc.h | 1 -
> hw/pckbd.c | 7 +++++--
> hw/vmmouse.c | 11 +++--------
> 4 files changed, 11 insertions(+), 15 deletions(-)
>
> diff --git a/hw/pc.c b/hw/pc.c
> index 7f3aa65..76787c7 100644
> --- a/hw/pc.c
> +++ b/hw/pc.c
> @@ -1181,13 +1181,12 @@ void pc_basic_device_init(ISABus *isa_bus, qemu_irq
> *gsi,
> if (!no_vmport) {
> vmport_init(isa_bus);
> vmmouse = isa_try_create(isa_bus, "vmmouse");
> + qdev_init_nofail(&vmmouse->qdev);
> + qdev_connect_gpio_out(DEVICE(vmmouse), 0,
> + qdev_get_gpio_in(DEVICE(i8042), 0));
Are you sue that vmmouse is always non-NULL? My understanding is that
all callers check if is_try_create() return NULL.
Could you clarify?
Thanks, Juan.