[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 11/13] mxs/imx23: Add the USB driver
From: |
Peter Crosthwaite |
Subject: |
Re: [Qemu-devel] [PATCH 11/13] mxs/imx23: Add the USB driver |
Date: |
Sat, 11 Jan 2014 19:57:28 +1000 |
On Wed, Dec 11, 2013 at 11:56 PM, Michel Pollet <address@hidden> wrote:
> Add the USB IO block, and the USB PHY IO block. This just wraps
> an ehci instance, and support some of the 'extra' mxs registers
>
> Signed-off-by: Michel Pollet <address@hidden>
> ---
> hw/usb/Makefile.objs | 1 +
> hw/usb/mxs_usb.c | 254
> +++++++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 255 insertions(+)
> create mode 100644 hw/usb/mxs_usb.c
>
> diff --git a/hw/usb/Makefile.objs b/hw/usb/Makefile.objs
> index a3eac3e..58d9cf1 100644
> --- a/hw/usb/Makefile.objs
> +++ b/hw/usb/Makefile.objs
> @@ -8,6 +8,7 @@ common-obj-$(CONFIG_USB_OHCI) += hcd-ohci.o
> common-obj-$(CONFIG_USB_EHCI) += hcd-ehci.o hcd-ehci-pci.o hcd-ehci-sysbus.o
> common-obj-$(CONFIG_USB_XHCI) += hcd-xhci.o
> common-obj-$(CONFIG_USB_MUSB) += hcd-musb.o
> +common-obj-$(CONFIG_MXS) += mxs_usb.o
>
> # emulated usb devices
> common-obj-y += dev-hub.o
> diff --git a/hw/usb/mxs_usb.c b/hw/usb/mxs_usb.c
> new file mode 100644
> index 0000000..1be5f37
> --- /dev/null
> +++ b/hw/usb/mxs_usb.c
> @@ -0,0 +1,254 @@
> +/*
> + * mxs_usb.c
> + *
> + * Copyright: Michel Pollet <address@hidden>
> + *
> + * QEMU Licence
> + */
> +
> +/*
> + * Implements the USB block of the mxs. This is just a case of
> + * instantiating a ehci block, and have a few read only registers
> + * for mxs specific bits
> + */
Sounds like a subclass of EHCI. There are several EHCI subclasses
already for this very purpose. Check the TEGRA2 or EXYNOS_4210 EHCI
variations for examples on how to sublcass EHCI.
Regards,
Peter
> +#include "hw/sysbus.h"
> +#include "hw/arm/mxs.h"
> +#include "hw/usb/hcd-ehci.h"
> +#include "hw/qdev.h"
> +
> +#define D(w)
> +
> +enum {
> + USB_MAX = 256 / 4,
> +};
> +
> +typedef struct mxs_usb_state {
> + SysBusDevice busdev;
> + MemoryRegion iomem;
> +
> + uint32_t r[USB_MAX];
> + qemu_irq irq_dma, irq_error;
> +
> + EHCIState ehci;
> +} mxs_usb_state;
> +
> +static uint64_t mxs_usb_read(
> + void *opaque, hwaddr offset, unsigned size)
> +{
> + mxs_usb_state *s = (mxs_usb_state *) opaque;
> + uint32_t res = 0;
> +
> + D(printf("%s %04x (%d) = ", __func__, (int)offset, size);)
> + switch (offset >> 2) {
> + case 0 ... USB_MAX:
> + res = s->r[offset >> 2];
> + break;
> + default:
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: bad offset 0x%x\n", __func__, (int) offset);
> + break;
> + }
> + D(printf("%08x\n", res);)
> +
> + return res;
> +}
> +
> +static void mxs_usb_write(void *opaque, hwaddr offset,
> + uint64_t value, unsigned size)
> +{
> + mxs_usb_state *s = (mxs_usb_state *) opaque;
> +
> + D(printf("%s %04x %08x(%d)\n", __func__, (int)offset, (int)value, size);)
> + switch (offset) {
> + case 0 ... USB_MAX:
> + s->r[offset] = value;
> + break;
> + default:
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: bad offset 0x%x\n", __func__, (int) offset);
> + break;
> + }
> +}
> +
> +static const MemoryRegionOps mxs_usb_ops = {
> + .read = mxs_usb_read,
> + .write = mxs_usb_write,
> + .endianness = DEVICE_NATIVE_ENDIAN,
> +};
> +
> +static int mxs_usb_init(SysBusDevice *dev)
> +{
> + mxs_usb_state *s = OBJECT_CHECK(mxs_usb_state, dev, "mxs_usb");
> + EHCIState *u = &s->ehci;
> +
> + memory_region_init_io(&s->iomem, OBJECT(s), &mxs_usb_ops, s,
> + "mxs_usb", 0x100);
> +
> + s->r[0] = 0xe241fa05;
> + s->r[0x04 >> 2] = 0x00000015;
> + s->r[0x08 >> 2] = 0x10020001;
> + s->r[0x0c >> 2] = 0x0000000b;
> + s->r[0x10 >> 2] = 0x40060910;
> + s->r[0x14 >> 2] = 0x00000710;
> +
> + u->capsbase = 0x100;
> + u->opregbase = 0x140;
This is nice and data driven in the subclassing framwork.
> + // FIXME ?!?!?
> +// u->dma = &dma_context_memory;
> +
> + usb_ehci_init(u, DEVICE(dev));
> + sysbus_init_irq(dev, &u->irq);
> +
> + memory_region_add_subregion(&u->mem, 0x0, &s->iomem);
> + sysbus_init_mmio(dev, &u->mem);
> +
And all you need to do it tack this on in your IMX_EHCI specific instance_init.
> + D(printf("%s created bus %s\n", __func__, u->bus.qbus.name);)
> +#if 0
> + /*
> + * This is suposed to make companion ports that will support
> + * slower speed devices (mouse/keyboard etc). It's inspired
> + * from ehci/pci however it doesn't work, right now...
> + */
> + int i;
> + for (i = 0; i < NB_PORTS; i += 2) {
> + DeviceState * d = qdev_create(NULL, "sysbus-ohci");
> + qdev_prop_set_string(d, "masterbus", u->bus.qbus.name);
> + qdev_prop_set_uint32(d, "firstport", i);
> + qdev_prop_set_uint32(d, "num-ports", 2);
> + qdev_init_nofail(d);
> + sysbus_connect_irq(SYS_BUS_DEVICE(d), 0, u->irq);
> + }
> +#endif
> + return 0;
> +}
> +
> +static void mxs_usb_class_init(ObjectClass *klass, void *data)
> +{
> + SysBusDeviceClass *sdc = SYS_BUS_DEVICE_CLASS(klass);
> +
> + sdc->init = mxs_usb_init;
> +}
> +
> +static TypeInfo mxs_usb_info = {
> + .name = "mxs_usb",
> + .parent = TYPE_SYS_BUS_DEVICE,
> + .instance_size = sizeof(mxs_usb_state),
> + .class_init = mxs_usb_class_init,
> +};
> +
> +static void mxs_usb_register(void)
> +{
> + type_register_static(&mxs_usb_info);
> +}
> +
> +type_init(mxs_usb_register)
> +
> +#undef D
> +#define D(w)
> +
AFAICT, there is no connectivity between you EHCI and PHY (probably a
good thing at this stage), leading me to believe your PHY is just a
"keep the guest happy" dummy.
With no interconnectivity, It should be a separate device in a
separate file and patch.
Regards,
Peter
> +enum {
> + USBPHY_PWD = 0x0,
> + USBPHY_TX = 0x1,
> + USBPHY_RX = 0x2,
> + USBPHY_CTRL = 0x3,
> + USBPHY_MAX = 10,
> +};
> +typedef struct mxs_usbphy_state {
> + SysBusDevice busdev;
> + MemoryRegion iomem;
> +
> + uint32_t r[USBPHY_MAX];
> +} mxs_usbphy_state;
> +
> +static uint64_t mxs_usbphy_read(void *opaque, hwaddr offset,
> + unsigned size)
> +{
> + mxs_usbphy_state *s = (mxs_usbphy_state *) opaque;
> + uint32_t res = 0;
> +
> + D(printf("%s %04x (%d) = ", __func__, (int)offset, size);)
> + switch (offset >> 4) {
> + case 0 ... USBPHY_MAX:
> + res = s->r[offset >> 4];
> + break;
> + default:
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: bad offset 0x%x\n", __func__, (int) offset);
> + break;
> + }
> + D(printf("%08x\n", res);)
> +
> + return res;
> +}
> +
> +static void mxs_usbphy_write(void *opaque, hwaddr offset,
> + uint64_t value, unsigned size)
> +{
> + mxs_usbphy_state *s = (mxs_usbphy_state *) opaque;
> + uint32_t oldvalue = 0;
> +
> + D(printf("%s %04x %08x(%d) = ", __func__, (int)offset, (int)value,
> size);)
> + switch (offset >> 4) {
> + case 0 ... USBPHY_MAX:
> + oldvalue = mxs_write(&s->r[offset >> 4], offset, value, size);
> + break;
> + default:
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: bad offset 0x%x\n", __func__, (int) offset);
> + return;
> + }
> + switch (offset >> 4) {
> + case USBPHY_CTRL:
> + if ((oldvalue ^ s->r[USBPHY_CTRL]) == 0x80000000
> + && !(oldvalue & 0x80000000)) {
> + D(printf("%s reseting, anding clockgate\n", __func__);)
> + s->r[USBPHY_CTRL] |= 0x40000000;
> + }
> + break;
> + }
> + D(printf("%08x\n", s->r[offset >> 4]);)
> +}
> +
> +
> +static const MemoryRegionOps mxs_usbphy_ops = {
> + .read = mxs_usbphy_read,
> + .write = mxs_usbphy_write,
> + .endianness = DEVICE_NATIVE_ENDIAN,
> +};
> +
> +static int mxs_usbphy_init(SysBusDevice *dev)
> +{
> + mxs_usbphy_state *s = OBJECT_CHECK(mxs_usbphy_state, dev, "mxs_usbphy");
> +
> + memory_region_init_io(&s->iomem, OBJECT(s), &mxs_usbphy_ops, s,
> + "mxs_usbphy", 0x2000);
> + sysbus_init_mmio(dev, &s->iomem);
> +
> + s->r[USBPHY_PWD] = 0x00860607;
> + s->r[USBPHY_PWD] = 0x00860607;
> + s->r[USBPHY_CTRL] = 0xc0000000;
> + return 0;
> +}
> +
> +
> +static void mxs_usbphy_class_init(ObjectClass *klass, void *data)
> +{
> + SysBusDeviceClass *sdc = SYS_BUS_DEVICE_CLASS(klass);
> +
> + sdc->init = mxs_usbphy_init;
> +}
> +
> +static TypeInfo usbphy_info = {
> + .name = "mxs_usbphy",
> + .parent = TYPE_SYS_BUS_DEVICE,
> + .instance_size = sizeof(mxs_usbphy_state),
> + .class_init = mxs_usbphy_class_init,
> +};
> +
> +static void mxs_usbphy_register(void)
> +{
> + type_register_static(&usbphy_info);
> +}
> +
> +type_init(mxs_usbphy_register)
> +
> --
> 1.8.5.1
>
>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 11/13] mxs/imx23: Add the USB driver,
Peter Crosthwaite <=