* [PATCH 4/6] IP100A Change search phy addr start form 0
@ 2006-08-17 19:07 Jesse Huang
2006-08-18 11:20 ` Jeff Garzik
0 siblings, 1 reply; 3+ messages in thread
From: Jesse Huang @ 2006-08-17 19:07 UTC (permalink / raw)
To: linux-kernel, netdev, akpm, jgarzik, jesse
From: Jesse Huang <jesse@icplus.com.tw>
Change search phy addr start form 0
Change Logs:
Change search phy addr start form 0
---
drivers/net/sundance.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
212cd4ffa21a57300eae4254bf02e5b33b96f544
diff --git a/drivers/net/sundance.c b/drivers/net/sundance.c
index 2bde1b3..f63871a 100755
--- a/drivers/net/sundance.c
+++ b/drivers/net/sundance.c
@@ -21,7 +21,7 @@
*/
#define DRV_NAME "sundance"
-#define DRV_VERSION "1.01+LK1.13"
+#define DRV_VERSION "1.01+LK1.14"
#define DRV_RELDATE "04-Aug-2006"
@@ -559,8 +559,9 @@ #endif
/*
* It seems some phys doesn't deal well with address 0 being accessed
* first, so leave address zero to the end of the loop (32 & 31).
+ * for IP100A the phy should start from 0
*/
- for (phy = 1; phy <= 32 && phy_idx < MII_CNT; phy++) {
+ for (phy = 0; phy <= 32 && phy_idx < MII_CNT; phy++) {
int phyx = phy & 0x1f;
int mii_status = mdio_read(dev, phyx, MII_BMSR);
if (mii_status != 0xffff && mii_status != 0x0000) {
--
1.3.GIT
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH 4/6] IP100A Change search phy addr start form 0
2006-08-17 19:07 [PATCH 4/6] IP100A Change search phy addr start form 0 Jesse Huang
@ 2006-08-18 11:20 ` Jeff Garzik
2006-08-21 5:40 ` Jesse Huang
0 siblings, 1 reply; 3+ messages in thread
From: Jeff Garzik @ 2006-08-18 11:20 UTC (permalink / raw)
To: Jesse Huang; +Cc: linux-kernel, netdev, akpm
Jesse Huang wrote:
> From: Jesse Huang <jesse@icplus.com.tw>
>
> Change search phy addr start form 0
>
> Change Logs:
> Change search phy addr start form 0
>
> ---
>
> drivers/net/sundance.c | 5 +++--
> 1 files changed, 3 insertions(+), 2 deletions(-)
>
> 212cd4ffa21a57300eae4254bf02e5b33b96f544
> diff --git a/drivers/net/sundance.c b/drivers/net/sundance.c
> index 2bde1b3..f63871a 100755
> --- a/drivers/net/sundance.c
> +++ b/drivers/net/sundance.c
> @@ -21,7 +21,7 @@
> */
>
> #define DRV_NAME "sundance"
> -#define DRV_VERSION "1.01+LK1.13"
> +#define DRV_VERSION "1.01+LK1.14"
> #define DRV_RELDATE "04-Aug-2006"
>
>
> @@ -559,8 +559,9 @@ #endif
> /*
> * It seems some phys doesn't deal well with address 0 being accessed
> * first, so leave address zero to the end of the loop (32 & 31).
> + * for IP100A the phy should start from 0
> */
> - for (phy = 1; phy <= 32 && phy_idx < MII_CNT; phy++) {
> + for (phy = 0; phy <= 32 && phy_idx < MII_CNT; phy++) {
> int phyx = phy & 0x1f;
> int mii_status = mdio_read(dev, phyx, MII_BMSR);
> if (mii_status != 0xffff && mii_status != 0x0000) {
For IP100A, is the phy built into the chip?
For a standard DP83840[A] phy, phy #0 is a "ghost" which mirrors another
phy. For this reason, we scan phy #0 last.
Does the above code not work?
Jeff
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH 4/6] IP100A Change search phy addr start form 0
2006-08-18 11:20 ` Jeff Garzik
@ 2006-08-21 5:40 ` Jesse Huang
0 siblings, 0 replies; 3+ messages in thread
From: Jesse Huang @ 2006-08-21 5:40 UTC (permalink / raw)
To: Jeff Garzik; +Cc: linux-kernel, netdev, akpm
Hi Jeff:
In IP100A, phy address is 0. Because IP100A is a single chip, the in
chip phy address is 0. so, we must search phy address for 0.
Jesse
----- Original Message -----
From: "Jeff Garzik" <jgarzik@pobox.com>
To: "Jesse Huang" <jesse@icplus.com.tw>
Cc: <linux-kernel@vger.kernel.org>; <netdev@vger.kernel.org>;
<akpm@osdl.org>
Sent: Friday, August 18, 2006 7:20 PM
Subject: Re: [PATCH 4/6] IP100A Change search phy addr start form 0
Jesse Huang wrote:
> From: Jesse Huang <jesse@icplus.com.tw>
>
> Change search phy addr start form 0
>
> Change Logs:
> Change search phy addr start form 0
>
> ---
>
> drivers/net/sundance.c | 5 +++--
> 1 files changed, 3 insertions(+), 2 deletions(-)
>
> 212cd4ffa21a57300eae4254bf02e5b33b96f544
> diff --git a/drivers/net/sundance.c b/drivers/net/sundance.c
> index 2bde1b3..f63871a 100755
> --- a/drivers/net/sundance.c
> +++ b/drivers/net/sundance.c
> @@ -21,7 +21,7 @@
> */
>
> #define DRV_NAME "sundance"
> -#define DRV_VERSION "1.01+LK1.13"
> +#define DRV_VERSION "1.01+LK1.14"
> #define DRV_RELDATE "04-Aug-2006"
>
>
> @@ -559,8 +559,9 @@ #endif
> /*
> * It seems some phys doesn't deal well with address 0 being accessed
> * first, so leave address zero to the end of the loop (32 & 31).
> + * for IP100A the phy should start from 0
> */
> - for (phy = 1; phy <= 32 && phy_idx < MII_CNT; phy++) {
> + for (phy = 0; phy <= 32 && phy_idx < MII_CNT; phy++) {
> int phyx = phy & 0x1f;
> int mii_status = mdio_read(dev, phyx, MII_BMSR);
> if (mii_status != 0xffff && mii_status != 0x0000) {
For IP100A, is the phy built into the chip?
For a standard DP83840[A] phy, phy #0 is a "ghost" which mirrors another
phy. For this reason, we scan phy #0 last.
Does the above code not work?
Jeff
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-08-21 5:40 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-08-17 19:07 [PATCH 4/6] IP100A Change search phy addr start form 0 Jesse Huang
2006-08-18 11:20 ` Jeff Garzik
2006-08-21 5:40 ` Jesse Huang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).