A little ctc cleanup

From: Pete Zaitcev
Date: Fri Jan 16 2004 - 15:26:59 EST


Hi, Martin:

With the current state machine, doing fsm_event(DEV_EVENT_STOP) is an
exact equivalent of dev_action_stop, only it adds a (superfluous) debugging
tracepoint, which is useless anyway, because dev_action_restart is an action,
so its caller does the same.

My compliments to Fritz and please ask him to accept this.

-- Pete

--- linux-2.6.1-mm3/drivers/s390/net/ctcmain.c 2004-01-14 16:17:02.000000000 -0500
+++ linux-2.6.1-mm3-s390/drivers/s390/net/ctcmain.c 2004-01-16 11:12:29.000000000 -0500
@@ -2075,10 +2075,9 @@
{
struct net_device *dev = (struct net_device *)arg;
struct ctc_priv *privptr = dev->priv;
-
+
printk(KERN_DEBUG "%s: Restarting\n", dev->name);
dev_action_stop(fi, event, arg);
- fsm_event(privptr->fsm, DEV_EVENT_STOP, dev);
fsm_addtimer(&privptr->restart_timer, CTC_TIMEOUT_5SEC,
DEV_EVENT_START, dev);
}

-
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/