Thread (5 messages) flat view 5 messages, 4 authors, 2016-01-14

Re: [PATCH] net: phy: turn carrier off on phy attach

From: Florian Fainelli <f.fainelli@gmail.com>
Date: 2016-01-13 01:31:13
Also in: lkml

On January 9, 2016 10:44:05 AM PST, Sjoerd Simons [off-list ref] wrote:
The operstate of a networking device initially IF_OPER_UNKNOWN aka
"unknown", updated on carrier state changes (with carrier state being
on
by default). This means it will stay unknown unless the carrier state
goes to off at some point, which is not the case if the phy is already
up/connected at startup.
Correct, drivers typically call netif_carrier_off prior to registering the network device to give a predictable link state, regardless of whether or not they use PHYLIB.
Explicitly turn off the carrier on phy attach, leaving the phy state
machine to turn the carrier on when it has done the initial
negotiation.
Same comment as Andrew on the comment below.

Out of curiosity, was there a particular driver you ran into issues with?

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