Skip to content

Commit

Permalink
[Tizen] Make sure that all used defines are defined (#29607)
Browse files Browse the repository at this point in the history
* Remove unused defines

* Do not leave undefined defines

* Add missing include

* Remove Tizen from exception
  • Loading branch information
arkq authored Oct 6, 2023
1 parent f4c2094 commit 2d07e21
Show file tree
Hide file tree
Showing 7 changed files with 9 additions and 12 deletions.
3 changes: 1 addition & 2 deletions build/config/compiler/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -258,8 +258,7 @@ config("strict_warnings") {
# to be more fine-grained than current_os, but it's not clear that
# we can access that here.
if (current_os != "android" && current_os != "freertos" &&
current_os != "linux" && current_os != "mbed" && current_os != "tizen" &&
current_os != "zephyr" &&
current_os != "linux" && current_os != "mbed" && current_os != "zephyr" &&
# cmsis-rtos is OpenIOT
current_os != "cmsis-rtos" &&
# cyw30739 is one of the Infineon builds
Expand Down
1 change: 1 addition & 0 deletions examples/platform/tizen/OptionsProxy.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@

#include <app_control.h>

#include <core/CHIPBuildConfig.h>
#include <platform/CHIPDeviceBuildConfig.h>

namespace {
Expand Down
1 change: 0 additions & 1 deletion src/platform/Beken/CHIPDevicePlatformConfig.h
Original file line number Diff line number Diff line change
Expand Up @@ -67,4 +67,3 @@
#define CHIP_DEVICE_CONFIG_CHIP_TASK_PRIORITY 2

#define CHIP_DEVICE_CONFIG_ENABLE_EXTENDED_DISCOVERY 1
#define CHIP_DEVICE_CONFIG_ENABLE_COMMISSIONABLE_DISCOVERY 1
7 changes: 5 additions & 2 deletions src/platform/Tizen/CHIPDevicePlatformConfig.h
Original file line number Diff line number Diff line change
Expand Up @@ -36,11 +36,14 @@

#if CHIP_ENABLE_OPENTHREAD
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD 1
#define CHIP_DEVICE_CONFIG_ENABLE_DNSSD 1
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD_SRP_CLIENT 1
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD_DNS_CLIENT 1
#define CHIP_DEVICE_CONFIG_ENABLE_EXTENDED_DISCOVERY 1
#define CHIP_DEVICE_CONFIG_ENABLE_COMMISSIONABLE_DISCOVERY 1
#else
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD 0
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD_SRP_CLIENT 0
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD_DNS_CLIENT 0
#define CHIP_DEVICE_CONFIG_ENABLE_EXTENDED_DISCOVERY 0
#endif

#ifndef CHIP_DEVICE_CONFIG_ENABLE_CHIPOBLE
Expand Down
2 changes: 2 additions & 0 deletions src/platform/Tizen/NetworkCommissioningDriver.h
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@

#include <platform/NetworkCommissioning.h>

#include "CHIPDevicePlatformConfig.h"

namespace chip {
namespace DeviceLayer {
namespace NetworkCommissioning {
Expand Down
2 changes: 0 additions & 2 deletions src/platform/mt793x/CHIPDevicePlatformConfig.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,11 +39,9 @@
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD_SRP_CLIENT 1
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD_DNS_CLIENT 1
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD_COMMISSIONABLE_DISCOVERY 1
#define CHIP_DEVICE_CONFIG_ENABLE_DNSSD 1
#endif /* CHIP_ENABLE_OPENTHREAD */

#define CHIP_DEVICE_CONFIG_ENABLE_EXTENDED_DISCOVERY 1
#define CHIP_DEVICE_CONFIG_ENABLE_COMMISSIONABLE_DISCOVERY 1

#ifndef CHIP_DEVICE_CONFIG_ENABLE_CHIPOBLE
#define CHIP_DEVICE_CONFIG_ENABLE_CHIPOBLE 1
Expand Down
5 changes: 0 additions & 5 deletions src/platform/nxp/mw320/CHIPDevicePlatformConfig.h
Original file line number Diff line number Diff line change
Expand Up @@ -59,8 +59,3 @@
#define CHIP_DEVICE_CONFIG_ENABLE_WIFI_TELEMETRY 0
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD_TELEMETRY 0
#define CHIP_DEVICE_CONFIG_ENABLE_THREAD_TELEMETRY_FULL 0

// mdns/mdns_sd ++
// #define CHIP_DEVICE_CONFIG_ENABLE_COMMISSIONABLE_DISCOVERY 1
#define CHIP_DEVICE_CONFIG_ENABLE_DNSSD 1
// mdns/dns_sd --

0 comments on commit 2d07e21

Please sign in to comment.