[meta-ti][dunfell][PATCH 1/2] optee-os: Upgrade to upstream 3.18.0

[meta-ti][dunfell][PATCH 1/2] optee-os: Upgrade to upstream 3.18.0

[Andrew Davis]

Update to the latest upstream master which is 3.18.0.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 recipes-security/optee/optee-os_%.bbappend | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/recipes-security/optee/optee-os_%.bbappend b/recipes-security/optee/optee-os_%.bbappend
index 5a25bd99..4081b324 100644
--- a/recipes-security/optee/optee-os_%.bbappend
+++ b/recipes-security/optee/optee-os_%.bbappend
@@ -1,5 +1,5 @@
-PV_ti-soc = "3.17.0+git${SRCPV}"
-SRCREV_ti-soc = "15a746d28d10df3d79d72bc9fe4a5a654b88bcca"
+PV_ti-soc = "3.18.0+git${SRCPV}"
+SRCREV_ti-soc = "8e155bae3a5eb9d1a3ed9260bd7281a7a35f5086"
 SRC_URI_ti-soc = "git://github.com/OP-TEE/optee_os.git;protocol=https"
 
 DEPENDS_append_ti-soc = " python3-cryptography-native"
-- 
2.36.1

[meta-ti][dunfell][PATCH 2/2] optee-os: Lower log level for AM62x

[Andrew Davis]

During resume from low power mode suspend OP-TEE prints some messages
on the UART. It seems this UART is not powered at this point in the
sequence breaking suspend/resume. We should track down the exact
prints and quiet them. Until we get that fix upstream, lets lower
the log level for this platform to unblock the LPM work.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 recipes-security/optee/optee-os_%.bbappend | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/recipes-security/optee/optee-os_%.bbappend b/recipes-security/optee/optee-os_%.bbappend
index 4081b324..3cda8559 100644
--- a/recipes-security/optee/optee-os_%.bbappend
+++ b/recipes-security/optee/optee-os_%.bbappend
@@ -6,7 +6,7 @@ DEPENDS_append_ti-soc = " python3-cryptography-native"
 
 EXTRA_OEMAKE_append_k3 = "${@ 'CFG_CONSOLE_UART='+ d.getVar('OPTEE_K3_USART') if d.getVar('OPTEE_K3_USART') else ''}"
 
-EXTRA_OEMAKE_append_am62xx-evm = " CFG_WITH_SOFTWARE_PRNG=y"
+EXTRA_OEMAKE_append_am62xx-evm = " CFG_WITH_SOFTWARE_PRNG=y CFG_TEE_CORE_LOG_LEVEL=1"
 EXTRA_OEMAKE_append_j721s2-evm = " CFG_WITH_SOFTWARE_PRNG=y"
 EXTRA_OEMAKE_append_j721s2-hs-evm = " CFG_WITH_SOFTWARE_PRNG=y"
 
-- 
2.36.1

RE: [meta-ti][dunfell][PATCH 2/2] optee-os: Lower log level for AM62x

[Gadiyar, Anand]

> From: Davis, Andrew <afd@ti.com>
> 
> During resume from low power mode suspend OP-TEE prints some messages
> on the UART. It seems this UART is not powered at this point in the
> sequence breaking suspend/resume. We should track down the exact
> prints and quiet them. Until we get that fix upstream, lets lower
> the log level for this platform to unblock the LPM work.
> 
> Signed-off-by: Andrew Davis <afd@ti.com>

Acked-by: Anand Gadiyar <gadiyar@ti.com>


> ---
>  recipes-security/optee/optee-os_%.bbappend | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/recipes-security/optee/optee-os_%.bbappend b/recipes-
> security/optee/optee-os_%.bbappend
> index 4081b324..3cda8559 100644
> --- a/recipes-security/optee/optee-os_%.bbappend
> +++ b/recipes-security/optee/optee-os_%.bbappend
> @@ -6,7 +6,7 @@ DEPENDS_append_ti-soc = " python3-cryptography-native"
> 
>  EXTRA_OEMAKE_append_k3 = "${@ 'CFG_CONSOLE_UART='+
> d.getVar('OPTEE_K3_USART') if d.getVar('OPTEE_K3_USART') else ''}"
> 
> -EXTRA_OEMAKE_append_am62xx-evm = " CFG_WITH_SOFTWARE_PRNG=y"
> +EXTRA_OEMAKE_append_am62xx-evm = " CFG_WITH_SOFTWARE_PRNG=y CFG_TEE_CORE_LOG_LEVEL=1"
>  EXTRA_OEMAKE_append_j721s2-evm = " CFG_WITH_SOFTWARE_PRNG=y"
>  EXTRA_OEMAKE_append_j721s2-hs-evm = " CFG_WITH_SOFTWARE_PRNG=y"
> 
> --
> 2.36.1

Re: [meta-ti][dunfell][PATCH 1/2] optee-os: Upgrade to upstream 3.18.0

[Bajjuri, Praneeth]

On 8/26/2022 2:16 PM, Andrew Davis via lists.yoctoproject.org wrote:
> Update to the latest upstream master which is 3.18.0.
> 
> Signed-off-by: Andrew Davis <afd@ti.com>

