All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] b44 enable interrupts after tx timeout (2.6-test version)
@ 2003-10-03 16:33 Pekka Pietikainen
  2003-10-03 17:35 ` [PATCH] b44 enable interrupts after tx timeout (2.4.23-pre6) Pekka Pietikainen
  0 siblings, 1 reply; 3+ messages in thread
From: Pekka Pietikainen @ 2003-10-03 16:33 UTC (permalink / raw)
  To: jgarzik; +Cc: linux-kernel

Resending the patch I sent some time ago for b44.c that nukes the
2.4 compatibility cruft as well. 

I'll do one for 2.4.23pre6 ASAP, hopefully being able sync the driver fully
with the one in 2.6 (free_netdev() etc.).

--- linux-2.6.0-0.test6.1.47/drivers/net/b44.c.orig	2003-09-28 03:50:18.000000000 +0300
+++ linux-2.6.0-0.test6.1.47/drivers/net/b44.c	2003-10-03 18:55:29.000000000 +0300
@@ -25,8 +25,8 @@
 
 #define DRV_MODULE_NAME		"b44"
 #define PFX DRV_MODULE_NAME	": "
-#define DRV_MODULE_VERSION	"0.9"
-#define DRV_MODULE_RELDATE	"Jul 14, 2003"
+#define DRV_MODULE_VERSION	"0.91"
+#define DRV_MODULE_RELDATE	"Oct 3, 2003"
 
 #define B44_DEF_MSG_ENABLE	  \
 	(NETIF_MSG_DRV		| \
@@ -80,15 +80,6 @@
 
 static int b44_debug = -1;	/* -1 == use B44_DEF_MSG_ENABLE as value */
 
-#ifndef PCI_DEVICE_ID_BCM4401
-#define PCI_DEVICE_ID_BCM4401      0x4401
-#endif
-
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0)
-#define IRQ_RETVAL(x) 
-#define irqreturn_t void
-#endif
-
 static struct pci_device_id b44_pci_tbl[] = {
 	{ PCI_VENDOR_ID_BROADCOM, PCI_DEVICE_ID_BCM4401,
 	  PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0UL },
@@ -870,6 +861,8 @@
 
 	spin_unlock_irq(&bp->lock);
 
+	b44_enable_ints(bp);
+
 	netif_wake_queue(dev);
 }
 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] b44 enable interrupts after tx timeout (2.4.23-pre6)
  2003-10-03 16:33 [PATCH] b44 enable interrupts after tx timeout (2.6-test version) Pekka Pietikainen
@ 2003-10-03 17:35 ` Pekka Pietikainen
  2003-10-14 18:40   ` Jeff Garzik
  0 siblings, 1 reply; 3+ messages in thread
From: Pekka Pietikainen @ 2003-10-03 17:35 UTC (permalink / raw)
  To: jgarzik; +Cc: linux-kernel

Here's the 2.4.23-pre6 version, which syncs the driver fully with 2.6.
Seems to work (tm).

--- linux-2.4.23-pre6/drivers/net/b44.c.orig	2003-08-25 14:44:42.000000000 +0300
+++ linux-2.4.23-pre6/drivers/net/b44.c	2003-10-03 19:39:49.000000000 +0300
@@ -25,8 +25,8 @@
 
 #define DRV_MODULE_NAME		"b44"
 #define PFX DRV_MODULE_NAME	": "
-#define DRV_MODULE_VERSION	"0.9"
-#define DRV_MODULE_RELDATE	"Jul 14, 2003"
+#define DRV_MODULE_VERSION	"0.91"
+#define DRV_MODULE_RELDATE	"Oct 3, 2003"
 
 #define B44_DEF_MSG_ENABLE	  \
 	(NETIF_MSG_DRV		| \
@@ -80,16 +80,7 @@
 
 static int b44_debug = -1;	/* -1 == use B44_DEF_MSG_ENABLE as value */
 
-#ifndef PCI_DEVICE_ID_BCM4401
-#define PCI_DEVICE_ID_BCM4401      0x4401
-#endif
-
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0)
-#define IRQ_RETVAL(x) 
-#define irqreturn_t void
-#endif
-
-static struct pci_device_id b44_pci_tbl[] __devinitdata = {
+static struct pci_device_id b44_pci_tbl[] = {
 	{ PCI_VENDOR_ID_BROADCOM, PCI_DEVICE_ID_BCM4401,
 	  PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0UL },
 	{ }	/* terminate list with empty entry */
@@ -870,6 +861,8 @@
 
 	spin_unlock_irq(&bp->lock);
 
+	b44_enable_ints(bp);
+
 	netif_wake_queue(dev);
 }
 
@@ -1393,7 +1386,7 @@
 		strcpy (info.driver, DRV_MODULE_NAME);
 		strcpy (info.version, DRV_MODULE_VERSION);
 		memset(&info.fw_version, 0, sizeof(info.fw_version));
-		strcpy (info.bus_info, pci_dev->slot_name);
+		strcpy (info.bus_info, pci_name(pci_dev));
 		info.eedump_len = 0;
 		info.regdump_len = 0;
 		if (copy_to_user (useraddr, &info, sizeof (info)))
@@ -1834,7 +1827,7 @@
 	iounmap((void *) bp->regs);
 
 err_out_free_dev:
-	kfree(dev);
+	free_netdev(dev);
 
 err_out_free_res:
 	pci_release_regions(pdev);
@@ -1852,7 +1845,7 @@
 	if (dev) {
 		unregister_netdev(dev);
 		iounmap((void *) ((struct b44 *)(dev->priv))->regs);
-		kfree(dev);
+		free_netdev(dev);
 		pci_release_regions(pdev);
 		pci_disable_device(pdev);
 		pci_set_drvdata(pdev, NULL);
--- linux-2.4.23-pre6/include/linux/pci_ids.h.orig	2003-10-03 20:30:14.000000000 +0300
+++ linux-2.4.23-pre6/include/linux/pci_ids.h	2003-10-03 19:41:01.000000000 +0300
@@ -1669,6 +1669,7 @@
 #define PCI_DEVICE_ID_TIGON3_5703A3	0x16c7
 #define PCI_DEVICE_ID_TIGON3_5901	0x170d
 #define PCI_DEVICE_ID_TIGON3_5901_2	0x170e
+#define PCI_DEVICE_ID_BCM4401		0x4401
 
 #define PCI_VENDOR_ID_SYBA		0x1592
 #define PCI_DEVICE_ID_SYBA_2P_EPP	0x0782

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] b44 enable interrupts after tx timeout (2.4.23-pre6)
  2003-10-03 17:35 ` [PATCH] b44 enable interrupts after tx timeout (2.4.23-pre6) Pekka Pietikainen
@ 2003-10-14 18:40   ` Jeff Garzik
  0 siblings, 0 replies; 3+ messages in thread
From: Jeff Garzik @ 2003-10-14 18:40 UTC (permalink / raw)
  To: Pekka Pietikainen; +Cc: linux-kernel

thanks, 2.4 and 2.6 patches to b44 applied



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2003-10-14 18:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-10-03 16:33 [PATCH] b44 enable interrupts after tx timeout (2.6-test version) Pekka Pietikainen
2003-10-03 17:35 ` [PATCH] b44 enable interrupts after tx timeout (2.4.23-pre6) Pekka Pietikainen
2003-10-14 18:40   ` Jeff Garzik

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.