2017-12-09 19:36:43

by Nishanth Menon

[permalink] [raw]
Subject: [PATCH] dt-bindings: mailbox: ti,message-manager: Fix interrupt name error

Message Manager's mailbox interrupts are queue based and not proxy
specific. The interrupt names are wrong in the binding, however
correctly reflected in the example provided. Remove the relation
to proxy ID in the documentation of binding. Existing device tree
descriptions follow the correct conventions already and documentation
update has been missed.

Signed-off-by: Nishanth Menon <[email protected]>
---
Documentation/devicetree/bindings/mailbox/ti,message-manager.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/mailbox/ti,message-manager.txt b/Documentation/devicetree/bindings/mailbox/ti,message-manager.txt
index c3b55b3ede8a..ebf0e3710cee 100644
--- a/Documentation/devicetree/bindings/mailbox/ti,message-manager.txt
+++ b/Documentation/devicetree/bindings/mailbox/ti,message-manager.txt
@@ -20,9 +20,9 @@ Required properties:
order referring to the transfer path.
- interrupt-names: Contains interrupt names matching the rx transfer path
for a given SoC. Receive interrupts shall be of the
- format: "rx_<QID>_<PID>".
+ format: "rx_<QID>".
For ti,k2g-message-manager, this shall contain:
- "rx_005_002", "rx_057_002"
+ "rx_005", "rx_057"
- interrupts: Contains the interrupt information corresponding to
interrupt-names property.

--
2.14.1


2017-12-13 08:03:19

by Lokesh Vutla

[permalink] [raw]
Subject: Re: dt-bindings: mailbox: ti,message-manager: Fix interrupt name error



On Sunday 10 December 2017 01:05 AM, Nishanth Menon wrote:
> Message Manager's mailbox interrupts are queue based and not proxy
> specific. The interrupt names are wrong in the binding, however
> correctly reflected in the example provided. Remove the relation
> to proxy ID in the documentation of binding. Existing device tree
> descriptions follow the correct conventions already and documentation
> update has been missed.
>
> Signed-off-by: Nishanth Menon <[email protected]>

Reviewed-by: Lokesh Vutla <[email protected]>

Thanks and regards,
Lokesh

2017-12-15 21:03:48

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mailbox: ti,message-manager: Fix interrupt name error

On Sat, Dec 09, 2017 at 01:35:28PM -0600, Nishanth Menon wrote:
> Message Manager's mailbox interrupts are queue based and not proxy
> specific. The interrupt names are wrong in the binding, however
> correctly reflected in the example provided. Remove the relation
> to proxy ID in the documentation of binding. Existing device tree
> descriptions follow the correct conventions already and documentation
> update has been missed.
>
> Signed-off-by: Nishanth Menon <[email protected]>
> ---
> Documentation/devicetree/bindings/mailbox/ti,message-manager.txt | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)

Applied, thanks.

Rob