From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E34233AAF65 for ; Fri, 31 Jul 2026 08:35:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785486939; cv=none; b=TyAhUJfkw3w7/ZVeozuBugK4kdbkQs0HYfYCQlxvf3w8Vq4jUh9u0huwgLK4cWa4FdRj5Sth8ChyejWJ6GvPsmXCqvrtAOxRh7d/CbBZlptGn0NPxUQfNuCoKXrSfxNRWsrooUcr56nzNxo2tji1y9x8uGkzLqERJjwq2Tn40sE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785486939; c=relaxed/simple; bh=0YcQTPpbAHI1H43ReAaIL7xaNtOaQA/qudDrLzE1Ka0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=RCkiztmUMRju8MCdUg2FCDUZ/xTpU2jnEEB8PmFC9GRN5WxA6NOfHtcfsbTjvArLf8oFhFgHVJFlGL9iqCnEWNJbGjBUMW6xorl1VQdf1ZfMhjOLxPEbZWjbyUUWVuXc3EFy6y3fTpjkxMllHnf6Zr6oxQ2zZg5dP2QQyCihf2E= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=Hh7DiGZq; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=Lqk+2rpc; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="Hh7DiGZq"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="Lqk+2rpc" Received: from pps.filterd (m0279868.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 66V8QRjb3473875 for ; Fri, 31 Jul 2026 08:35:36 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= sEyvAWBwmD7HLBGtEh2I3/0dgyK+rthm4FgKnWNYRvs=; b=Hh7DiGZqnxYb+izZ FV6GHwvrmEuAR+uR5S0754QjMDRfWE1tsG87MMymUskkoxR1cEdk5gnZ6jEQna9V XYEQamGOfvJud/49Be6AdZ8tL6/GNL51CTlWQqkNGRlWTiuyYMtSjtlYuZTgvTKR q1rwNU9hRHAi4LTX2P3MotztNIUBTDAjQ0vGxoiTcARceKqy7QmJ4PftiDzWghVn ylm91u4HjGIdq5H4P4v1/H7bBkym5JffZfTGZwZDGdSS5DOCsutjefw6XftgpfRh LZkxE3KDSNu6I6m5MkGBgd4BCgbmXyB0nK6ImwBSG35oGY7xxW/BhR/5pvE5cMNL /MU8HQ== Received: from mail-pg1-f199.google.com (mail-pg1-f199.google.com [209.85.215.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4frr6m013j-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Fri, 31 Jul 2026 08:35:36 +0000 (GMT) Received: by mail-pg1-f199.google.com with SMTP id 41be03b00d2f7-c860544c077so1635694a12.3 for ; Fri, 31 Jul 2026 01:35:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1785486935; x=1786091735; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:organization :from:content-language:references:cc:to:subject:user-agent :mime-version:date:message-id:from:to:cc:subject:date:message-id :reply-to:content-type; bh=sEyvAWBwmD7HLBGtEh2I3/0dgyK+rthm4FgKnWNYRvs=; b=Lqk+2rpctKtfQNqbJQmUR7xJeV9TQnPL5AAdtXvM+6ICu/dV3iLlLuZxCpSA33QW7B RW2Hs3wHAOTIvqe02pZzmm+qZkFJfzxLabG2UDbFGjAJt7yVncEsD8nnQ9X3HxUIstF6 mq7JfiUtCsJaYPg4WLro9lkgRflWm7AsM0tL1oAQv+V/xNrINNINkoviowGarPAxFMoB +l97OTVHPzuXc651Wrrst4qeLniUY4ZOMtPMQnfE1HQe3xgHbx0q5LmLX2Tbkv3VhtZL /pc0kdAcvVVrja5T1zQYkd8vlNqWzJJRRLZGhnX2LyOhedC7kOWOpmG3E+O/LIQMqZtN esUg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785486935; x=1786091735; h=content-transfer-encoding:content-type:in-reply-to:organization :from:content-language:references:cc:to:subject:user-agent :mime-version:date:message-id:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to:content-type; bh=sEyvAWBwmD7HLBGtEh2I3/0dgyK+rthm4FgKnWNYRvs=; b=hhSd4mU4RyidKkecse12xacc0BmikTvXWfNJV6uhe6EBoMXSKJyGn5wriaoS4f5axO O/XMsVdw/l+Qe4tbWT+LYJTxWhJ4LWIpjzckfWbaGQait68ELp2s6sb0YF/VU4Umw3cH /deHbCZlQ+VXI3iCsfMf0p/CZdoYZoXZCu9qV4MEoiwBxBnXGDvrFJQ9UeWZkPgrexdr 6rCJuX6NEXXe4jQWXfU44EEAaAEYc5dlxh5kCtfhl9OYFl40YqyRAJCix9AQapZdgBXI TJhGpdC8s4aXBq4Fdji96ESJWMG+TsYEl2zWnoxOoKV58QzeH0H+J9ORNskMawovbUeh KBgA== X-Forwarded-Encrypted: i=1; AHgh+RqaCYaEdppl1UHzdmuXoqWV0gA/RceS38Ift1ptxG7M8K365k5qkFO4l82PEFVsD9GLJpSJ6ij4D26k1YU=@vger.kernel.org X-Gm-Message-State: AOJu0Yw5L4K5uR73fF9CkNeHinDPzhdj6PRw39cheerVjTAU8Bm/a+zj TwnlrQGEQmpbr7qvM8o2QKAkbO8jIHM3eMmJijaQxrGJUGtClKxXhJdwD3Td2vfKgFyg3S9TNzF HxY01pnKKWh0s7zuxp58C1O8JZeMLbOM6qLM72KznXeCbPs4uRLsF/xS31EYkhg28pIk= X-Gm-Gg: AR+sD13cbU+G1NasJMDzJPEPOJprCLDzWtejmznZO0GLorlMoYQYuUeMGe5AOSevsqA n6ltRZ46fE/163LWqFGXtodd37+lkRv559v5c4pbumdcSteAsa+dxWrm+la76etvLGJpUE3Zrjj onqcLwJfmufGCK7+Tw2glAcmZKPrAXB9QN7UxcWWlBNekKwJkuCsTeEa+hRxl5UiLnX9FOdWHzJ yx8TIgcwY7miVFgFfIV0QhebyMYUu2/1jrLzQVCjZqIx60ywq7i0S2MDoB1iwx3ZEXKGpH7elnP VCvILfPMQPHHq3DZRpNW9ET7EvZYAeJfFIMq/eF4VVr6KolL/Dr9RyYzk7yqD/LaJ9mo0dKbf5O uaIDxtEDjj5atDhYmW0ktJDYQ/mo= X-Received: by 2002:a05:6a21:e097:b0:3c3:7d12:a98c with SMTP id adf61e73a8af0-3c91b3abb28mr1056432637.52.1785486935391; Fri, 31 Jul 2026 01:35:35 -0700 (PDT) X-Received: by 2002:a05:6a21:e097:b0:3c3:7d12:a98c with SMTP id adf61e73a8af0-3c91b3abb28mr1056414637.52.1785486934769; Fri, 31 Jul 2026 01:35:34 -0700 (PDT) Received: from [192.168.1.6] ([182.77.67.166]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-3153e18e107sm3602404eec.27.2026.07.31.01.35.29 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 31 Jul 2026 01:35:34 -0700 (PDT) Message-ID: <2e41d146-c641-444f-b406-875534c1cd98@oss.qualcomm.com> Date: Fri, 31 Jul 2026 14:05:27 +0530 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 5/6] tee: qcomtee: Add support for registering QTEE services on TEE bus To: Amirreza Zarrabi Cc: Basant Kumar , Apurupa Pattapu , Arun Kumar Neelakantam , op-tee@lists.trustedfirmware.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Dmitry Baryshkov , Konrad Dybcio , Jens Wiklander , Sumit Garg , Bjorn Andersson References: <20260722-qcom_uefisecapp_migrate_qcomtee-v2-0-b8a8fcbe4211@oss.qualcomm.com> <20260722-qcom_uefisecapp_migrate_qcomtee-v2-5-b8a8fcbe4211@oss.qualcomm.com> <886466f0-0df2-4ee0-90f7-bd7e7bd731d9@oss.qualcomm.com> Content-Language: en-US From: Harshal Dev Organization: Qualcomm In-Reply-To: <886466f0-0df2-4ee0-90f7-bd7e7bd731d9@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-GUID: MpXiZQvAHLFgFriWBB-55bt52yAxXyZy X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzMxMDA1OSBTYWx0ZWRfX8T9M06+BtSo6 I2bkaMOW6lZ1t2tdrxEDCnAY5jHSahx6qWcX10NgpjQApgrMhSdYEddKLdWim6G1/isqM3a04OB vxjAfzIfmdD1RggE9TNPB9llzRK/37DM9bbHvfeXjKpmKirPfxGiESFQQYEWyf9zxQlcJLVWysl slj8KLxjdPXu+dXHKRLPAuQq6aAdb/ICyNOI479eyo91U+Tn+uZE14lV691VCqomjc7f2bKzPyW NhbqHSBHjfPxh6BWJXYA3iPqGvpSYCbENu02XJLkm9zoXBxDU4hO9AwvvfrNTIdeA9/GsjJQRBO 3iA6Y9HupPkEemfNwwqKWAbXNobS9JWdY3HfWqrNqBxIQsusBHxpWKiOYxJuVY9i+HrEmfZmObm 5wHSFfPI8GTxA4FgMLHE1SgfHzFjD2LIR0m+M4XnX+RrbainD7Nhy8jQGP5GyvCm8Q2sQrxrSYR /XaKaG2R47oU9a8OErA== X-Authority-Analysis: v=2.4 cv=CqqPtH4D c=1 sm=1 tr=0 ts=6a6c5e58 cx=c_pps a=Oh5Dbbf/trHjhBongsHeRQ==:117 a=mrmDAJacfjHp5PIfN2e9YA==:17 a=IkcTkHD0fZMA:10 a=RAioF0-LDSMA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=ZpdpYltYx_vBUK5n70dp:22 a=EUspDBNiAAAA:8 a=-09G-BynkYMubqyH6yoA:9 a=QEXdDO2ut3YA:10 a=_Vgx9l1VpLgwpw_dHYaR:22 X-Proofpoint-Spam-Info: AW1haW4tMjYwNzMxMDA1OSBTYWx0ZWRfXwPwJi8j5QYpW HPrjyWjomAg5fnhXJqOYwPSoQopgWE6JhTManpPfmaBnlTywaRNzFdtaujpSpnkWd4TFpmfLFbM Oh/MMKvx2ZEzWCvy5dWL0+n4mRshfQk= X-Proofpoint-ORIG-GUID: MpXiZQvAHLFgFriWBB-55bt52yAxXyZy X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-07-31_03,2026-07-30_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0 lowpriorityscore=0 phishscore=0 impostorscore=0 priorityscore=1501 bulkscore=0 clxscore=1015 spamscore=0 adultscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2607310059 Hi Amir, On 29-07-2026 12:35 pm, Amirreza Zarrabi wrote: > Hi Harshal, > > On 7/22/2026 4:59 PM, Harshal Dev wrote: >> QTEE exposes certain secure services implemented either within the QTEE >> kernel or via pre-loaded Trusted Applications (TAs). Such always-available >> services can be readily accessed by TEE client drivers via QTEE's >> object-IPC protocol if the service is registered as a device on the TEE >> bus. >> >> One such service is the EFI-variables service, implemented by the >> uefisecapp TA which enables kernel clients to access EFI variables at >> runtime. >> >> Maintain a static list of such always-available secure services and add >> support for the QCOMTEE driver to register these services as devices on >> the TEE bus during probe. >> >> Signed-off-by: Harshal Dev >> --- >> drivers/tee/qcomtee/call.c | 160 ++++++++++++++++++++++++++++++++++- >> drivers/tee/qcomtee/core.c | 9 +- >> drivers/tee/qcomtee/qcomtee.h | 12 +++ >> drivers/tee/qcomtee/qcomtee_msg.h | 1 + >> drivers/tee/qcomtee/qcomtee_object.h | 3 +- >> 5 files changed, 177 insertions(+), 8 deletions(-) >> >> diff --git a/drivers/tee/qcomtee/call.c b/drivers/tee/qcomtee/call.c >> index c1bba5fbfa3e..e909955e6b21 100644 >> --- a/drivers/tee/qcomtee/call.c >> +++ b/drivers/tee/qcomtee/call.c >> @@ -662,7 +662,7 @@ static void qcomtee_get_qtee_feature_list(struct tee_context *ctx, u32 id, >> { >> struct qcomtee_object *client_env, *service; >> struct qcomtee_arg u[3] = { 0 }; >> - int result; >> + int result, error = 0; >> >> struct qcomtee_object_invoke_ctx *oic __free(kfree) = >> qcomtee_object_invoke_ctx_alloc(ctx, true); >> @@ -675,9 +675,13 @@ static void qcomtee_get_qtee_feature_list(struct tee_context *ctx, u32 id, >> >> /* Get ''FeatureVersions Service'' object. */ >> service = qcomtee_object_get_service(oic, client_env, >> - QCOMTEE_FEATURE_VER_UID); >> - if (service == NULL_QCOMTEE_OBJECT) >> + QCOMTEE_FEATURE_VER_UID, >> + &error); >> + if (service == NULL_QCOMTEE_OBJECT) { >> + if (error) >> + pr_err("Failed to get service! error: %d\n", error); >> goto out_failed; >> + } > > no need to check for !error (why new variable reuse result), Ack. will use 'result' everywhere where I am calling this. Makes sense. > just print "FeatureVersions Service unavailable (%d)", result > The message "Failed to get service! error: %d\n" is not helpful. Ack. > >> >> /* IB: Feature to query. */ >> u[0].b.addr = &id; >> @@ -697,6 +701,153 @@ static void qcomtee_get_qtee_feature_list(struct tee_context *ctx, u32 id, >> qcomtee_object_put(client_env); >> } >> >> +/** >> + * is_qcomtee_service_available() - Check if the QTEE service identified by the UID >> + * is available >> + * @ctx: TEE context. >> + * @uid: 32-bit UID of the service. >> + * >> + * Returns true if the service exists and is available. >> + * Returns false if a service is not exposed by QTEE. >> + */ >> +static bool is_qcomtee_service_available(struct tee_context *ctx, u32 uid) >> +{ >> + struct qcomtee_object *client_env; >> + struct qcomtee_object *service; >> + int error = 0; >> + bool ret = false; >> + >> + struct qcomtee_object_invoke_ctx *oic __free(kfree) = >> + qcomtee_object_invoke_ctx_alloc(ctx, true); >> + if (!oic) >> + return ret; >> + >> + client_env = qcomtee_object_get_client_env(oic); >> + if (client_env == NULL_QCOMTEE_OBJECT) >> + return ret; >> + >> + /* Get service object corresponding to the uid. */ >> + service = qcomtee_object_get_service(oic, client_env, uid, &error); >> + if (service != NULL_QCOMTEE_OBJECT) { >> + qcomtee_object_put(service); >> + ret = true; >> + } >> + >> + /* When we fail to get the service, QTEE provides the reason. */ >> + if (error) >> + pr_err("Failed to get service! error: %d\n", error); >> + > > This is not a useful message. We need to know which static service is missing: > e.g. print "%s is unavailable (%d)", qcom.tz.uefisecapp, error. Ack. > Also it is possible to re-org to avoid qcomtee_object_invoke_ctx_alloc() and > qcomtee_object_get_client_env() on each iteration? oic is reusable. > Agreed, since this is called in a loop, it results in unnecessary re-allocation on every iteration and call to QTEE for getting client_env(). Ack. >> + qcomtee_object_put(client_env); >> + return ret; >> +} >> + >> +/* >> + * QTEE Service UUID name space identifier >> + * >> + * A random UUID that is allocated as a name space identifier for forming UUID's >> + * representing secure services exposed by QTEE. >> + */ >> +static const uuid_t qtee_service_uuid_ns = UUID_INIT(0xe1b48857, 0x6154, 0x49f9, >> + 0x93, 0x4e, 0xa2, 0xf2, >> + 0x0a, 0xba, 0x98, 0x42); >> + >> +static const struct qtee_service qtee_services[] = { >> + { "qcom.tz.uefisecapp", >> + QCOMTEE_UEFI_SEC_UID } >> +}; >> + >> +static void qtee_release_service(struct device *dev) >> +{ >> + struct tee_client_device *qtee_service = to_tee_client_device(dev); >> + >> + kfree(qtee_service); >> +} >> + >> +/** >> + * qtee_enumerate_service() - Enumerate a given QTEE service and register >> + * it on the TEE bus as a TEE client device >> + * @ctx: TEE context. >> + * @service_uuid: UUID of the service to be registered on the TEE bus. >> + * @uid: 32-bit UID used by QTEE to identify the service. >> + * >> + * Returns 0 on success and < 0 on failure. >> + */ >> +static int qtee_enumerate_service(struct tee_context *ctx, const char *service_name, >> + const u32 uid) >> +{ >> + struct tee_client_device *qtee_service; >> + uuid_t service_uuid; >> + int rc; >> + >> + if (!is_qcomtee_service_available(ctx, uid)) >> + return -ENXIO; > > -EOPNOTSUPP? Ack. > >> + >> + tee_generate_uuid_v5(&service_uuid, &qtee_service_uuid_ns, service_name, >> + strlen(service_name)); >> + >> + qtee_service = kzalloc_obj(*qtee_service); >> + if (!qtee_service) >> + return -ENOMEM; >> + >> + qtee_service->dev.bus = &tee_bus_type; >> + qtee_service->dev.release = qtee_release_service; >> + if (dev_set_name(&qtee_service->dev, "qtee-svc-%pUb", &service_uuid)) { >> + kfree(qtee_service); >> + return -ENOMEM; >> + } >> + uuid_copy(&qtee_service->id.uuid, &service_uuid); >> + >> + rc = device_register(&qtee_service->dev); >> + if (rc) { >> + pr_err("QTEE service registration failed, err: %d\n", rc); >> + put_device(&qtee_service->dev); >> + kfree(qtee_service); > > It is double free!? is not put_device enough? > Thank you for catching this, I did not realize that put_device frees the entire struct. Ack. >> + return rc; >> + } >> + >> + return 0; >> +} >> + >> +/** >> + * qtee_enumerate_services() - Enumerate all the secure services exposed by QTEE >> + * from the static 'qtee_services' list and register them on the TEE bus as >> + * TEE client devices. >> + * >> + * Not all versions of QTEE support a given service. Hence, we try to >> + * enumerate as many services from the 'qtee_services' list as possible. >> + * Not being able to enumerate a service shouldn't cause the driver probe >> + * to fail since none of the services in the list are mandatory for >> + * establishing communication with QTEE. >> + * @ctx: TEE context. >> + */ >> +static void qtee_enumerate_services(struct tee_context *ctx) >> +{ >> + int rc; >> + u32 idx; >> + >> + for (idx = 0; idx < ARRAY_SIZE(qtee_services); idx++) { >> + rc = qtee_enumerate_service(ctx, qtee_services[idx].name, >> + qtee_services[idx].uid); >> + if (rc == -ENXIO) >> + pr_err("QTEE does not implement service %d.\n", >> + qtee_services[idx].uid); > > If you print in is_qcomtee_service_available(), why here again? > Ack, will remove from here. Regards, Harshal > - Amir > >> + } >> +} >> + >> +static int qtee_unregister_service(struct device *dev, void *data) >> +{ >> + if (!strncmp(dev_name(dev), "qtee-svc", strlen("qtee-svc"))) >> + device_unregister(dev); >> + >> + return 0; >> +} >> + >> +static void qtee_unregister_services(void) >> +{ >> + bus_for_each_dev(&tee_bus_type, NULL, NULL, >> + qtee_unregister_service); >> +} >> + >> static const struct tee_driver_ops qcomtee_ops = { >> .get_version = qcomtee_get_version, >> .open = qcomtee_open, >> @@ -778,6 +929,8 @@ static int qcomtee_probe(struct platform_device *pdev) >> QTEE_VERSION_GET_MINOR(qcomtee->qtee_version), >> QTEE_VERSION_GET_PATCH(qcomtee->qtee_version)); >> >> + qtee_enumerate_services(qcomtee->ctx); >> + >> return 0; >> >> err_dest_wq: >> @@ -807,6 +960,7 @@ static void qcomtee_remove(struct platform_device *pdev) >> { >> struct qcomtee *qcomtee = platform_get_drvdata(pdev); >> >> + qtee_unregister_services(); >> teedev_close_context(qcomtee->ctx); >> /* Wait for RELEASE operations to be processed for QTEE objects. */ >> tee_device_unregister(qcomtee->teedev); >> diff --git a/drivers/tee/qcomtee/core.c b/drivers/tee/qcomtee/core.c >> index b1cb50e434f0..4e39e867c3e9 100644 >> --- a/drivers/tee/qcomtee/core.c >> +++ b/drivers/tee/qcomtee/core.c >> @@ -896,19 +896,20 @@ qcomtee_object_get_client_env(struct qcomtee_object_invoke_ctx *oic) >> >> struct qcomtee_object * >> qcomtee_object_get_service(struct qcomtee_object_invoke_ctx *oic, >> - struct qcomtee_object *client_env, u32 uid) >> + struct qcomtee_object *client_env, u32 uid, >> + int *result) >> { >> struct qcomtee_arg u[3] = { 0 }; >> - int ret, result; >> + int ret; >> >> u[0].b.addr = &uid; >> u[0].b.size = sizeof(uid); >> u[0].type = QCOMTEE_ARG_TYPE_IB; >> u[1].type = QCOMTEE_ARG_TYPE_OO; >> ret = qcomtee_object_do_invoke(oic, client_env, QCOMTEE_CLIENT_ENV_OPEN, >> - u, &result); >> + u, result); >> >> - if (ret || result) >> + if (ret || *result) >> return NULL_QCOMTEE_OBJECT; >> >> return u[1].o; >> diff --git a/drivers/tee/qcomtee/qcomtee.h b/drivers/tee/qcomtee/qcomtee.h >> index f39bf63fd1c2..66d305a46c0a 100644 >> --- a/drivers/tee/qcomtee/qcomtee.h >> +++ b/drivers/tee/qcomtee/qcomtee.h >> @@ -17,6 +17,8 @@ >> #define QCOMTEE_OBJREF_FLAG_USER BIT(1) >> #define QCOMTEE_OBJREF_FLAG_MEM BIT(2) >> >> +#define QTEE_UUID_NS_NAME_SIZE 128 >> + >> /** >> * struct qcomtee - Main service struct. >> * @teedev: client device. >> @@ -39,6 +41,16 @@ struct qcomtee { >> u32 qtee_version; >> }; >> >> +/** >> + * struct qtee_service - A secure service exposed by QTEE identified by a 32-bit UID. >> + * @name: Name of the QTEE service. >> + * @uid: 32-bit UID used by QTEE to identify the service. >> + */ >> +struct qtee_service { >> + const char *name; >> + const u32 uid; >> +}; >> + >> void qcomtee_fetch_async_reqs(struct qcomtee_object_invoke_ctx *oic); >> struct qcomtee_object *qcomtee_idx_erase(struct qcomtee_object_invoke_ctx *oic, >> u32 idx); >> diff --git a/drivers/tee/qcomtee/qcomtee_msg.h b/drivers/tee/qcomtee/qcomtee_msg.h >> index 878f70178a5b..ecaf8db67d45 100644 >> --- a/drivers/tee/qcomtee/qcomtee_msg.h >> +++ b/drivers/tee/qcomtee/qcomtee_msg.h >> @@ -105,6 +105,7 @@ union qcomtee_msg_arg { >> #define QTEE_VERSION_GET_MINOR(x) (((x) >> 12) & 0xffU) >> #define QTEE_VERSION_GET_PATCH(x) ((x) >> 0 & 0xfffU) >> >> +#define QCOMTEE_UEFI_SEC_UID 413 >> /* Response types as returned from qcomtee_object_invoke_ctx_invoke(). */ >> >> /* The message contains a callback request. */ >> diff --git a/drivers/tee/qcomtee/qcomtee_object.h b/drivers/tee/qcomtee/qcomtee_object.h >> index 7bd6e23b038c..f4cb9b8fcbd4 100644 >> --- a/drivers/tee/qcomtee/qcomtee_object.h >> +++ b/drivers/tee/qcomtee/qcomtee_object.h >> @@ -316,6 +316,7 @@ qcomtee_object_get_client_env(struct qcomtee_object_invoke_ctx *oic); >> >> struct qcomtee_object * >> qcomtee_object_get_service(struct qcomtee_object_invoke_ctx *oic, >> - struct qcomtee_object *client_env, u32 uid); >> + struct qcomtee_object *client_env, u32 uid, >> + int *result); >> >> #endif /* QCOMTEE_OBJECT_H */ >> >