dt-bindings: phy: mvebu-comphy: extend the file to describe a3700 bindings
authorMiquel Raynal <miquel.raynal@bootlin.com>
Tue, 8 Jan 2019 16:31:21 +0000 (17:31 +0100)
committerKishon Vijay Abraham I <kishon@ti.com>
Thu, 7 Feb 2019 05:41:01 +0000 (11:11 +0530)
Current file describe COMPHY bindings for the IP available on the
CP110 of Armada 7k/8k. Bindings are very close (and serve the same
purpose) as the new Armada 3700 COMPHY driver so update this file to
describe both. Also add an example of how to use this second
compatible (same as for the ESPRESSObin).

While doing so, enhance a bit the file by adding upper case where
needed.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
Documentation/devicetree/bindings/phy/phy-mvebu-comphy.txt

index bfcf80341657ce6ba39891ff10b1c9c19d775a0a..cf2cd86db2674c62cfe162675abb572f262eca84 100644 (file)
@@ -1,16 +1,27 @@
-mvebu comphy driver
--------------------
+MVEBU comphy drivers
+--------------------
 
-A comphy controller can be found on Marvell Armada 7k/8k on the CP110. It
-provides a number of shared PHYs used by various interfaces (network, sata,
-usb, PCIe...).
+COMPHY controllers can be found on the following Marvell MVEBU SoCs:
+* Armada 7k/8k (on the CP110)
+* Armada 3700
+It provides a number of shared PHYs used by various interfaces (network, SATA,
+USB, PCIe...).
 
 Required properties:
 
-- compatible: should be "marvell,comphy-cp110"
-- reg: should contain the comphy register location and length.
-- marvell,system-controller: should contain a phandle to the
-                             system controller node.
+- compatible: should be one of:
+  * "marvell,comphy-cp110" for Armada 7k/8k
+  * "marvell,comphy-a3700" for Armada 3700
+- reg: should contain the COMPHY register(s) location(s) and length(s).
+  * 1 entry for Armada 7k/8k
+  * 4 entries for Armada 3700 along with the corresponding reg-names
+    properties, memory areas are:
+    * Generic COMPHY registers
+    * Lane 1 (PCIe/GbE)
+    * Lane 0 (USB3/GbE)
+    * Lane 2 (SATA/USB3)
+- marvell,system-controller: should contain a phandle to the system
+                            controller node (only for Armada 7k/8k)
 - #address-cells: should be 1.
 - #size-cells: should be 0.
 
@@ -18,11 +29,11 @@ A sub-node is required for each comphy lane provided by the comphy.
 
 Required properties (child nodes):
 
-- reg: comphy lane number.
-- #phy-cells : from the generic phy bindings, must be 1. Defines the
+- reg: COMPHY lane number.
+- #phy-cells : from the generic PHY bindings, must be 1. Defines the
                input port to use for a given comphy lane.
 
-Example:
+Examples:
 
        cpm_comphy: phy@120000 {
                compatible = "marvell,comphy-cp110";
@@ -41,3 +52,33 @@ Example:
                        #phy-cells = <1>;
                };
        };
+
+       comphy: phy@18300 {
+               compatible = "marvell,comphy-a3700";
+               reg = <0x18300 0x300>,
+               <0x1F000 0x400>,
+               <0x5C000 0x400>,
+               <0xe0178 0x8>;
+               reg-names = "comphy",
+               "lane1_pcie_gbe",
+               "lane0_usb3_gbe",
+               "lane2_sata_usb3";
+               #address-cells = <1>;
+               #size-cells = <0>;
+
+
+               comphy0: phy@0 {
+                       reg = <0>;
+                       #phy-cells = <1>;
+               };
+
+               comphy1: phy@1 {
+                       reg = <1>;
+                       #phy-cells = <1>;
+               };
+
+               comphy2: phy@2 {
+                       reg = <2>;
+                       #phy-cells = <1>;
+               };
+       };