Resolved review comments

Author: bhmanda-silabs

examples/platform/silabs/MatterConfig.cpp

@@ -46,12 +46,14 @@
 #endif
 
 // TODO: We shouldn't need any platform specific includes in this file
-#if ((defined(SLI_SI91X_MCU_INTERFACE) && SLI_SI91X_MCU_INTERFACE == 1) || defined(EXP_BOARD))
-#include <platform/silabs/wifi/wiseconnect-interface/WiseconnectWifiInterface.h>
-#if !defined(EXP_BOARD)
+#if (defined(SLI_SI91X_MCU_INTERFACE) && SLI_SI91X_MCU_INTERFACE == 1)
 #include <platform/silabs/SiWx917/SiWxPlatformInterface.h>
-#endif //! defined(EXP_BOARD)
-#endif //((defined(SLI_SI91X_MCU_INTERFACE) && SLI_SI91X_MCU_INTERFACE == 1 ) || defined(EXP_BOARD))
+#include <platform/silabs/wifi/wiseconnect-interface/WiseconnectWifiInterface.h>
+#endif // (defined(SLI_SI91X_MCU_INTERFACE) && SLI_SI91X_MCU_INTERFACE == 1 )
+
+#if defined(EXP_BOARD))
+#include <platform/silabs/wifi/wiseconnect-interface/WiseconnectWifiInterface.h>
+#endif // defined(EXP_BOARD)
 
 #include <crypto/CHIPCryptoPAL.h>
 // If building with the EFR32-provided crypto backend, we can use the

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

@@ -314,7 +314,6 @@ sl_status_t sl_wifi_siwx917_init(void)
     return status;
 }
 
-// TODO: this changes will be reverted back after the Silabs WiFi SDK team fix the scan API
 sl_status_t ScanCallback(sl_wifi_event_t event, sl_wifi_scan_result_t * scan_result, uint32_t result_length, void * arg)
 {
     sl_status_t status = SL_STATUS_OK;
@@ -344,10 +343,8 @@ sl_status_t InitiateScan()
 {
     sl_status_t status = SL_STATUS_OK;
 
-    // TODO: this changes will be reverted back after the Silabs WiFi SDK team fix the scan API
     sl_wifi_ssid_t ssid = { 0 };
 
-    // TODO: this changes will be reverted back after the Silabs WiFi SDK team fix the scan API
     sl_wifi_scan_configuration_t wifi_scan_configuration = default_wifi_scan_configuration;
 
     ssid.length = wfx_rsi.sec.ssid_length;