On 01/05/2018 08:00 AM, Sumit Garg wrote: > Signed-off-by: Udit Agarwal <udit.agar...@nxp.com> > Signed-off-by: Sumit Garg <sumit.g...@nxp.com> > --- > > Changes in v2: > Rebased to top of master > > arch/arm/include/asm/fsl_secure_boot.h | 16 ++++---- > board/freescale/ls1088a/MAINTAINERS | 5 +++ > .../ls1088ardb_sdcard_qspi_SECURE_BOOT_defconfig | 45 > ++++++++++++++++++++++ > include/configs/ls1088a_common.h | 15 +++++++- > 4 files changed, 72 insertions(+), 9 deletions(-) > create mode 100644 configs/ls1088ardb_sdcard_qspi_SECURE_BOOT_defconfig > > diff --git a/arch/arm/include/asm/fsl_secure_boot.h > b/arch/arm/include/asm/fsl_secure_boot.h > index ec6463d..3f30470 100644 > --- a/arch/arm/include/asm/fsl_secure_boot.h > +++ b/arch/arm/include/asm/fsl_secure_boot.h > @@ -26,6 +26,14 @@ > > #define CONFIG_KEY_REVOCATION > > +#if defined(CONFIG_FSL_LAYERSCAPE) > +/* > + * For fsl layerscape based platforms, ESBC image Address in Header > + * is 64 bit. > + */ > +#define CONFIG_ESBC_ADDR_64BIT > +#endif > + > #ifndef CONFIG_SPL_BUILD > #ifndef CONFIG_SYS_RAMBOOT > /* The key used for verification of next level images > @@ -42,14 +50,6 @@ > > #endif > > -#if defined(CONFIG_FSL_LAYERSCAPE) > -/* > - * For fsl layerscape based platforms, ESBC image Address in Header > - * is 64 bit. > - */ > -#define CONFIG_ESBC_ADDR_64BIT > -#endif > - > #ifdef CONFIG_ARCH_LS2080A > #define CONFIG_EXTRA_ENV \ > "setenv fdt_high 0xa0000000;" \ > diff --git a/board/freescale/ls1088a/MAINTAINERS > b/board/freescale/ls1088a/MAINTAINERS > index de3961d..371e5db 100644 > --- a/board/freescale/ls1088a/MAINTAINERS > +++ b/board/freescale/ls1088a/MAINTAINERS > @@ -27,3 +27,8 @@ M: Udit Agarwal <udit.agar...@nxp.com> > M: Vinitha Pillai-B57223 <vinitha.pil...@nxp.com> > S: Maintained > F: configs/ls1088ardb_qspi_SECURE_BOOT_defconfig > + > +LS1088ARDB_SD_SECURE_BOOT BOARD > +M: Sumit Garg <sumit.g...@nxp.com> > +S: Maintained > +F: configs/ls1088ardb_sdcard_qspi_SECURE_BOOT_defconfig > diff --git a/configs/ls1088ardb_sdcard_qspi_SECURE_BOOT_defconfig > b/configs/ls1088ardb_sdcard_qspi_SECURE_BOOT_defconfig > new file mode 100644 > index 0000000..ba90e64 > --- /dev/null > +++ b/configs/ls1088ardb_sdcard_qspi_SECURE_BOOT_defconfig > @@ -0,0 +1,45 @@ > +CONFIG_ARM=y > +CONFIG_TARGET_LS1088ARDB=y > +CONFIG_SPL_LIBCOMMON_SUPPORT=y > +CONFIG_SPL_LIBGENERIC_SUPPORT=y > +CONFIG_FSL_LS_PPA=y > +CONFIG_SPL_MMC_SUPPORT=y > +CONFIG_SPL_SERIAL_SUPPORT=y > +CONFIG_SPL_DRIVERS_MISC_SUPPORT=y > +CONFIG_DEFAULT_DEVICE_TREE="fsl-ls1088a-rdb" > +CONFIG_DISTRO_DEFAULTS=y > +# CONFIG_SYS_MALLOC_F is not set > +CONFIG_FIT_VERBOSE=y > +CONFIG_OF_BOARD_SETUP=y > +CONFIG_SYS_EXTRA_OPTIONS="SD_BOOT_QSPI" > +CONFIG_SECURE_BOOT=y > +CONFIG_SD_BOOT=y > +# CONFIG_USE_BOOTCOMMAND is not set > +# CONFIG_DISPLAY_BOARDINFO is not set > +CONFIG_SPL=y > +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y > +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x8b0 > +CONFIG_SPL_ENV_SUPPORT=y > +CONFIG_SPL_I2C_SUPPORT=y > +CONFIG_SPL_MPC8XXX_INIT_DDR_SUPPORT=y > +CONFIG_CMD_I2C=y > +CONFIG_CMD_MMC=y > +CONFIG_CMD_SF=y > +# CONFIG_CMD_SETEXPR is not set > +CONFIG_OF_CONTROL=y > +CONFIG_ENV_IS_IN_MMC=y
This looks wrong. For secure boot you don't use env, do you? If you agree I can drop this line when applying this patch. York _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
