On Sun, Apr 11, 2021 at 9:35 AM Marek Vasut <[email protected]> wrote: > > Enable USB host support on MX8MM Verdin. > > Signed-off-by: Marek Vasut <[email protected]> > Cc: Marcel Ziswiler <[email protected]> > Cc: Max Krummenacher <[email protected]> > Cc: Oleksandr Suvorov <[email protected]> > --- > V2: No change > --- > configs/verdin-imx8mm_defconfig | 8 +++++++- > include/configs/verdin-imx8mm.h | 5 +++++ > 2 files changed, 12 insertions(+), 1 deletion(-) > > diff --git a/configs/verdin-imx8mm_defconfig b/configs/verdin-imx8mm_defconfig > index ea0b5978f1..c8c3420b6a 100644 > --- a/configs/verdin-imx8mm_defconfig > +++ b/configs/verdin-imx8mm_defconfig > @@ -37,7 +37,6 @@ CONFIG_SPL_BOARD_INIT=y > CONFIG_SPL_SEPARATE_BSS=y > CONFIG_SPL_I2C_SUPPORT=y > CONFIG_SPL_POWER_SUPPORT=y > -CONFIG_SPL_USB_HOST_SUPPORT=y > CONFIG_SPL_WATCHDOG_SUPPORT=y > CONFIG_SYS_PROMPT="Verdin iMX8MM # " > # CONFIG_BOOTM_NETBSD is not set > @@ -50,6 +49,7 @@ CONFIG_CMD_FUSE=y > CONFIG_CMD_GPIO=y > CONFIG_CMD_I2C=y > CONFIG_CMD_MMC=y > +CONFIG_CMD_USB=y > CONFIG_CMD_CACHE=y > CONFIG_CMD_UUID=y > CONFIG_CMD_REGULATOR=y > @@ -89,6 +89,8 @@ CONFIG_MII=y > CONFIG_PINCTRL=y > CONFIG_SPL_PINCTRL=y > CONFIG_PINCTRL_IMX8M=y > +CONFIG_POWER_DOMAIN=y > +CONFIG_IMX8M_POWER_DOMAIN=y > CONFIG_DM_PMIC=y > CONFIG_SPL_DM_PMIC_PCA9450=y > CONFIG_DM_PMIC_PFUZE100=y > @@ -101,5 +103,9 @@ CONFIG_SPL_SYSRESET=y > CONFIG_SYSRESET_PSCI=y > CONFIG_SYSRESET_WATCHDOG=y > CONFIG_DM_THERMAL=y > +CONFIG_USB=y > +CONFIG_DM_USB=y > +# CONFIG_SPL_DM_USB is not set > +CONFIG_USB_EHCI_HCD=y > CONFIG_IMX_WATCHDOG=y > CONFIG_OF_LIBFDT_OVERLAY=y > diff --git a/include/configs/verdin-imx8mm.h b/include/configs/verdin-imx8mm.h > index 4751bf5a5a..e2a817891c 100644 > --- a/include/configs/verdin-imx8mm.h > +++ b/include/configs/verdin-imx8mm.h > @@ -117,5 +117,10 @@ > #define FEC_QUIRK_ENET_MAC > #define IMX_FEC_BASE 0x30BE0000 > > +/* 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 support for IMX8M! I'm attempting to add USB support to the venice board following this example but I think there are still some things missing from the dt to make this work. I find that mx6_parse_dt_adds failes; Looks like it is required to have an alias that points to the phy but then it fails because the phy doesn't have a reg. Also, it would see the CONFIG_MXC_USB_PORTSC is no longer needed as that is now the default. Best regards, Tim
