Thanks for the comments. Please see response inline.

-----Original Message-----
From: Rob Herring [mailto:robh at kernel.org]
Sent: Wednesday, May 30, 2018 11:44 PM
To: Liming Sun <redacted>
Cc: Olof Johansson <redacted>; Arnd Bergmann <arnd@arndb.de>;
David Woods [off-list ref]; Robin Murphy
[off-list ref]; devicetree at vger.kernel.org; linux-arm-
kernel at lists.infradead.org
Subject: Re: [PATCH v2 3/4] dt-bindings: soc: Add TmFifo binding for
Mellanox BlueField SoC

On Fri, May 25, 2018 at 04:17:15PM -0400, Liming Sun wrote:

Commit msg?

Updated in patch v3 3/4.

quoted

Reviewed-by: David Woods <redacted>
Signed-off-by: Liming Sun <redacted>
---
 .../devicetree/bindings/soc/mellanox/tmfifo.txt      | 20

++++++++++++++++++++

quoted

 1 file changed, 20 insertions(+)
 create mode 100644

Documentation/devicetree/bindings/soc/mellanox/tmfifo.txt

quoted

diff --git a/Documentation/devicetree/bindings/soc/mellanox/tmfifo.txt

b/Documentation/devicetree/bindings/soc/mellanox/tmfifo.txt

quoted

new file mode 100644
index 0000000..0a362f5

--- /dev/null
+++ b/Documentation/devicetree/bindings/soc/mellanox/tmfifo.txt

@@ -0,0 +1,20 @@
+* Mellanox BlueField SoC TmFifo
+
+BlueField TmFifo provides a shared FIFO between the target and the
+external host machine, which can be accessed via USB or PCIe.

A FIFO for what? I'd like to find a better spot than bindings/soc/

This is a generic HW FIFO which can be accessed by the SoC SW and external Host machine to exchange any data. In the current Linux tmfifo driver, this FIFO has been used (demuxed) to implement a virtual console and network interface based on virtio framework.  

Updated the tmfifo.txt in patch v3 3/4 with the above explanation. Please advise if there is a better place for this file.
 

quoted

+
+Required properties:
+
+- compatible:	Should be "mellanox,bf-tmfifo"
+- reg:		Physical base address and length of Rx/Tx block
+- interrupts:	The interrupt number of Rx low water mark, Rx high water

mark

quoted

+		Tx low water mark, Tx high water mark respectively.
+
+Example:
+
+tmfifo at 800a20 {
+	compatible = "mellanox,bf-tmfifo";
+	reg = <0x00800a20 0x00000018
+	       0x00800a40 0x00000018>;
+	interrupts = <41, 42, 43, 44>;
+};
--
1.8.3.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html