lkml.org 
[lkml]   [2022]   [Jan]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 4.19 54/59] net: phy: marvell: configure RGMII delays for 88E1118
    Date
    From: "Russell King (Oracle)" <rmk+kernel@armlinux.org.uk>

    [ Upstream commit f22725c95ececb703c3f741e8f946d23705630b7 ]

    Corentin Labbe reports that the SSI 1328 does not work when allowing
    the PHY to operate at gigabit speeds, but does work with the generic
    PHY driver.

    This appears to be because m88e1118_config_init() writes a fixed value
    to the MSCR register, claiming that this is to enable 1G speeds.
    However, this always sets bits 4 and 5, enabling RGMII transmit and
    receive delays. The suspicion is that the original board this was
    added for required the delays to make 1G speeds work.

    Add the necessary configuration for RGMII delays for the 88E1118 to
    bring this into line with the requirements for RGMII support, and thus
    make the SSI 1328 work.

    Corentin Labbe has tested this on gemini-ssi1328 and gemini-ns2502.

    Reported-by: Corentin Labbe <clabbe.montjoie@gmail.com>
    Tested-by: Corentin Labbe <clabbe.montjoie@gmail.com>
    Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
    Reviewed-by: Andrew Lunn <andrew@lunn.ch>
    Signed-off-by: Jakub Kicinski <kuba@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/net/phy/marvell.c | 6 ++++++
    1 file changed, 6 insertions(+)

    diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
    index 832a401c5fa56..484f6691b9c9f 100644
    --- a/drivers/net/phy/marvell.c
    +++ b/drivers/net/phy/marvell.c
    @@ -930,6 +930,12 @@ static int m88e1118_config_init(struct phy_device *phydev)
    if (err < 0)
    return err;

    + if (phy_interface_is_rgmii(phydev)) {
    + err = m88e1121_config_aneg_rgmii_delays(phydev);
    + if (err < 0)
    + return err;
    + }
    +
    /* Adjust LED Control */
    if (phydev->dev_flags & MARVELL_PHY_M1118_DNS323_LEDS)
    err = phy_write(phydev, 0x10, 0x1100);
    --
    2.34.1
    \
     
     \ /
      Last update: 2022-01-18 04:34    [W:2.071 / U:0.540 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site