From 481eb9795115b5ebbb8bbb3464760a0f3313bfc9 Mon Sep 17 00:00:00 2001 From: chbansal Date: Wed, 10 Jul 2024 19:22:17 +0530 Subject: [PATCH] addressing the review comments --- third_party/silabs/SiWx917_sdk.gni | 30 ++++++++++++++++-------------- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/third_party/silabs/SiWx917_sdk.gni b/third_party/silabs/SiWx917_sdk.gni index 2b20f6048cfc6b..5faba08acd3d3b 100644 --- a/third_party/silabs/SiWx917_sdk.gni +++ b/third_party/silabs/SiWx917_sdk.gni @@ -291,6 +291,14 @@ template("siwx917_sdk") { if (invoker.enable_dic) { _include_dirs += [ "${chip_root}/third_party/silabs/mqtt/stack" ] } + + if (chip_enable_icd_server || !disable_lcd) { + defines += [ + "SL_SLEEP_TIMER=1", + "SI91X_SYSRTC_COUNT=1", + ] + } + if (!disable_lcd) { defines += [ "CONFIG_ENABLE_UART", @@ -310,12 +318,6 @@ template("siwx917_sdk") { ] } - if (chip_enable_icd_server || !disable_lcd) { - defines += [ - "SL_SLEEP_TIMER=1", - "SI91X_SYSRTC_COUNT=1", - ] - } if (chip_enable_icd_server) { defines += [ "SL_ICD_ENABLED=1", @@ -759,6 +761,14 @@ template("siwx917_sdk") { ] } + if (chip_enable_icd_server || !disable_lcd) { + sources += [ + "${efr32_sdk_root}/platform/service/sleeptimer/src/sl_sleeptimer.c", + "${wifi_sdk_root}/components/device/silabs/si91x/mcu/drivers/peripheral_drivers/src/rsi_sysrtc.c", + "${wifi_sdk_root}/components/device/silabs/si91x/mcu/drivers/service/sleeptimer/src/sl_sleeptimer_hal_si91x_sysrtc.c", + ] + } + if (!disable_lcd) { sources += [ "${efr32_sdk_root}/platform/middleware/glib/dmd/display/dmd_memlcd.c", @@ -794,14 +804,6 @@ template("siwx917_sdk") { ] } - if (chip_enable_icd_server || !disable_lcd) { - sources += [ - "${efr32_sdk_root}/platform/service/sleeptimer/src/sl_sleeptimer.c", - "${wifi_sdk_root}/components/device/silabs/si91x/mcu/drivers/peripheral_drivers/src/rsi_sysrtc.c", - "${wifi_sdk_root}/components/device/silabs/si91x/mcu/drivers/service/sleeptimer/src/sl_sleeptimer_hal_si91x_sysrtc.c", - ] - } - public_deps = [ "${segger_rtt_root}:segger_rtt", "${segger_rtt_root}:segger_rtt_printf",