]> git.sur5r.net Git - u-boot/commitdiff
phy: marvell: a3700: Save/restore selector reg in SGMII init
authorMarek BehĂșn <marek.behun@nic.cz>
Tue, 24 Apr 2018 15:21:24 +0000 (17:21 +0200)
committerStefan Roese <sr@denx.de>
Mon, 14 May 2018 08:00:15 +0000 (10:00 +0200)
In SGMII initialization PIN_PIPE_SEL has to be zero when resetting
the PHY. Since comphy_mux already set the selector register to
correct values, we have to store it's value before setting it to 0
and restore it after SGMII init.

Signed-off-by: Marek Behun <marek.behun@nic.cz>
Reviewed-by: Stefan Roese <sr@denx.de>
Signed-off-by: Stefan Roese <sr@denx.de>
drivers/phy/marvell/comphy_a3700.c
drivers/phy/marvell/comphy_a3700.h

index bf68f5d6bea33071e6fcd62c06e71389152ef33c..50167a69afc761350c93589c54b9baf1e9b4745f 100644 (file)
@@ -697,13 +697,15 @@ static void comphy_sgmii_phy_init(u32 lane, u32 speed)
 static int comphy_sgmii_power_up(u32 lane, u32 speed, u32 invert)
 {
        int ret;
+       u32 saved_selector;
 
        debug_enter();
 
        /*
         * 1. Configure PHY to SATA/SAS mode by setting pin PIN_PIPE_SEL=0
         */
-       reg_set(COMPHY_SEL_ADDR, 0, rf_compy_select(lane));
+       saved_selector = readl(COMPHY_SEL_ADDR);
+       reg_set(COMPHY_SEL_ADDR, 0, 0xFFFFFFFF);
 
        /*
         * 2. Reset PHY by setting PHY input port PIN_RESET=1.
@@ -874,6 +876,11 @@ static int comphy_sgmii_power_up(u32 lane, u32 speed, u32 invert)
        if (!ret)
                printf("Failed to init RX of SGMII PHY %d\n", lane);
 
+       /*
+        * Restore saved selector.
+        */
+       reg_set(COMPHY_SEL_ADDR, saved_selector, 0xFFFFFFFF);
+
        debug_exit();
 
        return ret;
index 0f0138dc977209a99a82e7a5d92601cf5912599c..a14767d809b6ba35862c6efa600c330a084beb8b 100644 (file)
@@ -22,7 +22,6 @@
  * COMPHY SB definitions
  */
 #define COMPHY_SEL_ADDR                        MVEBU_REG(0x0183FC)
-#define rf_compy_select(lane)          (0x1 << (((lane) == 1) ? 4 : 0))
 
 #define COMPHY_PHY_CFG1_ADDR(lane)     MVEBU_REG(0x018300 + (1 - lane) * 0x28)
 #define rb_pin_pu_iveref               BIT(1)