From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexandru Gagniuc Date: Tue, 22 Dec 2020 17:54:42 -0600 Subject: [PATCH v2 1/8] spl: fit: Drop 'length' argument to board_spl_fit_post_load() In-Reply-To: <20201216000944.2832585-1-mr.nuke.me@gmail.com> References: <20201216000944.2832585-1-mr.nuke.me@gmail.com> Message-ID: <20201222235449.460100-2-mr.nuke.me@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de The size is derived from the FIT image itself. Any alignment requirements are machine-specific and known by the board code. Thus the total length can be derived from the FIT image and knowledge of the platform. The 'length' argument is redundant. Remove it. Signed-off-by: Alexandru Gagniuc Reviewed-by: Peng Fan Reviewed-by: Simon Glass --- arch/arm/mach-imx/spl.c | 5 +++-- common/spl/spl_fit.c | 4 ++-- include/spl.h | 4 ++-- 3 files changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/arm/mach-imx/spl.c b/arch/arm/mach-imx/spl.c index aa2686bb92..11255798d3 100644 --- a/arch/arm/mach-imx/spl.c +++ b/arch/arm/mach-imx/spl.c @@ -18,6 +18,7 @@ #include #include #include +#include DECLARE_GLOBAL_DATA_PTR; @@ -318,9 +319,9 @@ ulong board_spl_fit_size_align(ulong size) return size; } -void board_spl_fit_post_load(ulong load_addr, size_t length) +void board_spl_fit_post_load(const void *fit) { - u32 offset = length - CONFIG_CSF_SIZE; + u32 offset = ALIGN(fdt_totalsize(fit), 0x1000); if (imx_hab_authenticate_image(load_addr, offset + IVT_SIZE + CSF_PAD_SIZE, diff --git a/common/spl/spl_fit.c b/common/spl/spl_fit.c index 795e2922ce..1b4a7f6b15 100644 --- a/common/spl/spl_fit.c +++ b/common/spl/spl_fit.c @@ -26,7 +26,7 @@ DECLARE_GLOBAL_DATA_PTR; #define CONFIG_SYS_BOOTM_LEN (64 << 20) #endif -__weak void board_spl_fit_post_load(ulong load_addr, size_t length) +__weak void board_spl_fit_post_load(const void *fit) { } @@ -722,7 +722,7 @@ int spl_load_simple_fit(struct spl_image_info *spl_image, spl_image->flags |= SPL_FIT_FOUND; #ifdef CONFIG_IMX_HAB - board_spl_fit_post_load((ulong)fit, size); + board_spl_fit_post_load(fit); #endif return 0; diff --git a/include/spl.h b/include/spl.h index 374a295fa3..cb30ef0827 100644 --- a/include/spl.h +++ b/include/spl.h @@ -630,9 +630,9 @@ int board_return_to_bootrom(struct spl_image_info *spl_image, /** * board_spl_fit_post_load - allow process images after loading finished - * + * @fit: Pointer to a valid Flattened Image Tree blob */ -void board_spl_fit_post_load(ulong load_addr, size_t length); +void board_spl_fit_post_load(const void *fit); /** * board_spl_fit_size_align - specific size align before processing payload -- 2.26.2