U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] doc: board: ti: Add optee rng support
@ 2025-04-10 13:27 Udit Kumar
  2025-04-10 14:24 ` Nishanth Menon
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Udit Kumar @ 2025-04-10 13:27 UTC (permalink / raw)
  To: trini, u-boot
  Cc: nm, d-gole, n-francis, vigneshr, m-chawdhry, mwalle, kamlesh,
	j-choudhary, Udit Kumar

J722S has hw rng, which can be used by OPTEE.
So remove option to use SW TRNG by OPTEE.

Signed-off-by: Udit Kumar <u-kumar1@ti.com>
---
reference discussion 
https://lore.kernel.org/all/20250313144155.2382316-1-mwalle@kernel.org/

Bootlogs with optee trng 
https://gist.github.com/uditkumarti/5f90210b23e67cd4eb6d06a024031ee8#file-gistfile1-txt-L1970

 doc/board/ti/j722s_evm.rst | 1 -
 1 file changed, 1 deletion(-)

diff --git a/doc/board/ti/j722s_evm.rst b/doc/board/ti/j722s_evm.rst
index e5a1be50c4f..edc29a4f9e4 100644
--- a/doc/board/ti/j722s_evm.rst
+++ b/doc/board/ti/j722s_evm.rst
@@ -74,7 +74,6 @@ Set the variables corresponding to this platform:
  $ export UBOOT_CFG_CORTEXA=j722s_evm_a53_defconfig
  $ export TFA_BOARD=lite
  $ export OPTEE_PLATFORM=k3-am62x
- $ export OPTEE_EXTRA_ARGS="CFG_WITH_SOFTWARE_PRNG=y"
 
 .. j722s_evm_rst_include_start_build_steps
 
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] doc: board: ti: Add optee rng support
  2025-04-10 13:27 [PATCH] doc: board: ti: Add optee rng support Udit Kumar
@ 2025-04-10 14:24 ` Nishanth Menon
  2025-04-10 15:27 ` Dhruva Gole
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 7+ messages in thread
From: Nishanth Menon @ 2025-04-10 14:24 UTC (permalink / raw)
  To: Udit Kumar
  Cc: trini, u-boot, d-gole, n-francis, vigneshr, m-chawdhry, mwalle,
	kamlesh, j-choudhary

On 18:57-20250410, Udit Kumar wrote:
> J722S has hw rng, which can be used by OPTEE.
> So remove option to use SW TRNG by OPTEE.
> 
> Signed-off-by: Udit Kumar <u-kumar1@ti.com>
> ---
> reference discussion 
> https://lore.kernel.org/all/20250313144155.2382316-1-mwalle@kernel.org/
> 
> Bootlogs with optee trng 
> https://gist.github.com/uditkumarti/5f90210b23e67cd4eb6d06a024031ee8#file-gistfile1-txt-L1970
> 
>  doc/board/ti/j722s_evm.rst | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/doc/board/ti/j722s_evm.rst b/doc/board/ti/j722s_evm.rst
> index e5a1be50c4f..edc29a4f9e4 100644
> --- a/doc/board/ti/j722s_evm.rst
> +++ b/doc/board/ti/j722s_evm.rst
> @@ -74,7 +74,6 @@ Set the variables corresponding to this platform:
>   $ export UBOOT_CFG_CORTEXA=j722s_evm_a53_defconfig
>   $ export TFA_BOARD=lite
>   $ export OPTEE_PLATFORM=k3-am62x
> - $ export OPTEE_EXTRA_ARGS="CFG_WITH_SOFTWARE_PRNG=y"
>  
>  .. j722s_evm_rst_include_start_build_steps
>  
> -- 
> 2.34.1
> 

Reviewed-by: Nishanth Menon <nm@ti.com>

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] doc: board: ti: Add optee rng support
  2025-04-10 13:27 [PATCH] doc: board: ti: Add optee rng support Udit Kumar
  2025-04-10 14:24 ` Nishanth Menon
