Skip to content

Conversation

@kkasperczyk-no
Copy link
Owner

  • Currently CHIP does not fully support saving data
    in the persistent storage, so saving only part of it (Thread
    configuration) might be confusing. Added erasing Thread
    configuration on init then.
  • Added setting Thread device type to MED.

Signed-off-by: Kamil Kasperczyk [email protected]

* Currently CHIP does not fully support saving data
in the persistent storage, so saving only part of it (Thread
configuration) might be confusing. Added erasing Thread
configuration on init then.
* Added setting Thread device type to MED.

Signed-off-by: Kamil Kasperczyk <[email protected]>
/* Until all CHIP data needed to perform full communication are not saved in the persistent storage,
* Thread configuration should not be saved either.
* TODO: Remove it after having full support for persistent storage in CHIP */
if(otDatasetIsCommissioned(openthread_get_default_instance())) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if(otDatasetIsCommissioned(openthread_get_default_instance())) {
if (otDatasetIsCommissioned(openthread_get_default_instance())) {

@github-actions
Copy link

github-actions bot commented Apr 6, 2021

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Apr 6, 2021
kkasperczyk-no pushed a commit that referenced this pull request Aug 8, 2023
* Include HOSTAP_BASE to fix header file paths
* Select WEP automatically through Kconfig

Signed-off-by: Chaitanya Tata <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants