* [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI
@ 2020-08-28 10:15 Ross Burton
2020-08-28 10:15 ` [PATCH 2/4] arm-bsp/trusted-firmware-a: remove duplicated Mbed TLS tarball Ross Burton
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Ross Burton @ 2020-08-28 10:15 UTC (permalink / raw)
To: meta-arm
From: Ross Burton <ross.burton@arm.com>
trusted-firmware-a.inc was conditionally adding the Mbed TLS tarball to
SRC_URI, but all of the versioned recipes subsequently assigned over the
top of it so the Mbed TLS tarball was never actually in SRC_URI. This
resulted in machine overrides needing to add the tarball themselves.
Solve by using _append so that the SRC_URI is actually changed.
Change-Id: I05cf1dec2c6422a40a42c615fb2b9b6e8d6f1cb0
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc b/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc
index e1cffdf..7845dac 100644
--- a/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc
+++ b/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc
@@ -38,7 +38,7 @@ SRC_URI_MBEDTLS ??= ""
# This should be set to MBEDTLS LIC FILES checksum
LIC_FILES_CHKSUM_MBEDTLS ??= ""
# add MBEDTLS to our sources if activated
-SRC_URI += "${@bb.utils.contains('TFA_MBEDTLS', '1', '${SRC_URI_MBEDTLS}', '', d)}"
+SRC_URI_append = " ${@bb.utils.contains('TFA_MBEDTLS', '1', '${SRC_URI_MBEDTLS}', '', d)}"
# add mbed TLS chksum
LIC_FILES_CHKSUM += "${@bb.utils.contains('TFA_MBEDTLS', '1', '${LIC_FILES_CHKSUM_MBEDTLS}', '', d)}"
# add mbed TLS to version
--
2.28.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/4] arm-bsp/trusted-firmware-a: remove duplicated Mbed TLS tarball
2020-08-28 10:15 [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI Ross Burton
@ 2020-08-28 10:15 ` Ross Burton
2020-08-28 10:15 ` [PATCH 3/4] arm/trusted-firmware-a: don't build all targets in parallel Ross Burton
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Ross Burton @ 2020-08-28 10:15 UTC (permalink / raw)
To: meta-arm
From: Ross Burton <ross.burton@arm.com>
Now that trusted-firmware-a is actually adding the Mbed TLS tarball, the
machine-specific includes can stop doing it themselves.
Change-Id: I45754b8a4eba99c8fbf7c99e61d01f2a27da1b4a
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
.../trusted-firmware-a/trusted-firmware-a-sgi575.inc | 2 --
.../recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc | 2 --
2 files changed, 4 deletions(-)
diff --git a/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-sgi575.inc b/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-sgi575.inc
index 6d04fa3..5020dea 100644
--- a/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-sgi575.inc
+++ b/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-sgi575.inc
@@ -1,7 +1,5 @@
# SGI575 specific TFA support
-SRC_URI_append = " ${SRC_URI_MBEDTLS}"
-
COMPATIBLE_MACHINE = "sgi575"
TFA_PLATFORM = "sgi575"
TFA_BUILD_TARGET = "all fip"
diff --git a/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc b/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc
index d22108a..6049947 100644
--- a/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc
+++ b/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc
@@ -3,8 +3,6 @@
# Intermediate SHA with 2.3 baseline version
SRCREV_tfa = "2fe7d18b0d99fdf4c4dbf84863bb2036dfebc537"
-SRC_URI_append = " ${SRC_URI_MBEDTLS}"
-
DEPENDS += "scp-firmware"
COMPATIBLE_MACHINE = "tc0"
--
2.28.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 3/4] arm/trusted-firmware-a: don't build all targets in parallel
2020-08-28 10:15 [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI Ross Burton
2020-08-28 10:15 ` [PATCH 2/4] arm-bsp/trusted-firmware-a: remove duplicated Mbed TLS tarball Ross Burton
@ 2020-08-28 10:15 ` Ross Burton
2020-08-28 10:15 ` [PATCH 4/4] arm-bsp/trusted-firmware-a: remove redundant MBEDTLS_DIR assignment Ross Burton
2020-08-31 20:27 ` [meta-arm] [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI Jon Mason
3 siblings, 0 replies; 5+ messages in thread
From: Ross Burton @ 2020-08-28 10:15 UTC (permalink / raw)
To: meta-arm
From: Ross Burton <ross.burton@arm.com>
There are build races in the Makefiles when you build all targets at
once (missing dependencies):
| tools/fiptool/fiptool create [...]
| ERROR: fopen WORKDIR/build/juno/debug/fdts/juno_fw_config.dtb: No such file or directory
These are non-trivial to fix so whilst upstream works on this we can
just build each target serially.
Change-Id: Iba0d0ae7de6faf199efea073135fd3b7232b81a1
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
.../recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc b/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc
index 7845dac..8b19d68 100644
--- a/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc
+++ b/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc
@@ -122,7 +122,10 @@ do_compile() {
sed -i '/^INCLUDE_PATHS/ s,$, \$\{BUILD_CFLAGS},' ${S}/tools/fiptool/Makefile
sed -i 's^OPENSSL_DIR.*=.*$^OPENSSL_DIR = ${STAGING_DIR_NATIVE}/${prefix_native}^' ${S}/tools/*/Makefile
- oe_runmake ${TFA_BUILD_TARGET}
+ # Currently there are races if you build all the targets at once in parallel
+ for T in ${TFA_BUILD_TARGET}; do
+ oe_runmake $T
+ done
}
do_compile[cleandirs] = "${B}"
--
2.28.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 4/4] arm-bsp/trusted-firmware-a: remove redundant MBEDTLS_DIR assignment
2020-08-28 10:15 [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI Ross Burton
2020-08-28 10:15 ` [PATCH 2/4] arm-bsp/trusted-firmware-a: remove duplicated Mbed TLS tarball Ross Burton
2020-08-28 10:15 ` [PATCH 3/4] arm/trusted-firmware-a: don't build all targets in parallel Ross Burton
@ 2020-08-28 10:15 ` Ross Burton
2020-08-31 20:27 ` [meta-arm] [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI Jon Mason
3 siblings, 0 replies; 5+ messages in thread
From: Ross Burton @ 2020-08-28 10:15 UTC (permalink / raw)
To: meta-arm
From: Ross Burton <ross.burton@arm.com>
The base recipe already does this, so there's no need to do it again.
Change-Id: I04f679ab267db3bcdbb812dce6be2efc8a25b633
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
.../trusted-firmware-a/trusted-firmware-a-sgi575.inc | 2 +-
.../recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-sgi575.inc b/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-sgi575.inc
index 5020dea..e63be96 100644
--- a/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-sgi575.inc
+++ b/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-sgi575.inc
@@ -12,5 +12,5 @@ TFA_UBOOT = "0"
DEPENDS += "virtual/uefi-firmware"
EXTRA_OEMAKE += "BL33=${RECIPE_SYSROOT}/firmware/uefi.bin"
-EXTRA_OEMAKE += "MBEDTLS_DIR=mbedtls TRUSTED_BOARD_BOOT=1 GENERATE_COT=1 ARM_ROTPK_LOCATION=devel_rsa \
+EXTRA_OEMAKE += "TRUSTED_BOARD_BOOT=1 GENERATE_COT=1 ARM_ROTPK_LOCATION=devel_rsa \
ROT_KEY=plat/arm/board/common/rotpk/arm_rotprivk_rsa.pem"
diff --git a/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc b/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc
index 6049947..e2f41f1 100644
--- a/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc
+++ b/meta-arm-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-tc0.inc
@@ -15,5 +15,5 @@ TFA_MBEDTLS = "1"
TFA_DEBUG = "1"
EXTRA_OEMAKE += "SCP_BL2=${RECIPE_SYSROOT}/firmware/scp_ramfw.bin"
-EXTRA_OEMAKE += "MBEDTLS_DIR=mbedtls TRUSTED_BOARD_BOOT=1 GENERATE_COT=1 ARM_ROTPK_LOCATION=devel_rsa \
+EXTRA_OEMAKE += "TRUSTED_BOARD_BOOT=1 GENERATE_COT=1 ARM_ROTPK_LOCATION=devel_rsa \
ROT_KEY=plat/arm/board/common/rotpk/arm_rotprivk_rsa.pem"
--
2.28.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [meta-arm] [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI
2020-08-28 10:15 [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI Ross Burton
` (2 preceding siblings ...)
2020-08-28 10:15 ` [PATCH 4/4] arm-bsp/trusted-firmware-a: remove redundant MBEDTLS_DIR assignment Ross Burton
@ 2020-08-31 20:27 ` Jon Mason
3 siblings, 0 replies; 5+ messages in thread
From: Jon Mason @ 2020-08-31 20:27 UTC (permalink / raw)
To: Ross Burton; +Cc: meta-arm
On Fri, Aug 28, 2020 at 11:15:42AM +0100, Ross Burton wrote:
> From: Ross Burton <ross.burton@arm.com>
>
> trusted-firmware-a.inc was conditionally adding the Mbed TLS tarball to
> SRC_URI, but all of the versioned recipes subsequently assigned over the
> top of it so the Mbed TLS tarball was never actually in SRC_URI. This
> resulted in machine overrides needing to add the tarball themselves.
>
> Solve by using _append so that the SRC_URI is actually changed.
>
> Change-Id: I05cf1dec2c6422a40a42c615fb2b9b6e8d6f1cb0
> Signed-off-by: Ross Burton <ross.burton@arm.com>
Series pulled into master.
Thanks,
Jon
> ---
> meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc b/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc
> index e1cffdf..7845dac 100644
> --- a/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc
> +++ b/meta-arm/recipes-bsp/trusted-firmware-a/trusted-firmware-a.inc
> @@ -38,7 +38,7 @@ SRC_URI_MBEDTLS ??= ""
> # This should be set to MBEDTLS LIC FILES checksum
> LIC_FILES_CHKSUM_MBEDTLS ??= ""
> # add MBEDTLS to our sources if activated
> -SRC_URI += "${@bb.utils.contains('TFA_MBEDTLS', '1', '${SRC_URI_MBEDTLS}', '', d)}"
> +SRC_URI_append = " ${@bb.utils.contains('TFA_MBEDTLS', '1', '${SRC_URI_MBEDTLS}', '', d)}"
> # add mbed TLS chksum
> LIC_FILES_CHKSUM += "${@bb.utils.contains('TFA_MBEDTLS', '1', '${LIC_FILES_CHKSUM_MBEDTLS}', '', d)}"
> # add mbed TLS to version
> --
> 2.28.0
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-08-31 20:27 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-28 10:15 [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI Ross Burton
2020-08-28 10:15 ` [PATCH 2/4] arm-bsp/trusted-firmware-a: remove duplicated Mbed TLS tarball Ross Burton
2020-08-28 10:15 ` [PATCH 3/4] arm/trusted-firmware-a: don't build all targets in parallel Ross Burton
2020-08-28 10:15 ` [PATCH 4/4] arm-bsp/trusted-firmware-a: remove redundant MBEDTLS_DIR assignment Ross Burton
2020-08-31 20:27 ` [meta-arm] [PATCH 1/4] arm/trusted-firmware-a: actually add mbedtls to SRC_URI Jon Mason
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.