mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-12-17 22:46:40 +07:00
net: sfp: split the PHY probe from sfp_sm_mod_init()
Move the PHY probe into a separate function, splitting it from sfp_sm_mod_init(). This will allow us to eliminate the 50ms mdelay() inside the state machine. Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
8e210b6bdc
commit
181f29da15
@ -1353,14 +1353,10 @@ static void sfp_sm_fault(struct sfp *sfp, bool warn)
|
||||
static void sfp_sm_mod_init(struct sfp *sfp)
|
||||
{
|
||||
sfp_module_tx_enable(sfp);
|
||||
}
|
||||
|
||||
/* Wait t_init before indicating that the link is up, provided the
|
||||
* current state indicates no TX_FAULT. If TX_FAULT clears before
|
||||
* this time, that's fine too.
|
||||
*/
|
||||
sfp_sm_next(sfp, SFP_S_INIT, T_INIT_JIFFIES);
|
||||
sfp->sm_retries = 5;
|
||||
|
||||
static void sfp_sm_probe_for_phy(struct sfp *sfp)
|
||||
{
|
||||
/* Setting the serdes link mode is guesswork: there's no
|
||||
* field in the EEPROM which indicates what mode should
|
||||
* be used.
|
||||
@ -1645,8 +1641,17 @@ static void sfp_sm_main(struct sfp *sfp, unsigned int event)
|
||||
switch (sfp->sm_state) {
|
||||
case SFP_S_DOWN:
|
||||
if (sfp->sm_mod_state == SFP_MOD_PRESENT &&
|
||||
sfp->sm_dev_state == SFP_DEV_UP)
|
||||
sfp->sm_dev_state == SFP_DEV_UP) {
|
||||
sfp_sm_mod_init(sfp);
|
||||
sfp_sm_probe_for_phy(sfp);
|
||||
|
||||
/* Wait t_init before indicating that the link is up,
|
||||
* provided the current state indicates no TX_FAULT. If
|
||||
* TX_FAULT clears before this time, that's fine too.
|
||||
*/
|
||||
sfp_sm_next(sfp, SFP_S_INIT, T_INIT_JIFFIES);
|
||||
sfp->sm_retries = 5;
|
||||
}
|
||||
break;
|
||||
|
||||
case SFP_S_INIT:
|
||||
|
Loading…
Reference in New Issue
Block a user