Thread (39 messages) 39 messages, 4 authors, 2025-08-22

Re: [PATCH net-next v10 01/15] dt-bindings: net: Introduce the ethernet-connector description

From: Maxime Chevallier <maxime.chevallier@bootlin.com>
Date: 2025-07-31 13:35:09
Also in: linux-arm-kernel, linux-arm-msm, linux-devicetree, lkml

Hello Andrew,

First, thanks a lot for taking a look a this series !

On Sat, 26 Jul 2025 21:49:07 +0200
Andrew Lunn [off-list ref] wrote:
quoted
 - The number of lanes, which is a quite generic property that allows
   differentating between multiple similar technologies such as BaseT1
   and "regular" BaseT (which usually means BaseT4).  
quoted
+            mdi {
+                connector-0 {
+                    lanes = <2>;
+                    media = "BaseT";  
This is correct when the port is Fast Ethernet. However, as you point
out, now a days, the more normal case is 1G, with 4 lanes for BaseT.
To avoid developers just copy/pasting without engaging brain, maybe
add a comment about it being a Fast Ethernet port?
Absolutely :)

Maxime
	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