linux-kernel - r6040 mac driver issue and solution

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <07905E0D268758488D76D7D747F21BB6042DAAB0@rdc8.rdc>
Date:	Fri, 26 Apr 2013 20:29:02 +0800
From:	Bruce Ye (葉佳欣) <Bruce.Ye@....com.tw>
To:	<florian@...nwrt.org>
Cc:	<linux-kernel@...r.kernel.org>,
	Owen Peng (彭金隆) <Owen.Peng@....com.tw>,
	Bruce Ye (葉佳欣) <Bruce.Ye@....com.tw>
Subject: r6040 mac driver issue and solution

Dear Florian:
For RDC R6040 FAST ETHERNET DRIVER.
There's an issue in r6040 driver. It only occurs when there's more than one Ethernet port.
When initializing the MAC in boot up, r6040 driver set the same phy address to different ports.
It also display in boot log as below.
mii_bus:phy_addr=0:01
mii_bus:phy_addr=1:01

It will let the eth0 and eth1 work not in independent.
That is, if there is a connection on eth1 but not on eth0. The eth1 can not work only if link up the eth0.
To fix this problem is that giving the different phy address to different ports.
I already tested it on r6040.c version 0.27 and version 0.28 by the change and is worked.

Below is the change 

--- linux-3.0.4/drivers/net/r6040.c.orig	2013-04-26 04:36:51.736846552 -0700
+++ linux-3.0.4/drivers/net/r6040.c	2013-04-26 04:36:19.019588116 -0700
@@ -1030,12 +1030,25 @@ static void r6040_adjust_link(struct net
 	}
 }

+int phy_device_buffer[PHY_MAX_ADDR]={0};
+
 static int r6040_mii_probe(struct net_device *dev)
 {
 	struct r6040_private *lp = netdev_priv(dev);
 	struct phy_device *phydev = NULL;

-	phydev = phy_find_first(lp->mii_bus);
+	int addr;
+
+	for (addr = 0; addr < PHY_MAX_ADDR; addr++) {
+		
+		if (lp->mii_bus->phy_map[addr] && phy_device_buffer[addr]==0){
+			phy_device_buffer[addr]=1;
+			phydev = lp->mii_bus->phy_map[addr];
+			}
+		if(phydev)
+			break;
+	}
+
 	if (!phydev) {
 		dev_err(&lp->pdev->dev, "no PHY found\n");
 		return -ENODEV;

Signed-off-by: Bruce Ye Developer <Bruce.Ye@....com.tw>

Best Regards
Bruce
RDC Semiconductor
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/