linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] net: wireless: rtlwifi: Fix issues with Makefiles
@ 2014-11-24 18:34 Andreas Ruprecht
  2014-11-24 18:34 ` [PATCH 1/2] net: wireless: rtlwifi: Do not always include drivers in obj-m Andreas Ruprecht
  2014-11-24 18:34 ` [PATCH 2/2] net: wireless: rtlwifi: rtl8192ee: Fix compilation of the driver Andreas Ruprecht
  0 siblings, 2 replies; 3+ messages in thread
From: Andreas Ruprecht @ 2014-11-24 18:34 UTC (permalink / raw)
  To: Larry Finger
  Cc: Chaoming Li, John W. Linville, linux-wireless, netdev,
	linux-kernel, Andreas Ruprecht

This patch series fixes two issues in the Makefiles for different
rtlwifi drivers.

The first one lead to object files for the drivers always being
included in obj-m even if the corresponding Kconfig option is set to
'y'.

The second one is likely to be a copy-and-paste mistake, which
prevents compilation of the rtl8192ee driver unless the
completely unrelated Kconfig option for the rtl8821ae driver
(CONFIG_RTL8821AE) is also enabled.

Andreas Ruprecht (2):
  net: wireless: rtlwifi: Do not always include drivers in obj-m
  net: wireless: rtlwifi: rtl8192ee: Fix compilation of the driver

 drivers/net/wireless/rtlwifi/rtl8192ee/Makefile | 5 +----
 drivers/net/wireless/rtlwifi/rtl8723ae/Makefile | 3 ---
 drivers/net/wireless/rtlwifi/rtl8723be/Makefile | 3 ---
 drivers/net/wireless/rtlwifi/rtl8821ae/Makefile | 3 ---
 4 files changed, 1 insertion(+), 13 deletions(-)

--
1.9.1


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

* [PATCH 1/2] net: wireless: rtlwifi: Do not always include drivers in obj-m
  2014-11-24 18:34 [PATCH 0/2] net: wireless: rtlwifi: Fix issues with Makefiles Andreas Ruprecht
@ 2014-11-24 18:34 ` Andreas Ruprecht
  2014-11-24 18:34 ` [PATCH 2/2] net: wireless: rtlwifi: rtl8192ee: Fix compilation of the driver Andreas Ruprecht
  1 sibling, 0 replies; 3+ messages in thread
From: Andreas Ruprecht @ 2014-11-24 18:34 UTC (permalink / raw)
  To: Larry Finger
  Cc: Chaoming Li, John W. Linville, linux-wireless, netdev,
	linux-kernel, Andreas Ruprecht

In four of the rtlwifi drivers, the Makefile contains superfluous
statements indicating the compilation of the driver as an LKM
regardless of the corresponding Kconfig option.

If the corresponding option is set to 'y', the build system will then
see the object file in obj-m and obj-y, which leads to a compilation
as a built-in only. Even though this leads to the desired behavior,
the unconditional appearance in obj-m is confusing for someone reading
the Makefile.

This patch removes the superfluous Makefile statements.

Signed-off-by: Andreas Ruprecht <rupran@einserver.de>
---
 drivers/net/wireless/rtlwifi/rtl8192ee/Makefile | 3 ---
 drivers/net/wireless/rtlwifi/rtl8723ae/Makefile | 3 ---
 drivers/net/wireless/rtlwifi/rtl8723be/Makefile | 3 ---
 drivers/net/wireless/rtlwifi/rtl8821ae/Makefile | 3 ---
 4 files changed, 12 deletions(-)

diff --git a/drivers/net/wireless/rtlwifi/rtl8192ee/Makefile b/drivers/net/wireless/rtlwifi/rtl8192ee/Makefile
index 11952b9..6bd46a9 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192ee/Makefile
+++ b/drivers/net/wireless/rtlwifi/rtl8192ee/Makefile
@@ -1,6 +1,3 @@
-obj-m := rtl8192ee.o
-
-
 rtl8192ee-objs :=		\
 		dm.o		\
 		fw.o		\
