Re: [PATCH v4 1/3] tee: qcomtee: Register qcom.tz.tpm service for discovery
From: Amirreza Zarrabi
Date: Wed Oct 21 2026 - 17:17:32 EST
On 10/19/2026 8:21 PM, Kuldeep Singh wrote:
> Add "qcom.tz.tpm" (QCOMTEE_TPM_UID) to qtee_services[] so the TPM TA
> is enumerated as a TEE-bus device, allowing a client driver to bind to
> it via its generated UUID.
>
> Reviewed-by: Jarkko Sakkinen <jarkko@xxxxxxxxxx>
> Signed-off-by: Kuldeep Singh <kuldeep.singh@xxxxxxxxxxxxxxxx>
> ---
Looks fine to me.
Reviewed-by: Amirreza Zarrabi <amirreza.zarrabi@xxxxxxxxxxxxxxxx>
- Amir
> drivers/tee/qcomtee/call.c | 4 +++-
> drivers/tee/qcomtee/qcomtee_msg.h | 2 ++
> 2 files changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/tee/qcomtee/call.c b/drivers/tee/qcomtee/call.c
> index 47d8a4f9a159..1b887da83341 100644
> --- a/drivers/tee/qcomtee/call.c
> +++ b/drivers/tee/qcomtee/call.c
> @@ -747,7 +747,9 @@ static const uuid_t qtee_service_uuid_ns = UUID_INIT(0xe1b48857, 0x6154, 0x49f9,
>
> static const struct qtee_service qtee_services[] = {
> { "qcom.tz.uefisecapp",
> - QCOMTEE_UEFI_SEC_UID }
> + QCOMTEE_UEFI_SEC_UID },
> + { "qcom.tz.tpm",
> + QCOMTEE_TPM_UID }
> };
>
> static void qtee_release_service(struct device *dev)
> diff --git a/drivers/tee/qcomtee/qcomtee_msg.h b/drivers/tee/qcomtee/qcomtee_msg.h
> index 9278a361dc70..9fff9dc5fffb 100644
> --- a/drivers/tee/qcomtee/qcomtee_msg.h
> +++ b/drivers/tee/qcomtee/qcomtee_msg.h
> @@ -106,6 +106,8 @@ union qcomtee_msg_arg {
> #define QTEE_VERSION_GET_PATCH(x) ((x) >> 0 & 0xfffU)
>
> #define QCOMTEE_UEFI_SEC_UID 413
> +#define QCOMTEE_TPM_UID 81
> +
> /* Response types as returned from qcomtee_object_invoke_ctx_invoke(). */
>
> /* The message contains a callback request. */
>