Re: Linux 2.6.22.2

[Ayaz Abdulla]

Yes, you are right. Copy and paste error. I have attached a patch which 
will fix this issue.

Thanks for catching it.
Ayaz

Signed-off-by: Ayaz Abdulla <aabdulla@xxxxxxxxxx>


Prakash Punnoor wrote:
> Hi,
>
> I just noticed that PHY_OUI_VITESSE == PHY_OUI_REALTEK. Is that really 
> intentional?
>
>
> > diff --git a/drivers/net/forcedeth.c b/drivers/net/forcedeth.c
> > index 42ba1c0..a361dba 100644
> > --- a/drivers/net/forcedeth.c
> > +++ b/drivers/net/forcedeth.c
> > @@ -550,6 +550,8 @@ union ring_type {
> > /* PHY defines */
> > #define PHY_OUI_MARVELL	0x5043
> > #define PHY_OUI_CICADA	0x03f1
> > +#define PHY_OUI_VITESSE	0x01c1
> > +#define PHY_OUI_REALTEK	0x01c1
>
> - 
> (°=                 =°)
> //\ Prakash Punnoor /\\
> V_/                 \_V
--- old/drivers/net/forcedeth.c	2007-08-09 17:37:12.000000000 -0400
+++ new/drivers/net/forcedeth.c	2007-08-09 17:37:07.000000000 -0400
@@ -551,7 +551,7 @@
 #define PHY_OUI_MARVELL	0x5043
 #define PHY_OUI_CICADA	0x03f1
 #define PHY_OUI_VITESSE	0x01c1
-#define PHY_OUI_REALTEK	0x01c1
+#define PHY_OUI_REALTEK	0x0732
 #define PHYID1_OUI_MASK	0x03ff
 #define PHYID1_OUI_SHFT	6
 #define PHYID2_OUI_MASK	0xfc00