Commit 5b7ccaf3 authored by Johannes Berg's avatar Johannes Berg
Browse files

cfg80211/mac80211: re-add get_channel operation 



This essentially reverts commit 2e165b81 but
introduces the get_channel operation with a new
wireless_dev argument so that you can retrieve
the channel per interface. This is necessary as
even though we can track all interface channels
(except monitor) we can't track the channel type
used.

Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 075e0847

include/net/cfg80211.h

+9 −0
Original line number Diff line number Diff line
@@ -1612,6 +1612,10 @@ struct cfg80211_gtk_rekey_data { 
 * @get_et_strings:  Ethtool API to get a set of strings to describe stats

 *	and perhaps other supported types of ethtool data-sets.

 *	See @ethtool_ops.get_strings

 *

 * @get_channel: Get the current operating channel for the virtual interface.

 *	For monitor interfaces, it should return %NULL unless there's a single

 *	current monitoring channel.

 */

struct cfg80211_ops {

	int	(*suspend)(struct wiphy *wiphy, struct cfg80211_wowlan *wow);
@@ -1821,6 +1825,11 @@ struct cfg80211_ops { 
				  u32 sset, u8 *data);



	void (*set_monitor_enabled)(struct wiphy *wiphy, bool enabled);



	struct ieee80211_channel *

		(*get_channel)(struct wiphy *wiphy,

			       struct wireless_dev *wdev,

			       enum nl80211_channel_type *type);

};



/*

net/mac80211/cfg.c

+11 −0
Original line number Diff line number Diff line
@@ -2982,6 +2982,16 @@ static int ieee80211_probe_client(struct wiphy *wiphy, struct net_device *dev, 
	return 0;

}



static struct ieee80211_channel *

ieee80211_cfg_get_channel(struct wiphy *wiphy, struct wireless_dev *wdev,

			  enum nl80211_channel_type *type)

{

	struct ieee80211_local *local = wiphy_priv(wiphy);



	*type = local->_oper_channel_type;

	return local->oper_channel;

}



#ifdef CONFIG_PM

static void ieee80211_set_wakeup(struct wiphy *wiphy, bool enabled)

{
@@ -3062,4 +3072,5 @@ struct cfg80211_ops mac80211_config_ops = { 
	.get_et_sset_count = ieee80211_get_et_sset_count,

	.get_et_stats = ieee80211_get_et_stats,

	.get_et_strings = ieee80211_get_et_strings,

	.get_channel = ieee80211_cfg_get_channel,

};

net/wireless/nl80211.c

+11 −5
Original line number Diff line number Diff line
@@ -1759,11 +1759,17 @@ static int nl80211_send_iface(struct sk_buff *msg, u32 pid, u32 seq, int flags, 
			(cfg80211_rdev_list_generation << 2)))

		goto nla_put_failure;



	if (rdev->monitor_channel) {

		if (nla_put_u32(msg, NL80211_ATTR_WIPHY_FREQ,

				rdev->monitor_channel->center_freq) ||

	if (rdev->ops->get_channel) {

		struct ieee80211_channel *chan;

		enum nl80211_channel_type channel_type;



		chan = rdev->ops->get_channel(&rdev->wiphy, wdev,

					      &channel_type);

		if (chan &&

		    (nla_put_u32(msg, NL80211_ATTR_WIPHY_FREQ,

				 chan->center_freq) ||

		     nla_put_u32(msg, NL80211_ATTR_WIPHY_CHANNEL_TYPE,

				rdev->monitor_channel_type))

				 channel_type)))

			goto nla_put_failure;

	}

net/wireless/wext-compat.c

+7 −2
Original line number Diff line number Diff line
@@ -827,6 +827,8 @@ static int cfg80211_wext_giwfreq(struct net_device *dev, 
{

	struct wireless_dev *wdev = dev->ieee80211_ptr;

	struct cfg80211_registered_device *rdev = wiphy_to_dev(wdev->wiphy);

	struct ieee80211_channel *chan;

	enum nl80211_channel_type channel_type;



	switch (wdev->iftype) {

	case NL80211_IFTYPE_STATION:
@@ -834,10 +836,13 @@ static int cfg80211_wext_giwfreq(struct net_device *dev, 
	case NL80211_IFTYPE_ADHOC:

		return cfg80211_ibss_wext_giwfreq(dev, info, freq, extra);

	case NL80211_IFTYPE_MONITOR:

		if (!rdev->monitor_channel)

		if (!rdev->ops->get_channel)

			return -EINVAL;



		freq->m = rdev->monitor_channel->center_freq;

		chan = rdev->ops->get_channel(wdev->wiphy, wdev, &channel_type);

		if (!chan)

			return -EINVAL;

		freq->m = chan->center_freq;

		freq->e = 6;

		return 0;

	default:

CRA Git | Maintained and supported by SUSTech CRA and CCSE