Re: [PATCH] configs: stm32mp1: remove splashimage and add fdtoverlay_addr_r

[Patrice CHOTARD <patrice.chotard@foss.st.com>]

Hi Patrick

On 6/28/21 2:42 PM, Patrick Delaunay wrote:
> Add the variable used by PXE command for fdtoverlays support
> since the commit 69076dff2284 ("cmd: pxe: add support for FDT overlays").
> 
> Reused the unused "splashimage" address as CONFIG_SPLASH_SOURCE and
> CONFIG_VIDEO_LOGO are not activated and U-Boot display the "BACKGROUND"
> image found in extlinux.conf to manage splashscreen on stm32mp1 boards.
> 
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
> 
>  include/configs/stm32mp1.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/configs/stm32mp1.h b/include/configs/stm32mp1.h
> index 440efa1a55..2e7f49e7bb 100644
> --- a/include/configs/stm32mp1.h
> +++ b/include/configs/stm32mp1.h
> @@ -155,7 +155,7 @@
>  
>  /*
>   * memory layout for 32M uncompressed/compressed kernel,
> - * 1M fdt, 1M script, 1M pxe and 1M for splashimage
> + * 1M fdt, 1M script, 1M pxe and 1M for overlay
>   * and the ramdisk at the end.
>   */
>  #define CONFIG_EXTRA_ENV_SETTINGS \
> @@ -163,7 +163,7 @@
>  	"fdt_addr_r=0xc4000000\0" \
>  	"scriptaddr=0xc4100000\0" \
>  	"pxefile_addr_r=0xc4200000\0" \
> -	"splashimage=0xc4300000\0"  \
> +	"fdtoverlay_addr_r=0xc4300000\0" \
>  	"ramdisk_addr_r=0xc4400000\0" \
>  	"altbootcmd=run bootcmd\0" \
>  	"env_check=if env info -p -d -q; then env save; fi\0" \
> 
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

Thanks
Patrice