diff options
| author | Li Yang | 2007-10-15 17:01:12 +0200 |
|---|---|---|
| committer | Jeff Garzik | 2007-10-15 20:41:34 +0200 |
| commit | 293c8513398657f6263fcdb03c87f2760cf61be4 (patch) | |
| tree | 9c345b9c12a02e57d70a01bb881e5f7f9f91ce75 /drivers/net | |
| parent | gianfar: Cleanup compile warning caused by 0795af57 (diff) | |
| download | kernel-qcow2-linux-293c8513398657f6263fcdb03c87f2760cf61be4.tar.gz kernel-qcow2-linux-293c8513398657f6263fcdb03c87f2760cf61be4.tar.xz kernel-qcow2-linux-293c8513398657f6263fcdb03c87f2760cf61be4.zip | |
gianfar: Fix regression caused by new napi interface
Protect all new napi function calls with CONFIG_GFAR_NAPI. Otherwise
the driver will stop working when CONFIG_GFAR_NAPI disabled.
Signed-off-by: Li Yang <leoli@freescale.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net')
| -rw-r--r-- | drivers/net/gianfar.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c index 15099505915d..558440c15b6c 100644 --- a/drivers/net/gianfar.c +++ b/drivers/net/gianfar.c @@ -932,10 +932,14 @@ tx_skb_fail: /* Returns 0 for success. */ static int gfar_enet_open(struct net_device *dev) { +#ifdef CONFIG_GFAR_NAPI struct gfar_private *priv = netdev_priv(dev); +#endif int err; +#ifdef CONFIG_GFAR_NAPI napi_enable(&priv->napi); +#endif /* Initialize a bunch of registers */ init_registers(dev); @@ -945,13 +949,17 @@ static int gfar_enet_open(struct net_device *dev) err = init_phy(dev); if(err) { +#ifdef CONFIG_GFAR_NAPI napi_disable(&priv->napi); +#endif return err; } err = startup_gfar(dev); if (err) +#ifdef CONFIG_GFAR_NAPI napi_disable(&priv->napi); +#endif netif_start_queue(dev); @@ -1105,7 +1113,9 @@ static int gfar_close(struct net_device *dev) { struct gfar_private *priv = netdev_priv(dev); +#ifdef CONFIG_GFAR_NAPI napi_disable(&priv->napi); +#endif stop_gfar(dev); |