public inbox for op-tee@lists.trustedfirmware.org
 help / color / mirror / Atom feed
* [PATCH] optee: update outdated comment
@ 2026-01-04 17:01 Julia Lawall
  2026-01-05  5:26 ` Sumit Garg via OP-TEE
  0 siblings, 1 reply; 3+ messages in thread
From: Julia Lawall @ 2026-01-04 17:01 UTC (permalink / raw)
  To: Jens Wiklander; +Cc: kernel-janitors, Sumit Garg, op-tee, linux-kernel

The function cmd_alloc_suppl() was renamed as
optee_rpc_cmd_alloc_suppl() in commit c51a564a5b48
("optee: isolate smc abi").  Update the comment
accordingly.

Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>

---
 drivers/tee/optee/rpc.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/tee/optee/rpc.c b/drivers/tee/optee/rpc.c
index ebbbd42b0e3e..97fc5b14db0c 100644
--- a/drivers/tee/optee/rpc.c
+++ b/drivers/tee/optee/rpc.c
@@ -247,8 +247,8 @@ void optee_rpc_cmd_free_suppl(struct tee_context *ctx, struct tee_shm *shm)
 	param.u.value.c = 0;
 
 	/*
-	 * Match the tee_shm_get_from_id() in cmd_alloc_suppl() as secure
-	 * world has released its reference.
+	 * Match the tee_shm_get_from_id() in optee_rpc_cmd_alloc_suppl()
+	 * as secure world has released its reference.
 	 *
 	 * It's better to do this before sending the request to supplicant
 	 * as we'd like to let the process doing the initial allocation to


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

* Re: [PATCH] optee: update outdated comment
  2026-01-04 17:01 [PATCH] optee: update outdated comment Julia Lawall
@ 2026-01-05  5:26 ` Sumit Garg via OP-TEE
  2026-01-05 13:02   ` Jens Wiklander
  0 siblings, 1 reply; 3+ messages in thread
From: Sumit Garg via OP-TEE @ 2026-01-05  5:26 UTC (permalink / raw)
  To: Julia Lawall; +Cc: kernel-janitors, op-tee, linux-kernel

On Sun, Jan 04, 2026 at 06:01:19PM +0100, Julia Lawall wrote:
> The function cmd_alloc_suppl() was renamed as
> optee_rpc_cmd_alloc_suppl() in commit c51a564a5b48
> ("optee: isolate smc abi").  Update the comment
> accordingly.
> 
> Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>
> 
> ---
>  drivers/tee/optee/rpc.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Sumit Garg <sumit.garg@oss.qualcomm.com>

-Sumit

> 
> diff --git a/drivers/tee/optee/rpc.c b/drivers/tee/optee/rpc.c
> index ebbbd42b0e3e..97fc5b14db0c 100644
> --- a/drivers/tee/optee/rpc.c
> +++ b/drivers/tee/optee/rpc.c
> @@ -247,8 +247,8 @@ void optee_rpc_cmd_free_suppl(struct tee_context *ctx, struct tee_shm *shm)
>  	param.u.value.c = 0;
>  
>  	/*
> -	 * Match the tee_shm_get_from_id() in cmd_alloc_suppl() as secure
> -	 * world has released its reference.
> +	 * Match the tee_shm_get_from_id() in optee_rpc_cmd_alloc_suppl()
> +	 * as secure world has released its reference.
>  	 *
>  	 * It's better to do this before sending the request to supplicant
>  	 * as we'd like to let the process doing the initial allocation to
> 

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

* Re: [PATCH] optee: update outdated comment
  2026-01-05  5:26 ` Sumit Garg via OP-TEE
@ 2026-01-05 13:02   ` Jens Wiklander
  0 siblings, 0 replies; 3+ messages in thread
From: Jens Wiklander @ 2026-01-05 13:02 UTC (permalink / raw)
  To: Sumit Garg; +Cc: Julia Lawall, kernel-janitors, op-tee, linux-kernel

On Mon, Jan 5, 2026 at 6:26 AM Sumit Garg <sumit.garg@kernel.org> wrote:
>
> On Sun, Jan 04, 2026 at 06:01:19PM +0100, Julia Lawall wrote:
> > The function cmd_alloc_suppl() was renamed as
> > optee_rpc_cmd_alloc_suppl() in commit c51a564a5b48
> > ("optee: isolate smc abi").  Update the comment
> > accordingly.
> >
> > Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>
> >
> > ---
> >  drivers/tee/optee/rpc.c |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
>
> Reviewed-by: Sumit Garg <sumit.garg@oss.qualcomm.com>

I'm picking up this.

Thanks,
Jens

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

end of thread, other threads:[~2026-01-05 13:02 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-04 17:01 [PATCH] optee: update outdated comment Julia Lawall
2026-01-05  5:26 ` Sumit Garg via OP-TEE
2026-01-05 13:02   ` Jens Wiklander

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