On 09/18/2017 03:29 PM, Vivien Didelot wrote: > Hi Florian, > > Florian Fainelli <f.faine...@gmail.com> writes: > >> @@ -1000,6 +1005,9 @@ static void b53_adjust_link(struct dsa_switch *ds, int >> port, >> b53_write8(dev, B53_CTRL_PAGE, po_reg, gmii_po); >> } >> } >> + >> + /* Re-negotiate EEE if it was enabled already */ >> + p->eee_enabled = b53_eee_init(ds, port, phydev); >> } > > Same here, I think we can move this up to DSA core, maybe with a > eee_enabled mask in dsa_switch or a bool in dsa_port.
This can be done as a subsequent patch, sure. -- Florian
