Re: [PATCH 1/3] arm/arm-bsp: optee: add version 4.9.0

Hi Hugues,

On 1/21/26 3:03 PM, Hugues KAMBA MPIANA wrote:
[...]
> diff --git a/meta-arm/recipes-security/optee/optee-os_4.9.0.bb b/meta-arm/recipes-security/optee/optee-os_4.9.0.bb
> new file mode 100644
> index 00000000..8e64a636
> --- /dev/null
> +++ b/meta-arm/recipes-security/optee/optee-os_4.9.0.bb
> @@ -0,0 +1,8 @@
> +require recipes-security/optee/optee-os.inc
> +
> +DEPENDS += "dtc-native"
> +
> +FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:"
> +

We don't need that line anymore as there are no in-tree patches for 
optee-os anymore.

> +# 4.9.0
> +SRCREV = "c2b0684fcd89929976a8726e6e3af922b48dd2c7"
> diff --git a/meta-arm/recipes-security/optee/optee-test_4.9.0.bb b/meta-arm/recipes-security/optee/optee-test_4.9.0.bb
> new file mode 100644
> index 00000000..ddb574fc
> --- /dev/null
> +++ b/meta-arm/recipes-security/optee/optee-test_4.9.0.bb
> @@ -0,0 +1,14 @@
> +require recipes-security/optee/optee-test.inc
> +
> +# v4.9.0
> +SRCREV = "b27648ea8472cceceb8dda368a965c709066f7aa"
> +
> +LIC_FILES_CHKSUM = "file://LICENSE.md;md5=a8fa504109e4cd7ea575bc49ea4be560"
> +

This isn't the actual license but a file pointing to other files. I 
would suggest to add LICENSE-BSD and LICENSE-GPL as well so that we can 
catch modifications to those files.

I've checked all commit hashes, they match the tags.

I've run `xtest` on Rockchip PX30, RK3399 and RK3588 and they all pass, so:

Tested-by: Quentin Schulz <[email protected]> # xtest on PX30, 
RK3399, RK3588

We should probably think about also passing ;tag=${PV} to the SRC_URI 
via a variable such that we can not pass it when building the _git.bb 
variant of the recipes.

Thanks!
Quentin