Use mode specific APIs #40
Merged
Conversation
This file contains 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
sachinthegreen
force-pushed
the
nrf70-system-radiotest
branch
4 times, most recently
from
2476275 to
06886c0
Compare
sachinthegreen
force-pushed
the
nrf70-system-radiotest
branch
3 times, most recently
from
a7e059c to
4eef3d1
Compare
ioannisg
reviewed
Feb 6, 2025
| source "Kconfig.zephyr" | ||
| endmenu | ||
|
|
||
| module = NR70_SCAN_RT_BM_SAMPLE |
There was a problem hiding this comment.
Any reason why the nomenclature does not include COMBO or COMBINED and just lists the two modes, _SCAN_RT?
There was a problem hiding this comment.
No specific reason :). Felt that having both scan and RT in the name implied combined.
|
@sachinthegreen @krish2718 , not urgent to do on this PR, but I think we need to lightly enhance the architecture documentation to state the modes of operation and the new added capability to run-time switch between the two modes. We can do it after this pr is merged. |
sachinthegreen
force-pushed
the
nrf70-system-radiotest
branch
2 times, most recently
from
f63215e to
62585c9
Compare
Update the nrf70 Wi-Fi driver to use mode specific APIs (as a precursor to enable combining modes as required).
Separate out mode specific code into mode specific files for ease of readability and maintenance. This also acts as a precursor to be able to combine modes into a single build.
Changes to enable combining system and radio test code in a single build. Signed-off-by: Sachin D Kulkarni <[email protected]>
Add sample to demonstrate runtime switching between scan and radio test modes of operation for nRF70. Signed-off-by: Sachin D Kulkarni <[email protected]>
sachinthegreen
force-pushed
the
nrf70-system-radiotest
branch
from
62585c9 to
75e4f4a
Compare
Documentation changes for scan and radio test combo mode operation. Signed-off-by: Sachin D Kulkarni <[email protected]>
sachinthegreen
force-pushed
the
nrf70-system-radiotest
branch
from
75e4f4a to
a2c12f5
Compare
krish2718
approved these changes
Feb 7, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Update the nrf70 Wi-Fi driver to use mode specific APIs (as a precursor to enable combining modes as required).