staging: wfx: drop multicast filtering

The device allows to filter multicast frames. The driver has the
necessary code to take advantage of this feature. However, some bugs
has been reported on this feature. So, it was temporary disabled.

Finally, the things work well as-is for more than one year now. So there
is no plan to enable this feature in near future.

Since we dislike to maintain dead code, drop it.

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200907101521.66082-7-Jerome.Pouiller@silabs.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/wfx/hif_api_mib.h b/drivers/staging/wfx/hif_api_mib.h
index 6f1434795fa817320b9f336102d615ff7002dc92..d0e0a9e29afa6e0f814e11cb3873b5892de7b5af 100644 (file)
--- a/drivers/staging/wfx/hif_api_mib.h
+++ b/drivers/staging/wfx/hif_api_mib.h
@@ -82,42 +82,6 @@ struct hif_mib_gl_set_multi_msg {
        u8     reserved2[3];
 } __packed;
 
-enum hif_mac_addr_type {
-       HIF_MAC_ADDR_A1 = 0x0,
-       HIF_MAC_ADDR_A2 = 0x1,
-       HIF_MAC_ADDR_A3 = 0x2
-};
-
-struct hif_mib_mac_addr_data_frame_condition {
-       u8     condition_idx;
-       u8     address_type;
-       u8     mac_address[ETH_ALEN];
-} __packed;
-
-#define HIF_FILTER_UNICAST   0x1
-#define HIF_FILTER_MULTICAST 0x2
-#define HIF_FILTER_BROADCAST 0x4
-
-struct hif_mib_uc_mc_bc_data_frame_condition {
-       u8     condition_idx;
-       u8     allowed_frames;
-       u8     reserved[2];
-} __packed;
-
-struct hif_mib_config_data_filter {
-       u8     filter_idx;
-       u8     enable;
-       u8     reserved1[2];
-       u8     eth_type_cond;
-       u8     port_cond;
-       u8     magic_cond;
-       u8     mac_cond;
-       u8     ipv4_cond;
-       u8     ipv6_cond;
-       u8     uc_mc_bc_cond;
-       u8     reserved2;
-} __packed;
-
 struct hif_mib_set_data_filtering {
        u8     invert_matching:1;
        u8     reserved1:7;

diff --git a/drivers/staging/wfx/hif_tx_mib.c b/drivers/staging/wfx/hif_tx_mib.c
index 3b20b7486f0804b40b3262544b89a165c1cae9d6..7f24e9f77c2279a1dd43b924d090bf564694b8b8 100644 (file)
--- a/drivers/staging/wfx/hif_tx_mib.c
+++ b/drivers/staging/wfx/hif_tx_mib.c
@@ -228,46 +228,6 @@ int hif_set_tx_rate_retry_policy(struct wfx_vif *wvif,
        return ret;
 }
 
-int hif_set_mac_addr_condition(struct wfx_vif *wvif,
-                              int idx, const u8 *mac_addr)
-{
-       struct hif_mib_mac_addr_data_frame_condition val = {
-               .condition_idx = idx,
-               .address_type = HIF_MAC_ADDR_A1,
-       };
-
-       ether_addr_copy(val.mac_address, mac_addr);
-       return hif_write_mib(wvif->wdev, wvif->id,
-                            HIF_MIB_ID_MAC_ADDR_DATAFRAME_CONDITION,
-                            &val, sizeof(val));
-}
-
-int hif_set_uc_mc_bc_condition(struct wfx_vif *wvif, int idx, u8 allowed_frames)
-{
-       struct hif_mib_uc_mc_bc_data_frame_condition val = {
-               .condition_idx = idx,
-               .allowed_frames = allowed_frames,
-       };
-
-       return hif_write_mib(wvif->wdev, wvif->id,
-                            HIF_MIB_ID_UC_MC_BC_DATAFRAME_CONDITION,
-                            &val, sizeof(val));
-}
-
-int hif_set_config_data_filter(struct wfx_vif *wvif, bool enable, int idx,
-                              int mac_filters, int frames_types_filters)
-{
-       struct hif_mib_config_data_filter val = {
-               .enable = enable,
-               .filter_idx = idx,
-               .mac_cond = mac_filters,
-               .uc_mc_bc_cond = frames_types_filters,
-       };
-
-       return hif_write_mib(wvif->wdev, wvif->id,
-                            HIF_MIB_ID_CONFIG_DATA_FILTER, &val, sizeof(val));
-}
-
 int hif_set_data_filtering(struct wfx_vif *wvif, bool enable, bool invert)
 {
        struct hif_mib_set_data_filtering val = {

diff --git a/drivers/staging/wfx/hif_tx_mib.h b/drivers/staging/wfx/hif_tx_mib.h
index 1a6f4221bdeb84a39b40bb111078a822a216e7db..d4cac63164baa7d3fd84e7684ea012064c6b7f12 100644 (file)
--- a/drivers/staging/wfx/hif_tx_mib.h
+++ b/drivers/staging/wfx/hif_tx_mib.h
@@ -37,12 +37,6 @@ int hif_set_association_mode(struct wfx_vif *wvif, int ampdu_density,
                             bool greenfield, bool short_preamble);
 int hif_set_tx_rate_retry_policy(struct wfx_vif *wvif,
                                 int policy_index, u8 *rates);
-int hif_set_mac_addr_condition(struct wfx_vif *wvif,
-                              int idx, const u8 *mac_addr);
-int hif_set_uc_mc_bc_condition(struct wfx_vif *wvif,
-                              int idx, u8 allowed_frames);
-int hif_set_config_data_filter(struct wfx_vif *wvif, bool enable, int idx,
-                              int mac_filters, int frames_types_filters);
 int hif_set_data_filtering(struct wfx_vif *wvif, bool enable, bool invert);
 int hif_keep_alive_period(struct wfx_vif *wvif, int period);
 int hif_set_arp_ipv4_filter(struct wfx_vif *wvif, int idx, __be32 *addr);

diff --git a/drivers/staging/wfx/main.c b/drivers/staging/wfx/main.c
index 2af4914e905c5a13da992c764712d4daec3af6e6..1017a2290f08a5fbd537d49f54a057eb7e2bbd4e 100644 (file)
--- a/drivers/staging/wfx/main.c
+++ b/drivers/staging/wfx/main.c
@@ -142,7 +142,6 @@ static const struct ieee80211_ops wfx_ops = {
        .set_rts_threshold      = wfx_set_rts_threshold,
        .set_default_unicast_key = wfx_set_default_unicast_key,
        .bss_info_changed       = wfx_bss_info_changed,
-       .prepare_multicast      = wfx_prepare_multicast,
        .configure_filter       = wfx_configure_filter,
        .ampdu_action           = wfx_ampdu_action,
        .flush                  = wfx_flush,

diff --git a/drivers/staging/wfx/sta.c b/drivers/staging/wfx/sta.c
index feebb7c3bdfe35544a27ea5cf43589a3d3e34856..8700d2fc6a7739899822a0300daec5327b1a8b47 100644 (file)
--- a/drivers/staging/wfx/sta.c
+++ b/drivers/staging/wfx/sta.c
@@ -91,59 +91,12 @@ static void wfx_filter_beacon(struct wfx_vif *wvif, bool filter_beacon)
        }
 }
 
-static void wfx_filter_mcast(struct wfx_vif *wvif, bool filter_mcast)
-{
-       int i;
-
-       // Temporary workaround for filters
-       hif_set_data_filtering(wvif, false, true);
-       return;
-
-       if (!filter_mcast) {
-               hif_set_data_filtering(wvif, false, true);
-               return;
-       }
-       for (i = 0; i < wvif->filter_mcast_count; i++)
-               hif_set_mac_addr_condition(wvif, i, wvif->filter_mcast_addr[i]);
-       hif_set_uc_mc_bc_condition(wvif, 0,
-                                  HIF_FILTER_UNICAST | HIF_FILTER_BROADCAST);
-       hif_set_config_data_filter(wvif, true, 0, BIT(1),
-                                  BIT(wvif->filter_mcast_count) - 1);
-       hif_set_data_filtering(wvif, true, true);
-}
-
-u64 wfx_prepare_multicast(struct ieee80211_hw *hw,
-                         struct netdev_hw_addr_list *mc_list)
-{
-       int i;
-       struct netdev_hw_addr *ha;
-       struct wfx_vif *wvif = NULL;
-       struct wfx_dev *wdev = hw->priv;
-       int count = netdev_hw_addr_list_count(mc_list);
-
-       while ((wvif = wvif_iterate(wdev, wvif)) != NULL) {
-               if (count > ARRAY_SIZE(wvif->filter_mcast_addr)) {
-                       wvif->filter_mcast_count = 0;
-                       continue;
-               }
-               wvif->filter_mcast_count = count;
-
-               i = 0;
-               netdev_hw_addr_list_for_each(ha, mc_list) {
-                       ether_addr_copy(wvif->filter_mcast_addr[i], ha->addr);
-                       i++;
-               }
-       }
-
-       return 0;
-}
-
 void wfx_configure_filter(struct ieee80211_hw *hw, unsigned int changed_flags,
                          unsigned int *total_flags, u64 unused)
 {
        struct wfx_vif *wvif = NULL;
        struct wfx_dev *wdev = hw->priv;
-       bool filter_bssid, filter_prbreq, filter_beacon, filter_mcast;
+       bool filter_bssid, filter_prbreq, filter_beacon;
 
        // Notes:
        //   - Probe responses (FIF_BCN_PRBRESP_PROMISC) are never filtered
@@ -167,15 +120,7 @@ void wfx_configure_filter(struct ieee80211_hw *hw, unsigned int changed_flags,
                        filter_beacon = true;
                wfx_filter_beacon(wvif, filter_beacon);
 
-               if (*total_flags & FIF_ALLMULTI) {
-                       filter_mcast = false;
-               } else if (!wvif->filter_mcast_count) {
-                       dev_dbg(wdev->dev, "disabling unconfigured multicast filter");
-                       filter_mcast = false;
-               } else {
-                       filter_mcast = true;
-               }
-               wfx_filter_mcast(wvif, filter_mcast);
+               hif_set_data_filtering(wvif, false, true);
 
                if (*total_flags & FIF_OTHER_BSS)
                        filter_bssid = false;

diff --git a/drivers/staging/wfx/sta.h b/drivers/staging/wfx/sta.h
index 6b15a64ac9e28e251cf9a75314cafeca72c81419..610cfb0fcd02c6e3c01fcc5107d99fde7976fdfb 100644 (file)
--- a/drivers/staging/wfx/sta.h
+++ b/drivers/staging/wfx/sta.h
@@ -25,8 +25,6 @@ int wfx_config(struct ieee80211_hw *hw, u32 changed);
 int wfx_set_rts_threshold(struct ieee80211_hw *hw, u32 value);
 void wfx_set_default_unicast_key(struct ieee80211_hw *hw,
                                 struct ieee80211_vif *vif, int idx);
-u64 wfx_prepare_multicast(struct ieee80211_hw *hw,
-                         struct netdev_hw_addr_list *mc_list);
 void wfx_configure_filter(struct ieee80211_hw *hw, unsigned int changed_flags,
                          unsigned int *total_flags, u64 unused);
 

diff --git a/drivers/staging/wfx/wfx.h b/drivers/staging/wfx/wfx.h
index 241eaaf71f5ef9d40788ba2c958da110df6a3b15..56fbfab056511e717788138421a1ee2992b8f78c 100644 (file)
--- a/drivers/staging/wfx/wfx.h
+++ b/drivers/staging/wfx/wfx.h
@@ -79,9 +79,6 @@ struct wfx_vif {
 
        struct work_struct      update_tim_work;
 
-       int                     filter_mcast_count;
-       u8                      filter_mcast_addr[8][ETH_ALEN];
-
        unsigned long           uapsd_mask;
 
        /* avoid some operations in parallel with scan */