[PATCH v2] configs: am571x_hs_evm_defconfig: Reserve EMIF memory used by PPA

[Andrew Davis]

On 11/28/25 9:04 AM, Beleswar Padhi wrote:
> The AM571x SoC has 1 GB DDR space. As part of normal re-location process
> U-Boot copies itself to the top of DDR bank. However, on HS devices, the
> top 37 MB is used by PPA and is firewalled. This results in an exception
> and the boot fails.
> 
> Introduce a new defconfig for AM571x HS platforms and set
> CONFIG_SYS_MEM_TOP_HIDE to reserve the top 38 MB memory (aligned to 2MB
> as per page size for ARM32) for the same.
> 

This "SYS_MEM_TOP_HIDE" looks new, for other OMAP-class HS devices we
use "CFG_PRAM"[0], seems these two configs do the same thing, could probably
combine them in some way..

As CFG_PRAM is set based on TI_SECURE_EMIF_TOTAL_REGION_SIZE which is 32MB
for all other AM57x, why is this needed only for AM571? We build for all
the AM57x with one defconfig, so this new defconfig will go unused. Just set
the default CFG_PRAM size to 64MB (same as AM43x), should be enough for all
and doesn't hurt to have more than needed reserved to be safe.

Andrew

[0] https://github.com/u-boot/u-boot/blob/master/include/configs/am43xx_evm.h#L153

> Signed-off-by: Beleswar Padhi <b-padhi at ti.com>
> ---
> Fix boot for TI AM571x HS EVM board
> v2: Changelog:
> 1. Drop [PATCH v1 2/2] in v2 series. There is no need to add reserved
> memory node in DT as it is done by U-Boot as part of FDT fixups
> https://github.com/u-boot/u-boot/blob/master/arch/arm/mach-omap2/fdt-common.c#L72-L107
> 2. Add entry in MAINTAINERS for new config.
> 3. Update commit message to use memory size in MB than raw hex bytes
> 
> Link to v1:
> https://lore.kernel.org/all/20251128064353.1295063-1-b-padhi@ti.com/
> 
> Test logs:
> https://gist.github.com/3V3RYONE/e86b3f4551986b0400d0e0f66d6ecc66
> 
>   MAINTAINERS                     | 1 +
>   configs/am571x_hs_evm_defconfig | 3 +++
>   2 files changed, 4 insertions(+)
>   create mode 100644 configs/am571x_hs_evm_defconfig
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b43dae882b3..e5d564cd760 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1780,6 +1780,7 @@ F:	configs/am335x_hs_evm_spi_defconfig
>   F:	configs/am335x_hs_evm_uart_defconfig
>   F:	configs/am43xx_hs_evm_defconfig
>   F:	configs/am43xx_hs_evm_qspi_defconfig
> +F:	configs/am571x_hs_evm_defconfig
>   F:	configs/am57xx_hs_evm_defconfig
>   F:	configs/am57xx_hs_evm_usb_defconfig
>   F:	configs/dra7xx_hs_evm_defconfig
> diff --git a/configs/am571x_hs_evm_defconfig b/configs/am571x_hs_evm_defconfig
> new file mode 100644
> index 00000000000..7e44ee5221a
> --- /dev/null
> +++ b/configs/am571x_hs_evm_defconfig
> @@ -0,0 +1,3 @@
> +#include <configs/am57xx_hs_evm_defconfig>
> +
> +CONFIG_SYS_MEM_TOP_HIDE=0x2600000