This fixed most of the Sparse and W=1 warnings in drivers/net/phy. The Cavium code is still not fully clean, but it might actually be the strange code is confusing Sparse.
GregKH: Is it O.K. to take the last patch via netdev, not staging? Cc: Alexandru Ardelean <alexaundru.ardel...@analog.com> Cc: Richard Cochran <richardcoch...@gmail.com> Cc: Sunil Goutham <sgout...@marvell.com> Cc: Robert Richter <rrich...@marvell.com> Cc: Sunil Goutham <sgout...@marvell.com> Cc: Robert Richter <rrich...@marvell.com> Cc: Chris Packham <chris.pack...@alliedtelesis.co.nz> Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org> Andrew Lunn (7): net: phy: at803x: Avoid comparison is always false warning net: phy: Fixup parameters in kerneldoc net: phy: Properly define genphy_c45_driver net: phy: Make phy_10gbit_fec_features_array static net: phy: dp83640: Fixup cast to restricted __be16 warning net: phy: cavium: Improve __iomem mess net: phy: mdio-octeon: Cleanup module loading dependencies drivers/net/ethernet/cavium/octeon/octeon_mgmt.c | 6 +----- drivers/net/phy/adin.c | 12 ++++++------ drivers/net/phy/at803x.c | 7 +++---- drivers/net/phy/dp83640.c | 5 +++-- drivers/net/phy/mdio-boardinfo.c | 3 ++- drivers/net/phy/mdio-cavium.h | 14 +++++++------- drivers/net/phy/mdio-octeon.c | 11 ++--------- drivers/net/phy/mdio-thunder.c | 2 +- drivers/net/phy/mdio_device.c | 2 +- drivers/net/phy/phy_device.c | 4 +--- drivers/staging/octeon/ethernet-mdio.c | 2 +- drivers/staging/octeon/ethernet-mdio.h | 2 -- drivers/staging/octeon/ethernet.c | 2 -- include/linux/phy.h | 3 +++ 14 files changed, 31 insertions(+), 44 deletions(-) -- 2.27.0.rc2