• Russell King's avatar
    net: phy: avoid matching all-ones clause 45 PHY IDs · b95e86d8
    Russell King authored
    We currently match clause 45 PHYs using any ID read from a MMD marked
    as present in the "Devices in package" registers 5 and 6.  However,
    this is incorrect.  45.2 says:
    
      "The definition of the term package is vendor specific and could be
       a chip, module, or other similar entity."
    
    so a package could be more or less than the whole PHY - a PHY could be
    made up of several modules instantiated onto a single chip such as the
    Marvell 88x3310, or some of the MMDs could be disabled according to
    chip configuration, such as the Broadcom 84881.
    
    In the case of Broadcom 84881, the "Devices in package" registers
    contain 0xc000009b, meaning that there is a PHYXS present in the
    package, but all registers in MMD 4 return 0xffff.  This leads to our
    matching code incorrectly binding this PHY to one of our generic PHY
    drivers.
    
    This patch changes the way we determine whether to attempt to match a
    MMD identifier, or use it to request a module - if the identifier is
    all-ones, then we skip over it. When reading the identifiers, we
    initialise phydev->c45_ids.device_ids to all-ones, only reading the
    device ID if the "Devices in package" registers indicates we should.
    
    This avoids the generic drivers incorrectly matching on a PHY ID of
    0xffffffff.
    Signed-off-by: default avatarRussell King <rmk+kernel@armlinux.org.uk>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    b95e86d8
phy_device.c 68.1 KB