New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hostapd: add ubus call for ap infos #897

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@PolynomialDivision
Contributor

PolynomialDivision commented Apr 25, 2018

The call "get_infos" allows to gather hostapd config options
via ubus. As first infos we add the ht and vht support.
Although nl80211 supports to gather informations about
ht and vht capabilities, the hostapd configuration can disable
ht and vht. However, it is possible that the iw output is not
representing the actual hostapd configuration.

Signed-off-by: Nick Hainke vincent@systemli.org

@dedeckeh dedeckeh added the packages label Apr 26, 2018

@@ -909,6 +924,7 @@ static const struct ubus_method bss_methods[] = {
UBUS_METHOD_NOARG("wps_start", hostapd_bss_wps_start),
UBUS_METHOD_NOARG("wps_cancel", hostapd_bss_wps_cancel),
UBUS_METHOD_NOARG("update_beacon", hostapd_bss_update_beacon),
UBUS_METHOD_NOARG("get_infos", hostapd_bss_get_infos),

This comment has been minimized.

@blogic

blogic May 5, 2018

Contributor

would "features" or "caps" not be a better name to use ?

This comment has been minimized.

@PolynomialDivision

PolynomialDivision May 17, 2018

Contributor

I thought infos is more general and maybe someone else want to extend this call by different information elements.
I don't mind. I can rename it if you want.

hostapd: add ubus call for ap features
The call "get_features" allows to gather hostapd config options
via ubus. As first infos we add the ht and vht support.
Although nl80211 supports to gather informations about
ht and vht capabilities, the hostapd configuration can disable
vht and ht. However, it is possible that the iw output is not
representing the actual hostapd configuration.

Signed-off-by: Nick Hainke <vincent@systemli.org>
@@ -909,6 +924,7 @@ static const struct ubus_method bss_methods[] = {
UBUS_METHOD_NOARG("wps_start", hostapd_bss_wps_start),
UBUS_METHOD_NOARG("wps_cancel", hostapd_bss_wps_cancel),
UBUS_METHOD_NOARG("update_beacon", hostapd_bss_update_beacon),
UBUS_METHOD_NOARG("get_features", hostapd_bss_get_features),

This comment has been minimized.

@PolynomialDivision

PolynomialDivision May 17, 2018

Contributor

@blogic I renamed it to features. :)

@blogic

This comment has been minimized.

Contributor

blogic commented Jul 30, 2018

merged, Thanks !

@blogic blogic closed this Jul 30, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment