diff options
| author |   Russell King <rmk+kernel@armlinux.org.uk> | 2019-10-15 11:28:46 +0100 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-10-15 20:40:06 -0700 |
| commit | 554032cdfbf4491f38241a3f6b27459408d90df3 (patch) | |
| tree | c3e77da219f5f4f5e6614499b49be83e00c83c15 /drivers/net/phy/phylink.c | |
| parent | tc-testing: updated pedit test cases (diff) | |
| download | linux-dev-554032cdfbf4491f38241a3f6b27459408d90df3.tar.xz linux-dev-554032cdfbf4491f38241a3f6b27459408d90df3.zip | |
net: phylink: use more linkmode_*
Use more linkmode_* helpers rather than open-coding the bitmap
operations.
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/phy/phylink.c')
| -rw-r--r-- | drivers/net/phy/phylink.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/net/phy/phylink.c b/drivers/net/phy/phylink.c index a5a57ca94c1a..8e53ed90da3c 100644 --- a/drivers/net/phy/phylink.c +++ b/drivers/net/phy/phylink.c @@ -117,9 +117,7 @@ static int phylink_is_empty_linkmode(const unsigned long *linkmode) phylink_set(tmp, Pause); phylink_set(tmp, Asym_Pause); - bitmap_andnot(tmp, linkmode, tmp, __ETHTOOL_LINK_MODE_MASK_NBITS); - - return linkmode_empty(tmp); + return linkmode_subset(linkmode, tmp); } static const char *phylink_an_mode_str(unsigned int mode) @@ -1728,8 +1726,7 @@ static int phylink_sfp_module_insert(void *upstream, if (phy_interface_mode_is_8023z(iface) && pl->phydev) return -EINVAL; - changed = !bitmap_equal(pl->supported, support, - __ETHTOOL_LINK_MODE_MASK_NBITS); + changed = !linkmode_equal(pl->supported, support); if (changed) { linkmode_copy(pl->supported, support); linkmode_copy(pl->link_config.advertising, config.advertising); |