[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 4/8] usb/ehci: Add usb-ehci-sysbus
From: |
Peter Crosthwaite |
Subject: |
Re: [Qemu-devel] [PATCH v1 4/8] usb/ehci: Add usb-ehci-sysbus |
Date: |
Thu, 25 Oct 2012 22:39:06 +1000 |
On Thu, Oct 25, 2012 at 10:10 PM, Gerd Hoffmann <address@hidden> wrote:
>> typedef struct EHCIItfState {
>> - PCIDevice pcidev;
>> + /* FIXME: Figure out a better way to share one Property[] array between
>> two
>> + * QOM types with different parents
>> + */
>> + union {
>> + PCIDevice pcidev;
>> + SysBusDevice busdev;
>> + };
>
> Ah, I see where this hack comes from. I don't think they should share
> the properties in the first place ...
>
Kind of annoying to not share them as it really is a copy paste job
with the exception of the wrapping struct type.
> cheers,
> Gerd
>
- Re: [Qemu-devel] [PATCH v1 1/8] usb/ehci: parameterise the register region offsets, (continued)
[Qemu-devel] [PATCH v1 4/8] usb/ehci: Add usb-ehci-sysbus, Peter Crosthwaite, 2012/10/25
[Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Peter Crosthwaite, 2012/10/25
- Re: [Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Gerd Hoffmann, 2012/10/25
- Re: [Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Peter Crosthwaite, 2012/10/25
- Re: [Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Peter Maydell, 2012/10/25
- Re: [Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Avi Kivity, 2012/10/25
- Re: [Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Peter Maydell, 2012/10/25
- Re: [Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Avi Kivity, 2012/10/25
- Re: [Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Peter Maydell, 2012/10/25
- Re: [Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Avi Kivity, 2012/10/25
- Re: [Qemu-devel] [PATCH v1 8/8] usb/ehci: Put RAM in undefined MMIO regions, Peter Crosthwaite, 2012/10/25