[PATCH] board: beagle: beagleplay: enable OF_SYSTEM_SETUP

[Andrew Davis]

On 6/19/24 2:12 PM, Bryan Brattlof wrote:
> Unfortunately when enabling FDT fixups for the AM62x family of SoCs and
> moving TF-A to the bottom of RAM we missed the BeaglePlay. This is
> causing Linux's memory allocator to clobber TF-A and break its boot.
> 
> Enable OF_SYSTEM_SETUP to fixup the kernel's FDT to inform it of the
> actual location of the firmware
> 
> CC: Andrew Davis <afd at ti.com>
> CC: Nishanth Menon <nm at ti.com>
> CC: Robert Nelson <robertcnelson at gmail.com>
> Reported-by: Dhruva Gole <d-gole at ti.com>
> Signed-off-by: Bryan Brattlof <bb at ti.com>
> ---

Acked-by: Andrew Davis <afd at ti.com>

> Hello everyone,
> 
> Fair warning, this may turn into a philosophical discussion about the
> role of device-tree with SystemReady and U-Boot's role in enabling true
> distribution to be completely agnostic of the board it's running on.
> 
> However substantively this is simply fixing a boot regression Dhruva
> found while testing out the beagleplay.
> 
> Happy reviewing
> ~Bryan
> ---
>   board/beagle/beagleplay/Kconfig | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/board/beagle/beagleplay/Kconfig b/board/beagle/beagleplay/Kconfig
> index 7dbd833acb4cc..896a1c1be3010 100644
> --- a/board/beagle/beagleplay/Kconfig
> +++ b/board/beagle/beagleplay/Kconfig
> @@ -12,6 +12,7 @@ config TARGET_AM625_A53_BEAGLEPLAY
>   	bool "BeagleBoard.org AM625 BeaglePlay running on A53"
>   	select ARM64
>   	select BINMAN
> +	select OF_SYSTEM_SETUP
>   
>   config TARGET_AM625_R5_BEAGLEPLAY
>   	bool "BeagleBoard.org AM625 BeaglePlay running on R5"
> 
> ---
> base-commit: fe2ce09a0753634543c32cafe85eb87a625f76ca
> change-id: 20240619-play-fdt-fixup-a92e1ab872fc
> 
> Best regards,