On Tue, Jan 22, 2019 at 04:40:27PM -0500, John David Anglin wrote: > Hi Andrew, > > On 1/22/2019 3:28 PM, Andrew Lunn wrote: > >Does it make a difference if you do it by hand? Bring up the master > >interface, wan, lan0, lan1, add any bridge you need, etc. > > > >> From power on, none of the wan, lan0, lan1 or br0 achieve link > >>(LOWER_UP). > I can explore this but I don't know at the moment. I believe that all > interfaces are configured.
Hi John In what order? The master interface has to be up first before any slave interface is configured up. > In serdes.c, interrupts are used to monitor link changes. However, phy.c > doesn't do this and > it doesn't call phylink_mac_change(). It does not need to. There are two options here: 1) The PHY has no interrupt. phylib will poll the PHY once per second for link changes. 2) The PHY has in interrupt. Link changes will cause the interrupt to fire, and the phylib will then read the current state. For PHYs embedded within a switch driver by mv88e6xxx interrupts should always be used. I will do some testing on my espressobin. Andrew
