Re: [PATCH 2/2] imx8mm_evk: Add Serial Download Protocol support

Peng Fan
2022-09-20T06:56:37Z

Hi Fabio,

Just have a question, the tcpci driver is not supported, so how
do you manage to make SDP work?

Regards,
Peng.

On 9/20/2022 8:20 AM, Fabio Estevam wrote:
> Add Serial Download Protocol support as it is a useful method to
> load flash.bin to RAM and run it via 'uuu'.
>
> With this patch, it is possible to start both U-Boot SPL and U-Boot
> proper using the following 'uuu'command:
>
> $ uuu -brun spl flash.bin
>
> Based on a patch from Marek Vasut for the imx8mm-mx8menlo board.
>
> Also, to fit the SPL binary into the internal RAM, select CONFIG_LTO
> to reduce its size.
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
> arch/arm/dts/imx8mm-evk-u-boot.dtsi | 16 ++++++++++++++++
> configs/imx8mm_evk_defconfig | 21 +++++++++++++++++++++
> 2 files changed, 37 insertions(+)
>
> diff --git a/arch/arm/dts/imx8mm-evk-u-boot.dtsi b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
> index 36fbf56bc5..d82428f8fe 100644
> --- a/arch/arm/dts/imx8mm-evk-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
> @@ -20,6 +20,10 @@
> };
> };
>
> +&aips4 {
> + u-boot,dm-spl;
> +};
> +
> &reg_usdhc2_vmmc {
> u-boot,off-on-delay-us = <20000>;
> };
> @@ -84,6 +88,18 @@
> u-boot,dm-spl;
> };
>
> +&usbmisc1 {
> + u-boot,dm-spl;
> +};
> +
> +&usbphynop1 {
> + u-boot,dm-spl;
> +};
> +
> +&usbotg1 {
> + u-boot,dm-spl;
> +};
> +
> &usdhc1 {
> u-boot,dm-spl;
> };
> diff --git a/configs/imx8mm_evk_defconfig b/configs/imx8mm_evk_defconfig
> index 7bcedcd51f..2dff26b3eb 100644
> --- a/configs/imx8mm_evk_defconfig
> +++ b/configs/imx8mm_evk_defconfig
> @@ -17,6 +17,7 @@ CONFIG_SPL_SERIAL=y
> CONFIG_SPL_DRIVERS_MISC=y
> CONFIG_SPL=y
> CONFIG_SYS_LOAD_ADDR=0x40480000
> +CONFIG_LTO=y
> CONFIG_DISTRO_DEFAULTS=y
> CONFIG_FIT=y
> CONFIG_FIT_EXTERNAL_OFFSET=0x3000
> @@ -38,6 +39,9 @@ CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y
> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x300
> CONFIG_SPL_I2C=y
> CONFIG_SPL_POWER=y
> +CONFIG_SPL_USB_HOST=y
> +CONFIG_SPL_USB_GADGET=y
> +CONFIG_SPL_USB_SDP_SUPPORT=y
> CONFIG_SPL_WATCHDOG=y
> CONFIG_SYS_MAXARGS=64
> CONFIG_SYS_CBSIZE=2048
> @@ -50,6 +54,8 @@ CONFIG_CMD_FUSE=y
> CONFIG_CMD_GPIO=y
> CONFIG_CMD_I2C=y
> CONFIG_CMD_MMC=y
> +CONFIG_CMD_USB_SDP=y
> +CONFIG_CMD_USB_MASS_STORAGE=y
> CONFIG_CMD_CACHE=y
> CONFIG_CMD_REGULATOR=y
> CONFIG_CMD_EXT4_WRITE=y
> @@ -80,9 +86,13 @@ CONFIG_PHY_ATHEROS=y
> CONFIG_PHY_GIGE=y
> CONFIG_FEC_MXC=y
> CONFIG_MII=y
> +CONFIG_SPL_PHY=y
> +CONFIG_SPL_NOP_PHY=y
> CONFIG_PINCTRL=y
> CONFIG_SPL_PINCTRL=y
> CONFIG_PINCTRL_IMX8M=y
> +CONFIG_POWER_DOMAIN=y
> +CONFIG_IMX8M_POWER_DOMAIN=y
> CONFIG_DM_PMIC=y
> CONFIG_SPL_DM_PMIC_PCA9450=y
> CONFIG_DM_REGULATOR=y
> @@ -97,4 +107,15 @@ CONFIG_SPL_SYSRESET=y
> CONFIG_SYSRESET_PSCI=y
> CONFIG_SYSRESET_WATCHDOG=y
> CONFIG_DM_THERMAL=y
> +CONFIG_USB=y
> +CONFIG_USB_EHCI_HCD=y
> +CONFIG_MXC_USB_OTG_HACTIVE=y
> +# CONFIG_USB_STORAGE is not set
> +CONFIG_USB_GADGET=y
> +CONFIG_USB_GADGET_MANUFACTURER="FSL"
> +CONFIG_USB_GADGET_VENDOR_NUM=0x0525
> +CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5
> +CONFIG_CI_UDC=y
> +CONFIG_SDP_LOADADDR=0x40400000
> +CONFIG_USB_GADGET_DOWNLOAD=y
> CONFIG_IMX_WATCHDOG=y