-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Silabs] Refactor RCP WiFi initialization to remove platform-specific dependencies and unify initialization function #37135
[Silabs] Refactor RCP WiFi initialization to remove platform-specific dependencies and unify initialization function #37135
Conversation
PR #37135: Size comparison from 8370039 to e489f7b Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #37135: Size comparison from 8370039 to 43cd521 Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These changes allows us to unify the wifi init in MatterConfig. The changes should be done in this PR.
43cd521
to
fdc6069
Compare
PR #37135: Size comparison from 22cd392 to fdc6069 Full report (68 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
… and unify initialization function
PR #37135: Size comparison from 22cd392 to 1fe4ed6 Full report (68 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
…g across implementations
…ry allocation failure
PR #37135: Size comparison from 22cd392 to 1246983 Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the error codes, CHIP_ERROR_UNINITIALIZED
means A requested object is uninitialized.
It doesn't apply when we fail to init an object.
Co-authored-by: Mathieu Kardous <[email protected]>
PR #37135: Size comparison from 22cd392 to 12ce37e Full report (14 builds for cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, tizen)
|
…odes and update documentation
PR #37135: Size comparison from 22cd392 to 7d1d0d7 Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Co-authored-by: Mathieu Kardous <[email protected]>
PR #37135: Size comparison from 214e03e to 9010d38 Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Changelog
Refactor WiFi initialization to remove platform-specific dependencies and unify initialization function
Testing
Manually build the RCP combinations for RS9116, RCP 917, RCP WF200 with EFR32MG24.
Tested commissioning flow with RCP 917, RCP WF200 with EFR32MG24.