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