| Message ID | 20210721150141.1737124-1-arnd@kernel.org |
|---|---|
| State | New |
| Headers | show |
| Series | net: phy: at803x: fix at803x_match_phy_id mismatch | expand |
On Wed, Jul 21, 2021 at 5:01 PM Arnd Bergmann <arnd@kernel.org> wrote: > > From: Arnd Bergmann <arnd@arndb.de> > > There are two conflicting patches in net-next, one removed ... The email subject was meant to include [net-next], sorry for missing that. Arnd
Patch already sent by Vladimir, and just applied. On Wed, Jul 21, 2021 at 05:01:28PM +0200, Arnd Bergmann wrote: > From: Arnd Bergmann <arnd@arndb.de> > > There are two conflicting patches in net-next, one removed > the at803x_get_features() function and the other added another > user: > > drivers/net/phy/at803x.c: In function 'at803x_get_features': > drivers/net/phy/at803x.c:706:14: error: implicit declaration of function 'at803x_match_phy_id' [-Werror=implicit-function-declaration] > > Change the new caller over to an open-coded comparison as well. > > Fixes: 8887ca5474bd ("net: phy: at803x: simplify custom phy id matching") > Fixes: b856150c8098 ("net: phy: at803x: mask 1000 Base-X link mode") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > --- > drivers/net/phy/at803x.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/phy/at803x.c b/drivers/net/phy/at803x.c > index 0790ffcd3db6..bdac087058b2 100644 > --- a/drivers/net/phy/at803x.c > +++ b/drivers/net/phy/at803x.c > @@ -703,7 +703,7 @@ static int at803x_get_features(struct phy_device *phydev) > if (err) > return err; > > - if (!at803x_match_phy_id(phydev, ATH8031_PHY_ID)) > + if (phydev->drv->phy_id != ATH8031_PHY_ID) > return 0; > > /* AR8031/AR8033 have different status registers > -- > 2.29.2 > > -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!
diff --git a/drivers/net/phy/at803x.c b/drivers/net/phy/at803x.c index 0790ffcd3db6..bdac087058b2 100644 --- a/drivers/net/phy/at803x.c +++ b/drivers/net/phy/at803x.c @@ -703,7 +703,7 @@ static int at803x_get_features(struct phy_device *phydev) if (err) return err; - if (!at803x_match_phy_id(phydev, ATH8031_PHY_ID)) + if (phydev->drv->phy_id != ATH8031_PHY_ID) return 0; /* AR8031/AR8033 have different status registers