[PATCH V4 0/2] mailbox: tmel-qmp: Introduce QCOM TMEL QMP mailbox driver
From: Sricharan R
Date: Thu Mar 27 2025 - 14:18:26 EST
From: Sricharan Ramabadhran <quic_srichara@xxxxxxxxxxx>
The QMP mailbox is the primary means of communication between TMEL
(Trust Management Engine Lite) SS and other subsystem on the SoC.
A dedicated pair of inbound and outbound mailboxes is implemented for
each subsystem/external execution environment which needs to communicate
with TMEL for security services. The inbound mailboxes are used to send
IPC requests to TME-L, which are then processed by TME-L firmware and
accordingly the responses are sent to the requestor via outbound
mailboxes.
It is an IPC transport protocol which is light weight and supports
a subset of API's. It handles link initialization, negotiation,
establishment and communication across client(APPSS/BTSS/AUDIOSS)
and server(TME-L SS).
----------------------------------------------- ---------------------------------------------------
| | | |
| SOC CLIENT | SOC | TME-L SS |
| | AHB | |
| ---------- --------- --------- | | ------ ------- -------- ------------ |
| | | | | | | | WO | | | R | | | | |SERVICES | |
| | APPS |<-->| TMEL |<->| |------------->| | IN |-->| | | TMEL | |-------- | |
| | | | COM | | QMP | | RO | | | W | QMP |<--->| COM |<-->| a) ATTEST | |
| | | | | | |<-------------| | OUT |<--| | | | | b) CRYPTO | |
| | | | | | | | | | | | | | | | .. more | |
| --------- --------- --------- | | ------ ------- ------- ------------ |
| | | |
----------------------------------------------- --------------------------------------------------
TME-L SS provides different kinds of services like secureboot,
remote image authentication, key management, crypto, OEM provisioning etc.
This patch adds support for remote image authentication.
Support for rest of the services can be added.
Remote proc driver subscribes to this mailbox and uses the
mbox_send_message to use TME-L to securely authenticate/teardown the
images.
Since clients like same rproc driver use SCM/TMEL across socs, the goal
here was to abstract the TMEL-QMP SS functionality, so that clients should
be able to connect and send messages with a common API.
[V4]
Fixed TME-L naming in all places and expanded it.
Folded tmel_work in tmel.
Added more kernel doc as relevant.
Removed __packed in all places, as not required.
Renamed all functions to have tmel_ prefixes.
Used readl/writel in all places.
Added Inline for all required functions.
Removed redundant type conversions.
Removed redundant 'goto's
Added __free macro
Fixed Linux std errno in tmel_sec_boot_auth/teardown
Added spinlock in qmp_startup
Used of_mbox_index_xlate and dropped the tmel_qmp_mbox_xlate
Updated header file to have only mbox consumer required and moved rest to .c file
Fixed the TMEL_MSG macros to use standard GENMASK
Moved the irq registration to end of probe
Following tests were done and no issues.
*) Checkpatch
*) Codespell
*) Sparse
*) kerneldoc check
*) Kernel lock debugging
*) dt_binding_check and dtbs_check
[V3]
Fixed wrong code/comments wrappings.
Fixed Kconfig and Makefile entries to right place.
Removed unused headers inclusion.
Fixed locking, removed the mutexes and having only tx spinlock.
Removed the use of global ptr for tmel, made it as device specific.
Replaced pr_err/pr_debug with dev_err/dev_dbg in all places.
Fixed usage of dev_err_probe.
Fixed xlate callback as per comments.
Used devm equivalents and kcalloc version as per comments.
Removed all un-nessecary wrapper macros for register access, inlined it
as per comments.
Re-organised the function layout as per comments and make it more readable.
Removed the pictures in headers files as per comments.
Used Field_prep/get as per comments.
Fixed Kernel test reported issues.
Fixed all other comments as well.
Following tests were done and no issues.
*) Checkpatch
*) Codespell
*) Sparse
*) kerneldoc check
*) Kernel lock debugging
*) dt_binding_check and dtbs_check
[v2]
Added HW description in the bindings patch.
Fixed review comments for bindings from Krzysztof and Dmitry
Changed patch#2 driver to add work for mailbox tx processing
Cleaned up patch#2 for some checkpatch warnings.
There are some checkpatch [CHECK] like below, which looks like false positive.
CHECK: Macro argument 'm' may be better as '(m)' to avoid precedence issues
#1072: FILE: include/linux/mailbox/tmelcom-qmp.h:40:
+#define TMEL_MSG_UID_CREATE(m, a) ((u32)(((m & 0xff) << 8) | (a & 0xff)))
[v1]
RFC Post
Sricharan Ramabadhran (2):
dt-bindings: mailbox: Document qcom,ipq5424-tmel
mailbox: tmelite-qmp: Introduce TMEL QMP mailbox driver
.../bindings/mailbox/qcom,ipq5424-tmel.yaml | 60 ++
drivers/mailbox/Kconfig | 10 +
drivers/mailbox/Makefile | 2 +
drivers/mailbox/qcom-tmel-qmp.c | 947 ++++++++++++++++++
include/linux/mailbox/tmelcom-qmp.h | 65 ++
5 files changed, 1084 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mailbox/qcom,ipq5424-tmel.yaml
create mode 100644 drivers/mailbox/qcom-tmel-qmp.c
create mode 100644 include/linux/mailbox/tmelcom-qmp.h
--
2.34.1