Skip to content

Commit 70f2f3e

Browse files
[Silabs] Move Wi-Fi interface files to the final structure (project-chip#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 <commits@restyled.io>
1 parent 29c2647 commit 70f2f3e

30 files changed

+67
-91
lines changed

examples/platform/silabs/display/demo-ui.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
#include "sl_memlcd.h"
2828
#include <app/icd/server/ICDServerConfig.h>
2929
#if SL_WIFI && !SLI_SI91X_MCU_INTERFACE
30-
#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
30+
#include <platform/silabs/wifi/ncp/spi_multiplex.h>
3131
#endif // SL_WIFI && !SLI_SI91X_MCU_INTERFACE
3232
#include <string.h>
3333

src/platform/silabs/efr32/OTAImageProcessorImpl.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
#include <platform/silabs/SilabsConfig.h>
2323

2424
#if SL_WIFI
25-
#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
25+
#include <platform/silabs/wifi/ncp/spi_multiplex.h>
2626
#endif // SL_WIFI
2727

2828
extern "C" {

src/platform/silabs/wifi/BUILD.gn

+1-1
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ if (chip_enable_wifi && !wifi_soc) {
4343
if (use_rs9116) {
4444
import("${silabs_platform_dir}/wifi/rs911x/rs911x.gni")
4545
} else if (use_SiWx917) {
46-
import("${silabs_platform_dir}/wifi/rs911x/rs9117.gni")
46+
import("${silabs_platform_dir}/wifi/SiWx/ncp/rs9117.gni")
4747
}
4848
if (use_wf200) {
4949
import("${silabs_platform_dir}/wifi/wf200/wf200.gni")

src/platform/silabs/wifi/SiWx/WifiInterface.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ extern "C" {
6767

6868
#if (EXP_BOARD)
6969
#include "rsi_bt_common_apis.h"
70-
#include <platform/silabs/wifi/rs911x/platform/sl_board_configuration.h>
70+
#include <platform/silabs/wifi/SiWx/ncp/sl_board_configuration.h>
7171
#endif
7272

7373
#if CHIP_CONFIG_ENABLE_ICD_SERVER && SLI_SI91X_MCU_INTERFACE

src/platform/silabs/wifi/rs911x/platform/efx32_ncp_host.c src/platform/silabs/wifi/SiWx/ncp/efx32_ncp_host.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,15 @@
2222
#include "em_gpio.h"
2323
#include "em_usart.h"
2424
#include "gpiointerrupt.h"
25-
#include "sl_board_configuration_SiWx917.h"
2625
#include "sl_constants.h"
2726
#include "sl_rsi_utility.h"
2827
#include "sl_si91x_host_interface.h"
2928
#include "sl_si91x_ncp_utility.h"
3029
#include "sl_si91x_status.h"
3130
#include "sl_status.h"
3231
#include "sl_wifi_constants.h"
33-
#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
32+
#include <platform/silabs/wifi/SiWx/ncp/sl_board_configuration.h>
33+
#include <platform/silabs/wifi/ncp/spi_multiplex.h>
3434
#include <stdbool.h>
3535
#include <string.h>
3636

src/platform/silabs/wifi/rs911x/rs9117.gni src/platform/silabs/wifi/SiWx/ncp/rs9117.gni

+4-7
Original file line numberDiff line numberDiff line change
@@ -18,17 +18,14 @@ import("${silabs_sdk_build_root}/efr32_sdk.gni")
1818

1919
rs911x_src_plat = [
2020
"${chip_root}/src/platform/silabs/wifi/SiWx/WifiInterface.cpp",
21-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_interrupt.c",
22-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/sl_si91x_ncp_utility.c",
23-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/sl_board_configuration.h",
21+
"${chip_root}/src/platform/silabs/wifi/SiWx/ncp/sl_si91x_ncp_utility.c",
22+
"${chip_root}/src/platform/silabs/wifi/SiWx/ncp/sl_board_configuration.h",
2423
"${chip_root}/src/platform/silabs/wifi/wiseconnect-abstraction/WiseconnectInterfaceAbstraction.cpp",
2524
"${chip_root}/src/platform/silabs/wifi/wiseconnect-abstraction/WiseconnectInterfaceAbstraction.h",
26-
27-
# TODO: We shouldn't need a file form the WF200 for the SiWx917 NCP builds
28-
"${chip_root}/src/platform/silabs/wifi/wf200/platform/spi_multiplex.h",
25+
"${chip_root}/src/platform/silabs/wifi/ncp/spi_multiplex.h",
2926

3027
# TODO : We should be using the file from the Wiseconnect SDK and not our copy of it.
31-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/efx32_ncp_host.c",
28+
"${chip_root}/src/platform/silabs/wifi/SiWx/ncp/efx32_ncp_host.c",
3229
]
3330

3431
rs9117_inc_plat = [ "${wifi_sdk_root}/components/si91x/ble/inc" ]

src/platform/silabs/wifi/rs911x/platform/sl_board_configuration_SiWx917.h src/platform/silabs/wifi/SiWx/ncp/sl_board_configuration.h

+17
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,20 @@
1+
/*
2+
*
3+
* Copyright (c) 2024 Project CHIP Authors
4+
*
5+
* Licensed under the Apache License, Version 2.0 (the "License");
6+
* you may not use this file except in compliance with the License.
7+
* You may obtain a copy of the License at
8+
*
9+
* http://www.apache.org/licenses/LICENSE-2.0
10+
*
11+
* Unless required by applicable law or agreed to in writing, software
12+
* distributed under the License is distributed on an "AS IS" BASIS,
13+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
14+
* See the License for the specific language governing permissions and
15+
* limitations under the License.
16+
*/
17+
118
#pragma once
219

320
#ifdef SL_UART

src/platform/silabs/wifi/rs911x/platform/sl_si91x_ncp_utility.c src/platform/silabs/wifi/SiWx/ncp/sl_si91x_ncp_utility.c

-3
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,6 @@
1515
* limitations under the License.
1616
*/
1717

18-
/**
19-
* Includes
20-
*/
2118
#include <stdio.h>
2219
#include <stdlib.h>
2320
#include <string.h>

src/platform/silabs/wifi/rs911x/platform/sl_si91x_ncp_utility.h src/platform/silabs/wifi/SiWx/ncp/sl_si91x_ncp_utility.h

+5-14
Original file line numberDiff line numberDiff line change
@@ -21,24 +21,16 @@
2121
* devices such as External Flash and LCD.
2222
* That can be extended to other families as well.
2323
*/
24-
#ifndef SL_SI91X_NCP_UTILITY_H
25-
#define SL_SI91X_NCP_UTILITY_H
26-
2724
#pragma once
25+
2826
#include "FreeRTOS.h"
27+
#include "em_usart.h"
2928
#include "semphr.h"
3029
#include "silabs_utils.h"
31-
#include "sl_status.h"
32-
33-
// TODO: This is a WF200 specific include. It is not clear why we need this in the 917 NCP files.
34-
#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
35-
36-
#if defined(CHIP_9117)
37-
#include "em_usart.h"
38-
#include "sl_board_configuration_SiWx917.h"
3930
#include "sl_spidrv_exp_config.h"
40-
41-
#endif // CHIP_9117
31+
#include "sl_status.h"
32+
#include <platform/silabs/wifi/SiWx/ncp/sl_board_configuration.h>
33+
#include <platform/silabs/wifi/ncp/spi_multiplex.h>
4234

4335
#define USART_INITSYNC_BAUDRATE 12500000
4436

@@ -48,4 +40,3 @@ sl_status_t spi_board_init(void);
4840

4941
extern uint32_t rx_ldma_channel;
5042
extern uint32_t tx_ldma_channel;
51-
#endif // SL_SI91X_NCP_UTILITY_H

src/platform/silabs/wifi/rs911x/platform/efx_spi.c src/platform/silabs/wifi/rs911x/ncp/efx_spi.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
#include "sl_status.h"
3535
#include "spidrv.h"
3636
#include "task.h"
37-
#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
37+
#include <platform/silabs/wifi/ncp/spi_multiplex.h>
3838

3939
#ifdef SL_BOARD_NAME
4040
#include "sl_board_control.h"

src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_interrupt.c src/platform/silabs/wifi/rs911x/ncp/rsi_hal_mcu_interrupt.c

+7-19
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,7 @@
1515
* limitations under the License.
1616
*/
1717

18-
#include <stdio.h>
19-
#include <stdlib.h>
20-
#include <string.h>
21-
18+
#include "FreeRTOS.h"
2219
#include "dmadrv.h"
2320
#include "em_chip.h"
2421
#include "em_cmu.h"
@@ -27,25 +24,16 @@
2724
#include "em_gpio.h"
2825
#include "em_ldma.h"
2926
#include "em_usart.h"
27+
#include "event_groups.h"
3028
#include "gpiointerrupt.h"
29+
#include "rsi_board_configuration.h"
30+
#include "rsi_driver.h"
3131
#include "sl_device_init_clocks.h"
3232
#include "sl_status.h"
33-
34-
#include "FreeRTOS.h"
35-
#include "event_groups.h"
3633
#include "task.h"
37-
38-
#if (SLI_SI91X_MCU_INTERFACE | EXP_BOARD)
39-
#include "sl_board_configuration.h"
40-
41-
#include "sl_rsi_utility.h"
42-
#include "sl_si91x_host_interface.h"
43-
44-
void gpio_interrupt(uint8_t interrupt_number);
45-
#else
46-
#include "rsi_board_configuration.h"
47-
#include "rsi_driver.h"
48-
#endif
34+
#include <stdio.h>
35+
#include <stdlib.h>
36+
#include <string.h>
4937

5038
typedef void (*UserIntCallBack_t)(void);
5139
UserIntCallBack_t call_back, gpio_callback;

src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_ioports.c src/platform/silabs/wifi/rs911x/ncp/rsi_hal_mcu_ioports.c

+7-9
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,7 @@
1515
* limitations under the License.
1616
*/
1717

18-
#include <stdio.h>
19-
#include <stdlib.h>
20-
#include <string.h>
21-
18+
#include "FreeRTOS.h"
2219
#include "dmadrv.h"
2320
#include "em_chip.h"
2421
#include "em_cmu.h"
@@ -27,16 +24,17 @@
2724
#include "em_gpio.h"
2825
#include "em_ldma.h"
2926
#include "em_usart.h"
27+
#include "event_groups.h"
3028
#include "gpiointerrupt.h"
29+
#include "rsi_board_configuration.h"
30+
#include "rsi_driver.h"
3131
#include "sl_device_init_clocks.h"
3232
#include "sl_status.h"
33-
34-
#include "FreeRTOS.h"
35-
#include "event_groups.h"
3633
#include "task.h"
34+
#include <stdio.h>
35+
#include <stdlib.h>
36+
#include <string.h>
3737

38-
#include "rsi_board_configuration.h"
39-
#include "rsi_driver.h"
4038
/*===========================================================*/
4139
/**
4240
* @fn void rsi_hal_config_gpio(uint8_t gpio_number,uint8_t mode,uint8_t value)

src/platform/silabs/wifi/rs911x/rsi_wlan_config.h src/platform/silabs/wifi/rs911x/ncp/rsi_wlan_config.h

+1-10
Original file line numberDiff line numberDiff line change
@@ -46,18 +46,9 @@
4646

4747
//! To set Extended custom feature select bit map
4848
#if WIFI_ENABLE_SECURITY_WPA3_TRANSITION
49-
#ifdef CHIP_9117
50-
#define RSI_EXT_CUSTOM_FEATURE_BIT_MAP \
51-
(EXT_FEAT_448K_M4SS_256K | EXT_FEAT_LOW_POWER_MODE | EXT_FEAT_XTAL_CLK_ENABLE | EXT_FEAT_IEEE_80211W)
52-
#else /* !CHIP_9117 */
5349
#define RSI_EXT_CUSTOM_FEATURE_BIT_MAP (EXT_FEAT_384K_MODE | EXT_FEAT_IEEE_80211W)
54-
#endif /* CHIP_9117 */
55-
#else /* !WIFI_ENABLE_SECURITY_WPA3_TRANSITION */
56-
#ifdef CHIP_9117
57-
#define RSI_EXT_CUSTOM_FEATURE_BIT_MAP (EXT_FEAT_448K_M4SS_256K | EXT_FEAT_LOW_POWER_MODE | EXT_FEAT_XTAL_CLK_ENABLE)
58-
#else /* !CHIP_9117 */
50+
#else /* !WIFI_ENABLE_SECURITY_WPA3_TRANSITION */
5951
#define RSI_EXT_CUSTOM_FEATURE_BIT_MAP EXT_FEAT_384K_MODE
60-
#endif /* CHIP_9117 */
6152
#endif /* WIFI_ENABLE_SECURITY_WPA3_TRANSITION */
6253

6354
//! To set Extended TCPIP feature select bit map

src/platform/silabs/wifi/rs911x/rs911x.gni

+6-6
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,12 @@ import("${silabs_sdk_build_root}/efr32_sdk.gni")
1818

1919
rs911x_src_plat = [
2020
"${chip_root}/src/platform/silabs/wifi/rs911x/WifiInterface.cpp",
21-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_interrupt.c",
22-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_ioports.c",
23-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal_mcu_timer.c",
24-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/efx_spi.c",
25-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_board_configuration.h",
26-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform/rsi_hal.h",
21+
"${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_hal_mcu_interrupt.c",
22+
"${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_hal_mcu_ioports.c",
23+
"${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_hal_mcu_timer.c",
24+
"${chip_root}/src/platform/silabs/wifi/rs911x/ncp/efx_spi.c",
25+
"${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_board_configuration.h",
26+
"${chip_root}/src/platform/silabs/wifi/rs911x/ncp/rsi_hal.h",
2727
"${chip_root}/src/platform/silabs/wifi/wiseconnect-abstraction/WiseconnectInterfaceAbstraction.cpp",
2828
]
2929

src/platform/silabs/wifi/wf200/WifiInterface.cpp

+3-3
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,9 @@
3333
#include <platform/silabs/wifi/WifiInterfaceAbstraction.h>
3434
#include <platform/silabs/wifi/lwip-support/dhcp_client.h>
3535
#include <platform/silabs/wifi/lwip-support/ethernetif.h>
36-
#include <platform/silabs/wifi/wf200/platform/sl_wfx_board.h>
37-
#include <platform/silabs/wifi/wf200/platform/sl_wfx_host.h>
38-
#include <platform/silabs/wifi/wf200/platform/sl_wfx_task.h>
36+
#include <platform/silabs/wifi/wf200/ncp/sl_wfx_board.h>
37+
#include <platform/silabs/wifi/wf200/ncp/sl_wfx_host.h>
38+
#include <platform/silabs/wifi/wf200/ncp/sl_wfx_task.h>
3939
#include <stdio.h>
4040
#include <stdlib.h>
4141
#include <string.h>

src/platform/silabs/wifi/wf200/platform/efr_spi.c src/platform/silabs/wifi/wf200/ncp/efr_spi.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
#include "sl_wfx_host_api.h"
3333
#include "sl_wfx_task.h"
3434
#include "spidrv.h"
35-
#include <platform/silabs/wifi/wf200/platform/spi_multiplex.h>
35+
#include <platform/silabs/wifi/ncp/spi_multiplex.h>
3636
#include <stdio.h>
3737
#include <stdlib.h>
3838
#include <string.h>

src/platform/silabs/wifi/wf200/wf200.gni

+8-8
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,12 @@ import("${silabs_sdk_build_root}/efr32_sdk.gni")
1818

1919
wf200_plat_src = [
2020
"${chip_root}/src/platform/silabs/wifi/wf200/WifiInterface.cpp",
21-
"${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_wfx_task.c",
22-
"${chip_root}/src/platform/silabs/wifi/wf200/platform/wf200_init.c",
23-
"${chip_root}/src/platform/silabs/wifi/wf200/platform/efr_spi.c",
24-
"${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_wfx_board.h",
25-
"${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_wfx_host.h",
26-
"${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_wfx_task.h",
27-
"${chip_root}/src/platform/silabs/wifi/wf200/platform/sl_custom_board.h",
28-
"${chip_root}/src/platform/silabs/wifi/wf200/platform/spi_multiplex.h",
21+
"${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_wfx_task.c",
22+
"${chip_root}/src/platform/silabs/wifi/wf200/ncp/wf200_init.c",
23+
"${chip_root}/src/platform/silabs/wifi/wf200/ncp/efr_spi.c",
24+
"${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_wfx_board.h",
25+
"${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_wfx_host.h",
26+
"${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_wfx_task.h",
27+
"${chip_root}/src/platform/silabs/wifi/wf200/ncp/sl_custom_board.h",
28+
"${chip_root}/src/platform/silabs/wifi/ncp/spi_multiplex.h",
2929
]

third_party/silabs/efr32_sdk.gni

+1-4
Original file line numberDiff line numberDiff line change
@@ -370,10 +370,7 @@ template("efr32_sdk") {
370370

371371
if (use_rs9116) {
372372
# TODO: we should create a seperate directory for headers that are necessary for the underlying sdks
373-
_include_dirs += [
374-
"${chip_root}/src/platform/silabs/wifi/rs911x/platform",
375-
"${chip_root}/src/platform/silabs/wifi/rs911x",
376-
]
373+
_include_dirs += [ "${chip_root}/src/platform/silabs/wifi/rs911x/ncp" ]
377374
}
378375

379376
if (use_rs9116 || use_SiWx917) {

0 commit comments

Comments
 (0)