Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-16 Thread Jeff Garzik
Tom Rini wrote:
There's too many things in here that've sat too long (I'd been hoping to
just delete the driver, but that hasn't happened yet, so).  A cobbled
together list of changes is:
- Update MDIO support for workqueues.
- Make use of 
- Add RPX6 support.
- Comment out set_multicast_list (broken).
- Rework tx_ring stuff so we have tx_free, not tx_Full/n_pkts.
- Other PHY updates/fixes.
- Leo Li: Rework FCC clock configuration, make it easier.
- 2.4 : VLAN header room, other misc bits.
- Kill MII_REG_NNN in favor of defines from 
- DM9161 PHY support (2.4, Myself & [EMAIL PROTECTED])
- PQ2ADS and PQ2FADS support bits (Myself & [EMAIL PROTECTED]
Please make sure I and netdev@oss.sgi.com are CC'd on all net driver 
patches.

Also, it would be nice to move this into drivers/net/*
Jeff
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-16 Thread Balasaygun, Oray (Oray)
I concur, the two essential things in my patch (2.6 API change and fix for bug 
4310) are already picked up in your patch. I could have replied earlier but I 
thought I would take an hour or two to run with your patch on my board. But 
that can wait since you guys seem to be in a hurry with a decision on this.

Oray



-Original Message-
From: Tom Rini [mailto:[EMAIL PROTECTED]
Sent: Wednesday, March 16, 2005 9:55 AM
To: Bastos Fernandez Alexandre
Cc: 'Andrew Morton'; linux-kernel@vger.kernel.org; Leo Li; Balasaygun,
Oray (Oray)
Subject: Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again


On Wed, Mar 16, 2005 at 09:00:51AM +0100, Bastos Fernandez Alexandre wrote:

> What can we do?
> Has Oray's patch been commited? Or not yet?
> 
> I suggest commiting all the changes in this driver from linuxppc tree to
> linux tree
> and after this ask Oray to test again the driver and submmit new patch.
> Or ask Oray to submmit changes patch to linuxppc 

So, Oray's patch has two things in it, it looks like.  The basic
re-porting to 2.6 APIs, and support for the EON8260.  The EON8260
changes should be a bit cleaner to redo after the patch I sent goes in
as some of the other cleanups will help there.  But I'd also like to see
the full EON8260 changes :)  The 2.6 API parts look close, but I went
and expanded a good bit more on them.

I'd like to drop Oray's patch for now, and have the EON8260 work be
resubmitted to lkml/[EMAIL PROTECTED]

-- 
Tom Rini
http://gate.crashing.org/~trini/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-16 Thread Tom Rini
On Wed, Mar 16, 2005 at 09:00:51AM +0100, Bastos Fernandez Alexandre wrote:

> What can we do?
> Has Oray's patch been commited? Or not yet?
> 
> I suggest commiting all the changes in this driver from linuxppc tree to
> linux tree
> and after this ask Oray to test again the driver and submmit new patch.
> Or ask Oray to submmit changes patch to linuxppc 

So, Oray's patch has two things in it, it looks like.  The basic
re-porting to 2.6 APIs, and support for the EON8260.  The EON8260
changes should be a bit cleaner to redo after the patch I sent goes in
as some of the other cleanups will help there.  But I'd also like to see
the full EON8260 changes :)  The 2.6 API parts look close, but I went
and expanded a good bit more on them.

I'd like to drop Oray's patch for now, and have the EON8260 work be
resubmitted to lkml/[EMAIL PROTECTED]

-- 
Tom Rini
http://gate.crashing.org/~trini/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-16 Thread Bastos Fernandez Alexandre
What can we do?
Has Oray's patch been commited? Or not yet?

I suggest commiting all the changes in this driver from linuxppc tree to
linux tree
and after this ask Oray to test again the driver and submmit new patch.
Or ask Oray to submmit changes patch to linuxppc 

Best regards,

Alex

> -Original Message-
> From: Andrew Morton [SMTP:[EMAIL PROTECTED]
> Sent: Wednesday, March 16, 2005 7:32 AM
> To:   Tom Rini
> Cc:   linux-kernel@vger.kernel.org; Leo Li; Alexandre Bastos; Balasaygun,
> Oray (Oray)
> Subject:      Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function
> again
> 
> Tom Rini <[EMAIL PROTECTED]> wrote:
> >
> > There's too many things in here that've sat too long (I'd been hoping to
> >  just delete the driver, but that hasn't happened yet, so).  A cobbled
> >  together list of changes is:
> > 
> >  - Update MDIO support for workqueues.
> >  - Make use of 
> >  - Add RPX6 support.
> >  - Comment out set_multicast_list (broken).
> >  - Rework tx_ring stuff so we have tx_free, not tx_Full/n_pkts.
> >  - Other PHY updates/fixes.
> >  - Leo Li: Rework FCC clock configuration, make it easier.
> >  - 2.4 : VLAN header room, other misc bits.
> >  - Kill MII_REG_NNN in favor of defines from 
> >  - DM9161 PHY support (2.4, Myself & [EMAIL PROTECTED])
> >  - PQ2ADS and PQ2FADS support bits (Myself & [EMAIL PROTECTED]
> > 
> >  From: Leo Li <[EMAIL PROTECTED]>
> >  Signed-off-by: Tom Rini <[EMAIL PROTECTED]>
> >  Signed-off-by: Alexandre Bastos <[EMAIL PROTECTED]>
> 
> That's unfortunate - Oray sent a patch in just a few days ago which also
> fixes up this driver.  See
> 
> ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.11/2.6.1
> 1-mm3/broken-out/ppc-8260-fcc-ethernet-driver-cannot-read-lxt971-phy-id.pa
> tch
> 
> What should we do?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-16 Thread Bastos Fernandez Alexandre
What can we do?
Has Oray's patch been commited? Or not yet?

I suggest commiting all the changes in this driver from linuxppc tree to
linux tree
and after this ask Oray to test again the driver and submmit new patch.
Or ask Oray to submmit changes patch to linuxppc 

Best regards,

Alex

 -Original Message-
 From: Andrew Morton [SMTP:[EMAIL PROTECTED]
 Sent: Wednesday, March 16, 2005 7:32 AM
 To:   Tom Rini
 Cc:   linux-kernel@vger.kernel.org; Leo Li; Alexandre Bastos; Balasaygun,
 Oray (Oray)
 Subject:  Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function
 again
 
 Tom Rini [EMAIL PROTECTED] wrote:
 
  There's too many things in here that've sat too long (I'd been hoping to
   just delete the driver, but that hasn't happened yet, so).  A cobbled
   together list of changes is:
  
   - Update MDIO support for workqueues.
   - Make use of linux/mii.h
   - Add RPX6 support.
   - Comment out set_multicast_list (broken).
   - Rework tx_ring stuff so we have tx_free, not tx_Full/n_pkts.
   - Other PHY updates/fixes.
   - Leo Li: Rework FCC clock configuration, make it easier.
   - 2.4 : VLAN header room, other misc bits.
   - Kill MII_REG_NNN in favor of defines from linux/mii.h
   - DM9161 PHY support (2.4, Myself  [EMAIL PROTECTED])
   - PQ2ADS and PQ2FADS support bits (Myself  [EMAIL PROTECTED]
  
   From: Leo Li [EMAIL PROTECTED]
   Signed-off-by: Tom Rini [EMAIL PROTECTED]
   Signed-off-by: Alexandre Bastos [EMAIL PROTECTED]
 
 That's unfortunate - Oray sent a patch in just a few days ago which also
 fixes up this driver.  See
 
 ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.11/2.6.1
 1-mm3/broken-out/ppc-8260-fcc-ethernet-driver-cannot-read-lxt971-phy-id.pa
 tch
 
 What should we do?
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-16 Thread Tom Rini
On Wed, Mar 16, 2005 at 09:00:51AM +0100, Bastos Fernandez Alexandre wrote:

 What can we do?
 Has Oray's patch been commited? Or not yet?
 
 I suggest commiting all the changes in this driver from linuxppc tree to
 linux tree
 and after this ask Oray to test again the driver and submmit new patch.
 Or ask Oray to submmit changes patch to linuxppc 

So, Oray's patch has two things in it, it looks like.  The basic
re-porting to 2.6 APIs, and support for the EON8260.  The EON8260
changes should be a bit cleaner to redo after the patch I sent goes in
as some of the other cleanups will help there.  But I'd also like to see
the full EON8260 changes :)  The 2.6 API parts look close, but I went
and expanded a good bit more on them.

I'd like to drop Oray's patch for now, and have the EON8260 work be
resubmitted to lkml/[EMAIL PROTECTED]

-- 
Tom Rini
http://gate.crashing.org/~trini/
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-16 Thread Balasaygun, Oray (Oray)
I concur, the two essential things in my patch (2.6 API change and fix for bug 
4310) are already picked up in your patch. I could have replied earlier but I 
thought I would take an hour or two to run with your patch on my board. But 
that can wait since you guys seem to be in a hurry with a decision on this.

Oray



-Original Message-
From: Tom Rini [mailto:[EMAIL PROTECTED]
Sent: Wednesday, March 16, 2005 9:55 AM
To: Bastos Fernandez Alexandre
Cc: 'Andrew Morton'; linux-kernel@vger.kernel.org; Leo Li; Balasaygun,
Oray (Oray)
Subject: Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again


On Wed, Mar 16, 2005 at 09:00:51AM +0100, Bastos Fernandez Alexandre wrote:

 What can we do?
 Has Oray's patch been commited? Or not yet?
 
 I suggest commiting all the changes in this driver from linuxppc tree to
 linux tree
 and after this ask Oray to test again the driver and submmit new patch.
 Or ask Oray to submmit changes patch to linuxppc 

So, Oray's patch has two things in it, it looks like.  The basic
re-porting to 2.6 APIs, and support for the EON8260.  The EON8260
changes should be a bit cleaner to redo after the patch I sent goes in
as some of the other cleanups will help there.  But I'd also like to see
the full EON8260 changes :)  The 2.6 API parts look close, but I went
and expanded a good bit more on them.

I'd like to drop Oray's patch for now, and have the EON8260 work be
resubmitted to lkml/[EMAIL PROTECTED]

-- 
Tom Rini
http://gate.crashing.org/~trini/
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-16 Thread Jeff Garzik
Tom Rini wrote:
There's too many things in here that've sat too long (I'd been hoping to
just delete the driver, but that hasn't happened yet, so).  A cobbled
together list of changes is:
- Update MDIO support for workqueues.
- Make use of linux/mii.h
- Add RPX6 support.
- Comment out set_multicast_list (broken).
- Rework tx_ring stuff so we have tx_free, not tx_Full/n_pkts.
- Other PHY updates/fixes.
- Leo Li: Rework FCC clock configuration, make it easier.
- 2.4 : VLAN header room, other misc bits.
- Kill MII_REG_NNN in favor of defines from linux/mii.h
- DM9161 PHY support (2.4, Myself  [EMAIL PROTECTED])
- PQ2ADS and PQ2FADS support bits (Myself  [EMAIL PROTECTED]
Please make sure I and netdev@oss.sgi.com are CC'd on all net driver 
patches.

Also, it would be nice to move this into drivers/net/*
Jeff
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-15 Thread Andrew Morton
Tom Rini <[EMAIL PROTECTED]> wrote:
>
> There's too many things in here that've sat too long (I'd been hoping to
>  just delete the driver, but that hasn't happened yet, so).  A cobbled
>  together list of changes is:
> 
>  - Update MDIO support for workqueues.
>  - Make use of 
>  - Add RPX6 support.
>  - Comment out set_multicast_list (broken).
>  - Rework tx_ring stuff so we have tx_free, not tx_Full/n_pkts.
>  - Other PHY updates/fixes.
>  - Leo Li: Rework FCC clock configuration, make it easier.
>  - 2.4 : VLAN header room, other misc bits.
>  - Kill MII_REG_NNN in favor of defines from 
>  - DM9161 PHY support (2.4, Myself & [EMAIL PROTECTED])
>  - PQ2ADS and PQ2FADS support bits (Myself & [EMAIL PROTECTED]
> 
>  From: Leo Li <[EMAIL PROTECTED]>
>  Signed-off-by: Tom Rini <[EMAIL PROTECTED]>
>  Signed-off-by: Alexandre Bastos <[EMAIL PROTECTED]>

That's unfortunate - Oray sent a patch in just a few days ago which also
fixes up this driver.  See

ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.11/2.6.11-mm3/broken-out/ppc-8260-fcc-ethernet-driver-cannot-read-lxt971-phy-id.patch

What should we do?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-15 Thread Tom Rini
There's too many things in here that've sat too long (I'd been hoping to
just delete the driver, but that hasn't happened yet, so).  A cobbled
together list of changes is:

- Update MDIO support for workqueues.
- Make use of 
- Add RPX6 support.
- Comment out set_multicast_list (broken).
- Rework tx_ring stuff so we have tx_free, not tx_Full/n_pkts.
- Other PHY updates/fixes.
- Leo Li: Rework FCC clock configuration, make it easier.
- 2.4 : VLAN header room, other misc bits.
- Kill MII_REG_NNN in favor of defines from 
- DM9161 PHY support (2.4, Myself & [EMAIL PROTECTED])
- PQ2ADS and PQ2FADS support bits (Myself & [EMAIL PROTECTED]

From: Leo Li <[EMAIL PROTECTED]>
Signed-off-by: Tom Rini <[EMAIL PROTECTED]>
Signed-off-by: Alexandre Bastos <[EMAIL PROTECTED]>

 arch/ppc/8260_io/Kconfig|   11 
 arch/ppc/8260_io/fcc_enet.c |  969 +---
 include/asm-ppc/cpm2.h  |1 
 3 files changed, 301 insertions(+), 680 deletions(-)
--- linuxppc-2.6.11/arch/ppc/8260_io/Kconfig2005-03-15 09:00:42.0 
-0700
+++ linux-2.6.11/arch/ppc/8260_io/Kconfig   2005-03-02 00:38:26.0 
-0700
@@ -42,7 +42,7 @@
 choice
prompt "Type of PHY"
depends on 8260 && USE_MDIO
-   default FCC_GENERIC_PHY
+   default FCC_LXT971
 
 config FCC_LXT970
bool "LXT970"
@@ -53,13 +53,6 @@
 config FCC_QS6612
bool "QS6612"
 
-config FCC_DM9131
-   bool "DM9131"
-
-config FCC_DM9161
-   bool "DM9161"
-
-config FCC_GENERIC_PHY
-   bool "Generic"
 endchoice
 endmenu
+
--- linuxppc-2.6.11/arch/ppc/8260_io/fcc_enet.c 2005-03-15 08:57:26.0 
-0700
+++ linux-2.6.11/arch/ppc/8260_io/fcc_enet.c2005-03-02 00:38:08.0 
-0700
@@ -16,9 +16,6 @@
  * small packets.  Since this is a cache coherent processor and CPM,
  * I could also preallocate SKB's and use them directly on the interface.
  *
- * 2004-12 Leo Li ([EMAIL PROTECTED])
- * - Rework the FCC clock configuration part, make it easier to configure.
- *
  */
 
 #include 
@@ -37,8 +34,6 @@
 #include 
 #include 
 #include 
-#include 
-#include 
 #include 
 
 #include 
@@ -46,26 +41,6 @@
 #include 
 #include 
 #include 
-#include 
-
-/* We can't use the PHY interrupt if we aren't using MDIO. */
-#if !defined(CONFIG_USE_MDIO)
-#undef PHY_INTERRUPT
-#endif
-
-/* If we have a PHY interrupt, we will advertise both full-duplex and half-
- * duplex capabilities.  If we don't have a PHY interrupt, then we will only
- * advertise half-duplex capabilities.
- */
-#define MII_ADVERTISE_HALF (ADVERTISE_100HALF | ADVERTISE_10HALF | \
-ADVERTISE_CSMA)
-#define MII_ADVERTISE_ALL  (ADVERTISE_100FULL | ADVERTISE_10FULL | \
-MII_ADVERTISE_HALF)
-#ifdef PHY_INTERRUPT
-#define MII_ADVERTISE_DEFAULT  MII_ADVERTISE_ALL
-#else
-#define MII_ADVERTISE_DEFAULT  MII_ADVERTISE_HALF
-#endif
 #include 
 
 /* The transmitter timeout
@@ -90,6 +65,18 @@
const phy_cmd_t *shutdown;
 } phy_info_t;
 
+/* Register definitions for the PHY. */
+
+#define MII_REG_CR  0  /* Control Register */
+#define MII_REG_SR  1  /* Status Register  */
+#define MII_REG_PHYIR1  2  /* PHY Identification Register 1*/
+#define MII_REG_PHYIR2  3  /* PHY Identification Register 2*/
+#define MII_REG_ANAR4  /* A-N Advertisement Register   */
+#define MII_REG_ANLPAR  5  /* A-N Link Partner Ability Register*/
+#define MII_REG_ANER6  /* A-N Expansion Register   */
+#define MII_REG_ANNPTR  7  /* A-N Next Page Transmit Register  */
+#define MII_REG_ANLPRNPR8  /* A-N Link Partner Received Next Page Reg. */
+
 /* values for phy_status */
 
 #define PHY_CONF_ANE   0x0001  /* 1 auto-negotiation enabled */
