Hi Andrew
On 22/05/25 22:10, Andrew Davis wrote:
> There are common MMC args for TI plats in include/environment/ti/mmc.env.
> Since we already include this, there is no need to redefine these
> MMC vars. Use the defaults.
>
> This seems like something that could have been done while refactoring
> these vars in the first place as it happened after this AM62A file
> was available hence the fixes tag.
>
> Reported-by: Chirag Shilwant <c-shilw...@ti.com>
> Fixes: 3709b529156e ("env: ti: mmc.env: Move mmc related args to common
> place")
> Signed-off-by: Andrew Davis <a...@ti.com>
> ---
> board/ti/am62ax/am62ax.env | 15 +--------------
> 1 file changed, 1 insertion(+), 14 deletions(-)
>
> diff --git a/board/ti/am62ax/am62ax.env b/board/ti/am62ax/am62ax.env
> index 96d9e1e2797..dc7af56a6f7 100644
> --- a/board/ti/am62ax/am62ax.env
> +++ b/board/ti/am62ax/am62ax.env
> @@ -17,18 +17,5 @@ mmcdev=1
> bootpart=1:2
> bootdir=/boot
> rd_spec=-
> -init_mmc=run args_all args_mmc
> -get_overlay_mmc=
> - fdt address ${fdtaddr};
> - fdt resize 0x100000;
> - for overlay in $name_overlays;
> - do;
> - load mmc ${bootpart} ${dtboaddr} ${bootdir}/${overlay} &&
> - fdt apply ${dtboaddr};
> - done;
> -get_kern_mmc=load mmc ${bootpart} ${loadaddr}
> - ${bootdir}/${name_kern}
> -get_fit_mmc=load mmc ${bootpart} ${addr_fit}
> - ${bootdir}/${name_fit}
> -partitions=name=rootfs,start=0,size=-,uuid=${uuid_gpt_rootfs}
> +
> rproc_fw_binaries= 0 /lib/firmware/am62a-mcu-r5f0_0-fw 1
> /lib/firmware/am62a-c71_0-fw
Reviewed-by: Neha Malcom Francis <n-fran...@ti.com>
--
Thanking You
Neha Malcom Francis
