Hi Kever,
On Mon, 9 Nov 2020 at 20:43, Kever Yang <kever.y...@rock-chips.com> wrote:
>
> Rockchip has many 32bit SoCs and some of them are support SPL_OPTEE now,
> only boards with SPL_OPTEE support can fit BINMAN well, other boards
> will fail at initr_binman() in U-Boot proper after below patch,
> eg. rv1108 board.
> 83187546ae binman: Support multiple images in the library
Is the problem that binman_init() adds too much code? If so, we could
have a new BINMAN_RUNTIME option that enables binman.c and thus allow
boards to disable it.
I imagine that most boards will end up using binman, but many will not
need to access the image definition at runtime.
>
> Fixes: 79030a4861 ("rockchip: Add Single boot image (with binman, pad_cat)")
> Signed-off-by: Kever Yang <kever.y...@rock-chips.com>
> ---
>
> arch/arm/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index b2f7fcbd6e..5903c09370 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1721,7 +1721,7 @@ config ARCH_STM32MP
> config ARCH_ROCKCHIP
> bool "Support Rockchip SoCs"
> select BLK
> - select BINMAN if !ARM64
> + select BINMAN if SPL_OPTEE
> select DM
> select DM_GPIO
> select DM_I2C
> --
> 2.25.1
>
>
>
Regards,
Simon
