iwinfo: clean up rpc_iwinfo_call_hw_ht_mode()
authorAndre Heider <a.heider@gmail.com>
Wed, 23 Nov 2022 10:05:51 +0000 (11:05 +0100)
committerJo-Philipp Wich <jo@mein.io>
Thu, 15 Dec 2022 23:24:56 +0000 (00:24 +0100)
There's no need to call hwmodelist() again, just pass on the earlier
result.

Signed-off-by: Andre Heider <a.heider@gmail.com>
iwinfo.c

index d4177297e9acfa42ae777778e377de7b21fde5c2..cdbb8706f34eb8488e965966fc8742c3217f2e6d 100644 (file)
--- a/iwinfo.c
+++ b/iwinfo.c
@@ -241,30 +241,28 @@ rpc_iwinfo_call_htmodes(const char *name)
                                false, 0);
 }
 
-static void
+static int
 rpc_iwinfo_call_hwmodes(const char *name)
 {
        int modes;
 
        if (iw->hwmodelist(ifname, &modes))
-               return;
+               return -1;
 
        rpc_iwinfo_add_bit_array(name, modes,
                                IWINFO_80211_NAMES, IWINFO_80211_COUNT,
                                false, 0);
+
+       return modes;
 }
 
-static void rpc_iwinfo_call_hw_ht_mode()
+static void rpc_iwinfo_call_hw_ht_mode(int hwmodelist)
 {
        const char *hwmode_str;
        const char *htmode_str;
-       int32_t htmode = 0;
-       int modes;
+       int htmode;
 
-       if (iw->hwmodelist(ifname, &modes))
-               return;
-
-       if (modes == IWINFO_80211_AD)
+       if (hwmodelist == IWINFO_80211_AD)
        {
                blobmsg_add_string(&buf, "hwmode", "ad");
                return;
@@ -305,7 +303,7 @@ rpc_iwinfo_info(struct ubus_context *ctx, struct ubus_object *obj,
                 struct ubus_request_data *req, const char *method,
                 struct blob_attr *msg)
 {
-       int rv;
+       int rv, hwmodes;
        void *c;
 
        rv = rpc_iwinfo_open(msg);
@@ -342,9 +340,10 @@ rpc_iwinfo_info(struct ubus_context *ctx, struct ubus_object *obj,
 
        rpc_iwinfo_call_encryption("encryption");
        rpc_iwinfo_call_htmodes("htmodes");
-       rpc_iwinfo_call_hwmodes("hwmodes");
+       hwmodes = rpc_iwinfo_call_hwmodes("hwmodes");
 
-       rpc_iwinfo_call_hw_ht_mode();
+       if (hwmodes > 0)
+               rpc_iwinfo_call_hw_ht_mode(hwmodes);
 
        c = blobmsg_open_table(&buf, "hardware");
        rpc_iwinfo_call_hardware_id("id");