On Thu, Jul 1, 2021 at 5:36 PM Fabio Estevam festevam@gmail.com wrote:
Hi Tim,
On Thu, Jul 1, 2021 at 8:58 PM Tim Harvey tharvey@gateworks.com wrote:
Fabio,
I no longer have the patches/config around and am not sure I even have a working imx8mm-evk anymore.
Do you have the patches I submitted to enable USB OTG in general on IMX8MM? usb: ehci-mx6: add IMX8MM and IMX8MN OTG support usb: ehci-mx6: move mode set/detect to probe
Thanks for pointing out these patches. Does "ums 0 mmc 0" work for you?
Yes, this works on my board. I just sent a v2 of these patches (cc'd you).
By the way, I took a quick stab at enabling SDP on imx8mm-venice-gw73xx-0x and found compile issues so I configured for an imx8mm_evk using your diffs and hit the same issues.
drivers/usb/host/ehci-mx6.c: In function ‘ehci_hcd_init’: drivers/usb/host/ehci-mx6.c:360:46: error: ‘USB_BASE_ADDR’ undeclared (first use in this function); did you mean ‘SRC_BASE_ADDR’? struct usb_ehci *ehci = (struct usb_ehci *)(USB_BASE_ADDR + ^~~~~~~~~~~~~ SRC_BASE_ADDR drivers/usb/host/ehci-mx6.c:360:46: note: each undeclared identifier is reported only once for each function it appears in drivers/usb/host/ehci-mx6.c:361:4: error: ‘controller_spacing’ undeclared (first use in this function); did you mean ‘console_start’? (controller_spacing * index)); ^~~~~~~~~~~~~~~~~~ ...
The above is resolved with CONFIG_DM_USB=y and CONFIG_SPL_DM_USB=y
After enabling those I just run into other issues with things such as: WARNING 'mkimage.flash.mkimage' not found, resulting binary is not-functional ... aarch64-linux-ld.bfd: cmd/nvedit.o: in function `env_match': /usr/src/venice/uboot-master/cmd/nvedit.c:824: undefined reference to `env_get_char' aarch64-linux-ld.bfd: /usr/src/venice/uboot-master/cmd/nvedit.c:828: undefined reference to `env_get_char' ...
What branch were you working with here? I tried the above with v2021.07-rc5
Tim