Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-28 Thread Adam Ford
On Tue, Apr 27, 2021 at 10:50 AM Tim Harvey wrote: > > On Mon, Apr 26, 2021, 5:35 PM Marek Vasut wrote: > > > > On 4/27/21 2:01 AM, Tim Harvey wrote: > > [...] > > >>> Why would the power domain get probed/enabled for the usbotg2 > > >>> bus but not the usbotg1 bus? Here is some debugging: > > >>

Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-27 Thread Tim Harvey
On Mon, Apr 26, 2021, 5:35 PM Marek Vasut wrote: > > On 4/27/21 2:01 AM, Tim Harvey wrote: > [...] > >>> Why would the power domain get probed/enabled for the usbotg2 > >>> bus but not the usbotg1 bus? Here is some debugging: > >>> u-boot=> usb start > >>> starting USB... > >>> Bus usb@32e4: e

Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-26 Thread Marek Vasut
On 4/27/21 2:01 AM, Tim Harvey wrote: [...] Why would the power domain get probed/enabled for the usbotg2 bus but not the usbotg1 bus? Here is some debugging: u-boot=> usb start starting USB... Bus usb@32e4: ehci_usb_phy_mode usb@32e4 usb@32e4 probe ret=-22 probe failed, error -22 ^^^

Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-26 Thread Tim Harvey
On Fri, Apr 23, 2021 at 8:21 AM Marek Vasut wrote: > > On 4/23/21 5:04 PM, Tim Harvey wrote: > > On Thu, Apr 22, 2021 at 12:16 PM Marek Vasut wrote: > >> > >> On 4/20/21 2:33 AM, Tim Harvey wrote: > >> > >> [...] > >> > +/* USB Configs */ > +#define CONFIG_EHCI_HCD_INIT_AFTER_RESET > >>

Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-23 Thread Marek Vasut
On 4/23/21 5:04 PM, Tim Harvey wrote: On Thu, Apr 22, 2021 at 12:16 PM Marek Vasut wrote: On 4/20/21 2:33 AM, Tim Harvey wrote: [...] +/* USB Configs */ +#define CONFIG_EHCI_HCD_INIT_AFTER_RESET +#define CONFIG_MXC_USB_PORTSC (PORT_PTS_UTMI | PORT_PTS_PTW) +#define CONFIG_USB_MAX_CONTROLLE

Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-23 Thread Tim Harvey
On Thu, Apr 22, 2021 at 12:16 PM Marek Vasut wrote: > > On 4/20/21 2:33 AM, Tim Harvey wrote: > > [...] > > >> +/* USB Configs */ > >> +#define CONFIG_EHCI_HCD_INIT_AFTER_RESET > >> +#define CONFIG_MXC_USB_PORTSC (PORT_PTS_UTMI | PORT_PTS_PTW) > >> +#define CONFIG_USB_MAX_CONTROLLER_COUNT 2 > >>

Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-22 Thread Marek Vasut
On 4/20/21 2:33 AM, Tim Harvey wrote: [...] +/* USB Configs */ +#define CONFIG_EHCI_HCD_INIT_AFTER_RESET +#define CONFIG_MXC_USB_PORTSC (PORT_PTS_UTMI | PORT_PTS_PTW) +#define CONFIG_USB_MAX_CONTROLLER_COUNT 2 + #endif /*_VERDIN_IMX8MM_H */ -- 2.30.2 Marek, Thanks for your work on USB s

Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-22 Thread Tim Harvey
On Thu, Apr 22, 2021 at 3:34 AM Ying-Chun Liu (PaulLiu) wrote: > > Hi Marek, > > I have the same issue on my iMX8MM based board. > Are you continue working on the patch for iMX8MM or should I dig into > this problem? > > I'm asking this just for not duplicating efforts. > So if you are not working

Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-22 Thread Ying-Chun Liu (PaulLiu)
Hi Marek, I have the same issue on my iMX8MM based board. Are you continue working on the patch for iMX8MM or should I dig into this problem? I'm asking this just for not duplicating efforts. So if you are not working on this then I'll see if I can properly retrieve the necessary patches to make

Re: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

2021-04-19 Thread Tim Harvey
On Sun, Apr 11, 2021 at 9:35 AM Marek Vasut wrote: > > Enable USB host support on MX8MM Verdin. > > Signed-off-by: Marek Vasut > Cc: Marcel Ziswiler > Cc: Max Krummenacher > Cc: Oleksandr Suvorov > --- > V2: No change > --- > configs/verdin-imx8mm_defconfig | 8 +++- > include/configs/ver