2012-04-20 16:43:18

by Wey-Yi Guy

Subject: [PATCH v3 1/1] mac80211: declare ieee80211_ave_rssi as EXPORT

ieee80211_ave_rssi need to be declare as export for driver to use it.

Signed-off-by: Wey-Yi Guy <[email protected]>
---
net/mac80211/util.c | 9 +++++++++
1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/net/mac80211/util.c b/net/mac80211/util.c
index 89c1e5b..4a8cd29 100644
--- a/net/mac80211/util.c
+++ b/net/mac80211/util.c
@@ -1794,6 +1794,14 @@ int ieee80211_add_ext_srates_ie(struct ieee80211_vif *vif,
return 0;
}

+/**
+ * ieee80211_ave_rssi - report the average rssi for the specified interface
+ *
+ * @vif: the specified virtual interface
+ *
+ * This function return the average rssi value for the requested interface.
+ * It assumes that the given vif is valid.
+ */
int ieee80211_ave_rssi(struct ieee80211_vif *vif)
{
struct ieee80211_sub_if_data *sdata = vif_to_sdata(vif);
@@ -1801,3 +1809,4 @@ int ieee80211_ave_rssi(struct ieee80211_vif *vif)

return ifmgd->ave_beacon_signal;
}
+EXPORT_SYMBOL_GPL(ieee80211_ave_rssi);
--
1.7.0.4

2012-04-20 17:28:28

by Johannes Berg

Subject: Re: [PATCH v3 1/1] mac80211: declare ieee80211_ave_rssi as EXPORT

On Fri, 2012-04-20 at 09:38 -0700, Wey-Yi Guy wrote:
> ieee80211_ave_rssi need to be declare as export for driver to use it.
>
> Signed-off-by: Wey-Yi Guy <[email protected]>
> ---
> net/mac80211/util.c | 9 +++++++++
> 1 files changed, 9 insertions(+), 0 deletions(-)
>
> diff --git a/net/mac80211/util.c b/net/mac80211/util.c
> index 89c1e5b..4a8cd29 100644
> --- a/net/mac80211/util.c
> +++ b/net/mac80211/util.c
> @@ -1794,6 +1794,14 @@ int ieee80211_add_ext_srates_ie(struct ieee80211_vif *vif,
> return 0;
> }
>
> +/**
> + * ieee80211_ave_rssi - report the average rssi for the specified interface
> + *
> + * @vif: the specified virtual interface
> + *
> + * This function return the average rssi value for the requested interface.
> + * It assumes that the given vif is valid.
> + */
> int ieee80211_ave_rssi(struct ieee80211_vif *vif)

The documentation should be in the header file.

johannes