hostapd: make ACS dependent on nl80211

According to hostapd's defconfig file, ACS is currently only supported
through the nl80211 driver. Search through the source code also confirms
that ACS machinery is present in the nl80211 driver only.

Although hostapd can be built with ACS enabled and nl80211 disabled, an
attempt to use ACS with other drivers by setting a wireless channel to 0
results in a runtime failure (driver doesn't accept this value). So we
might save a user from selecting a meaningless combination by making ACS
dependent on nl80211.

Signed-off-by: Alexander Mukhin <alexander.i.mukhin@gmail.com>
Reviewed-by: Sam Voss <sam.voss@rockwellcollins.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
This commit is contained in:
Alexander Mukhin 2017-12-30 18:57:35 +03:00 committed by Thomas Petazzoni
parent 90d73e5f66
commit 87a941462d

View File

@ -53,6 +53,7 @@ config BR2_PACKAGE_HOSTAPD_HAS_WIFI_DRIVERS
config BR2_PACKAGE_HOSTAPD_ACS config BR2_PACKAGE_HOSTAPD_ACS
bool "Enable ACS" bool "Enable ACS"
default y default y
depends on BR2_PACKAGE_HOSTAPD_DRIVER_NL80211
help help
Enable support for standard ACS (Automatic Channel Selection). Enable support for standard ACS (Automatic Channel Selection).
Some propietary drivers use a custom algorithm which requires Some propietary drivers use a custom algorithm which requires
@ -60,6 +61,9 @@ config BR2_PACKAGE_HOSTAPD_ACS
causing hostapd to use the standard one which doesn't work causing hostapd to use the standard one which doesn't work
for those cases. for those cases.
comment "ACS is currently only supported through the nl80211 driver"
depends on !BR2_PACKAGE_HOSTAPD_DRIVER_NL80211
config BR2_PACKAGE_HOSTAPD_EAP config BR2_PACKAGE_HOSTAPD_EAP
bool "Enable EAP" bool "Enable EAP"
depends on !BR2_STATIC_LIBS depends on !BR2_STATIC_LIBS