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.

Sign up for GitHub

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

Jump to bottom

Register as a Native Wi-Fi driver #11534

Merged
merged 5 commits into from
Jul 6, 2023
Merged

Register as a Native Wi-Fi driver #11534

merged 5 commits into from
Jul 6, 2023

Conversation

krish2718
Copy link
Contributor

No description provided.

@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest labels Jun 14, 2023
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jun 14, 2023 •
edited
Loading

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hostap nrfconnect/sdk-hostap@2040db8 nrfconnect/sdk-hostap@8e2e758 (main) nrfconnect/sdk-hostap@2040db89..8e2e758b
zephyr nrfconnect/sdk-zephyr@743f999 nrfconnect/sdk-zephyr@52222dc nrfconnect/sdk-zephyr@743f9994..52222dc7

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@krish2718 krish2718 marked this pull request as ready for review June 14, 2023 21:42
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jun 14, 2023 •
edited
Loading

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
desktop52_verification X
test-fw-nrfconnect-ble X
test-fw-nrfconnect-ble_samples X
test-fw-nrfconnect-chip X
test-fw-nrfconnect-fem X
test-fw-nrfconnect-nfc X
test-fw-nrfconnect-nrf-iot_cloud X
test-fw-nrfconnect-nrf-iot_lwm2m X
test-fw-nrfconnect-nrf-iot_mosh X
test-fw-nrfconnect-nrf-iot_positioning X
test-fw-nrfconnect-nrf-iot_thingy91 X
test-fw-nrfconnect-nrf-iot_zephyr_lwm2m X
test-fw-nrfconnect-nrf_crypto X
test-fw-nrfconnect-rpc X
test-fw-nrfconnect-rs X
test-fw-nrfconnect-tfm X
test-fw-nrfconnect-thread X
test-sdk-find-my X
test-sdk-wifi X

test-sdk-wifi: added because there was no .github/test-spec.yml in 'hostap'

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@krish2718 krish2718 force-pushed the native_wifi branch 4 times, most recently from d37b797 to be45868 Compare June 22, 2023 09:14
@krish2718 krish2718 force-pushed the native_wifi branch 3 times, most recently from 3760494 to d42453c Compare July 4, 2023 08:18
sachinthegreen
sachinthegreen reviewed Jul 4, 2023
View reviewed changes
Copy link
Contributor

@sachinthegreen sachinthegreen left a comment

Choose a reason for hiding this comment

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

Do we need a Changelog entry for this?

@krish2718
Copy link
Contributor Author

Do we need a Changelog entry for this?

I don't think so, as this is transparent to applications.

@NordicBuilder NordicBuilder removed the DNM label Jul 5, 2023
@rlubos
Copy link
Contributor

rlubos commented Jul 6, 2023

This needs a rebase now (there was an earlier sdk-zephyr PR that had been merged).

@krish2718
manifest: sdk-zephyr: Pull in Native Wi-Fi support
abb12a3 
Zephyr Wi-Fi now has two flavours: native and offloaded.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
@krish2718
manifest: sdk-hostap: Pull in Native Wi-Fi support
b7a64a3 
WPA supplicant now registers as a Native Wi-Fi stack to Zephyr.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
@krish2718
drivers: wifi: Register as a Wi-Fi device
532bca6 
Tell the networking stack that this is a Native Wi-Fi device.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
@krish2718
drivers: wifi: Move to new Wi-Fi data structure
f5f46e5 
The Wi-Fi ops now are separated in to their own structure.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
@krish2718
drivers: wifi: Fix the build failure
2975a79 
The prototype has changed in Zephyr, this commit only fixes the build
failure, doesn't implement the feature, it will be done in a separate
commit.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
@rlubos rlubos merged commit ba58ca9 into nrfconnect:main Jul 6, 2023
30 checks passed
@krish2718 krish2718 deleted the native_wifi branch July 6, 2023 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sachinthegreen sachinthegreen sachinthegreen approved these changes

@rlubos rlubos rlubos approved these changes

@rado17 rado17 rado17 approved these changes

@VivekUppunda VivekUppunda VivekUppunda approved these changes

@ajayparida ajayparida Awaiting requested review from ajayparida

@carlescufi carlescufi Awaiting requested review from carlescufi

@tejlmand tejlmand Awaiting requested review from tejlmand

Assignees
No one assigned
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-hostap manifest-zephyr
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@krish2718 @NordicBuilder @rlubos @rado17 @sachinthegreen @VivekUppunda