@@ -124,15 +111,13 @@
 #define TX_RING_MOD_MASK   15  /*   for this to work */
 
 /* The FCC stores dest/src/type, data, and checksum for receive packets.
- * size includes support for VLAN
  */
-#define PKT_MAXBUF_SIZE1522
+#define PKT_MAXBUF_SIZE1518
 #define PKT_MINBUF_SIZE64
 
 /* Maximum input DMA size.  Must be a should(?) be a multiple of 4.
- * size includes support for VLAN
- */
-#define PKT_MAXDMA_SIZE1524
+*/
+#define PKT_MAXDMA_SIZE1520
 
 /* Maximum input buffer size.  Must be a multiple of 32.
 */
@@ -144,9 +129,8 @@
 static irqreturn_t fcc_enet_interrupt(int irq, void *dev_id, struct pt_regs *);
 static int fcc_enet_close(struct net_device *dev);
 static struct net_device_stats *fcc_enet_get_stats(struct net_device *dev);
-/* static void set_multicast_list(struct net_device *dev); */
+static void set_multicast_list(struct net_device *dev);
 static void fcc_restart(struct net_device *dev, int duplex);
-static void fcc_stop(struct net_device *dev);
 static int fcc_enet_set_mac_address(struct net_device 

[PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-15 Thread Tom Rini
There's too many things in here that've sat too long (I'd been hoping to
just delete the driver, but that hasn't happened yet, so).  A cobbled
together list of changes is:

- Update MDIO support for workqueues.
- Make use of linux/mii.h
- Add RPX6 support.
- Comment out set_multicast_list (broken).
- Rework tx_ring stuff so we have tx_free, not tx_Full/n_pkts.
- Other PHY updates/fixes.
- Leo Li: Rework FCC clock configuration, make it easier.
- 2.4 : VLAN header room, other misc bits.
- Kill MII_REG_NNN in favor of defines from linux/mii.h
- DM9161 PHY support (2.4, Myself  [EMAIL PROTECTED])
- PQ2ADS and PQ2FADS support bits (Myself  [EMAIL PROTECTED]

From: Leo Li [EMAIL PROTECTED]
Signed-off-by: Tom Rini [EMAIL PROTECTED]
Signed-off-by: Alexandre Bastos [EMAIL PROTECTED]

 arch/ppc/8260_io/Kconfig|   11 
 arch/ppc/8260_io/fcc_enet.c |  969 +---
 include/asm-ppc/cpm2.h  |1 
 3 files changed, 301 insertions(+), 680 deletions(-)
--- linuxppc-2.6.11/arch/ppc/8260_io/Kconfig2005-03-15 09:00:42.0 
-0700
+++ linux-2.6.11/arch/ppc/8260_io/Kconfig   2005-03-02 00:38:26.0 
-0700
@@ -42,7 +42,7 @@
 choice
prompt Type of PHY
depends on 8260  USE_MDIO
-   default FCC_GENERIC_PHY
+   default FCC_LXT971
 
 config FCC_LXT970
bool LXT970
@@ -53,13 +53,6 @@
 config FCC_QS6612
bool QS6612
 
-config FCC_DM9131
-   bool DM9131
-
-config FCC_DM9161
-   bool DM9161
-
-config FCC_GENERIC_PHY
-   bool Generic
 endchoice
 endmenu
+
--- linuxppc-2.6.11/arch/ppc/8260_io/fcc_enet.c 2005-03-15 08:57:26.0 
-0700
+++ linux-2.6.11/arch/ppc/8260_io/fcc_enet.c2005-03-02 00:38:08.0 
-0700
@@ -16,9 +16,6 @@
  * small packets.  Since this is a cache coherent processor and CPM,
  * I could also preallocate SKB's and use them directly on the interface.
  *
- * 2004-12 Leo Li ([EMAIL PROTECTED])
- * - Rework the FCC clock configuration part, make it easier to configure.
- *
  */
 
 #include linux/config.h
@@ -37,8 +34,6 @@
 #include linux/etherdevice.h
 #include linux/skbuff.h
 #include linux/spinlock.h
-#include linux/mii.h
-#include linux/workqueue.h
 #include linux/bitops.h
 
 #include asm/immap_cpm2.h
@@ -46,26 +41,6 @@
 #include asm/mpc8260.h
 #include asm/irq.h
 #include asm/uaccess.h
-#include asm/signal.h
-
-/* We can't use the PHY interrupt if we aren't using MDIO. */
-#if !defined(CONFIG_USE_MDIO)
-#undef PHY_INTERRUPT
-#endif
-
-/* If we have a PHY interrupt, we will advertise both full-duplex and half-
- * duplex capabilities.  If we don't have a PHY interrupt, then we will only
- * advertise half-duplex capabilities.
- */
-#define MII_ADVERTISE_HALF (ADVERTISE_100HALF | ADVERTISE_10HALF | \
-ADVERTISE_CSMA)
-#define MII_ADVERTISE_ALL  (ADVERTISE_100FULL | ADVERTISE_10FULL | \
-MII_ADVERTISE_HALF)
-#ifdef PHY_INTERRUPT
-#define MII_ADVERTISE_DEFAULT  MII_ADVERTISE_ALL
-#else
-#define MII_ADVERTISE_DEFAULT  MII_ADVERTISE_HALF
-#endif
 #include asm/cpm2.h
 
 /* The transmitter timeout
@@ -90,6 +65,18 @@
const phy_cmd_t *shutdown;
 } phy_info_t;
 
+/* Register definitions for the PHY. */
+
+#define MII_REG_CR  0  /* Control Register */
+#define MII_REG_SR  1  /* Status Register  */
+#define MII_REG_PHYIR1  2  /* PHY Identification Register 1*/
+#define MII_REG_PHYIR2  3  /* PHY Identification Register 2*/
+#define MII_REG_ANAR4  /* A-N Advertisement Register   */
+#define MII_REG_ANLPAR  5  /* A-N Link Partner Ability Register*/
+#define MII_REG_ANER6  /* A-N Expansion Register   */
+#define MII_REG_ANNPTR  7  /* A-N Next Page Transmit Register  */
+#define MII_REG_ANLPRNPR8  /* A-N Link Partner Received Next Page Reg. */
+
 /* values for phy_status */
 
 #define PHY_CONF_ANE   0x0001  /* 1 auto-negotiation enabled */
@@ -124,15 +111,13 @@
 #define TX_RING_MOD_MASK   15  /*   for this to work */
 
 /* The FCC stores dest/src/type, data, and checksum for receive packets.
- * size includes support for VLAN
  */
-#define PKT_MAXBUF_SIZE1522
+#define PKT_MAXBUF_SIZE1518
 #define PKT_MINBUF_SIZE64
 
 /* Maximum input DMA size.  Must be a should(?) be a multiple of 4.
- * size includes support for VLAN
- */
-#define PKT_MAXDMA_SIZE1524
+*/
+#define PKT_MAXDMA_SIZE1520
 
 /* Maximum input buffer size.  Must be a multiple of 32.
 */
@@ -144,9 +129,8 @@
 static irqreturn_t fcc_enet_interrupt(int irq, void *dev_id, struct pt_regs *);
 static int fcc_enet_close(struct net_device *dev);
 static struct net_device_stats *fcc_enet_get_stats(struct net_device *dev);
-/* static void set_multicast_list(struct net_device *dev); */
+static void set_multicast_list(struct 

Re: [PATCH] ppc32: Update 8260_io/fcc_enet.c to function again

2005-03-15 Thread Andrew Morton
Tom Rini [EMAIL PROTECTED] wrote:

 There's too many things in here that've sat too long (I'd been hoping to
  just delete the driver, but that hasn't happened yet, so).  A cobbled
  together list of changes is:
 
  - Update MDIO support for workqueues.
  - Make use of linux/mii.h
  - Add RPX6 support.
  - Comment out set_multicast_list (broken).
  - Rework tx_ring stuff so we have tx_free, not tx_Full/n_pkts.
  - Other PHY updates/fixes.
  - Leo Li: Rework FCC clock configuration, make it easier.
  - 2.4 : VLAN header room, other misc bits.
  - Kill MII_REG_NNN in favor of defines from linux/mii.h
  - DM9161 PHY support (2.4, Myself  [EMAIL PROTECTED])
  - PQ2ADS and PQ2FADS support bits (Myself  [EMAIL PROTECTED]
 
  From: Leo Li [EMAIL PROTECTED]
  Signed-off-by: Tom Rini [EMAIL PROTECTED]
  Signed-off-by: Alexandre Bastos [EMAIL PROTECTED]

That's unfortunate - Oray sent a patch in just a few days ago which also
fixes up this driver.  See

ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.11/2.6.11-mm3/broken-out/ppc-8260-fcc-ethernet-driver-cannot-read-lxt971-phy-id.patch

What should we do?
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/