($INBOX_DIR/description missing)
 help / color / mirror / Atom feed
From: "Sverdlin, Alexander" <alexander.sverdlin@siemens.com>
To: "meta-arm@lists.yoctoproject.org"
	<meta-arm@lists.yoctoproject.org>,
	"meng.li@windriver.com" <meng.li@windriver.com>
Subject: Re: [meta-arm] [PATCH] arm/optee-client: set parameter CFG_TEE_PLUGIN_LOAD_PATH as ${libdir}
Date: Mon, 4 Mar 2024 10:21:21 +0000	[thread overview]
Message-ID: <c47249fa9711f9000a5e73a3d1057640245c0e2b.camel@siemens.com> (raw)
In-Reply-To: <20240304035527.1281859-1-Meng.Li@windriver.com>

Hello Meng!

On Mon, 2024-03-04 at 11:55 +0800, Meng Li via lists.yoctoproject.org wrote:
> In optee-client package, the CFG_TEE_PLUGIN_LOAD_PATH is set as
> /usr/lib/tee-supplicant/plugins/. But when multilib is enabled in yocto,
> the ${libdir} is /usr/lib64, the optee-examples and optee-test package
> will install the plugins into usr/lib64. This will cause that optee feature
> doesn't work when it can't find out the necessary plugin. So, set parameter
> CFG_TEE_PLUGIN_LOAD_PATH as ${libdir} to support for multilib and
> non-multilib feature.
> 
> Signed-off-by: Meng Li <Meng.Li@windriver.com>

LGTM,
Reviewed-by: Alexander Sverdlin <alexander.sverdlin@siemens.com>

> ---
>  meta-arm/recipes-security/optee/optee-client.inc | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta-arm/recipes-security/optee/optee-client.inc b/meta-arm/recipes-security/optee/optee-client.inc
> index 3b9943cc..ef03045c 100644
> --- a/meta-arm/recipes-security/optee/optee-client.inc
> +++ b/meta-arm/recipes-security/optee/optee-client.inc
> @@ -20,6 +20,7 @@ S = "${WORKDIR}/git"
>  EXTRA_OECMAKE = " \
>      -DBUILD_SHARED_LIBS=ON \
>      -DCFG_TEE_FS_PARENT_PATH='${localstatedir}/lib/tee' \
> +    -DCFG_TEE_PLUGIN_LOAD_PATH='${libdir}/tee-supplicant/plugins/' \
>  "
>  EXTRA_OECMAKE:append:toolchain-clang = " -DCFG_WERROR=0"

-- 
Alexander Sverdlin
Siemens AG
www.siemens.com

      reply	other threads:[~2024-03-04 10:21 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-04  3:55 [PATCH] arm/optee-client: set parameter CFG_TEE_PLUGIN_LOAD_PATH as ${libdir} Meng Li
2024-03-04 10:21 ` Sverdlin, Alexander [this message]

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=c47249fa9711f9000a5e73a3d1057640245c0e2b.camel@siemens.com \
    --to=alexander.sverdlin@siemens.com \
    --cc=meng.li@windriver.com \
    --cc=meta-arm@lists.yoctoproject.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).