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

UserSelectedMode APIs are returning dummy values #109

Closed
pan-apple opened this issue Mar 24, 2020 · 2 comments
Closed

UserSelectedMode APIs are returning dummy values #109

pan-apple opened this issue Mar 24, 2020 · 2 comments
Labels
feature work p1 priority 1 work question Further information is requested
Milestone

Comments

@pan-apple
Copy link
Contributor

Problem

#104 removed support for DeviceDescriptor. Due to this UserSelectedMode APIs in GenericConnectivityManagerImpl.h are returning dummy values.

Proposed Solution

Identify the replacement APIs for DeviceDescriptor, and reimplemented the following functions.
_IsUserSelectedModeActive
_SetUserSelectedMode
_GetUserSelectedModeTimeout
_SetUserSelectedModeTimeout

@rwalker-apple rwalker-apple added this to the M4 milestone Mar 25, 2020
@rwalker-apple
Copy link
Contributor

@pan-apple can you please re-assess this and mark with an appropriate milestone?

@woody-apple woody-apple modified the milestones: Not sure when, V1.0 May 26, 2020
@woody-apple woody-apple added the p1 priority 1 work label Oct 16, 2020
@woody-apple woody-apple added the question Further information is requested label Nov 5, 2020
@pan-apple
Copy link
Contributor Author

We can close this issue, as this code is not relevant for CHIP.

shgutte pushed a commit to shgutte/connectedhomeip that referenced this issue Nov 22, 2024
mergify bot pushed a commit that referenced this issue Nov 28, 2024
…6628)

* [SL-UP] Remove two algo design for the wifi retry mechanism (#103)

* [SL-UP] Rename ot interval to transport interval (#108)

* [SL-UP] Delete unnecessary double abstraction for the wiseconnect power save functions (#109)

* [SL-UP] Update spi-multiplex header inclusion (#110)

* [SL-UP] Fix Builds after Wi-Fi interface refactor (#111)

* [SL-UP] Remove WF200 define duplication to avoid value conflicts at runtime (#126)

* fix spacing

* restyle

* Fix define name

* add missing function

* Fix wf200 ICD build

* Restyled by clang-format

* Add debug config

* Re added sem check

---------

Co-authored-by: Restyled.io <[email protected]>
yyzhong-g pushed a commit to yyzhong-g/connectedhomeip that referenced this issue Dec 12, 2024
…oject-chip#36628)

* [SL-UP] Remove two algo design for the wifi retry mechanism (project-chip#103)

* [SL-UP] Rename ot interval to transport interval (project-chip#108)

* [SL-UP] Delete unnecessary double abstraction for the wiseconnect power save functions (project-chip#109)

* [SL-UP] Update spi-multiplex header inclusion (project-chip#110)

* [SL-UP] Fix Builds after Wi-Fi interface refactor (project-chip#111)

* [SL-UP] Remove WF200 define duplication to avoid value conflicts at runtime (project-chip#126)

* fix spacing

* restyle

* Fix define name

* add missing function

* Fix wf200 ICD build

* Restyled by clang-format

* Add debug config

* Re added sem check

---------

Co-authored-by: Restyled.io <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature work p1 priority 1 work question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants