* [PATCH v2 2/4] dt-bindings: arm: scmi property mboxes is not mandatory [not found] <20210521134055.24271-1-etienne.carriere@linaro.org> @ 2021-05-21 13:40 ` Etienne Carriere 2021-06-02 10:40 ` Sudeep Holla 2021-05-21 13:40 ` [PATCH v2 3/4] dt-bindings: arm: OP-TEE as transport channel for SCMI messages Etienne Carriere 1 sibling, 1 reply; 4+ messages in thread From: Etienne Carriere @ 2021-05-21 13:40 UTC (permalink / raw) To: linux-kernel Cc: linux-arm-kernel, Sudeep Holla, Cristian Marussi, Vincent Guittot, Etienne Carriere, Rob Herring, devicetree Since "arm,scmi-smc" compatible support, mboxes property is not mandated. This change fixes the DT bindings description accordingly. Cc: Rob Herring <robh@kernel.org> Cc: devicetree@vger.kernel.org Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org> --- Changes since v1: - New patch is the series. Addresses mboxes propery description. --- Documentation/devicetree/bindings/arm/arm,scmi.txt | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/arm/arm,scmi.txt b/Documentation/devicetree/bindings/arm/arm,scmi.txt index 667d58e0a659..856d041b397e 100644 --- a/Documentation/devicetree/bindings/arm/arm,scmi.txt +++ b/Documentation/devicetree/bindings/arm/arm,scmi.txt @@ -15,10 +15,10 @@ 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 -- mboxes: List of phandle and mailbox channel specifiers. It should contain - exactly one or two mailboxes, one for transmitting messages("tx") - and another optional for receiving the notifications("rx") if - supported. +- 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 + supported. Mandated by compatible "arm,scmi". - shmem : List of phandle pointing to the shared memory(SHM) area as per generic mailbox client binding. - #address-cells : should be '1' if the device has sub-nodes, maps to -- 2.17.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 2/4] dt-bindings: arm: scmi property mboxes is not mandatory 2021-05-21 13:40 ` [PATCH v2 2/4] dt-bindings: arm: scmi property mboxes is not mandatory Etienne Carriere @ 2021-06-02 10:40 ` Sudeep Holla 0 siblings, 0 replies; 4+ messages in thread From: Sudeep Holla @ 2021-06-02 10:40 UTC (permalink / raw) To: Etienne Carriere Cc: linux-kernel, linux-arm-kernel, Cristian Marussi, Vincent Guittot, Rob Herring, devicetree On Fri, May 21, 2021 at 03:40:52PM +0200, Etienne Carriere wrote: > Since "arm,scmi-smc" compatible support, mboxes property is not mandated. > This change fixes the DT bindings description accordingly. > > Cc: Rob Herring <robh@kernel.org> > Cc: devicetree@vger.kernel.org > Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org> > --- > Changes since v1: > - New patch is the series. Addresses mboxes propery description. > --- > Documentation/devicetree/bindings/arm/arm,scmi.txt | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/arm,scmi.txt b/Documentation/devicetree/bindings/arm/arm,scmi.txt > index 667d58e0a659..856d041b397e 100644 > --- a/Documentation/devicetree/bindings/arm/arm,scmi.txt > +++ b/Documentation/devicetree/bindings/arm/arm,scmi.txt > @@ -15,10 +15,10 @@ 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 > -- mboxes: List of phandle and mailbox channel specifiers. It should contain > - exactly one or two mailboxes, one for transmitting messages("tx") > - and another optional for receiving the notifications("rx") if > - supported. > +- 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 > + supported. Mandated by compatible "arm,scmi". This change is not required with yaml as you can have right schema to reflect this dependency. I think I already took care of this. -- Regards, Sudeep ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v2 3/4] dt-bindings: arm: OP-TEE as transport channel for SCMI messages [not found] <20210521134055.24271-1-etienne.carriere@linaro.org> 2021-05-21 13:40 ` [PATCH v2 2/4] dt-bindings: arm: scmi property mboxes is not mandatory Etienne Carriere @ 2021-05-21 13:40 ` Etienne Carriere 2021-06-02 10:53 ` Sudeep Holla 1 sibling, 1 reply; 4+ messages in thread From: Etienne Carriere @ 2021-05-21 13:40 UTC (permalink / raw) To: linux-kernel Cc: linux-arm-kernel, Sudeep Holla, Cristian Marussi, Vincent Guittot, Etienne Carriere, Rob Herring, devicetree Introduce compatible "linaro,scmi-optee" for SCMI transport channel based on an OP-TEE service invocation. Cc: Rob Herring <robh@kernel.org> Cc: devicetree@vger.kernel.org Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org> --- 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 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 3/4] dt-bindings: arm: OP-TEE as transport channel for SCMI messages 2021-05-21 13:40 ` [PATCH v2 3/4] dt-bindings: arm: OP-TEE as transport channel for SCMI messages Etienne Carriere @ 2021-06-02 10:53 ` Sudeep Holla 0 siblings, 0 replies; 4+ messages in thread From: Sudeep Holla @ 2021-06-02 10:53 UTC (permalink / raw) To: Etienne Carriere Cc: linux-kernel, linux-arm-kernel, Cristian Marussi, Vincent Guittot, Rob Herring, devicetree 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 <robh@kernel.org> > Cc: devicetree@vger.kernel.org > Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org> > --- > 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/20210601224904.917990-9-sudeep.holla@arm.com/ ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-06-02 10:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20210521134055.24271-1-etienne.carriere@linaro.org>
2021-05-21 13:40 ` [PATCH v2 2/4] dt-bindings: arm: scmi property mboxes is not mandatory Etienne Carriere
2021-06-02 10:40 ` Sudeep Holla
2021-05-21 13:40 ` [PATCH v2 3/4] dt-bindings: arm: OP-TEE as transport channel for SCMI messages Etienne Carriere
2021-06-02 10:53 ` Sudeep Holla
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).