drivers: wifi: nrf7002: Fix formatting and style issues

This commit addresses code formatting and styling issues in the nRF7002 Wi-Fi driver based on review feedback. It includes fixes for inconsistent indentation, spacing, and general cleanup to align with Zephyr's coding guidelines and driver conventions. These changes are non-functional and do not affect the behavior of the driver, but improve code readability and maintainability.

Signed-off-by: Signed-off-by: Hanan Arshad <[email protected]>

Author: hanan619

drivers/wifi/nrf_wifi/Kconfig.nrfwifi

@@ -83,14 +83,13 @@ config NRF70_AP_MODE
 	default y if WIFI_NM_WPA_SUPPLICANT_AP
 
 config NRF70_WIFI_ENABLE_DUAL_VIF
-    bool "Enable dual virtual Wi-Fi interfaces"
-    default y if WIFI_NM_MAX_MANAGED_INTERFACES = 2
-    default n
-    depends on WIFI_NRF7002 && NET_L2_ETHERNET
-    help
-      Enable support for two virtual Wi-Fi interfaces (VIFs).
-      When enabled, the driver can operate two VIFs simultaneously,
-      allowing use cases such as one interface in AP mode and another in STA mode.
+  bool "Dual virtual Wi-Fi interfaces"
+  default y if WIFI_NM_MAX_MANAGED_INTERFACES = 2
+  depends on WIFI_NRF7002 && NET_L2_ETHERNET
+  help
+    Enable support for two virtual Wi-Fi interfaces (VIFs).
+    When enabled, the driver can operate two VIFs simultaneously,
+    allowing use cases such as one interface in AP mode and another in STA mode.
 
 config NRF70_P2P_MODE
 	bool "P2P support in driver"

drivers/wifi/nrf_wifi/src/fmac_main.c

@@ -446,11 +446,12 @@ void nrf_wifi_event_proc_cookie_rsp(void *vif_ctx,
 		cookie_rsp_event->mac_addr[5]);
 
 	/* Notify all vif */
-	for(int idx = 0; idx < vif_ctx_cnt; ++idx) {
+	for (int idx = 0; idx < vif_ctx_cnt; ++idx) {
 		vif_ctx_zep = nrf_wifi_get_vif_ctx_by_idx(idx);
-		if(vif_ctx_zep)
+		if (vif_ctx_zep) {
 			vif_ctx_zep->cookie_resp_received = true;
-	}	
+		}
+	}
 	/* TODO: When supp_callbk_fns.mgmt_tx_status is implemented, add logic
 	 * here to use the cookie and host_cookie to map requests to responses.
 	 */
@@ -726,13 +727,13 @@ static int nrf_wifi_drv_main_zep(const struct device *dev)
 	struct nrf_wifi_ctx_zep *rpu_ctx_zep = &rpu_drv_priv_zep.rpu_ctx_zep;
 	unsigned int vif_ctx_cnt = nrf_wifi_fmac_get_num_vifs(rpu_ctx_zep->rpu_ctx);
 
-	if(vif_ctx_cnt >= MAX_NUM_VIFS){
+	if (vif_ctx_cnt >= MAX_NUM_VIFS) {
 		LOG_ERR("%s: Max number of VIFs reached", __func__);
 		return -ENOMEM;
 	}
 
 	if (vif_ctx_cnt > 1) {
-		// FMAC is already initialized for VIF-0
+		/* FMAC is already initialized for VIF-0 */
 		return 0;
 	}
 
@@ -962,11 +963,11 @@ ETH_NET_DEVICE_DT_INST_DEFINE(0,
 		    &wifi_offload_ops, /* api */
 		    CONFIG_NRF_WIFI_IFACE_MTU); /*mtu */
 #ifdef CONFIG_NRF70_WIFI_ENABLE_DUAL_VIF
-// Register second interface
+/* Register second interface */
 ETH_NET_DEVICE_DT_INST_DEFINE(1,
-				nrf_wifi_drv_main_zep, /* init_fn */
-				NULL, /* pm_action_cb */
-				&rpu_drv_priv_zep.rpu_ctx_zep.vif_ctx_zep[1], /* data */
+		    nrf_wifi_drv_main_zep, /* init_fn */
+		    NULL, /* pm_action_cb */
+		    &rpu_drv_priv_zep.rpu_ctx_zep.vif_ctx_zep[1], /* data */
 #ifdef CONFIG_NRF70_STA_MODE
 		    &wpa_supp_ops, /* cfg */
 #else /* CONFIG_NRF70_STA_MODE */

drivers/wifi/nrf_wifi/src/net_if.c

@@ -63,10 +63,9 @@ void nrf_wifi_set_iface_event_handler(void *os_vif_ctx,
 	vif_ctx_cnt = nrf_wifi_fmac_get_num_vifs(rpu_ctx_zep->rpu_ctx);
 
 	/* Notify all vif */
-	for(int idx = 0; idx < vif_ctx_cnt; ++idx) {
+	for (int idx = 0; idx < vif_ctx_cnt; ++idx) {
 		vif_ctx_zep = nrf_wifi_get_vif_ctx_by_idx(idx);
-		if(vif_ctx_zep)
-		{
+		if (vif_ctx_zep) {
 			vif_ctx_zep->set_if_event_received = true;
 			vif_ctx_zep->set_if_status = event->return_value;
 		}

drivers/wifi/nrf_wifi/src/wpa_supp_if.c

@@ -443,8 +443,8 @@ void *nrf_wifi_wpa_supp_dev_init(void *supp_drv_if_ctx, const char *iface_name,
 				 struct zep_wpa_supp_dev_callbk_fns *supp_callbk_fns)
 {
 	struct nrf_wifi_vif_ctx_zep *vif_ctx_zep = NULL;
-	// Get device for Each interface
-	const int if_idx = net_if_get_by_name(iface_name);
+	/* Get device for Each interface */
+	int if_idx = net_if_get_by_name(iface_name);
 	struct net_if* iface = net_if_get_by_index(if_idx);
 	const struct device *device = net_if_get_device(iface);