commit 12866f10d9c2cdec2c6faba958b99a726f5ec120
author developer <developer@mediatek.com> Thu Aug 31 16:35:38 2023 +0800
committer developer <developer@mediatek.com> Thu Aug 31 16:40:45 2023 +0800
tree 6a34c7d10ae320c6aa26d720730dc702765ffc1f
parent 6bfd646d27af53293499cfeedb1439096a6a2706

[rdkb][common][hal][Fix dmcli disable ap and enable radio failed issue]

[Description]
Fix dmcli wifi_createVAP without check disable setting.
Fix setRadioOperatingParameters call setRadio11nGreenfieldEnable get error due to mt76 not support.

[Release-log]
N/A

Change-Id: Ib01a13c4dd97670866cd930153d3e316fa717ba0

src/wifi/wifi_hal.c

diff --git a/src/wifi/wifi_hal.c b/src/wifi/wifi_hal.c
index d6e9969..df13fc9 100644
--- a/src/wifi/wifi_hal.c
+++ b/src/wifi/wifi_hal.c
@@ -12362,12 +12362,14 @@
             return RETURN_ERR;
         }
     }
-    if (current_param.greenFieldEnable != operationParam->greenFieldEnable) {
-        if (wifi_setRadio11nGreenfieldEnable(index, operationParam->greenFieldEnable) != RETURN_OK) {
-            fprintf(stderr, "%s: wifi_setRadio11nGreenfieldEnable return error.\n", __func__);
-            return RETURN_ERR;
-        }
-    }
+
+    // mt76 not support yet
+    // if (current_param.greenFieldEnable != operationParam->greenFieldEnable) {
+    //     if (wifi_setRadio11nGreenfieldEnable(index, operationParam->greenFieldEnable) != RETURN_OK) {
+    //         fprintf(stderr, "%s: wifi_setRadio11nGreenfieldEnable return error.\n", __func__);
+    //         return RETURN_ERR;
+    //     }
+    // }
 
     // if enable is true, then restart the radio
     wifi_setRadioEnable(index, FALSE);
@@ -12536,10 +12538,12 @@
     else
         operationParam->stbcEnable = FALSE;
 
-    if (wifi_getRadio11nGreenfieldEnable(index, &operationParam->greenFieldEnable) != RETURN_OK) {
-        fprintf(stderr, "%s: wifi_getRadio11nGreenfieldEnable return error.\n", __func__);
-        return RETURN_ERR;
-    }
+    // mt76 not support yet
+    operationParam->greenFieldEnable = 0;
+    // if (wifi_getRadio11nGreenfieldEnable(index, &operationParam->greenFieldEnable) != RETURN_OK) {
+    //     fprintf(stderr, "%s: wifi_getRadio11nGreenfieldEnable return error.\n", __func__);
+    //     return RETURN_ERR;
+    // }
 
     // Below value is hardcoded
 
@@ -12979,7 +12983,8 @@
         }
 
         wifi_setApEnable(vap_info->vap_index, FALSE);
-        wifi_setApEnable(vap_info->vap_index, TRUE);
+        if (vap_info->u.bss_info.enabled == TRUE)
+            wifi_setApEnable(vap_info->vap_index, TRUE);
         multiple_set = FALSE;
 
         // If config use hostapd_cli to set, we calling these type of functions after enable the ap.