Linux wireless drivers development
 help / color / mirror / Atom feed
* [PATCH] libertas: remove one superfluous include
@ 2007-10-09 12:30 Holger Schurig
  2007-10-09 12:38 ` Holger Schurig
  0 siblings, 1 reply; 4+ messages in thread
From: Holger Schurig @ 2007-10-09 12:30 UTC (permalink / raw)
  To: linux-wireless, libertas-dev, Dan Williams

This makes scripts/checkincludes.pl happy.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>

Index: libertas-2.6/drivers/net/wireless/libertas/if_usb.c
===================================================================
--- libertas-2.6.orig/drivers/net/wireless/libertas/if_usb.c	2007-10-09 15:29:38.000000000 +0200
+++ libertas-2.6/drivers/net/wireless/libertas/if_usb.c	2007-10-09 15:29:59.000000000 +0200
@@ -15,7 +15,6 @@
 #include "defs.h"
 #include "dev.h"
 #include "if_usb.h"
-#include "decl.h"
 
 #define MESSAGE_HEADER_LEN	4
 

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

* Re: [PATCH] libertas: remove one superfluous include
  2007-10-09 12:30 [PATCH] libertas: remove one superfluous include Holger Schurig
@ 2007-10-09 12:38 ` Holger Schurig
  2007-10-09 20:10   ` Johannes Berg
  0 siblings, 1 reply; 4+ messages in thread
From: Holger Schurig @ 2007-10-09 12:38 UTC (permalink / raw)
  To: linux-wireless

> This makes scripts/checkincludes.pl happy.

Hehe, others have problems, too:

$ scripts/checkincludes.pl drivers/net/wireless/*.h
drivers/net/wireless/ipw2200.h: linux/dma-mapping.h is included 
more than once.



(in wireless-2.6 everything)

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

* Re: [PATCH] libertas: remove one superfluous include
  2007-10-09 12:38 ` Holger Schurig
@ 2007-10-09 20:10   ` Johannes Berg
  2007-10-11 10:17     ` [PATCH] janitorial: fix all double includes in drivers/net/wireless Holger Schurig
  0 siblings, 1 reply; 4+ messages in thread
From: Johannes Berg @ 2007-10-09 20:10 UTC (permalink / raw)
  To: Holger Schurig; +Cc: linux-wireless

