All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 0/2] fix build dependency for TI secure devices
@ 2016-07-20 18:55 Andreas Dannenberg
  2016-07-20 18:55 ` [U-Boot] [PATCH 1/2] arm: am4x: fix build dependency for " Andreas Dannenberg
  2016-07-20 18:55 ` [U-Boot] [PATCH 2/2] arm: omap5: " Andreas Dannenberg
  0 siblings, 2 replies; 7+ messages in thread
From: Andreas Dannenberg @ 2016-07-20 18:55 UTC (permalink / raw)
  To: u-boot

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot] [PATCH 1/2] arm: am4x: fix build dependency for secure devices
  2016-07-20 18:55 [U-Boot] [PATCH 0/2] fix build dependency for TI secure devices Andreas Dannenberg
@ 2016-07-20 18:55 ` Andreas Dannenberg
  2016-07-21 14:47   ` Tom Rini
  2016-07-23  0:13   ` [U-Boot] [U-Boot, " Tom Rini
  2016-07-20 18:55 ` [U-Boot] [PATCH 2/2] arm: omap5: " Andreas Dannenberg
  1 sibling, 2 replies; 7+ messages in thread
From: Andreas Dannenberg @ 2016-07-20 18:55 UTC (permalink / raw)
  To: u-boot

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

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [U-Boot] [PATCH 2/2] arm: omap5: fix build dependency for secure devices
  2016-07-20 18:55 [U-Boot] [PATCH 0/2] fix build dependency for TI secure devices Andreas Dannenberg
  2016-07-20 18:55 ` [U-Boot] [PATCH 1/2] arm: am4x: fix build dependency for " Andreas Dannenberg
@ 2016-07-20 18:55 ` Andreas Dannenberg
  2016-07-21 14:47   ` Tom Rini
  2016-07-23  0:13   ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 2 replies; 7+ messages in thread
From: Andreas Dannenberg @ 2016-07-20 18:55 UTC (permalink / raw)
  To: u-boot

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

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [U-Boot] [PATCH 1/2] arm: am4x: fix build dependency for secure devices
  2016-07-20 18:55 ` [U-Boot] [PATCH 1/2] arm: am4x: fix build dependency for " Andreas Dannenberg
@ 2016-07-21 14:47   ` Tom Rini
  2016-07-23  0:13   ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 0 replies; 7+ messages in thread
From: Tom Rini @ 2016-07-21 14:47 UTC (permalink / raw)
  To: u-boot

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>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot] [PATCH 2/2] arm: omap5: fix build dependency for secure devices
  2016-07-20 18:55 ` [U-Boot] [PATCH 2/2] arm: omap5: " Andreas Dannenberg
@ 2016-07-21 14:47   ` Tom Rini
  2016-07-23  0:13   ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 0 replies; 7+ messages in thread
From: Tom Rini @ 2016-07-21 14:47 UTC (permalink / raw)
  To: u-boot

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>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot] [U-Boot, 1/2] arm: am4x: fix build dependency for secure devices
  2016-07-20 18:55 ` [U-Boot] [PATCH 1/2] arm: am4x: fix build dependency for " Andreas Dannenberg
  2016-07-21 14:47   ` Tom Rini
@ 2016-07-23  0:13   ` Tom Rini
  1 sibling, 0 replies; 7+ messages in thread
From: Tom Rini @ 2016-07-23  0:13 UTC (permalink / raw)
  To: u-boot

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>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot] [U-Boot, 2/2] arm: omap5: fix build dependency for secure devices
  2016-07-20 18:55 ` [U-Boot] [PATCH 2/2] arm: omap5: " Andreas Dannenberg
  2016-07-21 14:47   ` Tom Rini
@ 2016-07-23  0:13   ` Tom Rini
  1 sibling, 0 replies; 7+ messages in thread
From: Tom Rini @ 2016-07-23  0:13 UTC (permalink / raw)
  To: u-boot

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>

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-07-23  0:13 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-20 18:55 [U-Boot] [PATCH 0/2] fix build dependency for TI secure devices Andreas Dannenberg
2016-07-20 18:55 ` [U-Boot] [PATCH 1/2] arm: am4x: fix build dependency for " Andreas Dannenberg
2016-07-21 14:47   ` Tom Rini
2016-07-23  0:13   ` [U-Boot] [U-Boot, " Tom Rini
2016-07-20 18:55 ` [U-Boot] [PATCH 2/2] arm: omap5: " Andreas Dannenberg
2016-07-21 14:47   ` Tom Rini
2016-07-23  0:13   ` [U-Boot] [U-Boot, " Tom Rini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.