[PATCHv3,3/5] ci/qemuarm64-secureboot.yml: add optee and tf-a

Message ID 20211115140100.24634-3-maxim.uvarov@linaro.org
State New
Headers show
Series [PATCHv3,1/5] optee: updae optee-os.inc to support external TAs | expand

Commit Message

Maxim Uvarov Nov. 15, 2021, 2 p.m. UTC
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
---
 ci/qemuarm64-secureboot.yml | 2 ++
 1 file changed, 2 insertions(+)

Comments

Jon Mason Nov. 15, 2021, 4:52 p.m. UTC | #1
On Mon, Nov 15, 2021 at 05:00:58PM +0300, Maxim Uvarov wrote:
> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
> ---
>  ci/qemuarm64-secureboot.yml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/ci/qemuarm64-secureboot.yml b/ci/qemuarm64-secureboot.yml
> index b7f3cb4..a7e89d4 100644
> --- a/ci/qemuarm64-secureboot.yml
> +++ b/ci/qemuarm64-secureboot.yml
> @@ -14,3 +14,5 @@ target:
>    - core-image-base
>    - perf
>    - optee-examples
> +  - optee-os
> +  - trusted-firmware-a

I don't think we should be added tf-a or optee-os as a target.  tf-a is
already being built as a dependency of the machine, and optee-os is a
dependency of tf-a for qemu machines.  So, this is not adding
anything, and I think the patch should be dropped.

Thanks,
Jon

> -- 
> 2.17.1
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#2386): https://lists.yoctoproject.org/g/meta-arm/message/2386
> Mute This Topic: https://lists.yoctoproject.org/mt/87069390/3616920
> Group Owner: meta-arm+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-arm/unsub [jdmason@kudzu.us]
> -=-=-=-=-=-=-=-=-=-=-=-
>

Patch

diff --git a/ci/qemuarm64-secureboot.yml b/ci/qemuarm64-secureboot.yml
index b7f3cb4..a7e89d4 100644
--- a/ci/qemuarm64-secureboot.yml
+++ b/ci/qemuarm64-secureboot.yml
@@ -14,3 +14,5 @@  target:
   - core-image-base
   - perf
   - optee-examples
+  - optee-os
+  - trusted-firmware-a