[PATCH net-next] Documentation: devicetree: revise ethernet device-tree binding about TRGMII

Subsystems: networking drivers, open firmware and flattened device tree bindings, the rest

STALE3675d

3 messages, 2 authors, 2016-09-23 · open the first message on its own page

[PATCH net-next] Documentation: devicetree: revise ethernet device-tree binding about TRGMII

From: <sean.wang@mediatek.com>
Date: 2016-09-22 16:16:47

From: Sean Wang <sean.wang@mediatek.com>

fix typo in mediatek-net.txt and add phy-mode "trgmii" to ethernet.txt

Cc: devicetree@vger.kernel.org
Reported-by: Sergei Shtylyov <redacted>
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
---
 Documentation/devicetree/bindings/net/ethernet.txt     | 4 ++--
 Documentation/devicetree/bindings/net/mediatek-net.txt | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/net/ethernet.txt b/Documentation/devicetree/bindings/net/ethernet.txt
index 5d88f37..e1d7681 100644
--- a/Documentation/devicetree/bindings/net/ethernet.txt
+++ b/Documentation/devicetree/bindings/net/ethernet.txt
@@ -11,8 +11,8 @@ The following properties are common to the Ethernet controllers:
   the maximum frame size (there's contradiction in ePAPR).
 - phy-mode: string, operation mode of the PHY interface; supported values are
   "mii", "gmii", "sgmii", "qsgmii", "tbi", "rev-mii", "rmii", "rgmii", "rgmii-id",
-  "rgmii-rxid", "rgmii-txid", "rtbi", "smii", "xgmii"; this is now a de-facto
-  standard property;
+  "rgmii-rxid", "rgmii-txid", "rtbi", "smii", "xgmii", "trgmii"; this is now a
+  de-facto standard property;
 - phy-connection-type: the same as "phy-mode" property but described in ePAPR;
 - phy-handle: phandle, specifies a reference to a node representing a PHY
   device; this property is described in ePAPR and so preferred;
diff --git a/Documentation/devicetree/bindings/net/mediatek-net.txt b/Documentation/devicetree/bindings/net/mediatek-net.txt
index bae848e..da05f8b 100644
--- a/Documentation/devicetree/bindings/net/mediatek-net.txt
+++ b/Documentation/devicetree/bindings/net/mediatek-net.txt
@@ -34,7 +34,7 @@ Required properties:
 - phy-handle: see ethernet.txt file in the same directory and
 	the phy-mode "trgmii" required being provided when reg
 	is equal to 0 and the MAC uses fixed-link to connect
-	with inernal switch such as MT7530.
+	with internal switch such as MT7530.
 
 Example:
 
-- 
1.9.1

Re: [PATCH net-next] Documentation: devicetree: revise ethernet device-tree binding about TRGMII

From: Sergei Shtylyov <hidden>
Date: 2016-09-22 16:48:55

On 09/22/2016 07:16 PM, sean.wang@mediatek.com wrote:
From: Sean Wang <sean.wang@mediatek.com>

fix typo in mediatek-net.txt and add phy-mode "trgmii" to ethernet.txt
   These changes are unrelated to each other, so there should be 2 separate patches. And have the patches I reviewed been merged already, why are you sending an incremental patch?
Cc: devicetree@vger.kernel.org
Reported-by: Sergei Shtylyov <redacted>
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
[...]

MBR, Sergei

Re: [PATCH net-next] Documentation: devicetree: revise ethernet device-tree binding about TRGMII

From: Sean Wang <sean.wang@mediatek.com>
Date: 2016-09-23 02:53:19

Date: Thu, 22 Sep 2016 19:48:47 +0300, Sergei Shtylyov [off-list ref] wrote:
On 09/22/2016 07:16 PM, sean.wang@mediatek.com wrote:
quoted
From: Sean Wang <sean.wang@mediatek.com>

fix typo in mediatek-net.txt and add phy-mode "trgmii" to ethernet.txt
   These changes are unrelated to each other, so there should be 2 separate 
patches. And have the patches I reviewed been merged already, why are you 
sending an incremental patch?
okay, I will make them into distinct patchs.

I saw they had been applied so I created an incremental
patch based on codebase after applied.
quoted
Cc: devicetree@vger.kernel.org
Reported-by: Sergei Shtylyov <redacted>
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
[...]

MBR, Sergei
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help