Skip to content

Commit 770aacd

Browse files
Adding config checks for functions specific to only wifi and thread in NetworkCommissioning Cluster code. (#33268) (#33412)
* Adding config checks for fucntions specific to only wifi and thread in NetworkCommissioning * Restlyed
1 parent cb6604f commit 770aacd

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

src/app/clusters/network-commissioning/network-commissioning.cpp

+11
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,9 @@ using namespace DeviceLayer::NetworkCommissioning;
5656
namespace {
5757

5858
// For WiFi and Thread scan results, each item will cost ~60 bytes in TLV, thus 15 is a safe upper bound of scan results.
59+
#if CHIP_DEVICE_CONFIG_ENABLE_WIFI_STATION || CHIP_DEVICE_CONFIG_ENABLE_WIFI_AP || CHIP_DEVICE_CONFIG_ENABLE_THREAD
5960
constexpr size_t kMaxNetworksInScanResponse = 15;
61+
#endif
6062

6163
constexpr uint16_t kCurrentClusterRevision = 2;
6264

@@ -544,6 +546,7 @@ bool CheckFailSafeArmed(CommandHandlerInterface::HandlerContext & ctx)
544546

545547
void Instance::HandleAddOrUpdateWiFiNetwork(HandlerContext & ctx, const Commands::AddOrUpdateWiFiNetwork::DecodableType & req)
546548
{
549+
#if CHIP_DEVICE_CONFIG_ENABLE_WIFI_STATION || CHIP_DEVICE_CONFIG_ENABLE_WIFI_AP
547550
MATTER_TRACE_SCOPE("HandleAddOrUpdateWiFiNetwork", "NetworkCommissioning");
548551

549552
VerifyOrReturn(CheckFailSafeArmed(ctx));
@@ -616,6 +619,7 @@ void Instance::HandleAddOrUpdateWiFiNetwork(HandlerContext & ctx, const Commands
616619
UpdateBreadcrumb(req.breadcrumb);
617620
ReportNetworksListChanged();
618621
}
622+
#endif
619623
}
620624

621625
#if CHIP_DEVICE_CONFIG_ENABLE_WIFI_PDC
@@ -726,6 +730,8 @@ void Instance::HandleAddOrUpdateWiFiNetworkWithPDC(HandlerContext & ctx,
726730

727731
void Instance::HandleAddOrUpdateThreadNetwork(HandlerContext & ctx, const Commands::AddOrUpdateThreadNetwork::DecodableType & req)
728732
{
733+
#if CHIP_DEVICE_CONFIG_ENABLE_THREAD
734+
729735
MATTER_TRACE_SCOPE("HandleAddOrUpdateThreadNetwork", "NetworkCommissioning");
730736

731737
VerifyOrReturn(CheckFailSafeArmed(ctx));
@@ -743,6 +749,7 @@ void Instance::HandleAddOrUpdateThreadNetwork(HandlerContext & ctx, const Comman
743749
ReportNetworksListChanged();
744750
UpdateBreadcrumb(req.breadcrumb);
745751
}
752+
#endif
746753
}
747754

748755
void Instance::UpdateBreadcrumb(const Optional<uint64_t> & breadcrumb)
@@ -978,6 +985,7 @@ void Instance::OnResult(Status commissioningError, CharSpan debugText, int32_t i
978985

979986
void Instance::OnFinished(Status status, CharSpan debugText, ThreadScanResponseIterator * networks)
980987
{
988+
#if CHIP_DEVICE_CONFIG_ENABLE_THREAD
981989
CHIP_ERROR err = CHIP_NO_ERROR;
982990
auto commandHandleRef = std::move(mAsyncCommandHandle);
983991
auto commandHandle = commandHandleRef.Get();
@@ -1088,10 +1096,12 @@ void Instance::OnFinished(Status status, CharSpan debugText, ThreadScanResponseI
10881096
CommitSavedBreadcrumb();
10891097
}
10901098
networks->Release();
1099+
#endif
10911100
}
10921101

10931102
void Instance::OnFinished(Status status, CharSpan debugText, WiFiScanResponseIterator * networks)
10941103
{
1104+
#if CHIP_DEVICE_CONFIG_ENABLE_WIFI_STATION || CHIP_DEVICE_CONFIG_ENABLE_WIFI_AP
10951105
CHIP_ERROR err = CHIP_NO_ERROR;
10961106
auto commandHandleRef = std::move(mAsyncCommandHandle);
10971107
auto commandHandle = commandHandleRef.Get();
@@ -1168,6 +1178,7 @@ void Instance::OnFinished(Status status, CharSpan debugText, WiFiScanResponseIte
11681178
{
11691179
networks->Release();
11701180
}
1181+
#endif
11711182
}
11721183

11731184
void Instance::OnPlatformEventHandler(const DeviceLayer::ChipDeviceEvent * event, intptr_t arg)

0 commit comments

Comments
 (0)