Set the RMII enable bit when an AST2700 port uses RMII so the MAC is programmed for the selected interface mode. Describe the requirement as a match-data quirk instead of checking the MAC generation in the data path, allowing later compatible controllers to opt into the behavior. Signed-off-by: Jacky Chou --- drivers/net/ethernet/faraday/ftgmac100.c | 16 ++++++++++++++-- drivers/net/ethernet/faraday/ftgmac100.h | 1 + 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/faraday/ftgmac100.c b/drivers/net/ethernet/faraday/ftgmac100.c index 67b1fa464a42..b835472da360 100644 --- a/drivers/net/ethernet/faraday/ftgmac100.c +++ b/drivers/net/ethernet/faraday/ftgmac100.c @@ -41,8 +41,11 @@ enum ftgmac100_mac_id { FTGMAC100_AST2700 }; +#define FTGMAC100_QUIRK_RMII_ENABLE BIT(0) + struct ftgmac100_match_data { enum ftgmac100_mac_id mac_id; + u32 quirks; }; /* Arbitrary values, I am not sure the HW has limits */ @@ -79,6 +82,7 @@ struct ftgmac100 { void __iomem *base; enum ftgmac100_mac_id mac_id; + u32 quirks; /* Rx ring */ unsigned int rx_q_entries; @@ -355,6 +359,7 @@ static void ftgmac100_init_hw(struct ftgmac100 *priv) static void ftgmac100_start_hw(struct ftgmac100 *priv) { u32 maccr = ioread32(priv->base + FTGMAC100_OFFSET_MACCR); + struct phy_device *phydev = priv->netdev->phydev; /* Keep the original GMAC and FAST bits */ maccr &= (FTGMAC100_MACCR_FAST_MODE | FTGMAC100_MACCR_GIGA_MODE); @@ -383,6 +388,11 @@ static void ftgmac100_start_hw(struct ftgmac100 *priv) if (priv->netdev->features & NETIF_F_HW_VLAN_CTAG_RX) maccr |= FTGMAC100_MACCR_RM_VLAN; + if ((priv->quirks & FTGMAC100_QUIRK_RMII_ENABLE) && + phydev->interface == PHY_INTERFACE_MODE_RMII) { + maccr |= FTGMAC100_MACCR_RMII_ENABLE; + } + /* Hit the HW */ iowrite32(maccr, priv->base + FTGMAC100_OFFSET_MACCR); } @@ -1942,7 +1952,7 @@ static int ftgmac100_probe_dt(struct net_device *netdev, static int ftgmac100_probe(struct platform_device *pdev) { - const struct ftgmac100_match_data *match_data; + const struct ftgmac100_match_data *match_data = NULL; enum ftgmac100_mac_id mac_id; struct resource *res; int irq; @@ -1987,6 +1997,7 @@ static int ftgmac100_probe(struct platform_device *pdev) priv->netdev = netdev; priv->dev = &pdev->dev; priv->mac_id = mac_id; + priv->quirks = match_data ? match_data->quirks : 0; INIT_WORK(&priv->reset_task, ftgmac100_reset_task); /* map io memory */ @@ -2134,7 +2145,8 @@ static const struct ftgmac100_match_data ftgmac100_match_data_ast2600 = { }; static const struct ftgmac100_match_data ftgmac100_match_data_ast2700 = { - .mac_id = FTGMAC100_AST2700 + .mac_id = FTGMAC100_AST2700, + .quirks = FTGMAC100_QUIRK_RMII_ENABLE, }; static const struct ftgmac100_match_data ftgmac100_match_data_faraday = { diff --git a/drivers/net/ethernet/faraday/ftgmac100.h b/drivers/net/ethernet/faraday/ftgmac100.h index 4968f6f0bdbc..8c35c895b146 100644 --- a/drivers/net/ethernet/faraday/ftgmac100.h +++ b/drivers/net/ethernet/faraday/ftgmac100.h @@ -167,6 +167,7 @@ #define FTGMAC100_MACCR_RX_BROADPKT (1 << 17) #define FTGMAC100_MACCR_DISCARD_CRCERR (1 << 18) #define FTGMAC100_MACCR_FAST_MODE (1 << 19) +#define FTGMAC100_MACCR_RMII_ENABLE BIT(20) #define FTGMAC100_MACCR_SW_RST (1 << 31) /* -- 2.34.1