On 6/24/20 2:43 PM, David Miller wrote: > From: Jisheng Zhang <jisheng.zh...@synaptics.com> > Date: Wed, 24 Jun 2020 11:25:16 +0800 > >> We face an issue with rtl8211f, a pin is shared between INTB and PMEB, >> and the PHY Register Accessible Interrupt is enabled by default, so >> the INTB/PMEB pin is always active in polling mode case. >> >> As Heiner pointed out "I was thinking about calling >> phy_disable_interrupts() in phy_init_hw(), to have a defined init >> state as we don't know in which state the PHY is if the PHY driver is >> loaded. We shouldn't assume that it's the chip power-on defaults, BIOS >> or boot loader could have changed this. Or in case of dual-boot >> systems the other OS could leave the PHY in whatever state." >> >> patch1 makes phy_disable_interrupts() non-static so that it could be used >> in phy_init_hw() to have a defined init state. >> >> patch2 calls phy_disable_interrupts() in phy_init_hw() to have a >> defined init state. >> >> Since v2: >> - Don't export phy_disable_interrupts() but just make it non-static >> >> Since v1: >> - EXPORT the correct symbol > > Series applied, thank you.
Did you mean that you applied v4? It does not look like you pushed your local changes to net-next yet, so I cannot tell for sure. -- Florian
