switch_chan method now could be called directly by hostapd_iface object.

Signed-off-by: Yury Shvedov <yshve...@wimarksystems.com>
---
 .../services/hostapd/src/src/ap/ubus.c        | 145 +++++++++++-------
 1 file changed, 86 insertions(+), 59 deletions(-)

diff --git a/package/network/services/hostapd/src/src/ap/ubus.c 
b/package/network/services/hostapd/src/src/ap/ubus.c
index a393451af2..6d12126a1a 100644
--- a/package/network/services/hostapd/src/src/ap/ubus.c
+++ b/package/network/services/hostapd/src/src/ap/ubus.c
@@ -145,9 +145,88 @@ hostapd_iface_get_bss(struct ubus_context *ctx, struct 
ubus_object *obj,
 
        return 0;
 }
+
+#ifdef NEED_AP_MLME
+enum {
+       CSA_FREQ,
+       CSA_BCN_COUNT,
+       CSA_CENTER_FREQ1,
+       CSA_CENTER_FREQ2,
+       CSA_BANDWIDTH,
+       CSA_SEC_CHANNEL_OFFSET,
+       CSA_HT,
+       CSA_VHT,
+       CSA_BLOCK_TX,
+       __CSA_MAX
+};
+
+static const struct blobmsg_policy csa_policy[__CSA_MAX] = {
+       [CSA_FREQ] = { "freq", BLOBMSG_TYPE_INT32 },
+       [CSA_BCN_COUNT] = { "bcn_count", BLOBMSG_TYPE_INT32 },
+       [CSA_CENTER_FREQ1] = { "center_freq1", BLOBMSG_TYPE_INT32 },
+       [CSA_CENTER_FREQ2] = { "center_freq2", BLOBMSG_TYPE_INT32 },
+       [CSA_BANDWIDTH] = { "bandwidth", BLOBMSG_TYPE_INT32 },
+       [CSA_SEC_CHANNEL_OFFSET] = { "sec_channel_offset", BLOBMSG_TYPE_INT32 },
+       [CSA_HT] = { "ht", BLOBMSG_TYPE_BOOL },
+       [CSA_VHT] = { "vht", BLOBMSG_TYPE_BOOL },
+       [CSA_BLOCK_TX] = { "block_tx", BLOBMSG_TYPE_BOOL },
+};
+
+static int
+hostapd_switch_chan(struct hostapd_data *hapd, struct blob_attr *msg)
+{
+       struct blob_attr *tb[__CSA_MAX];
+       struct csa_settings css;
+
+       blobmsg_parse(csa_policy, __CSA_MAX, tb, blob_data(msg), blob_len(msg));
+
+       if (!tb[CSA_FREQ])
+               return UBUS_STATUS_INVALID_ARGUMENT;
+
+       memset(&css, 0, sizeof(css));
+       css.freq_params.freq = blobmsg_get_u32(tb[CSA_FREQ]);
+
+#define SET_CSA_SETTING(name, field, type) \
+       do { \
+               if (tb[name]) \
+                       css.field = blobmsg_get_ ## type(tb[name]); \
+       } while(0)
+
+       SET_CSA_SETTING(CSA_BCN_COUNT, cs_count, u32);
+       SET_CSA_SETTING(CSA_CENTER_FREQ1, freq_params.center_freq1, u32);
+       SET_CSA_SETTING(CSA_CENTER_FREQ2, freq_params.center_freq2, u32);
+       SET_CSA_SETTING(CSA_BANDWIDTH, freq_params.bandwidth, u32);
+       SET_CSA_SETTING(CSA_SEC_CHANNEL_OFFSET, freq_params.sec_channel_offset, 
u32);
+       SET_CSA_SETTING(CSA_HT, freq_params.ht_enabled, bool);
+       SET_CSA_SETTING(CSA_VHT, freq_params.vht_enabled, bool);
+       SET_CSA_SETTING(CSA_BLOCK_TX, block_tx, bool);
+
+
+       if (hostapd_switch_channel(hapd, &css) != 0)
+               return UBUS_STATUS_NOT_SUPPORTED;
+       return UBUS_STATUS_OK;
+#undef SET_CSA_SETTING
+}
+
+static int
+hostapd_iface_switch_chan(struct ubus_context *ctx, struct ubus_object *obj,
+                       struct ubus_request_data *req, const char *method,
+                       struct blob_attr *msg)
+{
+       struct hostapd_iface *iface = container_of(obj, struct hostapd_iface,
+                       ubus.obj);
+       if (iface && iface->bss[0])
+               return hostapd_switch_chan(iface->bss[0], msg);
+       return UBUS_STATUS_INVALID_ARGUMENT;
+}
+#endif
+
 static const struct ubus_method iface_methods[] = {
        UBUS_METHOD_NOARG("get_state", hostapd_iface_get_state),
        UBUS_METHOD_NOARG("get_bss", hostapd_iface_get_bss),
+#ifdef NEED_AP_MLME
+       UBUS_METHOD("switch_chan", hostapd_iface_switch_chan, csa_policy),
+#endif
 };
 static struct ubus_object_type iface_object_type =
        UBUS_OBJECT_TYPE("hostapd_iface", iface_methods);
