From a67ba3b6f88acaca5ee19e583dcdcd9d9288b072 Mon Sep 17 00:00:00 2001 From: Paul Brook Date: Sun, 4 Apr 2010 21:18:26 +0100 Subject: Revert "Compile usb-ohci only once" This reverts commit f1698408f1dcb7548a21828a0b1e2b530fae3af3. PCI is always little-endian. Having a user-visible "be" property is just plain wrong. --- hw/devices.h | 2 +- hw/ppc_newworld.c | 2 +- hw/ppc_oldworld.c | 2 +- hw/ppc_prep.c | 2 +- hw/pxa.h | 4 +++ hw/pxa2xx.c | 13 ++------- hw/realview.c | 6 +--- hw/sm501.c | 7 +---- hw/usb-ohci.c | 86 +++++++++++++++++-------------------------------------- hw/usb-ohci.h | 5 ++-- hw/versatilepb.c | 6 +--- 11 files changed, 41 insertions(+), 94 deletions(-) (limited to 'hw') diff --git a/hw/devices.h b/hw/devices.h index e9e8ec7a77..0ac561d91b 100644 --- a/hw/devices.h +++ b/hw/devices.h @@ -69,5 +69,5 @@ void sm501_init(uint32_t base, uint32_t local_mem_bytes, qemu_irq irq, /* usb-ohci.c */ void usb_ohci_init_sm501(uint32_t mmio_base, uint32_t localmem_base, - int num_ports, int devfn, qemu_irq irq, int be); + int num_ports, int devfn, qemu_irq irq); #endif diff --git a/hw/ppc_newworld.c b/hw/ppc_newworld.c index 16d289c1f0..d131aa5e68 100644 --- a/hw/ppc_newworld.c +++ b/hw/ppc_newworld.c @@ -387,7 +387,7 @@ static void ppc_core99_init (ram_addr_t ram_size, escc_mem_index); if (usb_enabled) { - usb_ohci_init_pci(pci_bus, -1, 1); + usb_ohci_init_pci(pci_bus, -1); } /* U3 needs to use USB for input because Linux doesn't support via-cuda diff --git a/hw/ppc_oldworld.c b/hw/ppc_oldworld.c index d7ee858aaa..27ddc013e6 100644 --- a/hw/ppc_oldworld.c +++ b/hw/ppc_oldworld.c @@ -370,7 +370,7 @@ static void ppc_heathrow_init (ram_addr_t ram_size, escc_mem_index); if (usb_enabled) { - usb_ohci_init_pci(pci_bus, -1, 1); + usb_ohci_init_pci(pci_bus, -1); } if (graphic_depth != 15 && graphic_depth != 32 && graphic_depth != 8) diff --git a/hw/ppc_prep.c b/hw/ppc_prep.c index bb5967a7fc..a5e25b57fd 100644 --- a/hw/ppc_prep.c +++ b/hw/ppc_prep.c @@ -750,7 +750,7 @@ static void ppc_prep_init (ram_addr_t ram_size, #endif if (usb_enabled) { - usb_ohci_init_pci(pci_bus, -1, 1); + usb_ohci_init_pci(pci_bus, -1); } m48t59 = m48t59_init(i8259[8], 0, 0x0074, NVRAM_SIZE, 59); diff --git a/hw/pxa.h b/hw/pxa.h index 8d6a8c3348..2ca36c2f6a 100644 --- a/hw/pxa.h +++ b/hw/pxa.h @@ -213,4 +213,8 @@ struct PXA2xxI2SState { PXA2xxState *pxa270_init(unsigned int sdram_size, const char *revision); PXA2xxState *pxa255_init(unsigned int sdram_size); +/* usb-ohci.c */ +void usb_ohci_init_pxa(target_phys_addr_t base, int num_ports, int devfn, + qemu_irq irq); + #endif /* PXA_H */ diff --git a/hw/pxa2xx.c b/hw/pxa2xx.c index 4f9b95bc4d..ea62c0b002 100644 --- a/hw/pxa2xx.c +++ b/hw/pxa2xx.c @@ -9,7 +9,6 @@ #include "sysbus.h" #include "pxa.h" -#include "usb-ohci.h" #include "sysemu.h" #include "pc.h" #include "i2c.h" @@ -2129,11 +2128,7 @@ PXA2xxState *pxa270_init(unsigned int sdram_size, const char *revision) } if (usb_enabled) { -#ifdef TARGET_WORDS_BIGENDIAN - usb_ohci_init_pxa(0x4c000000, 3, -1, s->pic[PXA2XX_PIC_USBH1], 1); -#else - usb_ohci_init_pxa(0x4c000000, 3, -1, s->pic[PXA2XX_PIC_USBH1], 0); -#endif + usb_ohci_init_pxa(0x4c000000, 3, -1, s->pic[PXA2XX_PIC_USBH1]); } s->pcmcia[0] = pxa2xx_pcmcia_init(0x20000000); @@ -2252,11 +2247,7 @@ PXA2xxState *pxa255_init(unsigned int sdram_size) } if (usb_enabled) { -#ifdef TARGET_WORDS_BIGENDIAN - usb_ohci_init_pxa(0x4c000000, 3, -1, s->pic[PXA2XX_PIC_USBH1], 1); -#else - usb_ohci_init_pxa(0x4c000000, 3, -1, s->pic[PXA2XX_PIC_USBH1], 0); -#endif + usb_ohci_init_pxa(0x4c000000, 3, -1, s->pic[PXA2XX_PIC_USBH1]); } s->pcmcia[0] = pxa2xx_pcmcia_init(0x20000000); diff --git a/hw/realview.c b/hw/realview.c index c102ddca00..f786699a1c 100644 --- a/hw/realview.c +++ b/hw/realview.c @@ -266,11 +266,7 @@ static void realview_init(ram_addr_t ram_size, pic[48], pic[49], pic[50], pic[51], NULL); pci_bus = (PCIBus *)qdev_get_child_bus(dev, "pci"); if (usb_enabled) { -#ifdef TARGET_WORDS_BIGENDIAN - usb_ohci_init_pci(pci_bus, -1, 1); -#else - usb_ohci_init_pci(pci_bus, -1, 0); -#endif + usb_ohci_init_pci(pci_bus, -1); } n = drive_get_max_bus(IF_SCSI); while (n >= 0) { diff --git a/hw/sm501.c b/hw/sm501.c index 41ef3bff7a..1a342bdc58 100644 --- a/hw/sm501.c +++ b/hw/sm501.c @@ -1222,13 +1222,8 @@ void sm501_init(uint32_t base, uint32_t local_mem_bytes, qemu_irq irq, 0x1000, sm501_disp_ctrl_index); /* bridge to usb host emulation module */ -#ifdef TARGET_WORDS_BIGENDIAN - usb_ohci_init_sm501(base + MMIO_BASE_OFFSET + SM501_USB_HOST, base, - 2, -1, irq, 1); -#else usb_ohci_init_sm501(base + MMIO_BASE_OFFSET + SM501_USB_HOST, base, - 2, -1, irq, 0); -#endif + 2, -1, irq); /* bridge to serial emulation module */ if (chr) { diff --git a/hw/usb-ohci.c b/hw/usb-ohci.c index 61ad6142c5..ac23c44f65 100644 --- a/hw/usb-ohci.c +++ b/hw/usb-ohci.c @@ -30,6 +30,7 @@ #include "qemu-timer.h" #include "usb.h" #include "pci.h" +#include "pxa.h" #include "devices.h" #include "usb-ohci.h" @@ -1398,7 +1399,7 @@ static void ohci_port_set_status(OHCIState *ohci, int portnum, uint32_t val) return; } -static uint32_t ohci_mem_read_le(void *ptr, target_phys_addr_t addr) +static uint32_t ohci_mem_read(void *ptr, target_phys_addr_t addr) { OHCIState *ohci = ptr; uint32_t retval; @@ -1515,22 +1516,21 @@ static uint32_t ohci_mem_read_le(void *ptr, target_phys_addr_t addr) retval = 0xffffffff; } } - return retval; -} - -static uint32_t ohci_mem_read_be(void *ptr, target_phys_addr_t addr) -{ - uint32_t retval; - retval = ohci_mem_read_le(ptr, addr); +#ifdef TARGET_WORDS_BIGENDIAN retval = bswap32(retval); +#endif return retval; } -static void ohci_mem_write_le(void *ptr, target_phys_addr_t addr, uint32_t val) +static void ohci_mem_write(void *ptr, target_phys_addr_t addr, uint32_t val) { OHCIState *ohci = ptr; +#ifdef TARGET_WORDS_BIGENDIAN + val = bswap32(val); +#endif + /* Only aligned reads are allowed on OHCI */ if (addr & 3) { fprintf(stderr, "usb-ohci: Mis-aligned write\n"); @@ -1647,43 +1647,24 @@ static void ohci_mem_write_le(void *ptr, target_phys_addr_t addr, uint32_t val) } } -static void ohci_mem_write_be(void *ptr, target_phys_addr_t addr, uint32_t val) -{ - val = bswap32(val); - ohci_mem_write_le(ptr, addr, val); -} - /* Only dword reads are defined on OHCI register space */ -static CPUReadMemoryFunc * const ohci_readfn_be[3]={ - ohci_mem_read_be, - ohci_mem_read_be, - ohci_mem_read_be +static CPUReadMemoryFunc * const ohci_readfn[3]={ + ohci_mem_read, + ohci_mem_read, + ohci_mem_read }; /* Only dword writes are defined on OHCI register space */ -static CPUWriteMemoryFunc * const ohci_writefn_be[3]={ - ohci_mem_write_be, - ohci_mem_write_be, - ohci_mem_write_be -}; - -static CPUReadMemoryFunc * const ohci_readfn_le[3]={ - ohci_mem_read_le, - ohci_mem_read_le, - ohci_mem_read_le -}; - -static CPUWriteMemoryFunc * const ohci_writefn_le[3]={ - ohci_mem_write_le, - ohci_mem_write_le, - ohci_mem_write_le +static CPUWriteMemoryFunc * const ohci_writefn[3]={ + ohci_mem_write, + ohci_mem_write, + ohci_mem_write }; static void usb_ohci_init(OHCIState *ohci, DeviceState *dev, int num_ports, int devfn, qemu_irq irq, enum ohci_type type, - const char *name, uint32_t localmem_base, - int be) + const char *name, uint32_t localmem_base) { int i; @@ -1703,13 +1684,7 @@ static void usb_ohci_init(OHCIState *ohci, DeviceState *dev, usb_frame_time, usb_bit_time); } - if (be) { - ohci->mem = cpu_register_io_memory(ohci_readfn_be, ohci_writefn_be, - ohci); - } else { - ohci->mem = cpu_register_io_memory(ohci_readfn_le, ohci_writefn_le, - ohci); - } + ohci->mem = cpu_register_io_memory(ohci_readfn, ohci_writefn, ohci); ohci->localmem_base = localmem_base; ohci->name = name; @@ -1729,7 +1704,6 @@ static void usb_ohci_init(OHCIState *ohci, DeviceState *dev, typedef struct { PCIDevice pci_dev; OHCIState state; - uint32_t be; } OHCIPCIState; static void ohci_mapfunc(PCIDevice *pci_dev, int i, @@ -1754,7 +1728,7 @@ static int usb_ohci_initfn_pci(struct PCIDevice *dev) usb_ohci_init(&ohci->state, &dev->qdev, num_ports, ohci->pci_dev.devfn, ohci->pci_dev.irq[0], - OHCI_TYPE_PCI, ohci->pci_dev.name, 0, ohci->be); + OHCI_TYPE_PCI, ohci->pci_dev.name, 0); /* TODO: avoid cast below by using dev */ pci_register_bar((struct PCIDevice *)ohci, 0, 256, @@ -1762,33 +1736,29 @@ static int usb_ohci_initfn_pci(struct PCIDevice *dev) return 0; } -void usb_ohci_init_pci(struct PCIBus *bus, int devfn, int be) +void usb_ohci_init_pci(struct PCIBus *bus, int devfn) { - PCIDevice *dev; - - dev = pci_create(bus, devfn, "pci-ohci"); - qdev_prop_set_uint32(&dev->qdev, "be", be); - qdev_init_nofail(&dev->qdev); + pci_create_simple(bus, devfn, "pci-ohci"); } void usb_ohci_init_pxa(target_phys_addr_t base, int num_ports, int devfn, - qemu_irq irq, int be) + qemu_irq irq) { OHCIState *ohci = (OHCIState *)qemu_mallocz(sizeof(OHCIState)); usb_ohci_init(ohci, NULL /* FIXME */, num_ports, devfn, irq, - OHCI_TYPE_PXA, "OHCI USB", 0, be); + OHCI_TYPE_PXA, "OHCI USB", 0); cpu_register_physical_memory(base, 0x1000, ohci->mem); } void usb_ohci_init_sm501(uint32_t mmio_base, uint32_t localmem_base, - int num_ports, int devfn, qemu_irq irq, int be) + int num_ports, int devfn, qemu_irq irq) { OHCIState *ohci = (OHCIState *)qemu_mallocz(sizeof(OHCIState)); usb_ohci_init(ohci, NULL /* FIXME */, num_ports, devfn, irq, - OHCI_TYPE_SM501, "OHCI USB", localmem_base, be); + OHCI_TYPE_SM501, "OHCI USB", localmem_base); cpu_register_physical_memory(mmio_base, 0x1000, ohci->mem); } @@ -1798,10 +1768,6 @@ static PCIDeviceInfo ohci_info = { .qdev.desc = "Apple USB Controller", .qdev.size = sizeof(OHCIPCIState), .init = usb_ohci_initfn_pci, - .qdev.props = (Property[]) { - DEFINE_PROP_HEX32("be", OHCIPCIState, be, 0), - DEFINE_PROP_END_OF_LIST(), - } }; static void ohci_register(void) diff --git a/hw/usb-ohci.h b/hw/usb-ohci.h index aadc365160..eefcef3602 100644 --- a/hw/usb-ohci.h +++ b/hw/usb-ohci.h @@ -3,8 +3,7 @@ #include "qemu-common.h" -void usb_ohci_init_pci(struct PCIBus *bus, int devfn, int be); -void usb_ohci_init_pxa(target_phys_addr_t base, int num_ports, int devfn, - qemu_irq irq, int be); +void usb_ohci_init_pci(struct PCIBus *bus, int devfn); + #endif diff --git a/hw/versatilepb.c b/hw/versatilepb.c index a962106976..391f5b88c6 100644 --- a/hw/versatilepb.c +++ b/hw/versatilepb.c @@ -218,11 +218,7 @@ static void versatile_init(ram_addr_t ram_size, } } if (usb_enabled) { -#ifdef TARGET_WORDS_BIGENDIAN - usb_ohci_init_pci(pci_bus, -1, 1); -#else - usb_ohci_init_pci(pci_bus, -1, 0); -#endif + usb_ohci_init_pci(pci_bus, -1); } n = drive_get_max_bus(IF_SCSI); while (n >= 0) { -- cgit v1.2.3