From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ying-Chun Liu (PaulLiu) Date: Thu, 22 Apr 2021 18:34:55 +0800 Subject: [PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support In-Reply-To: References: <20210411162901.7238-1-marex@denx.de> <20210411162901.7238-24-marex@denx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 this work. I think something is missing from previous patch set https://lists.denx.de/pipermail/u-boot/2020-September/426757.html Yours, Paul Tim Harvey ? 2021/4/20 ??8:33 ??: > 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/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 -------------- next part -------------- A non-text attachment was scrubbed... Name: OpenPGP_signature Type: application/pgp-signature Size: 840 bytes Desc: OpenPGP digital signature URL: