On 16 April 2013 15:47, Andreas Färber <afaer...@suse.de> wrote:
> 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.

Yes, I abbreviated a bit.

> You could also remove the include now, right?

In theory, but given patch 2 it's a bit moot :-)

> Either way,
>
> Acked-by: Andreas Färber <afaer...@suse.de>

Thanks.

-- PMM

Reply via email to