Am 21.02.2018 um 05:27 schrieb David Miller: > From: Heiner Kallweit <hkallwe...@gmail.com> > Date: Tue, 20 Feb 2018 07:30:16 +0100 > >> rtl8169_init_phy() resets the PHY anyway after applying the chip-specific >> PHY configuration. So we don't need to soft-reset the PHY as part of the >> chip-specific configuration. >> >> Signed-off-by: Heiner Kallweit <hkallwe...@gmail.com> > > I'm not so comfortable with this. > > There are so many r8169 chip variants out there. > > And who knows, maybe one of them needs this second PHY reset or due to > some way the driver is coded it is necessary. > > Unless you can test this change on every r8169 chip type, I'm very > reluctant to apply this patch. > I understand the concern, the change however is in rtl8168e_2_hw_phy_config() which is specific to chip version 34 (RTL8168evl). On my system with this chip version the change didn't change system behavior.
> Sorry. >