Skip to content

Commit 90548e6

Browse files
committedApr 2, 2024·
Remove unused ThreadMode setting
1 parent 8049ff6 commit 90548e6

4 files changed

+0
-94
lines changed
 

‎src/include/platform/ConnectivityManager.h

-26
Original file line numberDiff line numberDiff line change
@@ -102,14 +102,6 @@ class ConnectivityManager
102102
kWiFiAPMode_OnDemand_NoStationProvision = 5,
103103
};
104104

105-
enum ThreadMode
106-
{
107-
kThreadMode_NotSupported = 0,
108-
kThreadMode_ApplicationControlled = 1,
109-
kThreadMode_Disabled = 2,
110-
kThreadMode_Enabled = 3,
111-
};
112-
113105
enum WiFiStationState
114106
{
115107
kWiFiStationState_NotConnected,
@@ -192,10 +184,7 @@ class ConnectivityManager
192184
void SetWiFiAPIdleTimeout(System::Clock::Timeout val);
193185

194186
// Thread Methods
195-
ThreadMode GetThreadMode();
196-
CHIP_ERROR SetThreadMode(ThreadMode val);
197187
bool IsThreadEnabled();
198-
bool IsThreadApplicationControlled();
199188
ThreadDeviceType GetThreadDeviceType();
200189
CHIP_ERROR SetThreadDeviceType(ThreadDeviceType deviceType);
201190
bool IsThreadAttached();
@@ -417,26 +406,11 @@ inline CHIP_ERROR ConnectivityManager::GetAndLogWiFiStatsCounters()
417406
return static_cast<ImplClass *>(this)->_GetAndLogWiFiStatsCounters();
418407
}
419408

420-
inline ConnectivityManager::ThreadMode ConnectivityManager::GetThreadMode()
421-
{
422-
return static_cast<ImplClass *>(this)->_GetThreadMode();
423-
}
424-
425-
inline CHIP_ERROR ConnectivityManager::SetThreadMode(ThreadMode val)
426-
{
427-
return static_cast<ImplClass *>(this)->_SetThreadMode(val);
428-
}
429-
430409
inline bool ConnectivityManager::IsThreadEnabled()
431410
{
432411
return static_cast<ImplClass *>(this)->_IsThreadEnabled();
433412
}
434413

435-
inline bool ConnectivityManager::IsThreadApplicationControlled()
436-
{
437-
return static_cast<ImplClass *>(this)->_IsThreadApplicationControlled();
438-
}
439-
440414
inline ConnectivityManager::ThreadDeviceType ConnectivityManager::GetThreadDeviceType()
441415
{
442416
return static_cast<ImplClass *>(this)->_GetThreadDeviceType();

‎src/include/platform/internal/GenericConnectivityManagerImpl_NoThread.h

-21
Original file line numberDiff line numberDiff line change
@@ -44,10 +44,7 @@ class GenericConnectivityManagerImpl_NoThread
4444
protected:
4545
// ===== Methods that implement the ConnectivityManager abstract interface.
4646

47-
ConnectivityManager::ThreadMode _GetThreadMode(void);
48-
CHIP_ERROR _SetThreadMode(ConnectivityManager::ThreadMode val);
4947
bool _IsThreadEnabled(void);
50-
bool _IsThreadApplicationControlled(void);
5148
ConnectivityManager::ThreadDeviceType _GetThreadDeviceType(void);
5249
CHIP_ERROR _SetThreadDeviceType(ConnectivityManager::ThreadDeviceType deviceType);
5350
bool _IsThreadAttached(void);
@@ -57,30 +54,12 @@ class GenericConnectivityManagerImpl_NoThread
5754
ImplClass * Impl() { return static_cast<ImplClass *>(this); }
5855
};
5956

60-
template <class ImplClass>
61-
inline ConnectivityManager::ThreadMode GenericConnectivityManagerImpl_NoThread<ImplClass>::_GetThreadMode(void)
62-
{
63-
return ConnectivityManager::kThreadMode_NotSupported;
64-
}
65-
66-
template <class ImplClass>
67-
inline CHIP_ERROR GenericConnectivityManagerImpl_NoThread<ImplClass>::_SetThreadMode(ConnectivityManager::ThreadMode val)
68-
{
69-
return CHIP_ERROR_UNSUPPORTED_CHIP_FEATURE;
70-
}
71-
7257
template <class ImplClass>
7358
inline bool GenericConnectivityManagerImpl_NoThread<ImplClass>::_IsThreadEnabled(void)
7459
{
7560
return false;
7661
}
7762

78-
template <class ImplClass>
79-
inline bool GenericConnectivityManagerImpl_NoThread<ImplClass>::_IsThreadApplicationControlled(void)
80-
{
81-
return false;
82-
}
83-
8463
template <class ImplClass>
8564
inline bool GenericConnectivityManagerImpl_NoThread<ImplClass>::_IsThreadAttached(void)
8665
{

‎src/include/platform/internal/GenericConnectivityManagerImpl_Thread.h

-10
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,7 @@ class GenericConnectivityManagerImpl_Thread
5858

5959
void _Init();
6060
void _OnPlatformEvent(const ChipDeviceEvent * event);
61-
ConnectivityManager::ThreadMode _GetThreadMode();
62-
CHIP_ERROR _SetThreadMode(ConnectivityManager::ThreadMode val);
6361
bool _IsThreadEnabled();
64-
bool _IsThreadApplicationControlled();
6562
ConnectivityManager::ThreadDeviceType _GetThreadDeviceType();
6663
CHIP_ERROR _SetThreadDeviceType(ConnectivityManager::ThreadDeviceType deviceType);
6764
#if CHIP_CONFIG_ENABLE_ICD_SERVER
@@ -82,7 +79,6 @@ class GenericConnectivityManagerImpl_Thread
8279
enum class Flags : uint8_t
8380
{
8481
kHaveServiceConnectivity = 0x01,
85-
kIsApplicationControlled = 0x02
8682
};
8783

8884
BitFlags<Flags> mFlags;
@@ -102,12 +98,6 @@ inline bool GenericConnectivityManagerImpl_Thread<ImplClass>::_IsThreadEnabled()
10298
return ThreadStackMgrImpl().IsThreadEnabled();
10399
}
104100

105-
template <class ImplClass>
106-
inline bool GenericConnectivityManagerImpl_Thread<ImplClass>::_IsThreadApplicationControlled()
107-
{
108-
return mFlags.Has(Flags::kIsApplicationControlled);
109-
}
110-
111101
template <class ImplClass>
112102
inline bool GenericConnectivityManagerImpl_Thread<ImplClass>::_IsThreadAttached()
113103
{

‎src/include/platform/internal/GenericConnectivityManagerImpl_Thread.ipp

-37
Original file line numberDiff line numberDiff line change
@@ -56,43 +56,6 @@ void GenericConnectivityManagerImpl_Thread<ImplClass>::_OnPlatformEvent(const Ch
5656
}
5757
}
5858

59-
template <class ImplClass>
60-
ConnectivityManager::ThreadMode GenericConnectivityManagerImpl_Thread<ImplClass>::_GetThreadMode()
61-
{
62-
if (mFlags.Has(Flags::kIsApplicationControlled))
63-
{
64-
return ConnectivityManager::kThreadMode_ApplicationControlled;
65-
}
66-
67-
return ThreadStackMgrImpl().IsThreadEnabled() ? ConnectivityManager::kThreadMode_Enabled
68-
: ConnectivityManager::kThreadMode_Disabled;
69-
}
70-
71-
template <class ImplClass>
72-
CHIP_ERROR GenericConnectivityManagerImpl_Thread<ImplClass>::_SetThreadMode(ConnectivityManager::ThreadMode val)
73-
{
74-
CHIP_ERROR err = CHIP_NO_ERROR;
75-
76-
VerifyOrExit(val == ConnectivityManager::kThreadMode_Enabled || val == ConnectivityManager::kThreadMode_Disabled ||
77-
val == ConnectivityManager::kThreadMode_ApplicationControlled,
78-
err = CHIP_ERROR_INVALID_ARGUMENT);
79-
80-
if (val == ConnectivityManager::kThreadMode_ApplicationControlled)
81-
{
82-
mFlags.Set(Flags::kIsApplicationControlled);
83-
}
84-
else
85-
{
86-
mFlags.Clear(Flags::kIsApplicationControlled);
87-
88-
err = ThreadStackMgrImpl().SetThreadEnabled(val == ConnectivityManager::kThreadMode_Enabled);
89-
SuccessOrExit(err);
90-
}
91-
92-
exit:
93-
return err;
94-
}
95-
9659
template <class ImplClass>
9760
void GenericConnectivityManagerImpl_Thread<ImplClass>::UpdateServiceConnectivity()
9861
{

0 commit comments

Comments
 (0)
Please sign in to comment.