[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/3] hw/arm/fsl-imx6: Wire up USB controllers
From: |
Guenter Roeck |
Subject: |
Re: [PATCH v2 3/3] hw/arm/fsl-imx6: Wire up USB controllers |
Date: |
Thu, 12 Mar 2020 09:56:56 -0700 |
User-agent: |
Mutt/1.9.4 (2018-02-28) |
On Thu, Mar 12, 2020 at 01:29:34PM +0000, Peter Maydell wrote:
> On Tue, 10 Mar 2020 at 21:04, Guenter Roeck <address@hidden> wrote:
> >
> > With this patch, the USB controllers on 'sabrelite' are detected
> > and can be used to boot the system.
> >
> > Signed-off-by: Guenter Roeck <address@hidden>
>
> > + for (i = 0; i < FSL_IMX6_NUM_USBS; i++) {
> > + static const int FSL_IMX6_USBn_IRQ[] = {
> > + FSL_IMX6_USB_OTG_IRQ,
> > + FSL_IMX6_USB_HOST1_IRQ,
> > + FSL_IMX6_USB_HOST2_IRQ,
> > + FSL_IMX6_USB_HOST3_IRQ,
> > + };
> > +
> > + object_property_set_bool(OBJECT(&s->usbphy[i]), true, "realized",
> > + &error_abort);
> > + sysbus_mmio_map(SYS_BUS_DEVICE(&s->usbphy[i]), 0,
> > + FSL_IMX6_USBPHY1_ADDR + i * 0x1000);
>
> Are you sure these addresses are right? Four of these starting
> at USBPHY1_ADDR means the last one clashes with what we define
> as "FSL_IMX6_SNVSHP_ADDR 0x020CC000".
>
> I only have the i.MX 6Dual/6Quad reference manual to hand,
> so maybe this imx6 variant is different, but that document
> says there are 4 USB controllers but only 2 PHY blocks.
>
Oops, I think you are correct. Good catch. I'll re-check the datsheet
and send an updated patch.
Thanks,
Guenter
[PATCH v2 1/3] hw/usb: Add basic i.MX USB Phy support, Guenter Roeck, 2020/03/10
[PATCH v2 3/3] hw/arm/fsl-imx6: Wire up USB controllers, Guenter Roeck, 2020/03/10
Re: [PATCH v2 0/3] Wire up USB controllers in i.MX6 emulations, no-reply, 2020/03/10