Thread (1 message) 1 message, 1 author, 2016-08-26

[PATCH 8/8] arm: orion5x: Configure WNR854T ethernet PHY LEDs

From: andrew@lunn.ch (Andrew Lunn)
Date: 2016-08-26 15:05:09
Also in: linux-devicetree

On Fri, Aug 26, 2016 at 10:21:04AM +0100, Jamie Lentin wrote:
The default PHY configuration disables most of the LEDs. The following
configures the ethernet activity LEDs as Netgear intended.

Signed-off-by: Jamie Lentin <redacted>
---
Andrew, note that the slave mdio bus is called "mdio", however there is
an example in bindings/net/dsa/dsa.txt using "mdio-bus". Is this a
mismatch to fix?
Hi Jamie

Yes, the documentation should match reality.

Unfortunately, there is a bit of a mess here. Different drivers have
different names for this node in device tree. And since device tree is
supposed to be stable, we cannot change them all to be the same.

I did intend to write some documentation defining what best practices
are for mdio busses, but never got around to it.

    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