[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 02/20] arm: add Faraday FUSBH200 EHCI control
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v2 02/20] arm: add Faraday FUSBH200 EHCI controller |
Date: |
Mon, 28 Jan 2013 08:27:25 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130105 Thunderbird/17.0.2 |
Am 25.01.2013 10:02, schrieb Gerd Hoffmann:
>>> +static const MemoryRegionOps ehci_mmio_faraday_ops = {
>>> + .read = ehci_faraday_read,
>>> + .write = ehci_faraday_write,
>>> + .valid.min_access_size = 4,
>>> + .valid.max_access_size = 4,
>>> + .endianness = DEVICE_LITTLE_ENDIAN,
>>> +};
>
> This should go to hcd-ehci-sysbus.c
>
>>> + memory_region_init_io(&s->mem_faraday, &ehci_mmio_faraday_ops, s,
>>> + "faraday", 0x4c);
>>
>> I don't think this is good design... Can't you do the Faraday part from
>> your own instance_init / initfn / realizefn function?
>
> Yes, please. Just add your own usb_ehci_sysbus_${board}_initfn, call
> usb_ehci_sysbus_initfn for the common stuff, then add your
> hardware-specific memory region.
I submitted a patch to facilitate this a little for you:
http://patchwork.ozlabs.org/patch/215919/
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
[Qemu-devel] [PATCH v2 03/20] arm: add Faraday FTAPBBRG020 APB DMA support, Kuo-Jung Su, 2013/01/25
[Qemu-devel] [PATCH v2 02/20] arm: add Faraday FUSBH200 EHCI controller, Kuo-Jung Su, 2013/01/25
[Qemu-devel] [PATCH v2 04/20] arm: add Faraday FTDMAC020 AHB DMA support, Kuo-Jung Su, 2013/01/25
[Qemu-devel] [PATCH v2 05/20] arm: add Faraday FTGMAC100 1Gbps ethernet support, Kuo-Jung Su, 2013/01/25
[Qemu-devel] [PATCH v2 06/20] arm: add Faraday FTMAC110 10/100Mbps ethernet support, Kuo-Jung Su, 2013/01/25
[Qemu-devel] [PATCH v2 07/20] arm: add Faraday FTI2C010 I2C controller support, Kuo-Jung Su, 2013/01/25
[Qemu-devel] [PATCH v2 09/20] arm: add Faraday FTNANDC021 nand flash controller support, Kuo-Jung Su, 2013/01/25
[Qemu-devel] [PATCH v2 08/20] arm: add Faraday FTLCDC200 LCD controller support, Kuo-Jung Su, 2013/01/25