diff mbox

wlcore: let AP support allmulticast for MDNS. It can be enabled by bringing up the interface with ip command with the argument allmulticast on

Message ID 20170130125417.6188-1-i-hunter1@ti.com (mailing list archive)
State Changes Requested
Delegated to: Kalle Valo
Headers show

Commit Message

Iain Hunter Jan. 30, 2017, 12:54 p.m. UTC
Let AP support allmulticast for MDNS.
It can be enabled by bringing up the interface with ip command with the argument allmulticast on

Signed-off-by: Iain Hunter <i-hunter1@ti.com>
---
 drivers/net/wireless/ti/wlcore/main.c | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Kalle Valo Jan. 30, 2017, 2:24 p.m. UTC | #1
Iain Hunter <drhunter95@gmail.com> writes:

> Let AP support allmulticast for MDNS.
> It can be enabled by bringing up the interface with ip command with the argument allmulticast on
>
> Signed-off-by: Iain Hunter <i-hunter1@ti.com>

I see your patch now in patchwork (twice actually):

https://patchwork.kernel.org/patch/9545223/

But there's a problem with the title, you just copied the commit log as
the title. The format should be something like this (also please word
wrap the commit log):

----------------------------------------------------------------------
wlcore: disable firmware multicast filter in AP mode

Let AP support allmulticast for MDNS. It can be enabled by bringing up
the interface with ip command with the argument allmulticast on.

Signed-off-by: Ed Example <ed@example.org>
----------------------------------------------------------------------

You can see examples from patchwork, like this iwlwifi patch:

https://patchwork.kernel.org/patch/9544763/
diff mbox

Patch

diff --git a/drivers/net/wireless/ti/wlcore/main.c b/drivers/net/wireless/ti/wlcore/main.c
index 3241e9eba73..1ab0561ab13 100644
--- a/drivers/net/wireless/ti/wlcore/main.c
+++ b/drivers/net/wireless/ti/wlcore/main.c
@@ -3281,6 +3281,20 @@  static void wl1271_op_configure_filter(struct ieee80211_hw *hw,
 			if (ret < 0)
 				goto out_sleep;
 		}
+
+        /*
+         * If interface in AP mode and created with allmulticast then disable
+         * the firmware filters so that all multicast packets are passed
+         * This is mandatory for MDNS based discovery protocols 
+         */
+ 		if (wlvif->bss_type == BSS_TYPE_AP_BSS) {
+ 			if (*total & FIF_ALLMULTI)
+				ret = wl1271_acx_group_address_tbl(wl, wlvif,
+								   false,
+								   NULL, 0);
+              
+                }
+
 	}
 
 	/*