[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f20833a4-1571-41f8-875a-d27086be3090@oss.qualcomm.com>
Date: Fri, 5 Dec 2025 13:08:41 +0530
From: Harshal Dev <harshal.dev@....qualcomm.com>
To: Krzysztof Kozlowski <krzk@...nel.org>,
Bjorn Andersson <bjorn.andersson@....qualcomm.com>,
Sumit Garg <sumit.garg@...nel.org>,
Jens Wiklander <jens.wiklander@...aro.org>
Cc: Amirreza Zarrabi <amirreza.zarrabi@....qualcomm.com>,
Arnd Bergmann <arnd@...db.de>,
Geert Uytterhoeven <geert+renesas@...der.be>,
op-tee@...ts.trustedfirmware.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: defconfig: Enable QCOMTEE for Qualcomm SoCs
Hi Krzysztof,
On 12/2/2025 3:13 PM, Krzysztof Kozlowski wrote:
> On 02/12/2025 10:37, Harshal Dev wrote:
>> Enable config for the QCOMTEE driver to facilitate communication with the
>> Qualcomm Trusted Execution Environment (QTEE) on Qualcomm platforms.
>
> Which platforms use it?
The Qualcomm Trusted Execution Environment (QTEE) firmware is available on all
Qualcomm platforms. However, certain platforms like db410c (APQ8016), where QTEE
does not support Object-IPC (the communication protocol implemented by QCOMTEE driver)
a graceful -EIO is returned by qcom_scm_qtee_init() which is in-charge of dynamically
registering the QCOMTEE driver as a platform driver during qcom_scm_probe().
More details available here: https://lore.kernel.org/op-tee/20250820-qcom-tee-using-tee-ss-without-mem-obj-v8-0-7066680f138a@oss.qualcomm.com/T/#m1eba97cb921c958034ffe7e2bfb776177ba48eb6
And so, the QCOMTEE driver can be safely enabled on all Qualcomm platforms regardless
of QTEE firmware version.
> Why this cannot be a module?
Upon a bit of reflection, I have realized that it is completely fine to have this driver
built as an in-tree module. I have also validated that this presents no issues during
loading/unloading of the driver.
I will send a new patch and update the CONFIG_QCOMTEE flag to m.
Regards,
Harshal
>
>>
>> Signed-off-by: Harshal Dev <harshal.dev@....qualcomm.com>
>> ---
>> arch/arm64/configs/defconfig | 1 +
>> 1 file changed, 1 insertion(+)
>>
> Best regards,
> Krzysztof
Powered by blists - more mailing lists