RE: [PATCH 3/6] QE: Add uqe_serial document to bindings

On Fri, Jan 09, 2016 at 04:12AM, Rob Herring [off-list ref] wrote:

-----Original Message-----
From: Rob Herring [mailto:robh@kernel.org]
Sent: Saturday, January 09, 2016 4:12 AM
To: Qiang Zhao <qiang.zhao@nxp.com>
Cc: <redacted>; <redacted>; linuxppc-
dev@lists.ozlabs.org; Priyanka.Jain@freescale.com; oss@buserror.net
Subject: Re: [PATCH 3/6] QE: Add uqe_serial document to bindings
=20
On Fri, Jan 08, 2016 at 10:18:11AM +0800, Zhao Qiang wrote:

quoted

Add uqe_serial document to
Documentation/devicetree/bindings/powerpc/fsl/cpm_qe/uqe_serial.txt

Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com>
---
 .../bindings/powerpc/fsl/cpm_qe/uqe_serial.txt       | 20

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

quoted

 1 file changed, 20 insertions(+)
 create mode 100644
Documentation/devicetree/bindings/powerpc/fsl/cpm_qe/uqe_serial.txt

diff --git
a/Documentation/devicetree/bindings/powerpc/fsl/cpm_qe/uqe_serial.txt
b/Documentation/devicetree/bindings/powerpc/fsl/cpm_qe/uqe_serial.txt
new file mode 100644
index 0000000..e677599

--- /dev/null
+++ b/Documentation/devicetree/bindings/powerpc/fsl/cpm_qe/uqe_serial.
+++ txt

@@ -0,0 +1,20 @@
+* Serial
+
+Currently defined compatibles:
+- ucc_uart
+
+Properties for ucc_uart:
+device_type : which type the device is

=20
Drop this please.

Yes, I will drop it in next version.

=20

quoted

+port-number : port number of UCC-UART

=20
Use aliases instead.

I don't understand, can you explain more?

=20

quoted

+rx-clock-name : which clock QE use for RX tx-clock-name : which clock
+QE use for TX

=20
These should use the clock binding.

This property means which clock source the UCC use,=20
the QE just use this property to route UCC clock to clock source.
The clock source maybe either internal or outside(from clock input pin).
So clock binding is not apply in this case.

=20

quoted

+
+Example:
+
+	serial: ucc@2200 {
+		device_type =3D "serial";
+		compatible =3D "ucc_uart";
+		port-number =3D <1>;
+		rx-clock-name =3D "brg2";
+		tx-clock-name =3D "brg2";
+	};
--
2.1.0.27.g96db324

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

Best Regards
Zhao Qiang