All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bin Meng <bmeng.cn@gmail.com>
To: u-boot@lists.denx.de
Subject: [PATCH v4 16/22] dt-bindings: net: Update Freescale TSEC to support "queue-group"
Date: Sun, 14 Mar 2021 20:15:00 +0800	[thread overview]
Message-ID: <20210314121506.18303-17-bmeng.cn@gmail.com> (raw)
In-Reply-To: <20210314121506.18303-1-bmeng.cn@gmail.com>

At present the Freescale TSEC node DT bindings doc requires a <reg>
property in the TSEC node. But this might not always be the case.
In the upstream Linux kernel, there is no DT bindings doc for it
but the kernel driver tests a subnode of a name prefixed with
"queue-group", as we can see from gfar_of_init():

  for_each_available_child_of_node(np, child) {
      if (!of_node_name_eq(child, "queue-group"))
  ...

in drivers/net/ethernet/freescale/gianfar.c

Update our DT bindings to describe this alternate description.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>

---

Changes in v4:
- describe "ranges" is required fo the alternate description

Changes in v3:
- add "ranges" in the alternate example

 doc/device-tree-bindings/net/fsl-tsec-phy.txt | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/doc/device-tree-bindings/net/fsl-tsec-phy.txt b/doc/device-tree-bindings/net/fsl-tsec-phy.txt
index a44c5fd9d9..f68980352b 100644
--- a/doc/device-tree-bindings/net/fsl-tsec-phy.txt
+++ b/doc/device-tree-bindings/net/fsl-tsec-phy.txt
@@ -3,12 +3,17 @@
 Properties:
 
   - compatible : Should be "fsl,etsec2" or "gianfar"
-  - reg : Offset and length of the register set for the device
+  - reg : Offset and length of the register set for the device. If this is
+    missing, a subnode with a name prefix "queue-group" must be provided to
+    provide the <reg> property.
   - phy-handle : See ethernet.txt file in the same directory.
   - phy-connection-type : See ethernet.txt file in the same directory. This
     property is only really needed if the connection is of type "rgmii-id",
     "rgmii-rxid" and "rgmii-txid" as all other connection types are detected
     by hardware.
+  - ranges : an <empty> value if subnode "queue-group" is present, specifying
+    that no address translation is required between them TSEC parent node and
+    the child "queue-group" node.
 
 Example:
 	ethernet at 24000 {
@@ -18,6 +23,18 @@ Example:
 		phy-connection-type = "sgmii";
 	};
 
+An alternate description with "queue-group" subnode example:
+	ethernet at 24000 {
+		compatible = "fsl,etsec2";
+		phy-handle = <&phy0>;
+		phy-connection-type = "sgmii";
+		ranges;
+
+		queue-group {
+			reg = <0x24000 0x1000>;
+		};
+	};
+
 Child nodes of the TSEC controller are typically the individual PHY devices
 connected via the MDIO bus (sometimes the MDIO bus controller is separate).
 
-- 
2.25.1

  parent reply	other threads:[~2021-03-14 12:15 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-14 12:14 [PATCH v4 00/22] ppc: qemu: Add eTSEC support Bin Meng
2021-03-14 12:14 ` Bin Meng
2021-03-14 12:14 ` [PATCH v4 01/22] dt-bindings: net: Add the old DT bindings for "fixed-link" Bin Meng
2021-03-14 12:14 ` [PATCH v4 02/22] of: extra: Introduce ofnode_phy_is_fixed_link() API Bin Meng
2021-03-14 12:14 ` [PATCH v4 03/22] dm: mdio: Use " Bin Meng
2021-03-14 12:14 ` [PATCH v4 04/22] net: phy: fixed: Be compatible with live OF tree Bin Meng
2021-03-14 12:14 ` [PATCH v4 05/22] net: phy: fixed: Drop #ifdef CONFIG_DM_ETH around phy_connect_fixed Bin Meng
2021-03-14 12:14 ` [PATCH v4 06/22] net: phy: xilinx: Be compatible with live OF tree Bin Meng
2021-04-23  7:35   ` Michal Simek
2021-04-23  9:14     ` Bin Meng
2021-04-23  9:23       ` Bin Meng
2021-04-23 11:52         ` Michal Simek
2021-03-14 12:14 ` [PATCH v4 07/22] net: phy: xilinx: Drop #ifdef CONFIG_DM_ETH around phy_connect_gmii2rgmii() Bin Meng
2021-03-14 12:14 ` [PATCH v4 08/22] net: phy: Simplify the logic of phy_connect_fixed() Bin Meng
2021-03-14 12:14 ` [PATCH v4 09/22] net: phy: fixed: Make driver ops static Bin Meng
2021-03-14 12:14 ` [PATCH v4 10/22] net: phy: fixed: Add the missing ending newline Bin Meng
2021-03-14 12:14 ` [PATCH v4 11/22] net: phy: fixed: Support the old DT binding Bin Meng
2021-03-14 12:14 ` [PATCH v4 12/22] net: tsec: Use dm_eth_phy_connect() directly for the DM case Bin Meng
2021-03-14 12:14 ` [PATCH v4 13/22] sandbox: Add a DSA sandbox driver and unit test Bin Meng
2021-03-14 12:14 ` [PATCH v4 14/22] test: dm: Add a case to test ofnode_phy_is_fixed_link() Bin Meng
2021-03-14 12:14 ` [PATCH v4 15/22] net: tsec: Use map_physmem() directly instead of dev_remap_addr() Bin Meng
2021-03-14 12:15 ` Bin Meng [this message]
2021-03-14 12:15 ` [PATCH v4 17/22] net: tsec: Support <reg> property from the subnode "queue-group" Bin Meng
2021-03-14 12:15 ` [PATCH v4 18/22] dm: core: Correctly read <ranges> of simple-bus Bin Meng
2021-03-14 12:15 ` [PATCH v4 19/22] test: dm: Add a test case for simple-bus <ranges> Bin Meng
2021-03-14 12:15 ` [PATCH v4 20/22] ppc: qemu: Create a virtual memory mapping of the platform bus Bin Meng
2021-03-14 12:15 ` [PATCH v4 21/22] ppc: qemu: Enable eTSEC support Bin Meng
2021-04-14 11:54   ` Priyanka Jain
2021-04-14 12:46     ` Tom Rini
2021-04-14 13:00     ` Bin Meng
2021-03-14 12:15 ` [PATCH v4 22/22] doc: board: qemu-ppce500: Document eTSEC usage Bin Meng
2021-04-06  8:48 ` [PATCH v4 00/22] ppc: qemu: Add eTSEC support Bin Meng
2021-04-06  8:48   ` Bin Meng
2021-04-08 15:29   ` Priyanka Jain
2021-04-08 15:29     ` Priyanka Jain

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210314121506.18303-17-bmeng.cn@gmail.com \
    --to=bmeng.cn@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.