@ 2025-04-10 15:27 ` Dhruva Gole
  2025-04-11 12:17 ` Michael Walle
  2025-04-16 14:10 ` Tom Rini
  3 siblings, 0 replies; 7+ messages in thread
From: Dhruva Gole @ 2025-04-10 15:27 UTC (permalink / raw)
  To: Udit Kumar
  Cc: trini, u-boot, nm, n-francis, vigneshr, m-chawdhry, mwalle,
	kamlesh, j-choudhary

On Apr 10, 2025 at 18:57:46 +0530, Udit Kumar wrote:
> J722S has hw rng, which can be used by OPTEE.
> So remove option to use SW TRNG by OPTEE.
> 
> Signed-off-by: Udit Kumar <u-kumar1@ti.com>
> ---
> reference discussion 
> https://lore.kernel.org/all/20250313144155.2382316-1-mwalle@kernel.org/
> 
> Bootlogs with optee trng 
> https://gist.github.com/uditkumarti/5f90210b23e67cd4eb6d06a024031ee8#file-gistfile1-txt-L1970
> 
>  doc/board/ti/j722s_evm.rst | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/doc/board/ti/j722s_evm.rst b/doc/board/ti/j722s_evm.rst
> index e5a1be50c4f..edc29a4f9e4 100644
> --- a/doc/board/ti/j722s_evm.rst
> +++ b/doc/board/ti/j722s_evm.rst
> @@ -74,7 +74,6 @@ Set the variables corresponding to this platform:
>   $ export UBOOT_CFG_CORTEXA=j722s_evm_a53_defconfig
>   $ export TFA_BOARD=lite
>   $ export OPTEE_PLATFORM=k3-am62x
> - $ export OPTEE_EXTRA_ARGS="CFG_WITH_SOFTWARE_PRNG=y"

Fine by me,
Reviewed-by: Dhruva Gole <d-gole@ti.com>

-- 
Best regards,
Dhruva Gole
Texas Instruments Incorporated

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] doc: board: ti: Add optee rng support
  2025-04-10 13:27 [PATCH] doc: board: ti: Add optee rng support Udit Kumar
  2025-04-10 14:24 ` Nishanth Menon
  2025-04-10 15:27 ` Dhruva Gole
@ 2025-04-11 12:17 ` Michael Walle
  2025-04-11 12:43   ` Nishanth Menon
  2025-04-16 14:10 ` Tom Rini
  3 siblings, 1 reply; 7+ messages in thread
From: Michael Walle @ 2025-04-11 12:17 UTC (permalink / raw)
  To: Udit Kumar, trini, u-boot
  Cc: nm, d-gole, n-francis, vigneshr, m-chawdhry, kamlesh, j-choudhary

[-- Attachment #1: Type: text/plain, Size: 474 bytes --]

Hi,

> J722S has hw rng, which can be used by OPTEE.
> So remove option to use SW TRNG by OPTEE.
>
> Signed-off-by: Udit Kumar <u-kumar1@ti.com>
> ---
> reference discussion 
> https://lore.kernel.org/all/20250313144155.2382316-1-mwalle@kernel.org/
>
> Bootlogs with optee trng 
> https://gist.github.com/uditkumarti/5f90210b23e67cd4eb6d06a024031ee8#file-gistfile1-txt-L1970


I can confirm the TRNG will work on u-boot as well as linux.
Thanks!

-michael

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 297 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] doc: board: ti: Add optee rng support
  2025-04-11 12:17 ` Michael Walle
@ 2025-04-11 12:43   ` Nishanth Menon
  2025-04-11 12:47     ` Michael Walle
  0 siblings, 1 reply; 7+ messages in thread
From: Nishanth Menon @ 2025-04-11 12:43 UTC (permalink / raw)
  To: Michael Walle
  Cc: Udit Kumar, trini, u-boot, d-gole, n-francis, vigneshr,
	m-chawdhry, kamlesh, j-choudhary

On 14:17-20250411, Michael Walle wrote:
> Hi,
> 
> > J722S has hw rng, which can be used by OPTEE.
> > So remove option to use SW TRNG by OPTEE.
> >
> > Signed-off-by: Udit Kumar <u-kumar1@ti.com>
> > ---
> > reference discussion 
> > https://lore.kernel.org/all/20250313144155.2382316-1-mwalle@kernel.org/
> >
> > Bootlogs with optee trng 
> > https://gist.github.com/uditkumarti/5f90210b23e67cd4eb6d06a024031ee8#file-gistfile1-txt-L1970
> 
> 
> I can confirm the TRNG will work on u-boot as well as linux.
> Thanks!

just curious: Did you have to change .configs for either?


-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] doc: board: ti: Add optee rng support
  2025-04-11 12:43   ` Nishanth Menon
