Hi Tim, On Thu, Jul 1, 2021 at 9:49 PM Tim Harvey <thar...@gateworks.com> wrote:
> Yes, this works on my board. I just sent a v2 of these patches (cc'd you). I have just tried "ums 0 mmc 1" and it works on a imx8mm-evk now after applying your patches, thanks! > 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 I am also testing against 2021.07-rc5. Please see the complete diff (your patches + imx8mm_evk defconfig changes against 2021.07-rc5): https://pastebin.com/raw/BtCD0xat Now the missing part is SDP. Hopefully we can get SDK working too. Thanks!