All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jon Mason" <jdmason@kudzu.us>
To: Yann Dirson <yann.dirson@blade-group.com>
Cc: meta-arm@lists.yoctoproject.org, Yann Dirson <yann@blade-group.com>
Subject: Re: [meta-arm] [PATCH v2 3/6] arm/optee-os: deploy tee.elf for u-boot
Date: Mon, 19 Apr 2021 22:30:19 -0400	[thread overview]
Message-ID: <20210420023019.GA27826@kudzu.us> (raw)
In-Reply-To: <20210414134726.4125294-4-yann@blade-group.com>

I applied the series, but didn't catch the warning/error that Ross
emntioned.  So, I had to revert this patch.  Could you please correct
it and resubmit?

Thanks,
Jon

On Wed, Apr 14, 2021 at 03:47:23PM +0200, Yann Dirson wrote:
> From: Yann Dirson <yann@blade-group.com>
> 
> Integration of OP-TEE in the u-boot image is done with `TEE=tee.elf`,
> so this build artifact must be exported.
> 
> U-boot integration can now be done in a BSP layer with something like
> this in `u-boot_%.bbappend`:
> 
>  # OP-TEE, dependent on "optee" DISTRO_FEATURES
> 
>  OPTEE_OEMAKE ?= " TEE=${DEPLOY_DIR_IMAGE}/optee/tee.elf"
> 
>  EXTRA_OEMAKE_append = " ${PACKAGECONFIG_CONFARGS}"
>  PACKAGECONFIG[optee] = "${OPTEE_OEMAKE},,optee-os"
>  PACKAGECONFIG_append = " ${@bb.utils.filter('DISTRO_FEATURES', 'optee', d)}"
> 
> Signed-off-by: Yann Dirson <yann@blade-group.com>
> ---
>  meta-arm/recipes-security/optee/optee-os.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-arm/recipes-security/optee/optee-os.inc b/meta-arm/recipes-security/optee/optee-os.inc
> index 34f586a..08cc2f0 100644
> --- a/meta-arm/recipes-security/optee/optee-os.inc
> +++ b/meta-arm/recipes-security/optee/optee-os.inc
> @@ -45,7 +45,7 @@ do_compile[cleandirs] = "${B}"
>  do_install() {
>      #install core in firmware
>      install -d ${D}${nonarch_base_libdir}/firmware/
> -    install -m 644 ${B}/core/*.bin ${D}${nonarch_base_libdir}/firmware/
> +    install -m 644 ${B}/core/*.bin ${B}/core/tee.elf ${D}${nonarch_base_libdir}/firmware/
>  
>      #install TA devkit
>      install -d ${D}${includedir}/optee/export-user_ta/
> -- 
> 2.30.2
> 

> 
> 
> 


  parent reply	other threads:[~2021-04-20  2:30 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-14 13:47 [PATCH v2 0/6] improving OP-TEE support for u-boot-based platforms Yann Dirson
2021-04-14 13:47 ` [PATCH v2 1/6] arm/optee: use pycryptodome rather pycryptodomex Yann Dirson
2021-04-14 13:47 ` [PATCH v2 2/6] arm/optee-os: mark LDFLAGS patch as submitted Yann Dirson
2021-04-14 13:47 ` [PATCH v2 3/6] arm/optee-os: deploy tee.elf for u-boot Yann Dirson
2021-04-19 16:34   ` [meta-arm] " Ross Burton
2021-04-20  0:17     ` Denys Dmytriyenko
2021-04-20  7:54       ` Yann Dirson
2021-04-20  9:35         ` Yann Dirson
2021-04-20 15:50           ` Denys Dmytriyenko
2021-04-20 17:06             ` Yann Dirson
2021-04-20  2:30   ` Jon Mason [this message]
2021-04-14 13:47 ` [PATCH v2 4/6] optee-client: use /var/lib/tee instead of /data/tee for persistent storage Yann Dirson
2021-04-14 13:47 ` [PATCH v2 5/6] optee-os: don't override upstream TA_LOG_LEVEL setting Yann Dirson
2021-04-14 13:47 ` [PATCH v2 6/6] RFC: optee-os: provide a "debug" PACKAGECONFIG flag Yann Dirson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210420023019.GA27826@kudzu.us \
    --to=jdmason@kudzu.us \
    --cc=meta-arm@lists.yoctoproject.org \
    --cc=yann.dirson@blade-group.com \
    --cc=yann@blade-group.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.