If baycom driver has never been opened, it will attempt to free
an IRQ that it never registered when removed. The problem is that hdlcdrv
does not keep track of open/close state.
diff -Nru a/drivers/net/hamradio/hdlcdrv.c b/drivers/net/hamradio/hdlcdrv.c
--- a/drivers/net/hamradio/hdlcdrv.c Fri Sep 19 13:16:29 2003
+++ b/drivers/net/hamradio/hdlcdrv.c Fri Sep 19 13:16:29 2003
@@ -554,6 +554,7 @@
/*
* initialise some variables
*/
+ s->opened = 1;
s->hdlcrx.hbuf.rd = s->hdlcrx.hbuf.wr = 0;
s->hdlcrx.in_hdlc_rx = 0;
s->hdlcrx.rx_state = 0;
@@ -593,6 +594,7 @@
if (s->skb)
dev_kfree_skb(s->skb);
s->skb = NULL;
+ s->opened = 0;
return i;
}
@@ -845,7 +847,7 @@
BUG_ON(s->magic != HDLCDRV_MAGIC);
- if (s->ops->close)
+ if (s->opened && s->ops->close)
s->ops->close(dev);
unregister_netdev(dev);
diff -Nru a/include/linux/hdlcdrv.h b/include/linux/hdlcdrv.h
--- a/include/linux/hdlcdrv.h Fri Sep 19 13:16:29 2003
+++ b/include/linux/hdlcdrv.h Fri Sep 19 13:16:29 2003
@@ -180,6 +180,7 @@
struct hdlcdrv_state {
int magic;
+ int opened;
const struct hdlcdrv_ops *ops;