Reviewed-by: Praneeth Bajjuri <praneeth@ti.com>

> ---
>   recipes-security/optee/optee-os_%.bbappend | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/recipes-security/optee/optee-os_%.bbappend b/recipes-security/optee/optee-os_%.bbappend
> index 5a25bd99..4081b324 100644
> --- a/recipes-security/optee/optee-os_%.bbappend
> +++ b/recipes-security/optee/optee-os_%.bbappend
> @@ -1,5 +1,5 @@
> -PV_ti-soc = "3.17.0+git${SRCPV}"
> -SRCREV_ti-soc = "15a746d28d10df3d79d72bc9fe4a5a654b88bcca"
> +PV_ti-soc = "3.18.0+git${SRCPV}"
> +SRCREV_ti-soc = "8e155bae3a5eb9d1a3ed9260bd7281a7a35f5086"
>   SRC_URI_ti-soc = "git://github.com/OP-TEE/optee_os.git;protocol=https"
>   
>   DEPENDS_append_ti-soc = " python3-cryptography-native"
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#14970): https://lists.yoctoproject.org/g/meta-ti/message/14970
> Mute This Topic: https://lists.yoctoproject.org/mt/93277310/2167429
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [praneeth@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 

Re: [meta-ti][dunfell][PATCH 2/2] optee-os: Lower log level for AM62x

[Andrew Davis]

On 8/26/22 2:16 PM, Andrew Davis wrote:
> During resume from low power mode suspend OP-TEE prints some messages
> on the UART. It seems this UART is not powered at this point in the
> sequence breaking suspend/resume. We should track down the exact
> prints and quiet them. Until we get that fix upstream, lets lower
> the log level for this platform to unblock the LPM work.
> 
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>   recipes-security/optee/optee-os_%.bbappend | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/recipes-security/optee/optee-os_%.bbappend b/recipes-security/optee/optee-os_%.bbappend
> index 4081b324..3cda8559 100644
> --- a/recipes-security/optee/optee-os_%.bbappend
> +++ b/recipes-security/optee/optee-os_%.bbappend
> @@ -6,7 +6,7 @@ DEPENDS_append_ti-soc = " python3-cryptography-native"
>   
>   EXTRA_OEMAKE_append_k3 = "${@ 'CFG_CONSOLE_UART='+ d.getVar('OPTEE_K3_USART') if d.getVar('OPTEE_K3_USART') else ''}"
>   
> -EXTRA_OEMAKE_append_am62xx-evm = " CFG_WITH_SOFTWARE_PRNG=y"
> +EXTRA_OEMAKE_append_am62xx-evm = " CFG_WITH_SOFTWARE_PRNG=y CFG_TEE_CORE_LOG_LEVEL=1"


ping.

Looks like there is now a merge conflict for this, easy enough to
fix, add CFG_TEE_CORE_LOG_LEVEL=1 to the new am62xx-lp-evm also.

If you want me to re-send let me know.

Andrew


>   EXTRA_OEMAKE_append_j721s2-evm = " CFG_WITH_SOFTWARE_PRNG=y"
>   EXTRA_OEMAKE_append_j721s2-hs-evm = " CFG_WITH_SOFTWARE_PRNG=y"
>   

RE: [meta-ti][dunfell][PATCH 2/2] optee-os: Lower log level for AM62x

[Gadiyar, Anand]

> From: Davis, Andrew <afd@ti.com>
> 
> On 8/26/22 2:16 PM, Andrew Davis wrote:
> > During resume from low power mode suspend OP-TEE prints some messages
> > on the UART. It seems this UART is not powered at this point in the
> > sequence breaking suspend/resume. We should track down the exact
> > prints and quiet them. Until we get that fix upstream, lets lower
> > the log level for this platform to unblock the LPM work.
> >
> > Signed-off-by: Andrew Davis <afd@ti.com>
> > ---
> >   recipes-security/optee/optee-os_%.bbappend | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/recipes-security/optee/optee-os_%.bbappend b/recipes-
> security/optee/optee-os_%.bbappend
> > index 4081b324..3cda8559 100644
> > --- a/recipes-security/optee/optee-os_%.bbappend
> > +++ b/recipes-security/optee/optee-os_%.bbappend
> > @@ -6,7 +6,7 @@ DEPENDS_append_ti-soc = " python3-cryptography-native"
> >
> >   EXTRA_OEMAKE_append_k3 = "${@ 'CFG_CONSOLE_UART='+ d.getVar('OPTEE_K3_USART') if d.getVar('OPTEE_K3_USART') else ''}"
> >
> > -EXTRA_OEMAKE_append_am62xx-evm = " CFG_WITH_SOFTWARE_PRNG=y"
> > +EXTRA_OEMAKE_append_am62xx-evm = " CFG_WITH_SOFTWARE_PRNG=y CFG_TEE_CORE_LOG_LEVEL=1"
> 
> 
> ping.
> 
> Looks like there is now a merge conflict for this, easy enough to
> fix, add CFG_TEE_CORE_LOG_LEVEL=1 to the new am62xx-lp-evm also.
> 
> If you want me to re-send let me know.
> 
> Andrew

Probably best to resend - I thought I had added it for am62xx-lp-evm
when I sent it out but it seems I did not. Sorry.