Skip to content

Commit 78830a0

Browse files
[Silabs] Wi-fi: Header and source dhcp files cleanup (#35143)
* dhcp client source and header files cleanup * Resolved build errors for Siwx917 BRD4338A board * Resolved build errors for efr boards * Resolved review comments * Build errors resolved * Added TO-DO cleanup for wifi files * Moved wifi include to common efr32_sdk.gni * Removed EFR platform headers from dhcp_client.cpp
1 parent 7d57f07 commit 78830a0

File tree

9 files changed

+21
-205
lines changed

9 files changed

+21
-205
lines changed

examples/platform/silabs/SiWx917/BUILD.gn

+5-2
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,10 @@ declare_args() {
5050

5151
# Sanity check
5252
assert(chip_enable_wifi)
53+
silabs_plat_dir = "${chip_root}/src/platform/silabs"
5354
silabs_plat_si91x_wifi_dir = "${chip_root}/src/platform/silabs/SiWx917/wifi"
54-
import("${silabs_common_plat_dir}/args.gni")
5555

56+
import("${silabs_common_plat_dir}/args.gni")
5657
config("chip_examples_project_config") {
5758
include_dirs = [ "project_include" ]
5859

@@ -181,9 +182,11 @@ source_set("siwx917-common") {
181182
include_dirs = [
182183
".",
183184
"SiWx917/",
185+
"${silabs_plat_dir}/wifi",
184186
"${silabs_plat_si91x_wifi_dir}",
185187
]
186188

189+
#TO-DO Cleanup to be done for all the wifi files into common folder
187190
sources = [
188191
"${silabs_common_plat_dir}/BaseApplication.cpp",
189192
"${silabs_common_plat_dir}/LEDWidget.cpp",
@@ -193,7 +196,7 @@ source_set("siwx917-common") {
193196
"${silabs_common_plat_dir}/syscalls_stubs.cpp",
194197
"${silabs_common_plat_dir}/wifi/wfx_notify.cpp",
195198
"${silabs_common_plat_dir}/wifi/wfx_rsi_host.cpp",
196-
"${silabs_plat_si91x_wifi_dir}/dhcp_client.cpp",
199+
"${silabs_plat_dir}/wifi/dhcp_client.cpp",
197200
"${silabs_plat_si91x_wifi_dir}/ethernetif.cpp",
198201
"${silabs_plat_si91x_wifi_dir}/lwip_netif.cpp",
199202
"SiWx917/sl_wifi_if.cpp",

examples/platform/silabs/efr32/wf200/host_if.cpp

+11-1
Original file line numberDiff line numberDiff line change
@@ -1197,9 +1197,19 @@ bool wfx_hw_ready(void)
11971197
******************************************************************************/
11981198
void wfx_dhcp_got_ipv4(uint32_t ip)
11991199
{
1200-
/* Acquire the new IP address
1200+
/*
1201+
* Acquire the new IP address
12011202
*/
1203+
uint8_t ip4_addr[4];
1204+
1205+
ip4_addr[0] = (ip) &HEX_VALUE_FF;
1206+
ip4_addr[1] = (ip >> 8) & HEX_VALUE_FF;
1207+
ip4_addr[2] = (ip >> 16) & HEX_VALUE_FF;
1208+
ip4_addr[3] = (ip >> 24) & HEX_VALUE_FF;
1209+
1210+
ChipLogDetail(DeviceLayer, "DHCP IP=%d.%d.%d.%d", ip4_addr[0], ip4_addr[1], ip4_addr[2], ip4_addr[3]);
12021211
sta_ip = ip;
1212+
12031213
wfx_ip_changed_notify(IP_STATUS_SUCCESS);
12041214
}
12051215
#endif /* CHIP_DEVICE_CONFIG_ENABLE_IPV4 */

src/platform/silabs/efr32/BUILD.gn

+1-2
Original file line numberDiff line numberDiff line change
@@ -166,8 +166,7 @@ static_library("efr32") {
166166

167167
if (use_wf200 || use_rs9116 || use_SiWx917) {
168168
sources += [
169-
"wifi/dhcp_client.cpp",
170-
"wifi/dhcp_client.h",
169+
"${silabs_platform_dir}/wifi/dhcp_client.cpp",
171170
"wifi/ethernetif.cpp",
172171
"wifi/ethernetif.h",
173172
"wifi/lwip_netif.cpp",

src/platform/silabs/efr32/wifi/dhcp_client.cpp

-150
This file was deleted.

src/platform/silabs/efr32/wifi/dhcp_client.h

-45
This file was deleted.

src/platform/silabs/SiWx917/wifi/dhcp_client.cpp src/platform/silabs/wifi/dhcp_client.cpp

-4
Original file line numberDiff line numberDiff line change
@@ -101,10 +101,6 @@ uint8_t dhcpclient_poll(void * arg)
101101
if (dhcp_supplied_address(netif))
102102
{
103103
dhcp_state = DHCP_ADDRESS_ASSIGNED;
104-
105-
uint64_t addr = netif->ip_addr.u_addr.ip4.addr;
106-
ChipLogProgress(DeviceLayer, "DHCP IP: %d.%d.%d.%d", NETIF_IPV4_ADDRESS(addr, 0), NETIF_IPV4_ADDRESS(addr, 1),
107-
NETIF_IPV4_ADDRESS(addr, 2), NETIF_IPV4_ADDRESS(addr, 3));
108104
}
109105
else
110106
{

third_party/silabs/SiWx917_sdk.gni

+1-1
Original file line numberDiff line numberDiff line change
@@ -256,7 +256,7 @@ template("siwx917_sdk") {
256256
}
257257

258258
defines += [ "LWIP_NETIF_API=1" ]
259-
if (lwip_ipv4) {
259+
if (chip_enable_wifi_ipv4) {
260260
defines += [
261261
"LWIP_IPV4=1",
262262

third_party/silabs/efr32_sdk.gni

+3
Original file line numberDiff line numberDiff line change
@@ -362,6 +362,9 @@ template("efr32_sdk") {
362362
if (use_rs9116 || use_SiWx917) {
363363
_include_dirs += [ "${chip_root}/src/platform/silabs/rs911x" ]
364364
}
365+
if (use_wf200 || use_rs9116 || use_SiWx917) {
366+
_include_dirs += [ "${chip_root}/src/platform/silabs/wifi" ]
367+
}
365368
if (silabs_family != "mgm24") {
366369
_include_dirs += [
367370
"${efr32_sdk_root}/platform/radio/rail_lib/hal",

0 commit comments

Comments
 (0)