diff --git a/drivers/net/wireless/rtlwifi/rtl8723ae/Makefile b/drivers/net/wireless/rtlwifi/rtl8723ae/Makefile
index 9c34a85..6220672 100644
--- a/drivers/net/wireless/rtlwifi/rtl8723ae/Makefile
+++ b/drivers/net/wireless/rtlwifi/rtl8723ae/Makefile
@@ -1,6 +1,3 @@
-obj-m := rtl8723ae.o
-
-
 rtl8723ae-objs :=		\
 		dm.o		\
 		fw.o		\
diff --git a/drivers/net/wireless/rtlwifi/rtl8723be/Makefile b/drivers/net/wireless/rtlwifi/rtl8723be/Makefile
index 59e416a..a77c341 100644
--- a/drivers/net/wireless/rtlwifi/rtl8723be/Makefile
+++ b/drivers/net/wireless/rtlwifi/rtl8723be/Makefile
@@ -1,6 +1,3 @@
-obj-m := rtl8723be.o
-
-
 rtl8723be-objs :=		\
 		dm.o		\
 		fw.o		\
diff --git a/drivers/net/wireless/rtlwifi/rtl8821ae/Makefile b/drivers/net/wireless/rtlwifi/rtl8821ae/Makefile
index 87ad604..f7a26f7 100644
--- a/drivers/net/wireless/rtlwifi/rtl8821ae/Makefile
+++ b/drivers/net/wireless/rtlwifi/rtl8821ae/Makefile
@@ -1,6 +1,3 @@
-obj-m := rtl8821ae.o
-
-
 rtl8821ae-objs :=		\
 		dm.o		\
 		fw.o		\
-- 
1.9.1


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

* [PATCH 2/2] net: wireless: rtlwifi: rtl8192ee: Fix compilation of the driver
  2014-11-24 18:34 [PATCH 0/2] net: wireless: rtlwifi: Fix issues with Makefiles Andreas Ruprecht
  2014-11-24 18:34 ` [PATCH 1/2] net: wireless: rtlwifi: Do not always include drivers in obj-m Andreas Ruprecht
@ 2014-11-24 18:34 ` Andreas Ruprecht
  1 sibling, 0 replies; 3+ messages in thread
From: Andreas Ruprecht @ 2014-11-24 18:34 UTC (permalink / raw)
  To: Larry Finger
  Cc: Chaoming Li, John W. Linville, linux-wireless, netdev,
	linux-kernel, Andreas Ruprecht

In the Makefile for this driver, the wrong Kconfig option is used
to trigger the compilation of the object file. This leads to the
driver only being included into the kernel when both CONFIG_RTL8821AE
and CONFIG_RTL8192AE are set to "y".

Signed-off-by: Andreas Ruprecht <rupran@einserver.de>
---
 drivers/net/wireless/rtlwifi/rtl8192ee/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/rtlwifi/rtl8192ee/Makefile b/drivers/net/wireless/rtlwifi/rtl8192ee/Makefile
index 6bd46a9..0315eed 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192ee/Makefile
+++ b/drivers/net/wireless/rtlwifi/rtl8192ee/Makefile
@@ -11,6 +11,6 @@ rtl8192ee-objs :=		\
 		trx.o		\
 
 
-obj-$(CONFIG_RTL8821AE) += rtl8192ee.o
+obj-$(CONFIG_RTL8192EE) += rtl8192ee.o
 
 ccflags-y += -D__CHECK_ENDIAN__
-- 
1.9.1


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

end of thread, other threads:[~2014-11-24 18:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-24 18:34 [PATCH 0/2] net: wireless: rtlwifi: Fix issues with Makefiles Andreas Ruprecht
2014-11-24 18:34 ` [PATCH 1/2] net: wireless: rtlwifi: Do not always include drivers in obj-m Andreas Ruprecht
2014-11-24 18:34 ` [PATCH 2/2] net: wireless: rtlwifi: rtl8192ee: Fix compilation of the driver Andreas Ruprecht

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).