Thread (101 messages) flat view 101 messages, 8 authors, 2021-01-09

Re: [PATCH v11 1/4] dt-bindings: net: Add 5GBASER phy interface mode

From: Andrew Lunn <andrew@lunn.ch>
Date: 2020-12-09 23:16:11
Also in: linux-devicetree, lkml, oe-kbuild-all

On Wed, Dec 09, 2020 at 03:03:47PM +1000, Pavana Sharma wrote:
quoted hunk ↗ jump to hunk
Add 5gbase-r PHY interface mode.

Signed-off-by: Pavana Sharma <redacted>
---
 Documentation/devicetree/bindings/net/ethernet-controller.yaml | 2 ++
 1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/net/ethernet-controller.yaml b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
index fdf709817218..aa6ae7851de9 100644
--- a/Documentation/devicetree/bindings/net/ethernet-controller.yaml
+++ b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
@@ -89,6 +89,8 @@ properties:
       - trgmii
       - 1000base-x
       - 2500base-x
+      # 5GBASE-R
+      - 5gbase-r
       - rxaui
       - xaui
Hi Pavana

For v10 i said:
What value does the comment add?
I don't remember you replying. Why is 5gbase-r special and it needs a
comment, saying the same thing in CAPS LETTERS? What value is there in
the CAPS LETTERS string?

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