Re: [PATCH 8/9] ARM: Rockchip: Use compressed OP-TEE binary

[Sascha Hauer <s.hauer@pengutronix.de>]

On Mon, Mar 16, 2026 at 06:21:21PM +0100, Sascha Hauer wrote:
> Rockchip images are often size constraint due to the ROM limiting the
> maximum image size to 2MiB, at least when redundant copies on SD/eMMC
> are desired. For this reason use the compressed variant of the OP-TEE
> image.
> 
> FIXME: Uncompress address currently hardcoded.

This comment is not true anymore. In the current patch I use the memory
directly behind the image aligned up to the next 1MiB boundary.

> 
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> ---
>  arch/arm/mach-rockchip/atf.c | 26 ++++++++++++++++++++------
>  1 file changed, 20 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/mach-rockchip/atf.c b/arch/arm/mach-rockchip/atf.c
> index 0d26dfb740..2ba7f074ce 100644
> --- a/arch/arm/mach-rockchip/atf.c
> +++ b/arch/arm/mach-rockchip/atf.c
> @@ -8,6 +8,7 @@
>  #include <asm/atf_common.h>
>  #include <asm/barebox-arm.h>
>  #include <asm-generic/memory_layout.h>
> +#include <asm-generic/sections.h>
>  #include <mach/rockchip/dmc.h>
>  #include <mach/rockchip/rockchip.h>
>  #include <mach/rockchip/bootrom.h>
> @@ -29,7 +30,12 @@ static void rk_scratch_save_optee_hdr(const struct optee_header *hdr)
>  	rk_scratch->optee_hdr = *hdr;
>  }
>  
> -static unsigned long load_elf64_image_phdr(const void *elf)
> +static void *free_mem(void)
> +{
> +	return (void *)PTR_ALIGN(&__image_end, SZ_1M);
> +}
> +
> +static unsigned long load_elf64_image_phdr(struct fwobj *bl31)

This last line should be changed in the next patch, not this one.

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |