Skip to content
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 supported rates and extended supported rates in ubus #726

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@PolynomialDivision
Copy link
Contributor

PolynomialDivision commented Feb 6, 2018

This pull request add the supported rates and extended supported rates in the mgmt frames.
This pull request contains: #648
Signed-off-by: Nick Hainke vincent@systemli.org

PolynomialDivision added some commits Jan 14, 2018

hostapd: add 802.11 elems in hostapd_ubus_request
Signed-off-by: Nick Hainke vincent@systemli.org
hostapd: add supported rates and extended in ubus
Signed-off-by: Nick Hainke vincent@systemli.org

@PolynomialDivision PolynomialDivision force-pushed the PolynomialDivision:feature/add_rates branch from 9995da0 to 7f9cace Feb 6, 2018

@pepe2k pepe2k added the packages label Feb 6, 2018

if (req->elems->supp_rates) {
supp_rates = blobmsg_open_array(&b, "supp_rates");
for (i = 0; req->elems->supp_rates && i < req->elems->supp_rates_len; i++) {
blobmsg_add_u8(&b, NULL, req->elems->supp_rates[i]);

This comment has been minimized.

@PolynomialDivision

PolynomialDivision Feb 7, 2018

Author Contributor

@nbd168: So here I should use e.g. u16?

This comment has been minimized.

@nbd168

nbd168 Feb 10, 2018

Member

Yes. They take up the same amount of memory anyway (due to alignment)

@PolynomialDivision

This comment has been minimized.

Copy link
Contributor Author

PolynomialDivision commented Feb 13, 2018

@blogic
I don't know if actually this patch is wanted. So I decided to let this pull request open #648.

@blogic

This comment has been minimized.

Copy link
Contributor

blogic commented Feb 13, 2018

its part of #724, please dont post duplicates. #724 has already received feedback.

@blogic blogic closed this Feb 13, 2018

@blogic blogic reopened this Feb 13, 2018

hostapd: chage supp and extended supp rates to u16
Signed-off-by: Nick Hainke vincent@systemli.org
@PolynomialDivision

This comment has been minimized.

Copy link
Contributor Author

PolynomialDivision commented Feb 14, 2018

@blogic Why is this part of #724 ? I think u mixed something?

@PolynomialDivision

This comment has been minimized.

Copy link
Contributor Author

PolynomialDivision commented Apr 24, 2018

Is this pull request still discussed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.