ar71xx: move phy interface setup into a separate function

SVN-Revision: 28987

target/linux/ar71xx/files/arch/mips/ar71xx/devices.c

@@ -585,58 +585,73 @@ static void __init ar71xx_init_eth_pll_data(unsigned int id)
 		pll_data->pll_1000 = pll_1000;
 }
 
-static int ar71xx_eth_instance __initdata;
-void __init ar71xx_add_device_eth(unsigned int id)
+static int __init ar71xx_setup_phy_if_mode(unsigned int id,
+					   struct ag71xx_platform_data *pdata)
 {
-	struct platform_device *pdev;
-	struct ag71xx_platform_data *pdata;
-
-	ar71xx_init_eth_pll_data(id);
-
 	switch (id) {
 	case 0:
-		switch (ar71xx_eth0_data.phy_if_mode) {
+		switch (pdata->phy_if_mode) {
 		case PHY_INTERFACE_MODE_MII:
-			ar71xx_eth0_data.mii_if = MII0_CTRL_IF_MII;
+			pdata->mii_if = MII0_CTRL_IF_MII;
 			break;
 		case PHY_INTERFACE_MODE_GMII:
-			ar71xx_eth0_data.mii_if = MII0_CTRL_IF_GMII;
+			pdata->mii_if = MII0_CTRL_IF_GMII;
 			break;
 		case PHY_INTERFACE_MODE_RGMII:
-			ar71xx_eth0_data.mii_if = MII0_CTRL_IF_RGMII;
+			pdata->mii_if = MII0_CTRL_IF_RGMII;
 			break;
 		case PHY_INTERFACE_MODE_RMII:
-			ar71xx_eth0_data.mii_if = MII0_CTRL_IF_RMII;
+			pdata->mii_if = MII0_CTRL_IF_RMII;
 			break;
 		default:
-			printk(KERN_ERR "ar71xx: invalid PHY interface mode "
-					"for eth0\n");
-			return;
+			return -EINVAL;
 		}
-		pdev = &ar71xx_eth0_device;
 		break;
 	case 1:
-		switch (ar71xx_eth1_data.phy_if_mode) {
+		switch (pdata->phy_if_mode) {
 		case PHY_INTERFACE_MODE_RMII:
-			ar71xx_eth1_data.mii_if = MII1_CTRL_IF_RMII;
+			pdata->mii_if = MII1_CTRL_IF_RMII;
 			break;
 		case PHY_INTERFACE_MODE_RGMII:
-			ar71xx_eth1_data.mii_if = MII1_CTRL_IF_RGMII;
+			pdata->mii_if = MII1_CTRL_IF_RGMII;
 			break;
 		default:
-			printk(KERN_ERR "ar71xx: invalid PHY interface mode "
-					"for eth1\n");
-			return;
+			return -EINVAL;
 		}
-		pdev = &ar71xx_eth1_device;
 		break;
-	default:
+	}
+
+	return 0;
+}
+
+static int ar71xx_eth_instance __initdata;
+void __init ar71xx_add_device_eth(unsigned int id)
+{
+	struct platform_device *pdev;
+	struct ag71xx_platform_data *pdata;
+	int err;
+
+	if (id > 1) {
 		printk(KERN_ERR "ar71xx: invalid ethernet id %d\n", id);
 		return;
 	}
 
+	ar71xx_init_eth_pll_data(id);
+
+	if (id == 0)
+		pdev = &ar71xx_eth0_device;
+	else
+		pdev = &ar71xx_eth1_device;
+
 	pdata = pdev->dev.platform_data;
 
+	err = ar71xx_setup_phy_if_mode(id, pdata);
+	if (err) {
+		printk(KERN_ERR
+		       "ar71xx: invalid PHY interface mode for GE%u\n", id);
+		return;
+	}
+
 	switch (ar71xx_soc) {
 	case AR71XX_SOC_AR7130:
 		pdata->ddr_flush = id ? ar71xx_ddr_flush_ge1