summaryrefslogtreecommitdiffstats
path: root/hw
diff options
context:
space:
mode:
authorPaul Brook <paul@codesourcery.com>2010-04-04 21:18:26 +0100
committerPaul Brook <paul@codesourcery.com>2010-04-04 21:18:26 +0100
commita67ba3b6f88acaca5ee19e583dcdcd9d9288b072 (patch)
tree74dc47a3e28269c9b14fe354892b8eca295c9f67 /hw
parent36368cf0d513efff45b0dbb4350cdc04a2bee0c8 (diff)
downloadhqemu-a67ba3b6f88acaca5ee19e583dcdcd9d9288b072.zip
hqemu-a67ba3b6f88acaca5ee19e583dcdcd9d9288b072.tar.gz
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.
Diffstat (limited to 'hw')
-rw-r--r--hw/devices.h2
-rw-r--r--hw/ppc_newworld.c2
-rw-r--r--hw/ppc_oldworld.c2
-rw-r--r--hw/ppc_prep.c2
-rw-r--r--hw/pxa.h4
-rw-r--r--hw/pxa2xx.c13
-rw-r--r--hw/realview.c6
-rw-r--r--hw/sm501.c7
-rw-r--r--hw/usb-ohci.c86
-rw-r--r--hw/usb-ohci.h5
-rw-r--r--hw/versatilepb.c6
11 files changed, 41 insertions, 94 deletions
diff --git a/hw/devices.h b/hw/devices.h
index e9e8ec7..0ac561d 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 16d289c..d131aa5 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 d7ee858..27ddc01 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 bb5967a..a5e25b5 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 8d6a8c3..2ca36c2 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 4f9b95b..ea62c0b 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 c102ddc..f786699 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 41ef3bf..1a342bd 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 61ad614..ac23c44 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 aadc365..eefcef3 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 a962106..391f5b8 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) {
OpenPOWER on IntegriCloud