From: Hans de Goede <hdegoede@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: Hans de Goede <hdegoede@redhat.com>, qemu-devel@nongnu.org
Subject: [Qemu-devel] [PATCH 10/11] usb-uhci: Add support for being a companion controller
Date: Fri, 24 Jun 2011 21:32:57 +0200 [thread overview]
Message-ID: <1308943978-6152-11-git-send-email-hdegoede@redhat.com> (raw)
In-Reply-To: <1308943978-6152-1-git-send-email-hdegoede@redhat.com>
To use as a companion controller set the masterbus property.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
hw/usb-uhci.c | 48 ++++++++++++++++++++++++++++++++++++++++++------
1 files changed, 42 insertions(+), 6 deletions(-)
diff --git a/hw/usb-uhci.c b/hw/usb-uhci.c
index b081b45..a7ab4a1 100644
--- a/hw/usb-uhci.c
+++ b/hw/usb-uhci.c
@@ -132,7 +132,7 @@ typedef struct UHCIPort {
struct UHCIState {
PCIDevice dev;
- USBBus bus;
+ USBBus bus; /* Note unused when we're a companion controller */
uint16_t cmd; /* cmd register */
uint16_t status;
uint16_t intr; /* interrupt enable register */
@@ -150,6 +150,10 @@ struct UHCIState {
/* Active packets */
QTAILQ_HEAD(,UHCIAsync) async_pending;
uint8_t num_ports_vmstate;
+
+ /* Properties */
+ char *masterbus;
+ uint32_t firstport;
};
typedef struct UHCI_TD {
@@ -1126,11 +1130,28 @@ static int usb_uhci_common_initfn(PCIDevice *dev)
pci_conf[PCI_INTERRUPT_PIN] = 4; // interrupt pin 3
pci_conf[USB_SBRN] = USB_RELEASE_1; // release number
- usb_bus_new(&s->bus, &uhci_bus_ops, &s->dev.qdev);
- for(i = 0; i < NB_PORTS; i++) {
- usb_register_port(&s->bus, &s->ports[i].port, s, i, &uhci_port_ops,
- USB_SPEED_MASK_LOW | USB_SPEED_MASK_FULL);
- usb_port_location(&s->ports[i].port, NULL, i+1);
+ if (s->masterbus) {
+ USBPort *ports[NB_PORTS];
+ for(i = 0; i < NB_PORTS; i++) {
+ s->ports[i].port.ops = &uhci_port_ops;
+ s->ports[i].port.opaque = s;
+ s->ports[i].port.index = i;
+ s->ports[i].port.speedmask =
+ USB_SPEED_MASK_LOW | USB_SPEED_MASK_FULL;
+ usb_port_location(&s->ports[i].port, NULL, i+1);
+ ports[i] = &s->ports[i].port;
+ }
+ if (usb_bus_register_companion(s->masterbus, ports, NB_PORTS,
+ s->firstport) != 0) {
+ return -1;
+ }
+ } else {
+ usb_bus_new(&s->bus, &uhci_bus_ops, &s->dev.qdev);
+ for(i = 0; i < NB_PORTS; i++) {
+ usb_register_port(&s->bus, &s->ports[i].port, s, i, &uhci_port_ops,
+ USB_SPEED_MASK_LOW | USB_SPEED_MASK_FULL);
+ usb_port_location(&s->ports[i].port, NULL, i+1);
+ }
}
s->frame_timer = qemu_new_timer_ns(vm_clock, uhci_frame_timer, s);
s->num_ports_vmstate = NB_PORTS;
@@ -1171,6 +1192,11 @@ static PCIDeviceInfo uhci_info[] = {
.device_id = PCI_DEVICE_ID_INTEL_82371SB_2,
.revision = 0x01,
.class_id = PCI_CLASS_SERIAL_USB,
+ .qdev.props = (Property[]) {
+ DEFINE_PROP_STRING("masterbus", UHCIState, masterbus),
+ DEFINE_PROP_UINT32("firstport", UHCIState, firstport, 0),
+ DEFINE_PROP_END_OF_LIST(),
+ },
},{
.qdev.name = "piix4-usb-uhci",
.qdev.size = sizeof(UHCIState),
@@ -1180,6 +1206,11 @@ static PCIDeviceInfo uhci_info[] = {
.device_id = PCI_DEVICE_ID_INTEL_82371AB_2,
.revision = 0x01,
.class_id = PCI_CLASS_SERIAL_USB,
+ .qdev.props = (Property[]) {
+ DEFINE_PROP_STRING("masterbus", UHCIState, masterbus),
+ DEFINE_PROP_UINT32("firstport", UHCIState, firstport, 0),
+ DEFINE_PROP_END_OF_LIST(),
+ },
},{
.qdev.name = "vt82c686b-usb-uhci",
.qdev.size = sizeof(UHCIState),
@@ -1189,6 +1220,11 @@ static PCIDeviceInfo uhci_info[] = {
.device_id = PCI_DEVICE_ID_VIA_UHCI,
.revision = 0x01,
.class_id = PCI_CLASS_SERIAL_USB,
+ .qdev.props = (Property[]) {
+ DEFINE_PROP_STRING("masterbus", UHCIState, masterbus),
+ DEFINE_PROP_UINT32("firstport", UHCIState, firstport, 0),
+ DEFINE_PROP_END_OF_LIST(),
+ },
},{
/* end of list */
}
--
1.7.5.1
next prev parent reply other threads:[~2011-06-24 19:32 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-24 19:32 [Qemu-devel] RFC/PULL: usb: add support for companion controllers Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 01/11] usb: Add a register_companion USB bus op Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 02/11] usb: Make port wakeup and complete ops take a USBPort instead of a Device Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 03/11] usb: Replace device_destroy bus op with a child_detach port op Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 04/11] usb-ehci: drop unused num-ports state member Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 05/11] usb-ehci: Connect Status bit is read only, don't allow changing it by the guest Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 06/11] usb-ehci: cleanup port reset handling Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 07/11] usb: assert on calling usb_attach(port, NULL) on a port without a dev Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 08/11] usb-ehci: Fix handling of PED and PEDC port status bits Hans de Goede
2011-06-25 0:15 ` Brad Hards
2011-06-25 6:08 ` Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 09/11] usb-ehci: Add support for registering companion controllers Hans de Goede
2011-06-24 19:32 ` Hans de Goede [this message]
2011-06-29 10:57 ` [Qemu-devel] [PATCH 10/11] usb-uhci: Add support for being a companion controller Gerd Hoffmann
2011-06-29 11:19 ` Hans de Goede
2011-06-29 12:29 ` Gerd Hoffmann
2011-06-30 11:09 ` Hans de Goede
2011-06-24 19:32 ` [Qemu-devel] [PATCH 11/11] usb-ohci: " Hans de Goede
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1308943978-6152-11-git-send-email-hdegoede@redhat.com \
--to=hdegoede@redhat.com \
--cc=kraxel@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).