@@ -472,69 +551,17 @@ hostapd_bss_update_beacon(struct ubus_context *ctx, 
struct ubus_object *obj,
        return 0;
 }
 
-enum {
-       CSA_FREQ,
-       CSA_BCN_COUNT,
-       CSA_CENTER_FREQ1,
-       CSA_CENTER_FREQ2,
-       CSA_BANDWIDTH,
-       CSA_SEC_CHANNEL_OFFSET,
-       CSA_HT,
-       CSA_VHT,
-       CSA_BLOCK_TX,
-       __CSA_MAX
-};
-
-static const struct blobmsg_policy csa_policy[__CSA_MAX] = {
-       [CSA_FREQ] = { "freq", BLOBMSG_TYPE_INT32 },
-       [CSA_BCN_COUNT] = { "bcn_count", BLOBMSG_TYPE_INT32 },
-       [CSA_CENTER_FREQ1] = { "center_freq1", BLOBMSG_TYPE_INT32 },
-       [CSA_CENTER_FREQ2] = { "center_freq2", BLOBMSG_TYPE_INT32 },
-       [CSA_BANDWIDTH] = { "bandwidth", BLOBMSG_TYPE_INT32 },
-       [CSA_SEC_CHANNEL_OFFSET] = { "sec_channel_offset", BLOBMSG_TYPE_INT32 },
-       [CSA_HT] = { "ht", BLOBMSG_TYPE_BOOL },
-       [CSA_VHT] = { "vht", BLOBMSG_TYPE_BOOL },
-       [CSA_BLOCK_TX] = { "block_tx", BLOBMSG_TYPE_BOOL },
-};
 
 #ifdef NEED_AP_MLME
 static int
-hostapd_switch_chan(struct ubus_context *ctx, struct ubus_object *obj,
-                   struct ubus_request_data *req, const char *method,
-                   struct blob_attr *msg)
+hostapd_bss_switch_chan(struct ubus_context *ctx, struct ubus_object *obj,
+                       struct ubus_request_data *req, const char *method,
+                       struct blob_attr *msg)
 {
-       struct blob_attr *tb[__CSA_MAX];
        struct hostapd_data *hapd = get_hapd_from_object(obj);
-       struct csa_settings css;
-
-       blobmsg_parse(csa_policy, __CSA_MAX, tb, blob_data(msg), blob_len(msg));
-
-       if (!tb[CSA_FREQ])
-               return UBUS_STATUS_INVALID_ARGUMENT;
-
-       memset(&css, 0, sizeof(css));
-       css.freq_params.freq = blobmsg_get_u32(tb[CSA_FREQ]);
-
-#define SET_CSA_SETTING(name, field, type) \
-       do { \
-               if (tb[name]) \
-                       css.field = blobmsg_get_ ## type(tb[name]); \
-       } while(0)
-
-       SET_CSA_SETTING(CSA_BCN_COUNT, cs_count, u32);
-       SET_CSA_SETTING(CSA_CENTER_FREQ1, freq_params.center_freq1, u32);
-       SET_CSA_SETTING(CSA_CENTER_FREQ2, freq_params.center_freq2, u32);
-       SET_CSA_SETTING(CSA_BANDWIDTH, freq_params.bandwidth, u32);
-       SET_CSA_SETTING(CSA_SEC_CHANNEL_OFFSET, freq_params.sec_channel_offset, 
u32);
-       SET_CSA_SETTING(CSA_HT, freq_params.ht_enabled, bool);
-       SET_CSA_SETTING(CSA_VHT, freq_params.vht_enabled, bool);
-       SET_CSA_SETTING(CSA_BLOCK_TX, block_tx, bool);
-
-
-       if (hostapd_switch_channel(hapd, &css) != 0)
-               return UBUS_STATUS_NOT_SUPPORTED;
-       return UBUS_STATUS_OK;
-#undef SET_CSA_SETTING
+       if (hapd)
+               return hostapd_switch_chan(hapd, msg);
+       return UBUS_STATUS_INVALID_ARGUMENT;
 }
 #endif
 
@@ -1050,7 +1077,7 @@ static const struct ubus_method bss_methods[] = {
        UBUS_METHOD_NOARG("update_beacon", hostapd_bss_update_beacon),
        UBUS_METHOD_NOARG("get_features", hostapd_bss_get_features),
 #ifdef NEED_AP_MLME
-       UBUS_METHOD("switch_chan", hostapd_switch_chan, csa_policy),
+       UBUS_METHOD("switch_chan", hostapd_bss_switch_chan, csa_policy),
 #endif
        UBUS_METHOD("set_vendor_elements", hostapd_vendor_elements, ve_policy),
        UBUS_METHOD("notify_response", hostapd_notify_response, notify_policy),
-- 
2.19.0


_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to