[Buildroot] [PATCH 1/2] hostapd: remove support for Realtek drivers

[Buildroot] [PATCH 1/2] hostapd: remove support for Realtek drivers

[Alexander Mukhin]

After hostapd update to 2.9, the patch provided no longer works,
although applies. Moreover, AP support for Realtek chips is broken
anyway in kernels > 4.9.

Signed-off-by: Alexander Mukhin <alexander.i.mukhin@gmail.com>
---
 package/hostapd/Config.in    | 7 -------
 package/hostapd/hostapd.hash | 1 -
 package/hostapd/hostapd.mk   | 5 -----
 3 files changed, 13 deletions(-)

diff --git a/package/hostapd/Config.in b/package/hostapd/Config.in
index 37c1126859..8a7abbb901 100644
--- a/package/hostapd/Config.in
+++ b/package/hostapd/Config.in
@@ -31,12 +31,6 @@ config BR2_PACKAGE_HOSTAPD_DRIVER_NL80211
 comment "nl80211 driver needs a toolchain w/ threads"
 	depends on !BR2_TOOLCHAIN_HAS_THREADS
 
-config BR2_PACKAGE_HOSTAPD_DRIVER_RTW
-	bool "Enable rtl871xdrv driver"
-	select BR2_PACKAGE_HOSTAPD_HAS_WIFI_DRIVERS
-	help
-	  Enable support for Realtek wireless chips.
-
 config BR2_PACKAGE_HOSTAPD_DRIVER_WIRED
 	bool "Enable wired driver"
 	help
@@ -47,7 +41,6 @@ config BR2_PACKAGE_HOSTAPD_DRIVER_NONE
 	default y
 	depends on !BR2_PACKAGE_HOSTAPD_DRIVER_HOSTAP
 	depends on !BR2_PACKAGE_HOSTAPD_DRIVER_NL80211
-	depends on !BR2_PACKAGE_HOSTAPD_DRIVER_RTW
 	depends on !BR2_PACKAGE_HOSTAPD_DRIVER_WIRED
 
 config BR2_PACKAGE_HOSTAPD_HAS_WIFI_DRIVERS
diff --git a/package/hostapd/hostapd.hash b/package/hostapd/hostapd.hash
index 50225cf6d7..bf5016acc3 100644
--- a/package/hostapd/hostapd.hash
+++ b/package/hostapd/hostapd.hash
@@ -1,4 +1,3 @@
 # Locally calculated
 sha256  881d7d6a90b2428479288d64233151448f8990ab4958e0ecaca7eeb3c9db2bd7  hostapd-2.9.tar.gz
-sha256  e204da659d0583c71af23cb9b55536fe99598ee26a44104344f456e4d17350c6  rtlxdrv.patch
 sha256  9da5dd0776da266b180b915e460ff75c6ff729aca1196ab396529510f24f3761  README
diff --git a/package/hostapd/hostapd.mk b/package/hostapd/hostapd.mk
index f55e716a7c..fb6ac8fb2e 100644
--- a/package/hostapd/hostapd.mk
+++ b/package/hostapd/hostapd.mk
@@ -39,11 +39,6 @@ ifeq ($(BR2_PACKAGE_HOSTAPD_DRIVER_NL80211),)
 HOSTAPD_CONFIG_DISABLE += CONFIG_DRIVER_NL80211
 endif
 
-ifeq ($(BR2_PACKAGE_HOSTAPD_DRIVER_RTW),y)
-HOSTAPD_PATCH += https://github.com/pritambaral/hostapd-rtl871xdrv/raw/master/rtlxdrv.patch
-HOSTAPD_CONFIG_SET += CONFIG_DRIVER_RTW
-endif
-
 ifeq ($(BR2_PACKAGE_HOSTAPD_DRIVER_WIRED),y)
 HOSTAPD_CONFIG_ENABLE += CONFIG_DRIVER_WIRED
 endif
-- 
2.20.1

[Buildroot] [PATCH 2/2] Remove myself from hostapd developers

[Alexander Mukhin]

Signed-off-by: Alexander Mukhin <alexander.i.mukhin@gmail.com>
---
 DEVELOPERS | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/DEVELOPERS b/DEVELOPERS
index 4b4a74ab3f..0905237790 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -106,9 +106,6 @@ F:	package/putty/
 N:	Alexander Lukichev <alexander.lukichev@espotel.com>
 F:	package/openpgm/
 
-N:	Alexander Mukhin <alexander.i.mukhin@gmail.com>
-F:	package/hostapd/
-
 N:	Alexander Sverdlin <alexander.sverdlin@gmail.com>
 F:	package/mini-snmpd/
 
-- 
2.20.1

[Buildroot] [PATCH 1/2] hostapd: remove support for Realtek drivers

[Thomas Petazzoni]

Hello Alexander,

On Fri,  6 Sep 2019 13:21:00 +0300
Alexander Mukhin <alexander.i.mukhin@gmail.com> wrote:

> After hostapd update to 2.9, the patch provided no longer works,
> although applies. Moreover, AP support for Realtek chips is broken
> anyway in kernels > 4.9.
> 
> Signed-off-by: Alexander Mukhin <alexander.i.mukhin@gmail.com>

The commit title prefix should have been "package/hostapd", and also
when removing an option, it must be added in Config.in.legacy. I fixed
both issues and applied to master.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH 2/2] Remove myself from hostapd developers

[Thomas Petazzoni]

On Fri,  6 Sep 2019 13:21:01 +0300
Alexander Mukhin <alexander.i.mukhin@gmail.com> wrote:

> Signed-off-by: Alexander Mukhin <alexander.i.mukhin@gmail.com>
> ---
>  DEVELOPERS | 3 ---
>  1 file changed, 3 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com