* [PATCH 1/3] staging: wean rtl8712 off of its ancient duplicate of if_ether.h
2012-05-10 3:47 [PATCH 0/3] staging: delete duplicated files/functionality in rtl8712 Paul Gortmaker
@ 2012-05-10 3:47 ` Paul Gortmaker
2012-05-10 3:47 ` [PATCH 2/3] staging: wean rtl8712 off of its ancient duplicate of ip.h Paul Gortmaker
` (2 subsequent siblings)
3 siblings, 0 replies; 7+ messages in thread
From: Paul Gortmaker @ 2012-05-10 3:47 UTC (permalink / raw)
To: linux-wireless
Cc: florian.c.schilhabel, ali, netdev, Paul Gortmaker, Larry Finger,
Greg Kroah-Hartman
This driver should not be carrying around ancient copies of
headers like <linux/if_ether.h> for its own use. Mapping it
onto the mainline one uncovers no build issues.
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
---
drivers/staging/rtl8712/if_ether.h | 141 --------------------------------
drivers/staging/rtl8712/rtl8712_recv.c | 3 +-
drivers/staging/rtl8712/rtl871x_recv.c | 2 +-
drivers/staging/rtl8712/xmit_linux.c | 2 +-
4 files changed, 4 insertions(+), 144 deletions(-)
delete mode 100644 drivers/staging/rtl8712/if_ether.h
diff --git a/drivers/staging/rtl8712/if_ether.h b/drivers/staging/rtl8712/if_ether.h
deleted file mode 100644
index 2bbe527..0000000
diff --git a/drivers/staging/rtl8712/rtl8712_recv.c b/drivers/staging/rtl8712/rtl8712_recv.c
index fa6dc9c..b2a7714 100644
--- a/drivers/staging/rtl8712/rtl8712_recv.c
+++ b/drivers/staging/rtl8712/rtl8712_recv.c
@@ -28,12 +28,13 @@
#define _RTL8712_RECV_C_
+#include <linux/if_ether.h>
+
#include "osdep_service.h"
#include "drv_types.h"
#include "recv_osdep.h"
#include "mlme_osdep.h"
#include "ip.h"
-#include "if_ether.h"
#include "ethernet.h"
#include "usb_ops.h"
#include "wifi.h"
diff --git a/drivers/staging/rtl8712/rtl871x_recv.c b/drivers/staging/rtl8712/rtl871x_recv.c
index 5b03b40..7376abb 100644
--- a/drivers/staging/rtl8712/rtl871x_recv.c
+++ b/drivers/staging/rtl8712/rtl871x_recv.c
@@ -29,6 +29,7 @@
#define _RTL871X_RECV_C_
#include <linux/slab.h>
+#include <linux/if_ether.h>
#include <linux/kmemleak.h>
#include "osdep_service.h"
@@ -36,7 +37,6 @@
#include "recv_osdep.h"
#include "mlme_osdep.h"
#include "ip.h"
-#include "if_ether.h"
#include "ethernet.h"
#include "usb_ops.h"
#include "wifi.h"
diff --git a/drivers/staging/rtl8712/xmit_linux.c b/drivers/staging/rtl8712/xmit_linux.c
index c970362..d27f652 100644
--- a/drivers/staging/rtl8712/xmit_linux.c
+++ b/drivers/staging/rtl8712/xmit_linux.c
@@ -29,12 +29,12 @@
#define _XMIT_OSDEP_C_
#include <linux/usb.h>
+#include <linux/if_ether.h>
#include "osdep_service.h"
#include "drv_types.h"
-#include "if_ether.h"
#include "ip.h"
#include "rtl871x_byteorder.h"
#include "wifi.h"
--
1.7.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH 2/3] staging: wean rtl8712 off of its ancient duplicate of ip.h
2012-05-10 3:47 [PATCH 0/3] staging: delete duplicated files/functionality in rtl8712 Paul Gortmaker
2012-05-10 3:47 ` [PATCH 1/3] staging: wean rtl8712 off of its ancient duplicate of if_ether.h Paul Gortmaker
@ 2012-05-10 3:47 ` Paul Gortmaker
2012-05-10 3:47 ` [PATCH 3/3] staging: delete all duplicated endian crap from rtl8712 driver Paul Gortmaker
2012-05-10 16:54 ` [PATCH 0/3] staging: delete duplicated files/functionality in rtl8712 Greg Kroah-Hartman
3 siblings, 0 replies; 7+ messages in thread
From: Paul Gortmaker @ 2012-05-10 3:47 UTC (permalink / raw)
To: linux-wireless
Cc: florian.c.schilhabel, ali, netdev, Paul Gortmaker, Larry Finger,
Greg Kroah-Hartman
This driver should not be carrying around ancient copies of
headers like <linux/ip.h> for its own use. Mapping it onto
the mainline one uncovers no build issues.
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
---
drivers/staging/rtl8712/ip.h | 137 --------------------------------
drivers/staging/rtl8712/rtl8712_recv.c | 2 +-
drivers/staging/rtl8712/rtl871x_recv.c | 2 +-
drivers/staging/rtl8712/xmit_linux.c | 3 +-
4 files changed, 3 insertions(+), 141 deletions(-)
delete mode 100644 drivers/staging/rtl8712/ip.h
diff --git a/drivers/staging/rtl8712/ip.h b/drivers/staging/rtl8712/ip.h
deleted file mode 100644
index f37b0f8..0000000
diff --git a/drivers/staging/rtl8712/rtl8712_recv.c b/drivers/staging/rtl8712/rtl8712_recv.c
index b2a7714..e975d0a 100644
--- a/drivers/staging/rtl8712/rtl8712_recv.c
+++ b/drivers/staging/rtl8712/rtl8712_recv.c
@@ -29,12 +29,12 @@
#define _RTL8712_RECV_C_
#include <linux/if_ether.h>
+#include <linux/ip.h>
#include "osdep_service.h"
#include "drv_types.h"
#include "recv_osdep.h"
#include "mlme_osdep.h"
-#include "ip.h"
#include "ethernet.h"
#include "usb_ops.h"
#include "wifi.h"
diff --git a/drivers/staging/rtl8712/rtl871x_recv.c b/drivers/staging/rtl8712/rtl871x_recv.c
index 7376abb..c9d1743 100644
--- a/drivers/staging/rtl8712/rtl871x_recv.c
+++ b/drivers/staging/rtl8712/rtl871x_recv.c
@@ -28,6 +28,7 @@
#define _RTL871X_RECV_C_
+#include <linux/ip.h>
#include <linux/slab.h>
#include <linux/if_ether.h>
#include <linux/kmemleak.h>
@@ -36,7 +37,6 @@
#include "drv_types.h"
#include "recv_osdep.h"
#include "mlme_osdep.h"
-#include "ip.h"
#include "ethernet.h"
#include "usb_ops.h"
#include "wifi.h"
diff --git a/drivers/staging/rtl8712/xmit_linux.c b/drivers/staging/rtl8712/xmit_linux.c
index d27f652..c6943c5 100644
--- a/drivers/staging/rtl8712/xmit_linux.c
+++ b/drivers/staging/rtl8712/xmit_linux.c
@@ -29,13 +29,12 @@
#define _XMIT_OSDEP_C_
#include <linux/usb.h>
+#include <linux/ip.h>
#include <linux/if_ether.h>
#include "osdep_service.h"
#include "drv_types.h"
-
-#include "ip.h"
#include "rtl871x_byteorder.h"
#include "wifi.h"
#include "mlme_osdep.h"
--
1.7.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/3] staging: delete all duplicated endian crap from rtl8712 driver
2012-05-10 3:47 [PATCH 0/3] staging: delete duplicated files/functionality in rtl8712 Paul Gortmaker
2012-05-10 3:47 ` [PATCH 1/3] staging: wean rtl8712 off of its ancient duplicate of if_ether.h Paul Gortmaker
2012-05-10 3:47 ` [PATCH 2/3] staging: wean rtl8712 off of its ancient duplicate of ip.h Paul Gortmaker
@ 2012-05-10 3:47 ` Paul Gortmaker
2012-05-10 16:54 ` [PATCH 0/3] staging: delete duplicated files/functionality in rtl8712 Greg Kroah-Hartman
3 siblings, 0 replies; 7+ messages in thread
From: Paul Gortmaker @ 2012-05-10 3:47 UTC (permalink / raw)
To: linux-wireless
Cc: florian.c.schilhabel, ali, netdev, Paul Gortmaker, Larry Finger,
Greg Kroah-Hartman
This driver had headers like big_endian.h, little_endian.h, swab.h
and yet we can throw them all in the trash can and the thing
still builds on x86-64 and ppc, just by deleting the references
to the deleted files.
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
---
drivers/staging/rtl8712/big_endian.h | 94 --------------
drivers/staging/rtl8712/generic.h | 178 ---------------------------
drivers/staging/rtl8712/hal_init.c | 1 -
drivers/staging/rtl8712/little_endian.h | 94 --------------
drivers/staging/rtl8712/osdep_service.h | 2 -
drivers/staging/rtl8712/rtl8712_cmd.c | 1 -
drivers/staging/rtl8712/rtl8712_xmit.c | 1 -
drivers/staging/rtl8712/rtl871x_byteorder.h | 32 -----
drivers/staging/rtl8712/rtl871x_cmd.c | 1 -
drivers/staging/rtl8712/rtl871x_xmit.c | 1 -
drivers/staging/rtl8712/swab.h | 131 --------------------
drivers/staging/rtl8712/usb_ops.c | 1 -
drivers/staging/rtl8712/wifi.h | 1 -
drivers/staging/rtl8712/xmit_linux.c | 1 -
14 files changed, 0 insertions(+), 539 deletions(-)
delete mode 100644 drivers/staging/rtl8712/big_endian.h
delete mode 100644 drivers/staging/rtl8712/generic.h
delete mode 100644 drivers/staging/rtl8712/little_endian.h
delete mode 100644 drivers/staging/rtl8712/rtl871x_byteorder.h
delete mode 100644 drivers/staging/rtl8712/swab.h
diff --git a/drivers/staging/rtl8712/big_endian.h b/drivers/staging/rtl8712/big_endian.h
deleted file mode 100644
index b16f8ec..0000000
diff --git a/drivers/staging/rtl8712/generic.h b/drivers/staging/rtl8712/generic.h
deleted file mode 100644
index 8868c9f..0000000
diff --git a/drivers/staging/rtl8712/hal_init.c b/drivers/staging/rtl8712/hal_init.c
index cc893c0..cb9d4cf 100644
--- a/drivers/staging/rtl8712/hal_init.c
+++ b/drivers/staging/rtl8712/hal_init.c
@@ -36,7 +36,6 @@
#include "osdep_service.h"
#include "drv_types.h"
-#include "rtl871x_byteorder.h"
#include "usb_osintf.h"
#define FWBUFF_ALIGN_SZ 512
diff --git a/drivers/staging/rtl8712/little_endian.h b/drivers/staging/rtl8712/little_endian.h
deleted file mode 100644
index cd57d6c..0000000
diff --git a/drivers/staging/rtl8712/osdep_service.h b/drivers/staging/rtl8712/osdep_service.h
index cabf774..f1ccc7e 100644
--- a/drivers/staging/rtl8712/osdep_service.h
+++ b/drivers/staging/rtl8712/osdep_service.h
@@ -106,8 +106,6 @@ static inline void _set_workitem(_workitem *pwork)
schedule_work(pwork);
}
-#include "rtl871x_byteorder.h"
-
#ifndef BIT
#define BIT(x) (1 << (x))
#endif
diff --git a/drivers/staging/rtl8712/rtl8712_cmd.c b/drivers/staging/rtl8712/rtl8712_cmd.c
index 9f6ebc4..b518505 100644
--- a/drivers/staging/rtl8712/rtl8712_cmd.c
+++ b/drivers/staging/rtl8712/rtl8712_cmd.c
@@ -50,7 +50,6 @@
#include "drv_types.h"
#include "recv_osdep.h"
#include "mlme_osdep.h"
-#include "rtl871x_byteorder.h"
#include "rtl871x_ioctl_set.h"
static void check_hw_pbc(struct _adapter *padapter)
diff --git a/drivers/staging/rtl8712/rtl8712_xmit.c b/drivers/staging/rtl8712/rtl8712_xmit.c
index 6933319..3d23514 100644
--- a/drivers/staging/rtl8712/rtl8712_xmit.c
+++ b/drivers/staging/rtl8712/rtl8712_xmit.c
@@ -30,7 +30,6 @@
#include "osdep_service.h"
#include "drv_types.h"
-#include "rtl871x_byteorder.h"
#include "wifi.h"
#include "osdep_intf.h"
#include "usb_ops.h"
diff --git a/drivers/staging/rtl8712/rtl871x_byteorder.h b/drivers/staging/rtl8712/rtl871x_byteorder.h
deleted file mode 100644
index bd3703b..0000000
diff --git a/drivers/staging/rtl8712/rtl871x_cmd.c b/drivers/staging/rtl8712/rtl871x_cmd.c
index d77388b..659683e 100644
--- a/drivers/staging/rtl8712/rtl871x_cmd.c
+++ b/drivers/staging/rtl8712/rtl871x_cmd.c
@@ -50,7 +50,6 @@
#include "drv_types.h"
#include "recv_osdep.h"
#include "mlme_osdep.h"
-#include "rtl871x_byteorder.h"
/*
Caller and the r8712_cmd_thread can protect cmd_q by spin_lock.
diff --git a/drivers/staging/rtl8712/rtl871x_xmit.c b/drivers/staging/rtl8712/rtl871x_xmit.c
index aa57e77..1d3f8b7 100644
--- a/drivers/staging/rtl8712/rtl871x_xmit.c
+++ b/drivers/staging/rtl8712/rtl871x_xmit.c
@@ -30,7 +30,6 @@
#include "osdep_service.h"
#include "drv_types.h"
-#include "rtl871x_byteorder.h"
#include "wifi.h"
#include "osdep_intf.h"
#include "usb_ops.h"
diff --git a/drivers/staging/rtl8712/swab.h b/drivers/staging/rtl8712/swab.h
deleted file mode 100644
index f127818..0000000
diff --git a/drivers/staging/rtl8712/usb_ops.c b/drivers/staging/rtl8712/usb_ops.c
index 5a8b0eb..c03508d 100644
--- a/drivers/staging/rtl8712/usb_ops.c
+++ b/drivers/staging/rtl8712/usb_ops.c
@@ -33,7 +33,6 @@
#include "osdep_intf.h"
#include "usb_ops.h"
#include "recv_osdep.h"
-#include "rtl871x_byteorder.h"
static u8 usb_read8(struct intf_hdl *pintfhdl, u32 addr)
{
diff --git a/drivers/staging/rtl8712/wifi.h b/drivers/staging/rtl8712/wifi.h
index 277398c..793443e 100644
--- a/drivers/staging/rtl8712/wifi.h
+++ b/drivers/staging/rtl8712/wifi.h
@@ -26,7 +26,6 @@
#ifndef _WIFI_H_
#define _WIFI_H_
-#include "rtl871x_byteorder.h"
#include <linux/compiler.h>
#ifdef BIT
diff --git a/drivers/staging/rtl8712/xmit_linux.c b/drivers/staging/rtl8712/xmit_linux.c
index c6943c5..65542cb 100644
--- a/drivers/staging/rtl8712/xmit_linux.c
+++ b/drivers/staging/rtl8712/xmit_linux.c
@@ -35,7 +35,6 @@
#include "osdep_service.h"
#include "drv_types.h"
-#include "rtl871x_byteorder.h"
#include "wifi.h"
#include "mlme_osdep.h"
#include "xmit_osdep.h"
--
1.7.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH 0/3] staging: delete duplicated files/functionality in rtl8712
2012-05-10 3:47 [PATCH 0/3] staging: delete duplicated files/functionality in rtl8712 Paul Gortmaker
` (2 preceding siblings ...)
2012-05-10 3:47 ` [PATCH 3/3] staging: delete all duplicated endian crap from rtl8712 driver Paul Gortmaker
@ 2012-05-10 16:54 ` Greg Kroah-Hartman
[not found] ` <20120510165451.GA12958-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
3 siblings, 1 reply; 7+ messages in thread
From: Greg Kroah-Hartman @ 2012-05-10 16:54 UTC (permalink / raw)
To: Paul Gortmaker
Cc: linux-wireless, florian.c.schilhabel, ali, netdev, Larry Finger
On Wed, May 09, 2012 at 11:47:18PM -0400, Paul Gortmaker wrote:
> A git grep happened to lead me into this dir, and once there I couldn't
> simply leave and pretend I didn't see the stuff that I saw.
>
> There were duplicated basic networking headers like if_ether.h and
> ip.h (of course, some ancient versions, too). And a whole whack of
> boilerplate endian handling functionality duplicated too.
>
> I forced myself to stop looking after that.
Yes, it's a mess, thanks for working on cleaning it up, even a little
bit, it all helps.
> Anyway, a trivial redirection onto mainline's networking headers
> in the proper include dir, and a deletion of any references to
> the endian headers and the thing still happily builds on x86_64
> and ppc even after shitcanning seven useless files.
>
> These three commits are against May 8th's linux-next tree. I've
> used "-D" to hide the line-by-line content of the deletions.
Can you regenerate these with the full deletion content? There have
been some spelling fixes in these files, and your patches do not apply
due to them. I can hand-edit the patches if you regenerate it.
Or if you rebase on the last linux-next tree, that will pick those
changes up and then I can apply these properly.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread