-
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
Implement a Generic Network commissioning Thread Driver #14713
Implement a Generic Network commissioning Thread Driver #14713
Conversation
PR #14713: Size comparison from 22f5d22 to be8c1d7 Increases (2 builds for esp32)
Full report (2 builds for esp32)
|
be8c1d7
to
854a46a
Compare
854a46a
to
5394554
Compare
PR #14713: Size comparison from 11182e6 to 5394554 Increases above 0.2%:
Increases (13 builds for cyw30739, efr32, k32w, linux, qpg, telink)
Decreases (1 build for qpg)
Full report (24 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
|
src/platform/OpenThread/GenericNetworkCommissioningThreadDriver.cpp
Outdated
Show resolved
Hide resolved
…rework to not use vector
PR #14713: Size comparison from 11182e6 to 7eee826 Increases above 0.2%:
Increases (36 builds for cyw30739, efr32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (4 builds for esp32, linux, qpg)
Full report (43 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
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.
The important parts to fix are the comparison of random data and the pointer-to-unwound-stack problem.
src/platform/OpenThread/GenericNetworkCommissioningThreadDriver.cpp
Outdated
Show resolved
Hide resolved
src/platform/OpenThread/GenericNetworkCommissioningThreadDriver.cpp
Outdated
Show resolved
Hide resolved
src/platform/OpenThread/GenericThreadStackManagerImpl_OpenThread.cpp
Outdated
Show resolved
Hide resolved
…ect-chip#14713)" This reverts commit a92d1a8.
…#14713) * Create a generic Network commissioning Thread Driver * Add OnThreadAttachFinished to send Connect network response * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Fix linux build do not build GenericNetworkCommissioningThreadDriver for linux * Fix a few sample app builds * fix shell app for all * cleaner fix for shell * Remove OtScanResponseIterator, Scan handling not yet implement. Need rework to not use vector * address PR comments Co-authored-by: Restyled.io <[email protected]>
* Implement a Generic Network commissioning Thread Driver (#14713) * Create a generic Network commissioning Thread Driver * Add OnThreadAttachFinished to send Connect network response * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Fix linux build do not build GenericNetworkCommissioningThreadDriver for linux * Fix a few sample app builds * fix shell app for all * cleaner fix for shell * Remove OtScanResponseIterator, Scan handling not yet implement. Need rework to not use vector * address PR comments Co-authored-by: Restyled.io <[email protected]> * support Thread Scan Network (#15105) fix malloc and release issue Cleanup + restyle Co-authored-by: Restyled.io <[email protected]>
Problem
Commissioning a Thread device is currently failing
Addresses #13170 #14595
Change overview
This adds a generic Network commissioning Thread Driver that can be used by all Thread end devices platforms
Instantiates the Thread Driver and the Network commissioning cluster in the thread Init
TODO Handle/execute ScanNetwork command. I will do it on a subsequent PR
Testing
Commissioned EFR32 light-app device successfully
I could not validate with any other platform than EFR32. CI can confirm builds