* [PATCH v3] dt-bindings: mailbox: sti-mailbox: convert to DT schema
@ 2023-02-14 20:49 Alain Volmat
2023-02-15 20:24 ` Krzysztof Kozlowski
0 siblings, 1 reply; 5+ messages in thread
From: Alain Volmat @ 2023-02-14 20:49 UTC (permalink / raw)
To: Jassi Brar, Rob Herring, Krzysztof Kozlowski, Patrice Chotard
Cc: Alain Volmat, linux-kernel, devicetree
Convert the sti-mailbox.txt file into st,sti-mailbox.yaml
Signed-off-by: Alain Volmat <avolmat@me.com>
---
v3: remove quotes around $ref
v2: update commit log
remove quotes around urls
add mbox-name $ref
remove mbox-name from required properties since not mandatory in the driver
fix IRQ type in example
.../bindings/mailbox/st,sti-mailbox.yaml | 53 +++++++++++++++++++
.../bindings/mailbox/sti-mailbox.txt | 51 ------------------
2 files changed, 53 insertions(+), 51 deletions(-)
create mode 100644 Documentation/devicetree/bindings/mailbox/st,sti-mailbox.yaml
delete mode 100644 Documentation/devicetree/bindings/mailbox/sti-mailbox.txt
diff --git a/Documentation/devicetree/bindings/mailbox/st,sti-mailbox.yaml b/Documentation/devicetree/bindings/mailbox/st,sti-mailbox.yaml
new file mode 100644
index 000000000000..a023c28dff49
--- /dev/null
+++ b/Documentation/devicetree/bindings/mailbox/st,sti-mailbox.yaml
@@ -0,0 +1,53 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mailbox/st,sti-mailbox.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: STMicroelectronics Mailbox Driver for STi platform
+
+description:
+ Each ST Mailbox IP currently consists of 4 instances of 32 channels.
+ Messages are passed between Application and Remote processors using
+ shared memory.
+
+maintainers:
+ - Patrice Chotard <patrice.chotard@foss.st.com>
+
+properties:
+ compatible:
+ const: st,stih407-mailbox
+
+ reg:
+ maxItems: 1
+
+ mbox-name:
+ $ref: /schemas/types.yaml#/definitions/string
+ description: name of the mailbox IP
+
+ interrupts:
+ description: the irq line for the RX mailbox
+ maxItems: 1
+
+ "#mbox-cells":
+ const: 2
+
+required:
+ - compatible
+ - reg
+ - "#mbox-cells"
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
+ mailbox0: mailbox@8f00000 {
+ compatible = "st,stih407-mailbox";
+ reg = <0x8f00000 0x1000>;
+ interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
+ #mbox-cells = <2>;
+ mbox-name = "a9";
+ };
+
+...
diff --git a/Documentation/devicetree/bindings/mailbox/sti-mailbox.txt b/Documentation/devicetree/bindings/mailbox/sti-mailbox.txt
deleted file mode 100644
index 351f612673fc..000000000000
--- a/Documentation/devicetree/bindings/mailbox/sti-mailbox.txt
+++ /dev/null
@@ -1,51 +0,0 @@
-ST Microelectronics Mailbox Driver
-
-Each ST Mailbox IP currently consists of 4 instances of 32 channels. Messages
-are passed between Application and Remote processors using shared memory.
-
-Controller
-----------
-
-Required properties:
-- compatible : Should be "st,stih407-mailbox"
-- reg : Offset and length of the device's register set
-- mbox-name : Name of the mailbox
-- #mbox-cells: : Must be 2
- <&phandle instance channel direction>
- phandle : Label name of controller
- instance : Instance number
- channel : Channel number
-
-Optional properties
-- interrupts : Contains the IRQ line for a Rx mailbox
-
-Example:
-
-mailbox0: mailbox@0 {
- compatible = "st,stih407-mailbox";
- reg = <0x08f00000 0x1000>;
- interrupts = <GIC_SPI 1 IRQ_TYPE_NONE>;
- #mbox-cells = <2>;
- mbox-name = "a9";
-};
-
-Client
-------
-
-Required properties:
-- compatible : Many (See the client docs)
-- reg : Shared (between Application and Remote) memory address
-- mboxes : Standard property to specify a Mailbox (See ./mailbox.txt)
- Cells must match 'mbox-cells' (See Controller docs above)
-
-Optional properties
-- mbox-names : Name given to channels seen in the 'mboxes' property.
-
-Example:
-
-mailbox_test {
- compatible = "mailbox-test";
- reg = <0x[shared_memory_address], [shared_memory_size]>;
- mboxes = <&mailbox2 0 1>, <&mailbox0 2 1>;
- mbox-names = "tx", "rx";
-};
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3] dt-bindings: mailbox: sti-mailbox: convert to DT schema
2023-02-14 20:49 [PATCH v3] dt-bindings: mailbox: sti-mailbox: convert to DT schema Alain Volmat
@ 2023-02-15 20:24 ` Krzysztof Kozlowski
2023-04-07 19:15 ` Alain Volmat
0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-15 20:24 UTC (permalink / raw)
To: Alain Volmat, Jassi Brar, Rob Herring, Krzysztof Kozlowski,
Patrice Chotard
Cc: linux-kernel, devicetree
On 14/02/2023 21:49, Alain Volmat wrote:
> Convert the sti-mailbox.txt file into st,sti-mailbox.yaml
>
> Signed-off-by: Alain Volmat <avolmat@me.com>
> ---
> v3: remove quotes around $ref
> v2: update commit log
> remove quotes around urls
> add mbox-name $ref
> remove mbox-name from required properties since not mandatory in the driver
> fix IRQ type in example
>
> .../bindings/mailbox/st,sti-mailbox.yaml | 53 +++++++++++++++++++
> .../bindings/mailbox/sti-mailbox.txt | 51 ------------------
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] dt-bindings: mailbox: sti-mailbox: convert to DT schema
2023-02-15 20:24 ` Krzysztof Kozlowski
@ 2023-04-07 19:15 ` Alain Volmat
2023-04-08 10:53 ` Krzysztof Kozlowski
2023-04-08 23:36 ` Jassi Brar
0 siblings, 2 replies; 5+ messages in thread
From: Alain Volmat @ 2023-04-07 19:15 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Jassi Brar, Rob Herring, Krzysztof Kozlowski, Patrice Chotard,
linux-kernel, devicetree
Gentle reminder, could someone take this patch into his branch ?
Regards,
Alain
On Wed, Feb 15, 2023 at 09:24:35PM +0100, Krzysztof Kozlowski wrote:
> On 14/02/2023 21:49, Alain Volmat wrote:
> > Convert the sti-mailbox.txt file into st,sti-mailbox.yaml
> >
> > Signed-off-by: Alain Volmat <avolmat@me.com>
> > ---
> > v3: remove quotes around $ref
> > v2: update commit log
> > remove quotes around urls
> > add mbox-name $ref
> > remove mbox-name from required properties since not mandatory in the driver
> > fix IRQ type in example
> >
> > .../bindings/mailbox/st,sti-mailbox.yaml | 53 +++++++++++++++++++
> > .../bindings/mailbox/sti-mailbox.txt | 51 ------------------
>
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] dt-bindings: mailbox: sti-mailbox: convert to DT schema
2023-04-07 19:15 ` Alain Volmat
@ 2023-04-08 10:53 ` Krzysztof Kozlowski
2023-04-08 23:36 ` Jassi Brar
1 sibling, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2023-04-08 10:53 UTC (permalink / raw)
To: Jassi Brar, Rob Herring, Krzysztof Kozlowski, Patrice Chotard,
linux-kernel, devicetree
On 07/04/2023 21:15, Alain Volmat wrote:
> Gentle reminder, could someone take this patch into his branch ?
You addressed the email to me, but I am not the maintainer to take this
patch.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] dt-bindings: mailbox: sti-mailbox: convert to DT schema
2023-04-07 19:15 ` Alain Volmat
2023-04-08 10:53 ` Krzysztof Kozlowski
@ 2023-04-08 23:36 ` Jassi Brar
1 sibling, 0 replies; 5+ messages in thread
From: Jassi Brar @ 2023-04-08 23:36 UTC (permalink / raw)
To: Krzysztof Kozlowski, Jassi Brar, Rob Herring,
Krzysztof Kozlowski, Patrice Chotard, linux-kernel, devicetree
On Fri, Apr 7, 2023 at 2:15 PM Alain Volmat <avolmat@me.com> wrote:
>
> Gentle reminder, could someone take this patch into his branch ?
>
Why do you think it's not picked yet?
Please have a look at v6.3-rc1
cheers
-jassi
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-04-08 23:36 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-14 20:49 [PATCH v3] dt-bindings: mailbox: sti-mailbox: convert to DT schema Alain Volmat
2023-02-15 20:24 ` Krzysztof Kozlowski
2023-04-07 19:15 ` Alain Volmat
2023-04-08 10:53 ` Krzysztof Kozlowski
2023-04-08 23:36 ` Jassi Brar
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).