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

[CSA-CP] Pull In Wi-Fi refactor #101

Merged
merged 6 commits into from
Nov 14, 2024

Conversation

mkardous-silabs
Copy link
Contributor

@mkardous-silabs mkardous-silabs commented Nov 13, 2024

Description

PR Cherry-picks the Wi-Fi refactor PRs that were merged into CSA.
This is the list of PRs that were cherry-picked :

PR also pulls in a 917 Platform fix that was done between refactor PRs that simplifies the cherry-pick process.
The Fix should have been pull in anywais.

Tests

  • CI
  • Manual tests

mkardous-silabs and others added 6 commits November 13, 2024 16:44
…(#36477)

* Restructure platform files

* Move all files to src/platform

* Build the wf200 in the new wifi structure

* rename directory

* Build the rs9116 in the new wifi structure

* Build the SiWx917 NCP in the new wifi structure

* Build the SiWx917 SoC in the new wifi structure

* Delete wfx_host_events header in favor of the new abstraction structure

* remove unnecessary includes in gn files and the cflag changes

* restyle

* add todo comment

* remove cflag inclusion

* add missing incllude

* Add missing includes

* Restyled by clang-format

* Move wifi_args.gni file to the wifi directory

* Rename Wf200WifiInterface to WifiInterface

* Rename SiWxWifiInterface to WifiInterface

* Rename Rsi91xWifiInterface.cpp to WifiInterface.cpp

---------

Co-authored-by: Restyled.io <[email protected]>
* Rename Wifi interface sources files

* restyle
…nager (#35808)

* Decouple UART power requirement handler

* Adds refactored BTN handler based on power manager

* Fixed incorrect API

* Adds changes for interrupt

* Replace use of RSI_xx APIs with sl_si91x_xx APIs

* Refactor button event handling and UART power requirement handler

* Adds comments based on the coding standards
Reduce into inline functions

* Renaming sl_btn0 -> btn0

* Adds changes to optimize code

* Adds CPP define in header

* Update function comments

* Replace with more readable condition

* Updates document to add BTN0 screen cycle action (#323)

* Restyled by prettier-markdown

* Adds compilation fix for EFR32 boards

* Addressing comments

---------

Co-authored-by: Restyled.io <[email protected]>
…rds (#36366)

* Move defines to files where they are used

* Refactor enums and use BitFlag wrapper instead of bit operations

* Refactor wifi eventing to use the Bitflag datatype

* Apply suggestions from code review

Co-authored-by: Junior Martinez <[email protected]>

* Remove unnecassry includes

* Restyled by clang-format

---------

Co-authored-by: Junior Martinez <[email protected]>
Co-authored-by: Restyled.io <[email protected]>
… files (#36460)

* Split source between wifi platforms

* Brake out wiseconnect specific code from the common abstraction

* move power save function to specific interface

* Remove hw state from extern C

* Remove power save call form extern C

* Remove unnecessary include

* Restyle

* Fix ifdef c++ include guards
…(#36477)

* Restructure platform files

* Move all files to src/platform

* Build the wf200 in the new wifi structure

* rename directory

* Build the rs9116 in the new wifi structure

* Build the SiWx917 NCP in the new wifi structure

* Build the SiWx917 SoC in the new wifi structure

* Delete wfx_host_events header in favor of the new abstraction structure

* remove unnecessary includes in gn files and the cflag changes

* restyle

* add todo comment

* remove cflag inclusion

* add missing incllude

* Add missing includes

* Restyled by clang-format

* Move wifi_args.gni file to the wifi directory

* Rename Wf200WifiInterface to WifiInterface

* Rename SiWxWifiInterface to WifiInterface

* Rename Rsi91xWifiInterface.cpp to WifiInterface.cpp

---------

Co-authored-by: Restyled.io <[email protected]>
@mkardous-silabs mkardous-silabs added changing-submodules-on-purpose csa-cp This TAG indicates that this commit is a cherry-pick from a CSA commit. labels Nov 13, 2024
@mkardous-silabs mkardous-silabs requested a review from a team as a code owner November 13, 2024 21:55
@mkardous-silabs mkardous-silabs marked this pull request as draft November 13, 2024 21:56
@mkardous-silabs mkardous-silabs marked this pull request as ready for review November 14, 2024 13:05
@mkardous-silabs mkardous-silabs merged commit 937321f into release_2.5-1.4 Nov 14, 2024
12 of 13 checks passed
@mkardous-silabs mkardous-silabs deleted the cherry-pick/wifi_refactor branch November 14, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changing-submodules-on-purpose csa-cp This TAG indicates that this commit is a cherry-pick from a CSA commit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants