[U-Boot] [PATCH 1/3] spl: update bl31_entry() with new name and parameter
Dr. Philipp Tomsich
philipp.tomsich at theobroma-systems.com
Thu Sep 7 08:36:40 UTC 2017
Kever,
> On 7 Sep 2017, at 09:00, Kever Yang <kever.yang at rock-chips.com> wrote:
>
> Rename the bl31_entry() function with 'spl_' prefix and add one
> parameter for bl31 entry address since we can get it from spl_image header.
Just as a heads-up, I also have a patch in my queue (i.e. to submit this week) that
integrates as IH_OS_ARM_TRUSTED_FIRMWARE and adds the option to pass
a pointer to the FDT as a platform-parameter.
I’ll copy you, so we can then figure out how to best merge your and my changes
and apply those at the same time.
Regards,
Philipp.
>
> Signed-off-by: Kever Yang <kever.yang at rock-chips.com>
> ---
>
> common/spl/spl.c | 2 +-
> common/spl/spl_atf.c | 4 ++--
> include/spl.h | 6 +++++-
> 3 files changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/common/spl/spl.c b/common/spl/spl.c
> index 6ff390c..1d8bbb2 100644
> --- a/common/spl/spl.c
> +++ b/common/spl/spl.c
> @@ -427,7 +427,7 @@ void board_init_r(gd_t *dummy1, ulong dummy2)
>
> if (CONFIG_IS_ENABLED(ATF_SUPPORT)) {
> debug("loaded - jumping to U-Boot via ATF BL31.\n");
> - bl31_entry();
> + spl_bl31_entry((void *)spl_image.entry_point);
> }
>
> if (CONFIG_IS_ENABLED(OPTEE_SUPPORT)) {
> diff --git a/common/spl/spl_atf.c b/common/spl/spl_atf.c
> index 6e8f928..2f6fff9 100644
> --- a/common/spl/spl_atf.c
> +++ b/common/spl/spl_atf.c
> @@ -82,13 +82,13 @@ void raw_write_daif(unsigned int daif)
> __asm__ __volatile__("msr DAIF, %0\n\t" : : "r" (daif) : "memory");
> }
>
> -void bl31_entry(void)
> +void spl_bl31_entry(void *entry_addr)
> {
> struct bl31_params *bl31_params;
> void (*entry)(struct bl31_params *params, void *plat_params) = NULL;
>
> bl31_params = bl2_plat_get_bl31_params();
> - entry = (void *)CONFIG_SPL_ATF_TEXT_BASE;
> + entry = entry_addr;
>
> raw_write_daif(SPSR_EXCEPTION_MASK);
> dcache_disable();
> diff --git a/include/spl.h b/include/spl.h
> index 13d46ed..d21a3d2 100644
> --- a/include/spl.h
> +++ b/include/spl.h
> @@ -267,7 +267,11 @@ int spl_dfu_cmd(int usbctrl, char *dfu_alt_info, char *interface, char *devstr);
> int spl_mmc_load_image(struct spl_image_info *spl_image,
> struct spl_boot_device *bootdev);
>
> -void bl31_entry(void);
> +/**
> + * spl_bl31_entry - entry function for ATF bl31
> + * @entry_addr - entry address of bl31 text
> + */
> +void spl_bl31_entry(void *entry_addr);
>
> /**
> * spl_optee_entry - entry function for optee
> --
> 1.9.1
>
More information about the U-Boot
mailing list