On 12.07.2018 21:09, Andrew Lunn wrote: >> Like r8169 also tg3 driver doesn't wait for the speed-down-renegotiation >> to finish. Therefore, even though I share Andrew's concerns, there seem >> to be chips where it's safe to not wait for the renegotiation to finish >> (e.g. because device is in PCI D3 already and can't generate an interrupt). >> Having said that I'd keep the sync parameter for phy_speed_down so that >> the driver can decide. > > Hi Heiner > > Please put a big fat comment about the dangers of sync=false in the > function header. We want people to known it is dangerous by default, > and should only be used in special conditions, when it is known to be > safe. > Andrew > OK ..
Heiner