Hi Florian,

Florian Fainelli <f.faine...@gmail.com> writes:

> @@ -649,7 +595,7 @@ static void bcm_sf2_sw_adjust_link(struct dsa_switch *ds, 
> int port,
>       core_writel(priv, reg, offset);
>  
>       if (!phydev->is_pseudo_fixed_link)
> -             p->eee_enabled = bcm_sf2_eee_init(ds, port, phydev);
> +             p->eee_enabled = b53_eee_init(ds, port, phydev);
>  }

I know this is a bit out-of-scope of this patch, but I have to say I am
not confortable with having still phy device stuffs in switch drivers...

Can this is_pseudo_fixed_link check + phy_eee_init + eee_enable be moved
up to dsa_slave_adjust_link in a future patch maybe?

Thanks,

        Vivien

Reply via email to