[PATCH 1/2] configs: am62x_r5: introduce fragment for USB MSC boot

[Siddharth Vadapalli]

On Fri, Feb 28, 2025 at 06:59:14PM -0600, Jon Humphreys wrote:

Hello Jon,

> Siddharth Vadapalli <s-vadapalli at 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-vadapalli at 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.