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

[Jon Humphreys]

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

thanks
Jon

> +# CONFIG_SPL_MMC is not set
> +# CONFIG_SPL_LIBDISK_SUPPORT is not set
> +# CONFIG_SPL_MTD is not set
> +# CONFIG_SPL_SPI is not set
> +# CONFIG_SPL_SYS_MALLOC is not set
> +# CONFIG_CMD_GPT is not set
> +# CONFIG_CMD_MMC is not set
> +# CONFIG_CMD_FAT is not set
> +# CONFIG_MMC_SDHCI is not set
> +# CONFIG_CMD_DFU is not set
> +# CONFIG_DFU is not set
> -- 
> 2.34.1