2021-05-21 13:44:32

by Etienne Carriere

Subject: [PATCH v2 3/4] dt-bindings: arm: OP-TEE as transport channel for SCMI messages

Introduce compatible "linaro,scmi-optee" for SCMI transport channel
based on an OP-TEE service invocation.

Cc: Rob Herring <[email protected]>
Cc: [email protected]
Signed-off-by: Etienne Carriere <[email protected]>
---
Changes since v1:
- Removed modification regarding mboxes property description.
---
Documentation/devicetree/bindings/arm/arm,scmi.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/arm/arm,scmi.txt b/Documentation/devicetree/bindings/arm/arm,scmi.txt
index 856d041b397e..ccab2ef6f6ec 100644
--- a/Documentation/devicetree/bindings/arm/arm,scmi.txt
+++ b/Documentation/devicetree/bindings/arm/arm,scmi.txt
@@ -14,7 +14,8 @@ Required properties:

The scmi node with the following properties shall be under the /firmware/ node.

-- compatible : shall be "arm,scmi" or "arm,scmi-smc" for smc/hvc transports
+- compatible : shall be "arm,scmi" or "arm,scmi-smc" for smc/hvc transports,
+ or "linaro,scmi-optee" for OP-TEE transport.
- mboxes: List of phandle and mailbox channel specifiers. When used, it should
contain exactly one or two mailboxes, one for transmitting messages
("tx") and another optional for receiving the notifications("rx") if
--
2.17.1

2021-06-02 10:55:15

by Sudeep Holla

Subject: Re: [PATCH v2 3/4] dt-bindings: arm: OP-TEE as transport channel for SCMI messages

On Fri, May 21, 2021 at 03:40:53PM +0200, Etienne Carriere wrote:
> Introduce compatible "linaro,scmi-optee" for SCMI transport channel
> based on an OP-TEE service invocation.
>
> Cc: Rob Herring <[email protected]>
> Cc: [email protected]
> Signed-off-by: Etienne Carriere <[email protected]>
> ---
> Changes since v1:
> - Removed modification regarding mboxes property description.
> ---
> Documentation/devicetree/bindings/arm/arm,scmi.txt | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,scmi.txt b/Documentation/devicetree/bindings/arm/arm,scmi.txt
> index 856d041b397e..ccab2ef6f6ec 100644
> --- a/Documentation/devicetree/bindings/arm/arm,scmi.txt
> +++ b/Documentation/devicetree/bindings/arm/arm,scmi.txt
> @@ -14,7 +14,8 @@ Required properties:
>
> The scmi node with the following properties shall be under the /firmware/ node.
>
> -- compatible : shall be "arm,scmi" or "arm,scmi-smc" for smc/hvc transports
> +- compatible : shall be "arm,scmi" or "arm,scmi-smc" for smc/hvc transports,
> + or "linaro,scmi-optee" for OP-TEE transport.

Please rebase this [1] when you post next version. It is not clear about
the mailbox or shmem properties here for this new OP-TEE transport but
I assume it will be easier to understand with yaml schema.

--
Regards,
Sudeep

[1] https://lore.kernel.org/r/[email protected]/