On 10/03/2018 04:39 AM, Rajesh Bhagat wrote: > Defines BOOTCOMMAND for TFABOOT configuration for > supported boot sources.
Please keep the commit message format consistent, to wrap back at or near 70th characters through the entire patch set. > > Signed-off-by: Pankit Garg <pankit.g...@nxp.com> > Signed-off-by: Rajesh Bhagat <rajesh.bha...@nxp.com> > --- > include/configs/ls1046aqds.h | 9 +++++++++ > include/configs/ls1046ardb.h | 7 +++++++ > 2 files changed, 16 insertions(+) > > diff --git a/include/configs/ls1046aqds.h b/include/configs/ls1046aqds.h > index 765a21e00f..2a30cef0e4 100644 > --- a/include/configs/ls1046aqds.h > +++ b/include/configs/ls1046aqds.h > @@ -432,6 +432,14 @@ unsigned long get_board_ddr_clk(void); > #define CONFIG_CMDLINE_TAG > > #undef CONFIG_BOOTCOMMAND > +#ifdef CONFIG_TFABOOT > +#define QSPI_NOR_BOOTCOMMAND "sf probe && sf read $kernel_load " \ > + "e0000 f00000 && bootm $kernel_load" > +#define IFC_NOR_BOOTCOMMAND "cp.b $kernel_start $kernel_load " \ > + "$kernel_size && bootm $kernel_load" > +#define SD_BOOTCOMMAND "mmc info; mmc read $kernel_load" \ > + "$kernel_addr_sd $kernel_size_sd && > bootm $kernel_load" > +#else > #if defined(CONFIG_QSPI_BOOT) || defined(CONFIG_SD_BOOT_QSPI) > #define CONFIG_BOOTCOMMAND "sf probe && sf read $kernel_load " \ > "e0000 f00000 && bootm $kernel_load" > @@ -439,6 +447,7 @@ unsigned long get_board_ddr_clk(void); > #define CONFIG_BOOTCOMMAND "cp.b $kernel_start $kernel_load " \ > "$kernel_size && bootm $kernel_load" > #endif > +#endif > > #include <asm/fsl_secure_boot.h> > > diff --git a/include/configs/ls1046ardb.h b/include/configs/ls1046ardb.h > index b8d9419b48..cc1f5f5f55 100644 > --- a/include/configs/ls1046ardb.h > +++ b/include/configs/ls1046ardb.h > @@ -216,6 +216,12 @@ > > #ifndef SPL_NO_MISC > #undef CONFIG_BOOTCOMMAND > +#ifdef CONFIG_TFABOOT > +#define QSPI_NOR_BOOTCOMMAND "run distro_bootcmd; run qspi_bootcmd; " > \ > + "env exists secureboot && esbc_halt;;" > +#define SD_BOOTCOMMAND "run distro_bootcmd;run sd_bootcmd; " \ > + "env exists secureboot && esbc_halt;" > +#else > #if defined(CONFIG_QSPI_BOOT) > #define CONFIG_BOOTCOMMAND "run distro_bootcmd; run qspi_bootcmd; " \ > "env exists secureboot && esbc_halt;;" > @@ -224,6 +230,7 @@ > "env exists secureboot && esbc_halt;" > #endif > #endif > +#endif > > #include <asm/fsl_secure_boot.h> > > Shouldn't this patch be put before patch #19? Did you compile each patch incrementally? York _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot