* [U-Boot] [REGRESSION] commit e04916a "SECURE_BOOT : enable esbc_validate..." breaks MX6 [not found] <mailman.970.1424900134.22659.u-boot@lists.denx.de> @ 2015-04-02 16:19 ` Ulises Cardenas 2015-04-02 20:18 ` Tom Rini 0 siblings, 1 reply; 3+ messages in thread From: Ulises Cardenas @ 2015-04-02 16:19 UTC (permalink / raw) To: u-boot Hi, The aforementioned commit e04916a721a2069fc770412c57974d02e153ad18, causes MX6 boards to break Configuring CONFIG_SECURE_BOOT makes the u-boot build fail for any MX6 board. This was reproduced in mx6dl, mx6q and mx6sx. The log for the mx6q build is: ============== In file included from /home/ulises/Security/u-boot-imx/include/fsl_validate.h:11:0, from /home/ulises/Security/u-boot-imx/board/freescale/common/fsl_validate.c:8: /home/ulises/Security/u-boot-imx/include/fsl_sec_mon.h:27:2: error: #error Neither CONFIG_SYS_FSL_SEC_MON_LE nor CONFIG_SYS_FSL_SEC_MON_BE defined #error Neither CONFIG_SYS_FSL_SEC_MON_LE nor CONFIG_SYS_FSL_SEC_MON_BE defined ^ In file included from /home/ulises/Security/u-boot-imx/board/freescale/common/fsl_validate.c:10:0: /home/ulises/Security/u-boot-imx/include/fsl_sfp.h:29:2: error: #error Neither CONFIG_SYS_FSL_SFP_LE nor CONFIG_SYS_FSL_SFP_BE is defined #error Neither CONFIG_SYS_FSL_SFP_LE nor CONFIG_SYS_FSL_SFP_BE is defined ^ /home/ulises/Security/u-boot-imx/board/freescale/common/fsl_validate.c:19:36: fatal error: asm/arch/immap_ls102xa.h: No such file or directory #include <asm/arch/immap_ls102xa.h> ^ compilation terminated. AS arch/arm/lib/vectors.o /home/ulises/Security/u-boot-imx/scripts/Makefile.build:276: recipe for target 'board/freescale/common/fsl_validate.o' failed make[2]: *** [board/freescale/common/fsl_validate.o] Error 1 /home/ulises/Security/u-boot-imx/Makefile:1112: recipe for target 'board/freescale/common' failed make[1]: *** [board/freescale/common] Error 2 make[1]: *** Waiting for unfinished jobs.... ========== If I revert the commit, the build successfully builds and the I am able to boot the boards on secure mode. Thanks, Ulises ^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [REGRESSION] commit e04916a "SECURE_BOOT : enable esbc_validate..." breaks MX6 2015-04-02 16:19 ` [U-Boot] [REGRESSION] commit e04916a "SECURE_BOOT : enable esbc_validate..." breaks MX6 Ulises Cardenas @ 2015-04-02 20:18 ` Tom Rini 2015-04-20 19:42 ` Ulises Cardenas 0 siblings, 1 reply; 3+ messages in thread From: Tom Rini @ 2015-04-02 20:18 UTC (permalink / raw) To: u-boot On Thu, Apr 02, 2015 at 10:19:11AM -0600, Ulises Cardenas wrote: > Hi, > The aforementioned commit e04916a721a2069fc770412c57974d02e153ad18, causes MX6 boards to break > > Configuring CONFIG_SECURE_BOOT makes the u-boot build fail for any MX6 board. This was reproduced in mx6dl, mx6q and mx6sx. The log for the mx6q build is: > > ============== > In file included from /home/ulises/Security/u-boot-imx/include/fsl_validate.h:11:0, > from /home/ulises/Security/u-boot-imx/board/freescale/common/fsl_validate.c:8: > /home/ulises/Security/u-boot-imx/include/fsl_sec_mon.h:27:2: error: #error Neither CONFIG_SYS_FSL_SEC_MON_LE nor CONFIG_SYS_FSL_SEC_MON_BE defined > #error Neither CONFIG_SYS_FSL_SEC_MON_LE nor CONFIG_SYS_FSL_SEC_MON_BE defined > ^ > In file included from /home/ulises/Security/u-boot-imx/board/freescale/common/fsl_validate.c:10:0: > /home/ulises/Security/u-boot-imx/include/fsl_sfp.h:29:2: error: #error Neither CONFIG_SYS_FSL_SFP_LE nor CONFIG_SYS_FSL_SFP_BE is defined > #error Neither CONFIG_SYS_FSL_SFP_LE nor CONFIG_SYS_FSL_SFP_BE is defined > ^ > /home/ulises/Security/u-boot-imx/board/freescale/common/fsl_validate.c:19:36: fatal error: asm/arch/immap_ls102xa.h: No such file or directory > #include <asm/arch/immap_ls102xa.h> > ^ > compilation terminated. > AS arch/arm/lib/vectors.o > /home/ulises/Security/u-boot-imx/scripts/Makefile.build:276: recipe for target 'board/freescale/common/fsl_validate.o' failed > make[2]: *** [board/freescale/common/fsl_validate.o] Error 1 > /home/ulises/Security/u-boot-imx/Makefile:1112: recipe for target 'board/freescale/common' failed > make[1]: *** [board/freescale/common] Error 2 > make[1]: *** Waiting for unfinished jobs.... > ========== > > If I revert the commit, the build successfully builds and the I am able to boot the boards on secure mode. Well, lets start with seeing which one of those needs to be set as well for secure boot to continue to work. From there we can find a good spot to whack that default in. -- > > Thanks, > Ulises > > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150402/9f06c38f/attachment.sig> ^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [REGRESSION] commit e04916a "SECURE_BOOT : enable esbc_validate..." breaks MX6 2015-04-02 20:18 ` Tom Rini @ 2015-04-20 19:42 ` Ulises Cardenas 0 siblings, 0 replies; 3+ messages in thread From: Ulises Cardenas @ 2015-04-20 19:42 UTC (permalink / raw) To: u-boot On 04/02/2015 02:18 PM, Tom Rini wrote: > On Thu, Apr 02, 2015 at 10:19:11AM -0600, Ulises Cardenas wrote: >> Hi, >> The aforementioned commit e04916a721a2069fc770412c57974d02e153ad18, causes MX6 boards to break >> >> Configuring CONFIG_SECURE_BOOT makes the u-boot build fail for any MX6 board. This was reproduced in mx6dl, mx6q and mx6sx. The log for the mx6q build is: >> >> ============== >> In file included from /home/ulises/Security/u-boot-imx/include/fsl_validate.h:11:0, >> from /home/ulises/Security/u-boot-imx/board/freescale/common/fsl_validate.c:8: >> /home/ulises/Security/u-boot-imx/include/fsl_sec_mon.h:27:2: error: #error Neither CONFIG_SYS_FSL_SEC_MON_LE nor CONFIG_SYS_FSL_SEC_MON_BE defined >> #error Neither CONFIG_SYS_FSL_SEC_MON_LE nor CONFIG_SYS_FSL_SEC_MON_BE defined >> ^ >> In file included from /home/ulises/Security/u-boot-imx/board/freescale/common/fsl_validate.c:10:0: >> /home/ulises/Security/u-boot-imx/include/fsl_sfp.h:29:2: error: #error Neither CONFIG_SYS_FSL_SFP_LE nor CONFIG_SYS_FSL_SFP_BE is defined >> #error Neither CONFIG_SYS_FSL_SFP_LE nor CONFIG_SYS_FSL_SFP_BE is defined >> ^ >> /home/ulises/Security/u-boot-imx/board/freescale/common/fsl_validate.c:19:36: fatal error: asm/arch/immap_ls102xa.h: No such file or directory >> #include <asm/arch/immap_ls102xa.h> >> ^ >> compilation terminated. >> AS arch/arm/lib/vectors.o >> /home/ulises/Security/u-boot-imx/scripts/Makefile.build:276: recipe for target 'board/freescale/common/fsl_validate.o' failed >> make[2]: *** [board/freescale/common/fsl_validate.o] Error 1 >> /home/ulises/Security/u-boot-imx/Makefile:1112: recipe for target 'board/freescale/common' failed >> make[1]: *** [board/freescale/common] Error 2 >> make[1]: *** Waiting for unfinished jobs.... >> ========== >> >> If I revert the commit, the build successfully builds and the I am able to boot the boards on secure mode. > Well, lets start with seeing which one of those needs to be set as well > for secure boot to continue to work. From there we can find a good spot > to whack that default in. > The issue was solved by Gaurav with the following patch.: [PATCH][v2] iMX: Fix compilation error when enabling SECURE_BOOT > Move the compilation of file fsl_validate.c in MACRO CONFIG_CMD_ESBC_VALIDATE. > This file should be compiled only when the above MACRO is defined > > This caused a break in compilation of iMX platforms when compiling for SECURE_BOOT > > Signed-off-by: Gaurav Rana <gaurav.rana@freescale.com> > --- > Changes in V2: > Corrected the commit message. > > board/freescale/common/Makefile | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/board/freescale/common/Makefile b/board/freescale/common/Makefile > index 7181cac..87d0578 100644 > --- a/board/freescale/common/Makefile > +++ b/board/freescale/common/Makefile > @@ -74,8 +74,7 @@ obj-$(CONFIG_P5040DS) += p_corenet/ > obj-$(CONFIG_LS102XA_NS_ACCESS) += ns_access.o > > ifdef CONFIG_SECURE_BOOT > -obj-y += fsl_validate.o > -obj-$(CONFIG_CMD_ESBC_VALIDATE) += cmd_esbc_validate.o > +obj-$(CONFIG_CMD_ESBC_VALIDATE) += fsl_validate.o cmd_esbc_validate.o > endif > > endif ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-04-20 19:42 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <mailman.970.1424900134.22659.u-boot@lists.denx.de>
2015-04-02 16:19 ` [U-Boot] [REGRESSION] commit e04916a "SECURE_BOOT : enable esbc_validate..." breaks MX6 Ulises Cardenas
2015-04-02 20:18 ` Tom Rini
2015-04-20 19:42 ` Ulises Cardenas
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox