-
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
Import generic implementation of connection manager #52
Conversation
#endif // CHIP_DEVICE_CONFIG_ENABLE_THREAD | ||
|
||
namespace chip { | ||
namespace DeviceLayer { | ||
|
||
struct chipDeviceEvent; | ||
|
||
/* |
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.
Let's delete rather than comment?
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.
+1
class ConnectivityManagerImpl final | ||
: public ConnectivityManager, | ||
public Internal::GenericConnectivityManagerImpl<ConnectivityManagerImpl>, | ||
#if CHIP_DEVICE_CONFIG_ENABLE_WOBLE |
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.
WOBLE means Weave Over BLE. Can we name this DOBLE? (Data over BLE?)... or something else - given we're removing input technology specific references. Thanks!
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.
I'd recommend CHIPOBLE for the time being for consistency's sake.
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.
I'm fine with that... 🤣
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.
Further rationale: xoBLE connections are initiated based on the specific contents of a BLE advertisement. Thread over BLE connections are in response to a Thread-specific BLE advertisement. WoBLE in response to a Weave-specific BLE advertisement. Likewise, CHIP connections would be in response to a CHIP-specific BLE advertisement.
* Provides control over WoBLE services and connectivity for a chip device. | ||
* | ||
* BLEManager defines the abstract interface of a singleton object that provides | ||
* control over WoBLE services and connectivity for a chip device. BLEManager |
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.
WoBLE
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.
I'd recommend CHIPOBLE for the time being for consistency's sake.
|
||
// ===== Methods that implement the ConnectivityManager abstract interface. | ||
|
||
ConnectivityManager::WoBLEServiceMode _GetWoBLEServiceMode(void); |
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.
WoBLE
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.
I'd recommend CHIPOBLE for the time being for consistency's sake.
{ | ||
switch (mode) | ||
{ | ||
case ConnectivityManager::kWoBLEServiceMode_NotSupported: |
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.
WoBLE
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.
I'd recommend CHIPOBLE for the time being for consistency's sake.
}; | ||
|
||
template<class ImplClass> | ||
inline ConnectivityManager::WoBLEServiceMode GenericConnectivityManagerImpl_NoBLE<ImplClass>::_GetWoBLEServiceMode(void) |
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.
WoBLE
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.
I'd recommend CHIPOBLE for the time being for consistency's sake.
#include <platform/CHIPDeviceError.h> | ||
#include <platform/GeneralUtils.h> | ||
// #include <core/CHIPGlobals.h> | ||
#include <CHIPDeviceConfig.h> |
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.
I'd recommend maintaining a multi-level include approach or we'll need to come up with a naming scheme that avoids header name collisions.
@woody-apple , @gerickson the latest commit should address the comments |
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.
👍
103d894
to
4cf23e9
Compare
Rebased and resolved merge conflicts |
Add BLEManagerImpl for mbed-os
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
Merge in WMN_TOOLS/matter from credentials_example to silabs Squashed commit of the following: commit c90ed96cefc627f8e85584b587d06cf9f0e3210c Author: Rehan Rasool <[email protected]> Date: Fri Sep 2 14:07:04 2022 -0400 Fix numbers sequence commit de19989a9006d55fb102154dfc51ba6ac8ea24e5 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:53:22 2022 -0400 Fix formatting commit 574f7a95c4d2c2c03a4a944f86b79a69763aa396 Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:51:09 2022 -0400 Address review comments commit b91a03a3da9a870736f96be9cc8af11ddb58eeef Author: Sergei Lissianoi <[email protected]> Date: Fri Sep 2 13:15:49 2022 -0400 Fix typo and update README commit 8cfe380f59ce4c0dff769ae4e7ff614bcf64113a Author: Ezra Hale <[email protected]> Date: Wed Aug 31 10:09:56 2022 -0400 added rest of information from confluence page to flush out credentials example README.md commit df1d5c8129d3999a83ba64edf4341757a9ef09dc Author: Ricardo Casallas <[email protected]> Date: Fri Aug 26 13:42:32 2022 -0400 Added example of credentials' injection.
No description provided.