[PATCH] Re: Next March 25: staging/epl build break.

From: Subrata Modak
Date: Wed Mar 25 2009 - 10:07:31 EST


Hi Greg,

>Today's next x86 randconfig build failed with
>
>drivers/staging/epl/VirtualEthernetLinux.c: In function VEthAddInstance:
>drivers/staging/epl/VirtualEthernetLinux.c:302: error: struct net_device has no member named open
>drivers/staging/epl/VirtualEthernetLinux.c:303: error: struct net_device has no member named stop
>drivers/staging/epl/VirtualEthernetLinux.c:304: error: struct net_device has no member named get_stats
>drivers/staging/epl/VirtualEthernetLinux.c:305: error: struct net_device has no member named hard_start_xmit
>drivers/staging/epl/VirtualEthernetLinux.c:306: error: struct net_device has no member named tx_timeout
>

The above build problem reported by Sachin can be fixed with the
patch below. However, it still will leave with the following
warnings, which i think can be ignored:

drivers/staging/epl/VirtualEthernetLinux.c:164: warning: â??VEthOpenâ?? defined but not used
drivers/staging/epl/VirtualEthernetLinux.c:183: warning: â??VEthCloseâ?? defined but not used
drivers/staging/epl/VirtualEthernetLinux.c:196: warning: â??VEthXmitâ?? defined but not used
drivers/staging/epl/VirtualEthernetLinux.c:231: warning: â??VEthGetStatsâ?? defined but not used
drivers/staging/epl/VirtualEthernetLinux.c:238: warning: â??VEthTimeoutâ?? defined but not used

>.config attached.
>
>Thanks
>-Sachin

Signed-Off-By: Subrata Modak <subrata@xxxxxxxxxxxxxxxxxx>
---

--- a/linux-2.6.29/drivers/staging/epl/Kconfig 2009-03-24 04:42:14.000000000 +0530
+++ b/linux-2.6.29/drivers/staging/epl/Kconfig 2009-03-25 19:01:18.000000000 +0530
@@ -1,6 +1,7 @@
config EPL
tristate "openPOWERLINK protocol stack"
depends on NET && HIGH_RES_TIMERS && X86
+ depends on COMPAT_NET_DEV_OPS
default N
---help---
Enable support for the openPOWERLINK network protocol stack.
--- a/linux-2.6.29/drivers/staging/epl/VirtualEthernetLinux.c 2009-03-25 18:38:06.000000000 +0530
+++ b/linux-2.6.29/drivers/staging/epl/VirtualEthernetLinux.c 2009-03-25 19:03:16.000000000 +0530
@@ -299,11 +299,13 @@ tEplKernel VEthAddInstance(tEplDllkInitP
goto Exit;
}

+#ifdef CONFIG_COMPAT_NET_DEV_OPS
pVEthNetDevice_g->open = VEthOpen;
pVEthNetDevice_g->stop = VEthClose;
pVEthNetDevice_g->get_stats = VEthGetStats;
pVEthNetDevice_g->hard_start_xmit = VEthXmit;
pVEthNetDevice_g->tx_timeout = VEthTimeout;
+#endif
pVEthNetDevice_g->watchdog_timeo = EPL_VETH_TX_TIMEOUT;
pVEthNetDevice_g->destructor = free_netdev;

--
Regards--
Subrata

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