commit 514406b4461d3896689ae2202614d0a632b5547b
author developer <developer@mediatek.com> Mon Dec 05 17:20:21 2022 +0800
committer developer <developer@mediatek.com> Mon Dec 05 17:53:42 2022 +0800
tree 4244b32dcaeb013672e1b9365e3a9bc4b1988bd3
parent 3042373b5a63cf8720aa3e759b10cd7545746864

[rdkb][common][hal][Refactor setApRtsThreshold]

[Description]
Refactor setApRtsThreshold to use hostapd to control.

[Release-log]
N/A

Change-Id: Ic5407b457579403ec8176ec2fbe16fe392f44d78

src/wifi/wifi_hal.c

diff --git a/src/wifi/wifi_hal.c b/src/wifi/wifi_hal.c
index 88bebf3..d308918 100644
--- a/src/wifi/wifi_hal.c
+++ b/src/wifi/wifi_hal.c
@@ -5004,21 +5004,29 @@
     //save config and apply instantly
     if (NULL == output_bool)
         return RETURN_ERR;
-    *output_bool = FALSE;
+    *output_bool = TRUE;
     return RETURN_OK;
 }
 
 // sets the packet size threshold in bytes to apply RTS/CTS backoff rules.
 INT wifi_setApRtsThreshold(INT apIndex, UINT threshold)
 {
-    char cmd[128];
-    char buf[512];
+    char buf[16] = {0};
+    char config_file[128] = {0};
+    struct params param = {0};
+
+    if (threshold > 65535) {
+        fprintf(stderr, "%s: rts threshold %u is too big.\n", __func__, threshold);
+        return RETURN_ERR;
+    }
 
-    if (threshold > 0)
-        snprintf(cmd, sizeof(cmd), "iwconfig %s%d rts %d", AP_PREFIX, apIndex, threshold);
-    else
-        snprintf(cmd, sizeof(cmd), "iwconfig %s%d rts off", AP_PREFIX, apIndex);
-    _syscmd(cmd, buf, sizeof(buf));
+    snprintf(config_file, sizeof(config_file), "%s%d.conf", AP_PREFIX, apIndex);
+    snprintf(buf, sizeof(buf), "%u", threshold);
+    param.name = "rts_threshold";
+    param.value = buf;
+    wifi_hostapdWrite(config_file, &param, 1);
+    wifi_hostapdProcessUpdate(apIndex, &param, 1);
+    wifi_reloadAp(apIndex);
 
     return RETURN_OK;
 }