From 0976b6c4262a11a8d0dab9aeb64f5cdee266c44a Mon Sep 17 00:00:00 2001 From: Michael Pratt Date: Sat, 3 Apr 2021 14:42:51 -0400 Subject: ramips: mt7620: use DTS to set PHY base address for external PHYs Set the PHY base address to 12 for mt7530 and 8 for others, which is based on the default setting for some devices from printing the register with the following command after it is written to by uboot during the boot cycle. `md 0x10117014 1` PHY_BASE option only uses 5 bits of the register, bits 16 to 20, so use 8-bit integer type. Set the option using the DTS property mediatek,ephy-base and create the gsw node if missing. Also, added a kernel message to display the EPHY base address. Note: If anything is written to a PHY address that is greater than 1 hex char (greater than 0xf) then there is adverse effects with Atheros switches. Signed-off-by: Michael Pratt --- .../ramips/files/drivers/net/ethernet/ralink/gsw_mt7620.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'target/linux/ramips/files') diff --git a/target/linux/ramips/files/drivers/net/ethernet/ralink/gsw_mt7620.c b/target/linux/ramips/files/drivers/net/ethernet/ralink/gsw_mt7620.c index 4b030f457b..54b8b204eb 100644 --- a/target/linux/ramips/files/drivers/net/ethernet/ralink/gsw_mt7620.c +++ b/target/linux/ramips/files/drivers/net/ethernet/ralink/gsw_mt7620.c @@ -98,9 +98,6 @@ static void mt7620_hw_init(struct mt7620_gsw *gsw, int mdio_mode) mtk_switch_w32(gsw, mtk_switch_r32(gsw, GSW_REG_MIB_CNT_EN) | (1 << 1), GSW_REG_MIB_CNT_EN); if (mdio_mode) { - if (!gsw->ephy_base) - gsw->ephy_base = 12; - /* set MT7530 central align */ val = mt7530_mdio_r32(gsw, 0x7830); val &= ~BIT(0); @@ -115,11 +112,12 @@ static void mt7620_hw_init(struct mt7620_gsw *gsw, int mdio_mode) } if (gsw->ephy_base) { - /* set phy base addr to ephy_base */ mtk_switch_w32(gsw, mtk_switch_r32(gsw, GSW_REG_GPC1) | (gsw->ephy_base << 16), GSW_REG_GPC1); fe_reset(BIT(24)); /* Resets the Ethernet PHY block. */ + + pr_info("gsw: ephy base address: %d\n", gsw->ephy_base); } /* global page 4 */ @@ -246,7 +244,7 @@ static int mt7620_gsw_probe(struct platform_device *pdev) struct resource *res = platform_get_resource(pdev, IORESOURCE_MEM, 0); struct mt7620_gsw *gsw; struct device_node *np = pdev->dev.of_node; - u16 val; + u8 val; gsw = devm_kzalloc(&pdev->dev, sizeof(struct mt7620_gsw), GFP_KERNEL); if (!gsw) @@ -260,7 +258,7 @@ static int mt7620_gsw_probe(struct platform_device *pdev) gsw->port4_ephy = !of_property_read_bool(np, "mediatek,port4-gmac"); - if (of_property_read_u16(np, "mediatek,ephy-base-address", &val) == 0) + if (of_property_read_u8(np, "mediatek,ephy-base", &val) == 0) gsw->ephy_base = val; else gsw->ephy_base = 0; -- cgit v1.2.3