lkml.org 
[lkml]   [2019]   [Dec]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.19 052/243] net: dsa: mv88e6xxx: Work around mv886e6161 SERDES missing MII_PHYSID2
    Date
    From: Andrew Lunn <andrew@lunn.ch>

    [ Upstream commit ddc49acb659a2d8bfc5fdb0de0ef197712c11d75 ]

    We already have a workaround for a couple of switches whose internal
    PHYs only have the Marvel OUI, but no model number. We detect such
    PHYs and give them the 6390 ID as the model number. However the
    mv88e6161 has two SERDES interfaces in the same address range as its
    internal PHYs. These suffer from the same problem, the Marvell OUI,
    but no model number. As a result, these SERDES interfaces were getting
    the same PHY ID as the mv88e6390, even though they are not PHYs, and
    the Marvell PHY driver was trying to drive them.

    Add a special case to stop this from happen.

    Reported-by: Chris Healy <Chris.Healy@zii.aero>
    Signed-off-by: Andrew Lunn <andrew@lunn.ch>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/net/dsa/mv88e6xxx/chip.c | 21 ++++++++++++++++-----
    1 file changed, 16 insertions(+), 5 deletions(-)

    diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net/dsa/mv88e6xxx/chip.c
    index 411ae9961bf4f..43b00e8bcdcd7 100644
    --- a/drivers/net/dsa/mv88e6xxx/chip.c
    +++ b/drivers/net/dsa/mv88e6xxx/chip.c
    @@ -2645,11 +2645,22 @@ static int mv88e6xxx_mdio_read(struct mii_bus *bus, int phy, int reg)
    mutex_unlock(&chip->reg_lock);

    if (reg == MII_PHYSID2) {
    - /* Some internal PHYS don't have a model number. Use
    - * the mv88e6390 family model number instead.
    - */
    - if (!(val & 0x3f0))
    - val |= MV88E6XXX_PORT_SWITCH_ID_PROD_6390 >> 4;
    + /* Some internal PHYs don't have a model number. */
    + if (chip->info->family != MV88E6XXX_FAMILY_6165)
    + /* Then there is the 6165 family. It gets is
    + * PHYs correct. But it can also have two
    + * SERDES interfaces in the PHY address
    + * space. And these don't have a model
    + * number. But they are not PHYs, so we don't
    + * want to give them something a PHY driver
    + * will recognise.
    + *
    + * Use the mv88e6390 family model number
    + * instead, for anything which really could be
    + * a PHY,
    + */
    + if (!(val & 0x3f0))
    + val |= MV88E6XXX_PORT_SWITCH_ID_PROD_6390 >> 4;
    }

    return err ? err : val;
    --
    2.20.1


    \
     
     \ /
      Last update: 2022-09-17 16:06    [W:2.234 / U:0.072 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site