Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5eb5a52d authored by Johannes Berg's avatar Johannes Berg Committed by John W. Linville
Browse files

mac80211: fix mesh advertisement 



When a mac80211-based driver advertises mesh mode
support, this will be advertised to userspace.
However, if mac80211 was compiled without mesh
support, then that won't actually be true. Fix
this by removing the bit for mesh if mesh isn't
compiled in.

Since this synchronizes what we advertise to
cfg80211 and actually support, it means we can
now rely on cfg80211's interface type checks
and need not check again in mac80211.

Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 45f7e311

net/mac80211/cfg.c

+0 −33
Original line number Diff line number Diff line
@@ -19,33 +19,6 @@ 
#include "rate.h"

#include "mesh.h"



static bool nl80211_type_check(enum nl80211_iftype type)

{

	switch (type) {

	case NL80211_IFTYPE_ADHOC:

	case NL80211_IFTYPE_STATION:

	case NL80211_IFTYPE_MONITOR:

#ifdef CONFIG_MAC80211_MESH

	case NL80211_IFTYPE_MESH_POINT:

#endif

	case NL80211_IFTYPE_AP:

	case NL80211_IFTYPE_AP_VLAN:

	case NL80211_IFTYPE_WDS:

		return true;

	default:

		return false;

	}

}



static bool nl80211_params_check(enum nl80211_iftype type,

				 struct vif_params *params)

{

	if (!nl80211_type_check(type))

		return false;



	return true;

}



static int ieee80211_add_iface(struct wiphy *wiphy, char *name,

			       enum nl80211_iftype type, u32 *flags,

			       struct vif_params *params)
@@ -55,9 +28,6 @@ static int ieee80211_add_iface(struct wiphy *wiphy, char *name, 
	struct ieee80211_sub_if_data *sdata;

	int err;



	if (!nl80211_params_check(type, params))

		return -EINVAL;



	err = ieee80211_if_add(local, name, &dev, type, params);

	if (err || type != NL80211_IFTYPE_MONITOR || !flags)

		return err;
@@ -85,9 +55,6 @@ static int ieee80211_change_iface(struct wiphy *wiphy, 
	if (ieee80211_sdata_running(sdata))

		return -EBUSY;



	if (!nl80211_params_check(type, params))

		return -EINVAL;



	ret = ieee80211_if_change_type(sdata, type);

	if (ret)

		return ret;

net/mac80211/main.c

+5 −0
Original line number Diff line number Diff line
@@ -622,6 +622,11 @@ int ieee80211_register_hw(struct ieee80211_hw *hw) 
	/* mac80211 always supports monitor */

	local->hw.wiphy->interface_modes |= BIT(NL80211_IFTYPE_MONITOR);



#ifndef CONFIG_MAC80211_MESH

	/* mesh depends on Kconfig, but drivers should set it if they want */

	local->hw.wiphy->interface_modes &= ~BIT(NL80211_IFTYPE_MESH_POINT);

#endif



	if (local->hw.flags & IEEE80211_HW_SIGNAL_DBM)

		local->hw.wiphy->signal_type = CFG80211_SIGNAL_TYPE_MBM;

	else if (local->hw.flags & IEEE80211_HW_SIGNAL_UNSPEC)