[U-Boot] [PATCH 0/2] fix build dependency for TI secure devices

[U-Boot] [PATCH 0/2] fix build dependency for TI secure devices

[Andreas Dannenberg]

The recent "Secure Boot by Authenticating/Decrypting SPL FIT blobs"
patch series [1] introduced an undesired dependency on
CONFIG_SPL_LOAD_FIT for TI secure devices, breaking the build when
that option is not active. While this is not an immediately visible
issue as we migrated all applicable device defconfig files to use
CONFIG_SPL_LOAD_FIT, it does create an issue if someone choses to
disable that option

[1] https://www.mail-archive.com/u-boot%40lists.denx.de/msg217284.html

Andreas Dannenberg (2):
  arm: am4x: fix build dependency for secure devices
  arm: omap5: fix build dependency for secure devices

 arch/arm/cpu/armv7/am33xx/config.mk | 2 +-
 arch/arm/cpu/armv7/omap5/config.mk  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
2.7.2

[U-Boot] [PATCH 1/2] arm: am4x: fix build dependency for secure devices

[Andreas Dannenberg]

Commit e29878f introduces an undesired dependency on CONFIG_SPL_LOAD_FIT
when building U-Boot for AM43xx high-security (HS) devices that causes the
build to break when that option is not active. Fix this issue by only
building the u-boot_HS.img target when building U-Boot into an actual
FIT image.

Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
---
 arch/arm/cpu/armv7/am33xx/config.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/cpu/armv7/am33xx/config.mk b/arch/arm/cpu/armv7/am33xx/config.mk
index ab94708..d4eb21c 100644
--- a/arch/arm/cpu/armv7/am33xx/config.mk
+++ b/arch/arm/cpu/armv7/am33xx/config.mk
@@ -26,7 +26,7 @@ endif
 else
 ifeq ($(CONFIG_TI_SECURE_DEVICE),y)
 ALL-$(CONFIG_QSPI_BOOT) += u-boot_HS_XIP_X-LOADER
-ALL-y	+= u-boot_HS.img
+ALL-$(CONFIG_SPL_LOAD_FIT) += u-boot_HS.img
 endif
 ALL-y	+= u-boot.img
 endif
-- 
2.7.2

[U-Boot] [PATCH 1/2] arm: am4x: fix build dependency for secure devices

[Tom Rini]

On Wed, Jul 20, 2016 at 01:55:58PM -0500, Andreas Dannenberg wrote:

> Commit e29878f introduces an undesired dependency on CONFIG_SPL_LOAD_FIT
> when building U-Boot for AM43xx high-security (HS) devices that causes the
> build to break when that option is not active. Fix this issue by only
> building the u-boot_HS.img target when building U-Boot into an actual
> FIT image.
> 
> Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160721/0c995af3/attachment.sig>

[U-Boot] [U-Boot, 1/2] arm: am4x: fix build dependency for secure devices

[Tom Rini]

On Wed, Jul 20, 2016 at 01:55:58PM -0500, Andreas Dannenberg wrote:

> Commit e29878f introduces an undesired dependency on CONFIG_SPL_LOAD_FIT
> when building U-Boot for AM43xx high-security (HS) devices that causes the
> build to break when that option is not active. Fix this issue by only
> building the u-boot_HS.img target when building U-Boot into an actual
> FIT image.
> 
> Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160722/4e20bb20/attachment.sig>

[U-Boot] [PATCH 2/2] arm: omap5: fix build dependency for secure devices

[Andreas Dannenberg]

Commit 17c2987 introduces an undesired dependency on CONFIG_SPL_LOAD_FIT
when building U-Boot for AM57xx and DRA7xx high-security (HS) devices that
causes the build to break when that option is not active. Fix this issue
by only building the u-boot_HS.img target when building U-Boot into an
actual FIT image.

Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
---
 arch/arm/cpu/armv7/omap5/config.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/cpu/armv7/omap5/config.mk b/arch/arm/cpu/armv7/omap5/config.mk
index d245572..286ca86 100644
--- a/arch/arm/cpu/armv7/omap5/config.mk
+++ b/arch/arm/cpu/armv7/omap5/config.mk
@@ -16,7 +16,7 @@ ALL-y	+= MLO
 endif
 else
 ifeq ($(CONFIG_TI_SECURE_DEVICE),y)
-ALL-y   += u-boot_HS.img
+ALL-$(CONFIG_SPL_LOAD_FIT) += u-boot_HS.img
 endif
 ALL-y	+= u-boot.img
 endif
-- 
2.7.2

[U-Boot] [PATCH 2/2] arm: omap5: fix build dependency for secure devices

[Tom Rini]

On Wed, Jul 20, 2016 at 01:55:59PM -0500, Andreas Dannenberg wrote:

> Commit 17c2987 introduces an undesired dependency on CONFIG_SPL_LOAD_FIT
> when building U-Boot for AM57xx and DRA7xx high-security (HS) devices that
> causes the build to break when that option is not active. Fix this issue
> by only building the u-boot_HS.img target when building U-Boot into an
> actual FIT image.
> 
> Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160721/b35e1cd3/attachment.sig>

[U-Boot] [U-Boot, 2/2] arm: omap5: fix build dependency for secure devices

[Tom Rini]

On Wed, Jul 20, 2016 at 01:55:59PM -0500, Andreas Dannenberg wrote:

> Commit 17c2987 introduces an undesired dependency on CONFIG_SPL_LOAD_FIT
> when building U-Boot for AM57xx and DRA7xx high-security (HS) devices that
> causes the build to break when that option is not active. Fix this issue
> by only building the u-boot_HS.img target when building U-Boot into an
> actual FIT image.
> 
> Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160722/238e75be/attachment.sig>