[Silabs] Move Wi-Fi interface files to the final structure (#36644)

* Move files * Move wifi files to their final location & cleanup * Restyled by clang-format * rename directories * rename paths for new locations * Restyled by clang-format * update include path with the structure --------- Co-authored-by: Restyled.io <[email protected]>
project-chip · Nov 28, 2024 · 70f2f3e · 70f2f3e
1 parent 29c2647
commit 70f2f3e
Show file tree

Hide file tree

Showing 30 changed files with 67 additions and 91 deletions.
diff --git a/examples/platform/silabs/display/demo-ui.c b/examples/platform/silabs/display/demo-ui.c
@@ -27,7 +27,7 @@
 #include "sl_memlcd.h"
 #include <app/icd/server/ICDServerConfig.h>
 #if SL_WIFI && !SLI_SI91X_MCU_INTERFACE
-#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
+#include <platform/silabs/wifi/ncp/spi_multiplex.h>
 #endif // SL_WIFI && !SLI_SI91X_MCU_INTERFACE
 #include <string.h>
 

diff --git a/src/platform/silabs/efr32/OTAImageProcessorImpl.cpp b/src/platform/silabs/efr32/OTAImageProcessorImpl.cpp
@@ -22,7 +22,7 @@
 #include <platform/silabs/SilabsConfig.h>
 
 #if SL_WIFI
-#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
+#include <platform/silabs/wifi/ncp/spi_multiplex.h>
 #endif // SL_WIFI
 
 extern "C" {

diff --git a/src/platform/silabs/wifi/BUILD.gn b/src/platform/silabs/wifi/BUILD.gn
@@ -43,7 +43,7 @@ if (chip_enable_wifi && !wifi_soc) {
   if (use_rs9116) {
     import("${silabs_platform_dir}/wifi/rs911x/rs911x.gni")
   } else if (use_SiWx917) {
-    import("${silabs_platform_dir}/wifi/rs911x/rs9117.gni")
+    import("${silabs_platform_dir}/wifi/SiWx/ncp/rs9117.gni")
   }
   if (use_wf200) {
     import("${silabs_platform_dir}/wifi/wf200/wf200.gni")

diff --git a/src/platform/silabs/wifi/SiWx/WifiInterface.cpp b/src/platform/silabs/wifi/SiWx/WifiInterface.cpp
@@ -67,7 +67,7 @@ extern "C" {
 
 #if (EXP_BOARD)
 #include "rsi_bt_common_apis.h"
-#include <platform/silabs/wifi/rs911x/platform/sl_board_configuration.h>
+#include <platform/silabs/wifi/SiWx/ncp/sl_board_configuration.h>
 #endif
 
 #if CHIP_CONFIG_ENABLE_ICD_SERVER && SLI_SI91X_MCU_INTERFACE

diff --git a/...abs/wifi/rs911x/platform/efx32_ncp_host.c → ...orm/silabs/wifi/SiWx/ncp/efx32_ncp_host.c b/...abs/wifi/rs911x/platform/efx32_ncp_host.c → ...orm/silabs/wifi/SiWx/ncp/efx32_ncp_host.c
@@ -22,15 +22,15 @@
 #include "em_gpio.h"
 #include "em_usart.h"
 #include "gpiointerrupt.h"
-#include "sl_board_configuration_SiWx917.h"
 #include "sl_constants.h"
 #include "sl_rsi_utility.h"
 #include "sl_si91x_host_interface.h"
 #include "sl_si91x_ncp_utility.h"
 #include "sl_si91x_status.h"
 #include "sl_status.h"
 #include "sl_wifi_constants.h"
-#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
+#include <platform/silabs/wifi/SiWx/ncp/sl_board_configuration.h>
+#include <platform/silabs/wifi/ncp/spi_multiplex.h>
 #include <stdbool.h>
 #include <string.h>
 

diff --git a/src/platform/silabs/wifi/rs911x/rs9117.gni → src/platform/silabs/wifi/SiWx/ncp/rs9117.gni b/src/platform/silabs/wifi/rs911x/rs9117.gni → src/platform/silabs/wifi/SiWx/ncp/rs9117.gni
@@ -18,17 +18,14 @@ import("${silabs_sdk_build_root}/efr32_sdk.gni")
 
 rs911x_src_plat = [
   "${chip_root}/src/platform/silabs/wifi/SiWx/WifiInterface.cpp",
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_interrupt.c",
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/sl_si91x_ncp_utility.c",
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/sl_board_configuration.h",
+  "${chip_root}/src/platform/silabs/wifi/SiWx/ncp/sl_si91x_ncp_utility.c",
+  "${chip_root}/src/platform/silabs/wifi/SiWx/ncp/sl_board_configuration.h",
   "${chip_root}/src/platform/silabs/wifi/wiseconnect-abstraction/WiseconnectInterfaceAbstraction.cpp",
   "${chip_root}/src/platform/silabs/wifi/wiseconnect-abstraction/WiseconnectInterfaceAbstraction.h",
-
-  # TODO: We shouldn't need a file form the WF200 for the SiWx917 NCP builds
-  "${chip_root}/src/platform/silabs/wifi/wf200/platform/spi_multiplex.h",
+  "${chip_root}/src/platform/silabs/wifi/ncp/spi_multiplex.h",
 
   # TODO : We should be using the file from the Wiseconnect SDK and not our copy of it.
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/efx32_ncp_host.c",
+  "${chip_root}/src/platform/silabs/wifi/SiWx/ncp/efx32_ncp_host.c",
 ]
 
 rs9117_inc_plat = [ "${wifi_sdk_root}/components/si91x/ble/inc" ]

diff --git a/...platform/sl_board_configuration_SiWx917.h → ...bs/wifi/SiWx/ncp/sl_board_configuration.h b/...platform/sl_board_configuration_SiWx917.h → ...bs/wifi/SiWx/ncp/sl_board_configuration.h
@@ -1,3 +1,20 @@
+/*
+ *
+ *    Copyright (c) 2024 Project CHIP Authors
+ *
+ *    Licensed under the Apache License, Version 2.0 (the "License");
+ *    you may not use this file except in compliance with the License.
+ *    You may obtain a copy of the License at
+ *
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *    Unless required by applicable law or agreed to in writing, software
+ *    distributed under the License is distributed on an "AS IS" BASIS,
+ *    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *    See the License for the specific language governing permissions and
+ *    limitations under the License.
+ */
+
 #pragma once
 
 #ifdef SL_UART

diff --git a/...fi/rs911x/platform/sl_si91x_ncp_utility.c → ...labs/wifi/SiWx/ncp/sl_si91x_ncp_utility.c b/...fi/rs911x/platform/sl_si91x_ncp_utility.c → ...labs/wifi/SiWx/ncp/sl_si91x_ncp_utility.c
@@ -15,9 +15,6 @@
  *    limitations under the License.
  */
 
-/**
- * Includes
- */
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>

diff --git a/...fi/rs911x/platform/sl_si91x_ncp_utility.h → ...labs/wifi/SiWx/ncp/sl_si91x_ncp_utility.h b/...fi/rs911x/platform/sl_si91x_ncp_utility.h → ...labs/wifi/SiWx/ncp/sl_si91x_ncp_utility.h
@@ -21,24 +21,16 @@
  * devices such as External Flash and LCD.
  * That can be extended to other families as well.
  */
-#ifndef SL_SI91X_NCP_UTILITY_H
-#define SL_SI91X_NCP_UTILITY_H
-
 #pragma once
+
 #include "FreeRTOS.h"
+#include "em_usart.h"
 #include "semphr.h"
 #include "silabs_utils.h"
-#include "sl_status.h"
-
-// TODO: This is a WF200 specific include. It is not clear why we need this in the 917 NCP files.
-#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
-
-#if defined(CHIP_9117)
-#include "em_usart.h"
-#include "sl_board_configuration_SiWx917.h"
 #include "sl_spidrv_exp_config.h"
-
-#endif // CHIP_9117
+#include "sl_status.h"
+#include <platform/silabs/wifi/SiWx/ncp/sl_board_configuration.h>
+#include <platform/silabs/wifi/ncp/spi_multiplex.h>
 
 #define USART_INITSYNC_BAUDRATE 12500000
 
@@ -48,4 +40,3 @@ sl_status_t spi_board_init(void);
 
 extern uint32_t rx_ldma_channel;
 extern uint32_t tx_ldma_channel;
-#endif // SL_SI91X_NCP_UTILITY_H
diff --git a/...ilabs/wifi/wf200/platform/spi_multiplex.h → src/platform/silabs/wifi/ncp/spi_multiplex.h b/...ilabs/wifi/wf200/platform/spi_multiplex.h → src/platform/silabs/wifi/ncp/spi_multiplex.h
diff --git a/...orm/silabs/wifi/rs911x/platform/efx_spi.c → ...platform/silabs/wifi/rs911x/ncp/efx_spi.c b/...orm/silabs/wifi/rs911x/platform/efx_spi.c → ...platform/silabs/wifi/rs911x/ncp/efx_spi.c
@@ -34,7 +34,7 @@
 #include "sl_status.h"
 #include "spidrv.h"
 #include "task.h"
-#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
+#include <platform/silabs/wifi/ncp/spi_multiplex.h>
 
 #ifdef SL_BOARD_NAME
 #include "sl_board_control.h"

diff --git a/...rs911x/platform/rsi_board_configuration.h → ...wifi/rs911x/ncp/rsi_board_configuration.h b/...rs911x/platform/rsi_board_configuration.h → ...wifi/rs911x/ncp/rsi_board_configuration.h
diff --git a/...orm/silabs/wifi/rs911x/platform/rsi_hal.h → ...platform/silabs/wifi/rs911x/ncp/rsi_hal.h b/...orm/silabs/wifi/rs911x/platform/rsi_hal.h → ...platform/silabs/wifi/rs911x/ncp/rsi_hal.h
diff --git a/...i/rs911x/platform/rsi_hal_mcu_interrupt.c → ...s/wifi/rs911x/ncp/rsi_hal_mcu_interrupt.c b/...i/rs911x/platform/rsi_hal_mcu_interrupt.c → ...s/wifi/rs911x/ncp/rsi_hal_mcu_interrupt.c
@@ -15,10 +15,7 @@
  *    limitations under the License.
  */
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-
+#include "FreeRTOS.h"
 #include "dmadrv.h"
 #include "em_chip.h"
 #include "em_cmu.h"
@@ -27,25 +24,16 @@
 #include "em_gpio.h"
 #include "em_ldma.h"
 #include "em_usart.h"
+#include "event_groups.h"
 #include "gpiointerrupt.h"
+#include "rsi_board_configuration.h"
+#include "rsi_driver.h"
 #include "sl_device_init_clocks.h"
 #include "sl_status.h"
-
-#include "FreeRTOS.h"
-#include "event_groups.h"
 #include "task.h"
-
-#if (SLI_SI91X_MCU_INTERFACE | EXP_BOARD)
-#include "sl_board_configuration.h"
-
-#include "sl_rsi_utility.h"
-#include "sl_si91x_host_interface.h"
-
-void gpio_interrupt(uint8_t interrupt_number);
-#else
-#include "rsi_board_configuration.h"
-#include "rsi_driver.h"
-#endif
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
 
 typedef void (*UserIntCallBack_t)(void);
 UserIntCallBack_t call_back, gpio_callback;

diff --git a/...ifi/rs911x/platform/rsi_hal_mcu_ioports.c → ...abs/wifi/rs911x/ncp/rsi_hal_mcu_ioports.c b/...ifi/rs911x/platform/rsi_hal_mcu_ioports.c → ...abs/wifi/rs911x/ncp/rsi_hal_mcu_ioports.c
@@ -15,10 +15,7 @@
  *    limitations under the License.
  */
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-
+#include "FreeRTOS.h"
 #include "dmadrv.h"
 #include "em_chip.h"
 #include "em_cmu.h"
@@ -27,16 +24,17 @@
 #include "em_gpio.h"
 #include "em_ldma.h"
 #include "em_usart.h"
+#include "event_groups.h"
 #include "gpiointerrupt.h"
+#include "rsi_board_configuration.h"
+#include "rsi_driver.h"
 #include "sl_device_init_clocks.h"
 #include "sl_status.h"
-
-#include "FreeRTOS.h"
-#include "event_groups.h"
 #include "task.h"
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
 
-#include "rsi_board_configuration.h"
-#include "rsi_driver.h"
 /*===========================================================*/
 /**
  * @fn            void rsi_hal_config_gpio(uint8_t gpio_number,uint8_t mode,uint8_t value)

diff --git a/...bs/wifi/rs911x/platform/rsi_hal_mcu_rtc.c → .../silabs/wifi/rs911x/ncp/rsi_hal_mcu_rtc.c b/...bs/wifi/rs911x/platform/rsi_hal_mcu_rtc.c → .../silabs/wifi/rs911x/ncp/rsi_hal_mcu_rtc.c
diff --git a/.../wifi/rs911x/platform/rsi_hal_mcu_timer.c → ...ilabs/wifi/rs911x/ncp/rsi_hal_mcu_timer.c b/.../wifi/rs911x/platform/rsi_hal_mcu_timer.c → ...ilabs/wifi/rs911x/ncp/rsi_hal_mcu_timer.c
diff --git a/...form/silabs/wifi/rs911x/rsi_wlan_config.h → .../silabs/wifi/rs911x/ncp/rsi_wlan_config.h b/...form/silabs/wifi/rs911x/rsi_wlan_config.h → .../silabs/wifi/rs911x/ncp/rsi_wlan_config.h
@@ -46,18 +46,9 @@
 
 //! To set Extended custom feature select bit map
 #if WIFI_ENABLE_SECURITY_WPA3_TRANSITION
-#ifdef CHIP_9117
-#define RSI_EXT_CUSTOM_FEATURE_BIT_MAP                                                                                             \
-    (EXT_FEAT_448K_M4SS_256K | EXT_FEAT_LOW_POWER_MODE | EXT_FEAT_XTAL_CLK_ENABLE | EXT_FEAT_IEEE_80211W)
-#else /* !CHIP_9117 */
 #define RSI_EXT_CUSTOM_FEATURE_BIT_MAP (EXT_FEAT_384K_MODE | EXT_FEAT_IEEE_80211W)
-#endif /* CHIP_9117 */
-#else  /* !WIFI_ENABLE_SECURITY_WPA3_TRANSITION */
-#ifdef CHIP_9117
-#define RSI_EXT_CUSTOM_FEATURE_BIT_MAP (EXT_FEAT_448K_M4SS_256K | EXT_FEAT_LOW_POWER_MODE | EXT_FEAT_XTAL_CLK_ENABLE)
-#else /* !CHIP_9117 */
+#else /* !WIFI_ENABLE_SECURITY_WPA3_TRANSITION */
 #define RSI_EXT_CUSTOM_FEATURE_BIT_MAP EXT_FEAT_384K_MODE
-#endif /* CHIP_9117 */
 #endif /* WIFI_ENABLE_SECURITY_WPA3_TRANSITION */
 
 //! To set Extended TCPIP feature select bit map

diff --git a/.../rs911x/platform/sl_board_configuration.h → .../wifi/rs911x/ncp/sl_board_configuration.h b/.../rs911x/platform/sl_board_configuration.h → .../wifi/rs911x/ncp/sl_board_configuration.h
diff --git a/src/platform/silabs/wifi/rs911x/rs911x.gni b/src/platform/silabs/wifi/rs911x/rs911x.gni
@@ -18,12 +18,12 @@ import("${silabs_sdk_build_root}/efr32_sdk.gni")
 
 rs911x_src_plat = [
   "${chip_root}/src/platform/silabs/wifi/rs911x/WifiInterface.cpp",
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_interrupt.c",
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_ioports.c",
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_timer.c",
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/efx_spi.c",
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_board_configuration.h",
-  "${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal.h",
+  "${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_hal_mcu_interrupt.c",
+  "${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_hal_mcu_ioports.c",
+  "${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_hal_mcu_timer.c",
+  "${chip_root}/src/platform/silabs/wifi/rs911x/ncp/efx_spi.c",
+  "${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_board_configuration.h",
+  "${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_hal.h",
   "${chip_root}/src/platform/silabs/wifi/wiseconnect-abstraction/WiseconnectInterfaceAbstraction.cpp",
 ]
 

diff --git a/src/platform/silabs/wifi/wf200/WifiInterface.cpp b/src/platform/silabs/wifi/wf200/WifiInterface.cpp
@@ -33,9 +33,9 @@
 #include <platform/silabs/wifi/WifiInterfaceAbstraction.h>
 #include <platform/silabs/wifi/lwip-support/dhcp_client.h>
 #include <platform/silabs/wifi/lwip-support/ethernetif.h>
-#include <platform/silabs/wifi/wf200/platform/sl_wfx_board.h>
-#include <platform/silabs/wifi/wf200/platform/sl_wfx_host.h>
-#include <platform/silabs/wifi/wf200/platform/sl_wfx_task.h>
+#include <platform/silabs/wifi/wf200/ncp/sl_wfx_board.h>
+#include <platform/silabs/wifi/wf200/ncp/sl_wfx_host.h>
+#include <platform/silabs/wifi/wf200/ncp/sl_wfx_task.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>

diff --git a/...form/silabs/wifi/wf200/platform/efr_spi.c → src/platform/silabs/wifi/wf200/ncp/efr_spi.c b/...form/silabs/wifi/wf200/platform/efr_spi.c → src/platform/silabs/wifi/wf200/ncp/efr_spi.c
@@ -32,7 +32,7 @@
 #include "sl_wfx_host_api.h"
 #include "sl_wfx_task.h"
 #include "spidrv.h"
-#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
+#include <platform/silabs/wifi/ncp/spi_multiplex.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>

diff --git a/...abs/wifi/wf200/platform/sl_custom_board.h → ...m/silabs/wifi/wf200/ncp/sl_custom_board.h b/...abs/wifi/wf200/platform/sl_custom_board.h → ...m/silabs/wifi/wf200/ncp/sl_custom_board.h
diff --git a/...silabs/wifi/wf200/platform/sl_wfx_board.h → ...form/silabs/wifi/wf200/ncp/sl_wfx_board.h b/...silabs/wifi/wf200/platform/sl_wfx_board.h → ...form/silabs/wifi/wf200/ncp/sl_wfx_board.h
diff --git a/.../silabs/wifi/wf200/platform/sl_wfx_host.h → ...tform/silabs/wifi/wf200/ncp/sl_wfx_host.h b/.../silabs/wifi/wf200/platform/sl_wfx_host.h → ...tform/silabs/wifi/wf200/ncp/sl_wfx_host.h
diff --git a/.../silabs/wifi/wf200/platform/sl_wfx_task.c → ...tform/silabs/wifi/wf200/ncp/sl_wfx_task.c b/.../silabs/wifi/wf200/platform/sl_wfx_task.c → ...tform/silabs/wifi/wf200/ncp/sl_wfx_task.c
diff --git a/.../silabs/wifi/wf200/platform/sl_wfx_task.h → ...tform/silabs/wifi/wf200/ncp/sl_wfx_task.h b/.../silabs/wifi/wf200/platform/sl_wfx_task.h → ...tform/silabs/wifi/wf200/ncp/sl_wfx_task.h
diff --git a/...m/silabs/wifi/wf200/platform/wf200_init.c → ...atform/silabs/wifi/wf200/ncp/wf200_init.c b/...m/silabs/wifi/wf200/platform/wf200_init.c → ...atform/silabs/wifi/wf200/ncp/wf200_init.c
diff --git a/src/platform/silabs/wifi/wf200/wf200.gni b/src/platform/silabs/wifi/wf200/wf200.gni
@@ -18,12 +18,12 @@ import("${silabs_sdk_build_root}/efr32_sdk.gni")
 
 wf200_plat_src = [
   "${chip_root}/src/platform/silabs/wifi/wf200/WifiInterface.cpp",
-  "${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_wfx_task.c",
-  "${chip_root}/src/platform/silabs/wifi/wf200/platform/wf200_init.c",
-  "${chip_root}/src/platform/silabs/wifi/wf200/platform/efr_spi.c",
-  "${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_wfx_board.h",
-  "${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_wfx_host.h",
-  "${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_wfx_task.h",
-  "${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_custom_board.h",
-  "${chip_root}/src/platform/silabs/wifi/wf200/platform/spi_multiplex.h",
+  "${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_wfx_task.c",
+  "${chip_root}/src/platform/silabs/wifi/wf200/ncp/wf200_init.c",
+  "${chip_root}/src/platform/silabs/wifi/wf200/ncp/efr_spi.c",
+  "${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_wfx_board.h",
+  "${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_wfx_host.h",
+  "${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_wfx_task.h",
+  "${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_custom_board.h",
+  "${chip_root}/src/platform/silabs/wifi/ncp/spi_multiplex.h",
 ]
diff --git a/third_party/silabs/efr32_sdk.gni b/third_party/silabs/efr32_sdk.gni
@@ -370,10 +370,7 @@ template("efr32_sdk") {
 
     if (use_rs9116) {
       # TODO: we should create a seperate directory for headers that are necessary for the underlying sdks
-      _include_dirs += [
-        "${chip_root}/src/platform/silabs/wifi/rs911x/platform",
-        "${chip_root}/src/platform/silabs/wifi/rs911x",
-      ]
+      _include_dirs += [ "${chip_root}/src/platform/silabs/wifi/rs911x/ncp" ]
     }
 
     if (use_rs9116 || use_SiWx917) {