On Fri, Feb 28, 2025 at 06:59:14PM -0600, Jon Humphreys wrote: Hello Jon,
> Siddharth Vadapalli <s-vadapa...@ti.com> writes: > > > Introduce the config fragment for enabling USB MSC boot. USB MSC boot > > involves fetching the next stage of the bootloader from a USB Mass Storage > > device such as a USB Flash Drive with the USB controller on the SoC acting > > as the USB Host. > > > > Signed-off-by: Siddharth Vadapalli <s-vadapa...@ti.com> > > --- > > configs/am62x_r5_usbmsc.config | 22 ++++++++++++++++++++++ > > 1 file changed, 22 insertions(+) > > create mode 100644 configs/am62x_r5_usbmsc.config > > > > diff --git a/configs/am62x_r5_usbmsc.config b/configs/am62x_r5_usbmsc.config > > new file mode 100644 > > index 0000000000..2ce6afb662 > > --- /dev/null > > +++ b/configs/am62x_r5_usbmsc.config > > @@ -0,0 +1,22 @@ > > +CONFIG_USB=y > > +CONFIG_SPL_USB_HOST=y > > +CONFIG_SPL_SYSCON=y > > +CONFIG_SPL_MISC=y > > +CONFIG_USB_XHCI_HCD=y > > +CONFIG_USB_GADGET=y > > +CONFIG_SPL_USB_GADGET=y > > +CONFIG_USB_DWC3=y > > +CONFIG_SPL_USB_DWC3_GENERIC=y > > +CONFIG_SPL_USB_DWC3_AM62=y > > +CONFIG_SPL_USB_STORAGE=y > > Hi Siddharth, > > You either need to set CONFIG_USB_STORAGE as well in > am62x_r5_usbmsc.config, or otherwise modify usb.h to include the > declaration of usb_stor_info(). > > Eg use: > > #if defined CONFIG_SPL_USB_STORAGE || defined CONFIG_USB_STORAGE > > instead of > > #ifdef CONFIG_USB_STORAGE > > See the build log: > > spl_usb.c:41:29: error: implicit declaration of function usb_stor_scan Thank you for reviewing the patch and pointing this out. I will enable CONFIG_USB_STORAGE in am62x_r5_usbmsc.config to fix this and post the v2 series. Regards, Siddharth.
