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 ap status in l2 wifi #79159

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

GaofengZhangNXP
Copy link
Contributor

@GaofengZhangNXP GaofengZhangNXP commented Sep 29, 2024

add ap status in l2 wifi

@GaofengZhangNXP GaofengZhangNXP changed the title hostapd: support ap status hostapd: add ap status in l2 wifi Sep 29, 2024
@GaofengZhangNXP GaofengZhangNXP force-pushed the gaofeng_main_local branch 5 times, most recently from cb14733 to 190ad3d Compare September 29, 2024 07:21
Comment on lines 1416 to 1419
struct hostapd_iface *iface = NULL;
struct hostapd_config *conf = NULL;
struct hostapd_data *hapd = NULL;
struct hostapd_bss_config *bss = NULL;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to set these to null because you are setting values for the variables below.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

modified

Comment on lines 1447 to 1448
int proto;
int key_mgmt;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please introduce variables at the start of the function

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

modified

int key_mgmt;

status->state = iface->state;
struct hostapd_ssid *ssid = &bss->ssid;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same for this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

modified

modules/hostap/src/supp_api.c Show resolved Hide resolved
modules/hostap/src/supp_api.c Show resolved Hide resolved
modules/hostap/src/supp_api.c Show resolved Hide resolved
modules/hostap/src/supp_api.c Show resolved Hide resolved
modules/hostap/src/supp_api.c Outdated Show resolved Hide resolved
subsys/net/l2/wifi/wifi_shell.c Outdated Show resolved Hide resolved
subsys/net/l2/wifi/wifi_shell.c Outdated Show resolved Hide resolved
@GaofengZhangNXP GaofengZhangNXP force-pushed the gaofeng_main_local branch 3 times, most recently from 4ee9b2f to bf7f9e0 Compare October 8, 2024 06:23
add ap status in l2 wifi

Signed-off-by: Gaofeng Zhang <[email protected]>
@nashif nashif merged commit 0c54a3f into zephyrproject-rtos:main Oct 15, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants