net: dsa: microchip: ksz9477: remove MII_CTRL1000 check from ksz9477_w_phy()
authorOleksij Rempel <o.rempel@pengutronix.de>
Fri, 26 Aug 2022 10:56:28 +0000 (12:56 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 31 Aug 2022 08:41:23 +0000 (09:41 +0100)
The reason why PHYlib may access MII_CTRL1000 on the chip without GBit
support is only if chip provides wrong information about extended caps
register. This issue is now handled by ksz9477_r_phy_quirks()

With proper regmap_ranges provided for all chips we will be able to
catch this kind of bugs any way. So, remove this sanity check.

Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Reviewed-by: Vladimir Oltean <olteanv@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/microchip/ksz9477.c

index fb9de6b447b12edc71a2caac5ecc5aa87d2e1143..f0c81d90c99f41fe71020f92106b3bffe239fb39 100644 (file)
@@ -341,10 +341,6 @@ int ksz9477_w_phy(struct ksz_device *dev, u16 addr, u16 reg, u16 val)
        if (addr >= dev->phy_port_cnt)
                return 0;
 
-       /* No gigabit support.  Do not write to this register. */
-       if (!dev->info->gbit_capable[addr] && reg == MII_CTRL1000)
-               return -ENXIO;
-
        return ksz_pwrite16(dev, addr, 0x100 + (reg << 1), val);
 }