| Message ID | 20231106-enable-trng-v3-1-e93d7d41ccd4@ti.com |
|---|---|
| State | Superseded |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [v3] optee-os: Enable OP-TEE TRNG in AM62X and AM62PX | expand |
On 11/8/2023 8:35 AM, Kamlesh Gurudasani wrote: > Enable TRNG in AM62X and AM62PX, as bugs related to TRNG has been fixed now. > > TF-A fix: > https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git/commit/?id=d7a7135d32a8c7da004c0c19b75bd4e2813f9759 > > OP-TEE fix: > https://github.com/OP-TEE/optee_os/commit/e313f4765fd0478bb66985827441411793433773 > > Signed-off-by: Kamlesh Gurudasani <kamlesh@ti.com> > --- > Changes in V3: > -updated commit title and message > > Changes in V2: > -updated commit message > -included changes for AM62p > --- > meta-ti-bsp/recipes-security/optee/optee-os-ti.inc | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/meta-ti-bsp/recipes-security/optee/optee-os-ti.inc b/meta-ti-bsp/recipes-security/optee/optee-os-ti.inc > index 91b66a36..435efced 100644 > --- a/meta-ti-bsp/recipes-security/optee/optee-os-ti.inc > +++ b/meta-ti-bsp/recipes-security/optee/optee-os-ti.inc You missed my request on the first patch: When you send the V2 series, can you base the commit off of the kirkstone-wip branch. I have applied some changes to the file you are changing above (renamed it in this case) so your patch would not apply currently. Thanks. Right now I cannot apply this patch as I have already accepted the renaming patches into the next branch and it is staged for prod release as soon as CICD passes. Please rebase your patch on the kirkstone-wip branch and move your changes to the optee-os-ti-overrides.inc file. > @@ -6,8 +6,8 @@ inherit ti-secdev > > EXTRA_OEMAKE:append:k3 = "${@ ' CFG_CONSOLE_UART='+ d.getVar('OPTEE_K3_USART') if d.getVar('OPTEE_K3_USART') else ''}" > > -EXTRA_OEMAKE:append:am62xx = " CFG_WITH_SOFTWARE_PRNG=y CFG_TEE_CORE_LOG_LEVEL=1" > -EXTRA_OEMAKE:append:am62pxx = " CFG_WITH_SOFTWARE_PRNG=y CFG_TEE_CORE_LOG_LEVEL=1" > +EXTRA_OEMAKE:append:am62xx = " CFG_TEE_CORE_LOG_LEVEL=1" > +EXTRA_OEMAKE:append:am62pxx = " CFG_TEE_CORE_LOG_LEVEL=1" > EXTRA_OEMAKE:append:am62axx = " CFG_TEE_CORE_LOG_LEVEL=1" > > do_compile:append:k3() { > > --- > base-commit: d5e768e29b3f8091cbdedb702ff4185f73af4f3d > change-id: 20231106-enable-trng-e56f79d668e8 > > Best regards,
diff --git a/meta-ti-bsp/recipes-security/optee/optee-os-ti.inc b/meta-ti-bsp/recipes-security/optee/optee-os-ti.inc index 91b66a36..435efced 100644 --- a/meta-ti-bsp/recipes-security/optee/optee-os-ti.inc +++ b/meta-ti-bsp/recipes-security/optee/optee-os-ti.inc @@ -6,8 +6,8 @@ inherit ti-secdev EXTRA_OEMAKE:append:k3 = "${@ ' CFG_CONSOLE_UART='+ d.getVar('OPTEE_K3_USART') if d.getVar('OPTEE_K3_USART') else ''}" -EXTRA_OEMAKE:append:am62xx = " CFG_WITH_SOFTWARE_PRNG=y CFG_TEE_CORE_LOG_LEVEL=1" -EXTRA_OEMAKE:append:am62pxx = " CFG_WITH_SOFTWARE_PRNG=y CFG_TEE_CORE_LOG_LEVEL=1" +EXTRA_OEMAKE:append:am62xx = " CFG_TEE_CORE_LOG_LEVEL=1" +EXTRA_OEMAKE:append:am62pxx = " CFG_TEE_CORE_LOG_LEVEL=1" EXTRA_OEMAKE:append:am62axx = " CFG_TEE_CORE_LOG_LEVEL=1" do_compile:append:k3() {
Enable TRNG in AM62X and AM62PX, as bugs related to TRNG has been fixed now. TF-A fix: https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git/commit/?id=d7a7135d32a8c7da004c0c19b75bd4e2813f9759 OP-TEE fix: https://github.com/OP-TEE/optee_os/commit/e313f4765fd0478bb66985827441411793433773 Signed-off-by: Kamlesh Gurudasani <kamlesh@ti.com> --- Changes in V3: -updated commit title and message Changes in V2: -updated commit message -included changes for AM62p --- meta-ti-bsp/recipes-security/optee/optee-os-ti.inc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- base-commit: d5e768e29b3f8091cbdedb702ff4185f73af4f3d change-id: 20231106-enable-trng-e56f79d668e8 Best regards,