Changeset 45954


Ignore:
Timestamp:
2015-06-14T19:41:01+02:00 (3 years ago)
Author:
nbd
Message:

ar71xx: fix 100/10mbps ethernet link issues on mynet range extender

The mynet range extender hardware is suffering from ethernet
link loss when booting with a recent openwrt image. This only
happens on 100mbps links, with 1Gbps speed the link was fine.

The cause of the problem is that the AR8035 PHY (aka F1E)
requires turning on and off the special TX delay setting
depending on the speed of the link.

The 10mbps mode only needed the proper pll value, which was
extracted from the vendor code.

Reported-by: Pascal Paradis
Signed-off-by: Christian Lamparter <chunkeey@…>

Location:
trunk/target/linux/ar71xx
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/target/linux/ar71xx/files/arch/mips/ath79/mach-mynet-rext.c

    r44665 r45954  
    1515#include <linux/ath9k_platform.h> 
    1616#include <linux/ar8216_platform.h> 
     17#include <linux/platform_data/phy-at803x.h> 
    1718 
    1819#include <asm/mach-ath79/ar71xx_regs.h> 
     
    125126}; 
    126127 
     128static struct at803x_platform_data mynet_rext_at803x_data = { 
     129        .disable_smarteee = 0, 
     130        .enable_rgmii_rx_delay = 1, 
     131        .enable_rgmii_tx_delay = 0, 
     132        .fixup_rgmii_tx_delay = 1, 
     133}; 
     134 
     135static struct mdio_board_info mynet_rext_mdio0_info[] = { 
     136        { 
     137                .bus_id = "ag71xx-mdio.0", 
     138                .phy_addr = 4, 
     139                .platform_data = &mynet_rext_at803x_data, 
     140        }, 
     141}; 
     142 
    127143static void mynet_rext_get_mac(const char *name, char *mac) 
    128144{ 
     
    170186        ath79_register_mdio(0, 0x0); 
    171187 
     188        mdiobus_register_board_info(mynet_rext_mdio0_info, 
     189                                    ARRAY_SIZE(mynet_rext_mdio0_info)); 
     190 
    172191        /* LAN */ 
    173192        mynet_rext_get_mac("et0macaddr=", ath79_eth0_data.mac_addr); 
     
    176195        ath79_eth0_data.phy_if_mode = PHY_INTERFACE_MODE_RGMII; 
    177196        ath79_eth0_data.phy_mask = BIT(4); 
     197        ath79_eth0_pll_data.pll_10   = 0x00001313; /* athrs_mac.c */ 
    178198        ath79_eth0_pll_data.pll_1000 = 0x0e000000; /* athrs_mac.c */ 
    179199        ath79_eth0_data.mii_bus_dev = &ath79_mdio0_device.dev; 
  • trunk/target/linux/ar71xx/patches-3.18/425-net-phy-at803x-allow-to-configure-via-pdata.patch

    r44456 r45954  
    3333 #define AT803X_DEBUG_RGMII_TX_CLK_DLY          BIT(8) 
    3434  
     35@@ -50,6 +60,7 @@ MODULE_LICENSE("GPL"); 
     36 struct at803x_priv { 
     37        bool phy_reset:1; 
     38        struct gpio_desc *gpiod_reset; 
     39+       int prev_speed; 
     40 }; 
     41  
     42 struct at803x_context { 
    3543@@ -61,6 +71,43 @@ struct at803x_context { 
    3644        u16 led_control; 
     
    121129 } 
    122130  
     131@@ -258,6 +334,8 @@ static int at803x_config_intr(struct phy 
     132 static void at803x_link_change_notify(struct phy_device *phydev) 
     133 { 
     134        struct at803x_priv *priv = phydev->priv; 
     135+       struct at803x_platform_data *pdata; 
     136+       pdata = dev_get_platdata(&phydev->dev); 
     137  
     138        /* 
     139         * Conduct a hardware reset for AT8030 every time a link loss is 
     140@@ -287,6 +365,26 @@ static void at803x_link_change_notify(st 
     141                } else { 
     142                        priv->phy_reset = false; 
     143                } 
     144+       } 
     145+       if (pdata->fixup_rgmii_tx_delay && 
     146+           phydev->speed != priv->prev_speed) { 
     147+               switch (phydev->speed) { 
     148+               case SPEED_10: 
     149+               case SPEED_100: 
     150+                       at803x_dbg_reg_set(phydev, 
     151+                               AT803X_DEBUG_SYSTEM_MODE_CTRL, 
     152+                               AT803X_DEBUG_RGMII_TX_CLK_DLY); 
     153+                       break; 
     154+               case SPEED_1000: 
     155+                       at803x_dbg_reg_clr(phydev, 
     156+                               AT803X_DEBUG_SYSTEM_MODE_CTRL, 
     157+                               AT803X_DEBUG_RGMII_TX_CLK_DLY); 
     158+                       break; 
     159+               default: 
     160+                       break; 
     161+               } 
     162+ 
     163+               priv->prev_speed = phydev->speed; 
     164        } 
     165 } 
     166  
    123167--- /dev/null 
    124168+++ b/include/linux/platform_data/phy-at803x.h 
    125 @@ -0,0 +1,10 @@ 
     169@@ -0,0 +1,11 @@ 
    126170+#ifndef _PHY_AT803X_PDATA_H 
    127171+#define _PHY_AT803X_PDATA_H 
    128172+ 
    129173+struct at803x_platform_data { 
    130 +       int disable_smarteee:1; 
    131 +       int enable_rgmii_tx_delay:1; 
    132 +       int enable_rgmii_rx_delay:1; 
     174+       int disable_smarteee:1; 
     175+       int enable_rgmii_tx_delay:1; 
     176+       int enable_rgmii_rx_delay:1; 
     177+       int fixup_rgmii_tx_delay:1; 
    133178+}; 
    134179+ 
Note: See TracChangeset for help on using the changeset viewer.