[-- Attachment #1: Type: text/plain, Size: 305 bytes --]

On Tue, 2007-10-09 at 14:38 +0200, Holger Schurig wrote:

> $ scripts/checkincludes.pl drivers/net/wireless/*.h
> drivers/net/wireless/ipw2200.h: linux/dma-mapping.h is included 
> more than once.

Feel free to post a patch. It doesn't look like anybody is maintaining
those any more.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

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

* [PATCH] janitorial: fix all double includes in drivers/net/wireless
  2007-10-09 20:10   ` Johannes Berg
@ 2007-10-11 10:17     ` Holger Schurig
  0 siblings, 0 replies; 4+ messages in thread
From: Holger Schurig @ 2007-10-11 10:17 UTC (permalink / raw)
  To: linux-wireless

This patch removes all double includes of the same file. This
makes scripts/checkincludes.pl happy.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>

---

Note: as I don't have most of the hardware, I wasn't able to 
check the functionality of the patch. However, all drivers 
survived a compile check.

And, after all, it's a quite trivial patch ...


Index: wireless-2.6/drivers/net/wireless/b43/main.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/b43/main.c	2007-10-11 
13:08:37.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/b43/main.c	2007-10-11 
13:08:42.000000000 +0200
@@ -49,7 +49,6 @@
 #include "pio.h"
 #include "sysfs.h"
 #include "xmit.h"
-#include "sysfs.h"
 #include "lo.h"
 #include "pcmcia.h"
 
Index: wireless-2.6/drivers/net/wireless/ipw2200.h
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/ipw2200.h	2007-10-11 
13:05:20.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/ipw2200.h	2007-10-11 
13:05:24.000000000 +0200
@@ -45,7 +45,6 @@
 
 #include <linux/firmware.h>
 #include <linux/wireless.h>
-#include <linux/dma-mapping.h>
 #include <linux/jiffies.h>
 #include <asm/io.h>
 
Index: wireless-2.6/drivers/net/wireless/iwlwifi/iwl-3945-rs.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/iwlwifi/iwl-3945-rs.c	
2007-10-11 13:07:13.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/iwlwifi/iwl-3945-rs.c	
2007-10-11 13:07:43.000000000 +0200
@@ -37,9 +37,6 @@
 
 #include <linux/workqueue.h>
 
-#include <net/mac80211.h>
-#include <linux/wireless.h>
-
 #define IWL 3945
 
 #include "../net/mac80211/ieee80211_rate.h"
Index: wireless-2.6/drivers/net/wireless/iwlwifi/iwl-3945.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/iwlwifi/iwl-3945.c	
2007-10-11 13:05:57.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/iwlwifi/iwl-3945.c	
2007-10-11 13:06:01.000000000 +0200
@@ -38,7 +38,6 @@
 #include <net/mac80211.h>
 
 #include <linux/etherdevice.h>
-#include <linux/delay.h>
 
 #define IWL 3945
 
Index: wireless-2.6/drivers/net/wireless/iwlwifi/iwl-4965-rs.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/iwlwifi/iwl-4965-rs.c	
2007-10-11 13:08:19.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/iwlwifi/iwl-4965-rs.c	
2007-10-11 13:08:29.000000000 +0200
@@ -36,9 +36,6 @@
 
 #include <linux/workqueue.h>
 
-#include <net/mac80211.h>
-#include <linux/wireless.h>
-
 #define IWL 4965
 
 #include "../net/mac80211/ieee80211_rate.h"
Index: wireless-2.6/drivers/net/wireless/iwlwifi/iwl-4965.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/iwlwifi/iwl-4965.c	
2007-10-11 13:08:03.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/iwlwifi/iwl-4965.c	
2007-10-11 13:08:09.000000000 +0200
@@ -35,9 +35,7 @@
 #include <linux/netdevice.h>
 #include <linux/wireless.h>
 #include <net/mac80211.h>
-#include <linux/netdevice.h>
 #include <linux/etherdevice.h>
-#include <linux/delay.h>
 
 #define IWL 4965
 
Index: wireless-2.6/drivers/net/wireless/iwlwifi/iwl3945-base.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/iwlwifi/iwl3945-base.c	
2007-10-11 13:07:49.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/iwlwifi/iwl3945-base.c	
2007-10-11 13:07:56.000000000 +0200
@@ -48,8 +48,6 @@
 #include <linux/netdevice.h>
 #include <linux/wireless.h>
 #include <linux/firmware.h>
-#include <linux/skbuff.h>
-#include <linux/netdevice.h>
 #include <linux/etherdevice.h>
 #include <linux/if_arp.h>
 
Index: wireless-2.6/drivers/net/wireless/iwlwifi/iwl4965-base.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/iwlwifi/iwl4965-base.c	
2007-10-11 13:06:08.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/iwlwifi/iwl4965-base.c	
2007-10-11 13:06:37.000000000 +0200
@@ -48,8 +48,6 @@
 #include <linux/netdevice.h>
 #include <linux/wireless.h>
 #include <linux/firmware.h>
-#include <linux/skbuff.h>
-#include <linux/netdevice.h>
 #include <linux/etherdevice.h>
 #include <linux/if_arp.h>
 
Index: wireless-2.6/drivers/net/wireless/iwlwifi/iwlwifi.h
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/iwlwifi/iwlwifi.h	
2007-10-11 13:04:25.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/iwlwifi/iwlwifi.h	
2007-10-11 13:09:53.000000000 +0200
@@ -39,18 +39,13 @@ struct iwl_priv;
 /* Hardware specific file defines the PCI IDs table for that 
hardware module */
 extern struct pci_device_id iwl_hw_card_ids[];
 
+#include "iwl-hw.h"
 #if IWL == 3945
-
 #define DRV_NAME	"iwl3945"
-#include "iwl-hw.h"
 #include "iwl-3945-hw.h"
-
 #elif IWL == 4965
-
 #define DRV_NAME        "iwl4965"
-#include "iwl-hw.h"
 #include "iwl-4965-hw.h"
-
 #endif
 
 #include "iwl-prph.h"
Index: 
wireless-2.6/drivers/net/wireless/zd1211rw-mac80211/zd_def.h
===================================================================
--- 
wireless-2.6.orig/drivers/net/wireless/zd1211rw-mac80211/zd_def.h	
2007-10-11 13:04:56.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/zd1211rw-mac80211/zd_def.h	
2007-10-11 13:05:06.000000000 +0200
@@ -21,7 +21,6 @@
 #include <linux/kernel.h>
 #include <linux/stringify.h>
 #include <linux/device.h>
-#include <linux/kernel.h>
 
 typedef u16 __nocast zd_addr_t;
 
Index: wireless-2.6/drivers/net/wireless/zd1211rw/zd_def.h
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/zd1211rw/zd_def.h	
2007-10-11 13:05:37.000000000 +0200
+++ wireless-2.6/drivers/net/wireless/zd1211rw/zd_def.h	
2007-10-11 13:05:40.000000000 +0200
@@ -21,7 +21,6 @@
 #include <linux/kernel.h>
 #include <linux/stringify.h>
 #include <linux/device.h>
-#include <linux/kernel.h>
 
 typedef u16 __nocast zd_addr_t;
 


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

end of thread, other threads:[~2007-10-11 10:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-10-09 12:30 [PATCH] libertas: remove one superfluous include Holger Schurig
2007-10-09 12:38 ` Holger Schurig
2007-10-09 20:10   ` Johannes Berg
2007-10-11 10:17     ` [PATCH] janitorial: fix all double includes in drivers/net/wireless Holger Schurig

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox