| Message ID | 20221207192426.45591-3-marex@denx.de |
|---|---|
| State | Superseded |
| Delegated to: | Patrice Chotard |
| Headers | show |
| Series | [v3,1/4] ARM: stm32: Fix ECDSA authentication with Dcache enabled | expand |
Hi, On 12/7/22 20:24, Marek Vasut wrote: > The ROM API table pointer is no longer accessible from U-Boot, fix > this by passing the ROM API pointer through. This makes it possible > for U-Boot to call ROM API functions to authenticate payload like > signed fitImages. > > Signed-off-by: Marek Vasut <marex@denx.de> > --- > Cc: Alexandru Gagniuc <mr.nuke.me@gmail.com> > Cc: Patrice Chotard <patrice.chotard@foss.st.com> > Cc: Patrick Delaunay <patrick.delaunay@foss.st.com> > --- > V2: - Rename image_entry_noargs_t to image_entry_stm32_t > - Add missing __noreturn > V3: No change > --- > arch/arm/mach-stm32mp/cpu.c | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/arch/arm/mach-stm32mp/cpu.c b/arch/arm/mach-stm32mp/cpu.c > index ee59866bb73..dc4112d5e6c 100644 > --- a/arch/arm/mach-stm32mp/cpu.c > +++ b/arch/arm/mach-stm32mp/cpu.c > @@ -22,6 +22,7 @@ > #include <dm/device.h> > #include <dm/uclass.h> > #include <linux/bitops.h> > +#include <spl.h> > > /* > * early TLB into the .data section so that it not get cleared > @@ -413,3 +414,17 @@ uintptr_t get_stm32mp_bl2_dtb(void) > { > return nt_fw_dtb; > } > + > +#ifdef CONFIG_SPL_BUILD > +void __noreturn jump_to_image_no_args(struct spl_image_info *spl_image) > +{ > + typedef void __noreturn (*image_entry_stm32_t)(u32 romapi); > + uintptr_t romapi = get_stm32mp_rom_api_table(); > + > + image_entry_stm32_t image_entry = > + (image_entry_stm32_t)spl_image->entry_point; > + > + printf("image entry point: 0x%lx\n", spl_image->entry_point); > + image_entry(romapi); > +} > +#endif Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com> Thanks Patrick
Hi Marek On 12/7/22 20:24, Marek Vasut wrote: > The ROM API table pointer is no longer accessible from U-Boot, fix > this by passing the ROM API pointer through. This makes it possible > for U-Boot to call ROM API functions to authenticate payload like > signed fitImages. > > Signed-off-by: Marek Vasut <marex@denx.de> > --- > Cc: Alexandru Gagniuc <mr.nuke.me@gmail.com> > Cc: Patrice Chotard <patrice.chotard@foss.st.com> > Cc: Patrick Delaunay <patrick.delaunay@foss.st.com> > --- > V2: - Rename image_entry_noargs_t to image_entry_stm32_t > - Add missing __noreturn > V3: No change > --- > arch/arm/mach-stm32mp/cpu.c | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/arch/arm/mach-stm32mp/cpu.c b/arch/arm/mach-stm32mp/cpu.c > index ee59866bb73..dc4112d5e6c 100644 > --- a/arch/arm/mach-stm32mp/cpu.c > +++ b/arch/arm/mach-stm32mp/cpu.c > @@ -22,6 +22,7 @@ > #include <dm/device.h> > #include <dm/uclass.h> > #include <linux/bitops.h> > +#include <spl.h> > > /* > * early TLB into the .data section so that it not get cleared > @@ -413,3 +414,17 @@ uintptr_t get_stm32mp_bl2_dtb(void) > { > return nt_fw_dtb; > } > + > +#ifdef CONFIG_SPL_BUILD > +void __noreturn jump_to_image_no_args(struct spl_image_info *spl_image) > +{ > + typedef void __noreturn (*image_entry_stm32_t)(u32 romapi); > + uintptr_t romapi = get_stm32mp_rom_api_table(); > + > + image_entry_stm32_t image_entry = > + (image_entry_stm32_t)spl_image->entry_point; > + > + printf("image entry point: 0x%lx\n", spl_image->entry_point); > + image_entry(romapi); > +} > +#endif Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com> Thanks Patrice
diff --git a/arch/arm/mach-stm32mp/cpu.c b/arch/arm/mach-stm32mp/cpu.c index ee59866bb73..dc4112d5e6c 100644 --- a/arch/arm/mach-stm32mp/cpu.c +++ b/arch/arm/mach-stm32mp/cpu.c @@ -22,6 +22,7 @@ #include <dm/device.h> #include <dm/uclass.h> #include <linux/bitops.h> +#include <spl.h> /* * early TLB into the .data section so that it not get cleared @@ -413,3 +414,17 @@ uintptr_t get_stm32mp_bl2_dtb(void) { return nt_fw_dtb; } + +#ifdef CONFIG_SPL_BUILD +void __noreturn jump_to_image_no_args(struct spl_image_info *spl_image) +{ + typedef void __noreturn (*image_entry_stm32_t)(u32 romapi); + uintptr_t romapi = get_stm32mp_rom_api_table(); + + image_entry_stm32_t image_entry = + (image_entry_stm32_t)spl_image->entry_point; + + printf("image entry point: 0x%lx\n", spl_image->entry_point); + image_entry(romapi); +} +#endif
The ROM API table pointer is no longer accessible from U-Boot, fix this by passing the ROM API pointer through. This makes it possible for U-Boot to call ROM API functions to authenticate payload like signed fitImages. Signed-off-by: Marek Vasut <marex@denx.de> --- Cc: Alexandru Gagniuc <mr.nuke.me@gmail.com> Cc: Patrice Chotard <patrice.chotard@foss.st.com> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com> --- V2: - Rename image_entry_noargs_t to image_entry_stm32_t - Add missing __noreturn V3: No change --- arch/arm/mach-stm32mp/cpu.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+)