On 9/16/20 2:57 PM, peng....@nxp.com wrote:
[...]
> +++ b/drivers/usb/host/ehci-mx6.c
> @@ -735,13 +735,16 @@ static int ehci_usb_bind(struct udevice *dev)
>        * the driver is fully converted to DT probing.
>        */
>       u32 controller_spacing;
> -     if (IS_ENABLED(CONFIG_MX6))
> -             controller_spacing = 0x200;
> -     else
> -             controller_spacing = 0x10000;
> -     fdt_addr_t addr = devfdt_get_addr_index(dev, 0);
>  
> -     dev->req_seq = (addr - USB_BASE_ADDR) / controller_spacing;
> +     if (dev->req_seq == -1) {
> +             if (IS_ENABLED(CONFIG_MX6))
> +                     controller_spacing = 0x200;
> +             else
> +                     controller_spacing = 0x10000;
> +             fdt_addr_t addr = devfdt_get_addr_index(dev, 0);

Can we get rid of the whole req_seq stuff ?
It is a workaround, see
501547cec1 ("usb: ehci-mx6: Fix bus enumeration for DM case")

Reply via email to