@ 2025-04-11 12:47     ` Michael Walle
  0 siblings, 0 replies; 7+ messages in thread
From: Michael Walle @ 2025-04-11 12:47 UTC (permalink / raw)
  To: Nishanth Menon
  Cc: Udit Kumar, trini, u-boot, d-gole, n-francis, vigneshr,
	m-chawdhry, kamlesh, j-choudhary

[-- Attachment #1: Type: text/plain, Size: 832 bytes --]

Hi,

> > > J722S has hw rng, which can be used by OPTEE.
> > > So remove option to use SW TRNG by OPTEE.
> > >
> > > Signed-off-by: Udit Kumar <u-kumar1@ti.com>
> > > ---
> > > reference discussion 
> > > https://lore.kernel.org/all/20250313144155.2382316-1-mwalle@kernel.org/
> > >
> > > Bootlogs with optee trng 
> > > https://gist.github.com/uditkumarti/5f90210b23e67cd4eb6d06a024031ee8#file-gistfile1-txt-L1970
> > 
> > 
> > I can confirm the TRNG will work on u-boot as well as linux.
> > Thanks!
>
> just curious: Did you have to change .configs for either?

We are using the (arm64) defconfig in linux, so no.

Regarding u-boot, I don't know because we are using our own config
and I had to enable CONFIG_OPTEE there (as well as
CONFIG_OPTEE_SERVICE_DISCOVERY, but that one is default y).

-michael

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 297 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] doc: board: ti: Add optee rng support
  2025-04-10 13:27 [PATCH] doc: board: ti: Add optee rng support Udit Kumar
                   ` (2 preceding siblings ...)
  2025-04-11 12:17 ` Michael Walle
@ 2025-04-16 14:10 ` Tom Rini
  3 siblings, 0 replies; 7+ messages in thread
From: Tom Rini @ 2025-04-16 14:10 UTC (permalink / raw)
  To: u-boot, Udit Kumar
  Cc: nm, d-gole, n-francis, vigneshr, m-chawdhry, mwalle, kamlesh,
	j-choudhary

On Thu, 10 Apr 2025 18:57:46 +0530, Udit Kumar wrote:

> J722S has hw rng, which can be used by OPTEE.
> So remove option to use SW TRNG by OPTEE.
> 
> 

Applied to u-boot/master, thanks!

[1/1] doc: board: ti: Add optee rng support
      commit: a8baac546fd9bfbf3f36af3d35b5a284a9664f4c
-- 
Tom



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2025-04-16 14:12 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-04-10 13:27 [PATCH] doc: board: ti: Add optee rng support Udit Kumar
2025-04-10 14:24 ` Nishanth Menon
2025-04-10 15:27 ` Dhruva Gole
2025-04-11 12:17 ` Michael Walle
2025-04-11 12:43   ` Nishanth Menon
2025-04-11 12:47     ` Michael Walle
2025-04-16 14:10 ` Tom Rini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox