nrf_wifi: Restructuring #81101
Merged
nrf_wifi: Restructuring #81101
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 1 added project Note: This message is automatically posted and updated by the Manifest GitHub Action. |
zephyrbot
added
manifest
manifest-nrf_wifi
DNM
krish2718
force-pushed
the
nrf_wifi_restructure
branch
7 times, most recently
from
9aa9a0e to
0d430f4
Compare
zephyrbot
requested review from
Kludentwo,
MaureenHelm,
jukkar,
nashif,
rlubos and
stephanosio
krish2718
force-pushed
the
nrf_wifi_restructure
branch
from
0d430f4 to
02f2473
Compare
sachinthegreen
previously approved these changes
Nov 12, 2024
carlescufi
reviewed
Nov 14, 2024
krish2718
force-pushed
the
nrf_wifi_restructure
branch
from
02f2473 to
e146f42
Compare
sachinthegreen
previously approved these changes
Nov 15, 2024
jukkar
previously approved these changes
Nov 17, 2024
krish2718
dismissed stale reviews from jukkar and sachinthegreen
via
2683639
krish2718
force-pushed
the
nrf_wifi_restructure
branch
from
e146f42 to
2683639
Compare
jukkar
previously approved these changes
Nov 22, 2024
krish2718
force-pushed
the
nrf_wifi_restructure
branch
from
2683639 to
fd359bc
Compare
Following the naming convention add an underscore and update path in the maintainers file too. Signed-off-by: Chaitanya Tata <[email protected]>
The OSAL code now lives in it's own repo nrf_wifi. Signed-off-by: Chaitanya Tata <[email protected]>
This is a Zephyr shim for the nrf-wifi module. Signed-off-by: Chaitanya Tata <[email protected]>
This is now taken care by the Zephyr module shim. Signed-off-by: Chaitanya Tata <[email protected]>
The shim files should be part of the Zephyr module directory within Zephyr, as they are coupled with the module implementation which os OS agnostic. Signed-off-by: Chaitanya Tata <[email protected]>
The blobs are now part of nrf_wifi module, and remove the manual check and add verifying the blobs and their integrity using helper function. Signed-off-by: Chaitanya Tata <[email protected]>
Pulls PR that removed nRF Wi-Fi. Signed-off-by: Chaitanya Tata <[email protected]>
krish2718
force-pushed
the
nrf_wifi_restructure
branch
from
fd359bc to
bb2dfa1
Compare
rlubos
approved these changes
Nov 25, 2024
sachinthegreen
approved these changes
Nov 25, 2024
jukkar
approved these changes
Nov 25, 2024
Member
|
Removed DNM, since the added project has been approved by the TSC. |
carlescufi
approved these changes
Nov 25, 2024
Contributor
|
What is the we file added in the root (bb2dfa1)? It looks like a mistake to me. |
Contributor
Author
Oops, it is a mistake, will fix it, thanks. |
Contributor
Author
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
nrf_wifiout ofhal_nordicto it's own repomodules/nrf_wifiand move all the OSAL handling from Zephyr driver