[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/2] hw/sm501: Use correct setter for sysbus-
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/2] hw/sm501: Use correct setter for sysbus-ohci dma-address property |
Date: |
Tue, 16 Apr 2013 16:47:05 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 |
Am 09.04.2013 13:27, schrieb Peter Maydell:
> The sysbus-ohci dma-address property is declared as a HEX64
> property, not a TADDR, so use the correct setter for it.
Actually, it's declared as
DEFINE_PROP_DMAADDR("dma-offset", OHCISysBusState, dma_offset, 3),
which is in turn defined as DEFINE_PROP_HEX64 in include/hw/qdev-dma.h.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> hw/display/sm501.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/display/sm501.c b/hw/display/sm501.c
> index 6b660ac..3840bce 100644
> --- a/hw/display/sm501.c
> +++ b/hw/display/sm501.c
> @@ -1430,7 +1430,7 @@ void sm501_init(MemoryRegion *address_space_mem,
> uint32_t base,
> /* bridge to usb host emulation module */
> dev = qdev_create(NULL, "sysbus-ohci");
> qdev_prop_set_uint32(dev, "num-ports", 2);
> - qdev_prop_set_taddr(dev, "dma-offset", base);
> + qdev_prop_set_uint64(dev, "dma-offset", base);
> qdev_init_nofail(dev);
> sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0,
> base + MMIO_BASE_OFFSET + SM501_USB_HOST);
You could also remove the include now, right?
Either way,
Acked-by: Andreas Färber <address@hidden>
CC'ing Gerd.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg