From: Arend Van Spriel <arend.vanspriel@broadcom.com>
To: Arend van Spriel <arend@broadcom.com>,
Johannes Berg <johannes@sipsolutions.net>
Cc: linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH] nl80211: use ieee80211_band in kernel space
Date: Thu, 25 Feb 2016 13:59:08 +0100 [thread overview]
Message-ID: <56CEFA9C.1080201@broadcom.com> (raw)
In-Reply-To: <1456399941-8408-1-git-send-email-arend@broadcom.com>
On 25-2-2016 12:32, Arend van Spriel wrote:
> The enumeration nl80211_band intends to expose the public part of
> ieee80211_band enumeration. As such it should not be used in the
> kernel. This patch changes nl80211_band usage to ieee80211_band.
>
Fixes: 59ad2160c285 ("nl80211: add feature for BSS selection support")
Not sure if it is any of the trees already.
> Reported-by: kbuild robot <lkp@intel.com>
> Signed-off-by: Arend van Spriel <arend@broadcom.com>
> ---
> include/net/cfg80211.h | 4 ++--
> net/wireless/nl80211.c | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h
> index 1dfb890..c6ddb69 100644
> --- a/include/net/cfg80211.h
> +++ b/include/net/cfg80211.h
> @@ -1863,7 +1863,7 @@ struct cfg80211_ibss_params {
> * @delta: value of RSSI level adjustment.
> */
> struct cfg80211_bss_select_adjust {
> - enum nl80211_band band;
> + enum ieee80211_band band;
> s8 delta;
> };
>
> @@ -1878,7 +1878,7 @@ struct cfg80211_bss_select_adjust {
> struct cfg80211_bss_selection {
> enum nl80211_bss_select_attr behaviour;
> union {
> - enum nl80211_band band_pref;
> + enum ieee80211_band band_pref;
> struct cfg80211_bss_select_adjust adjust;
> } param;
> };
> diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c
> index 52ce26f..1f6d8e4 100644
> --- a/net/wireless/nl80211.c
> +++ b/net/wireless/nl80211.c
> @@ -5786,7 +5786,7 @@ static int validate_scan_freqs(struct nlattr *freqs)
> return n_channels;
> }
>
> -static bool is_band_valid(struct wiphy *wiphy, enum nl80211_band b)
> +static bool is_band_valid(struct wiphy *wiphy, enum ieee80211_band b)
> {
> return b < IEEE80211_NUM_BANDS && wiphy->bands[b];
> }
>
next prev parent reply other threads:[~2016-02-25 12:59 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-25 11:32 [PATCH] nl80211: use ieee80211_band in kernel space Arend van Spriel
2016-02-25 12:59 ` Arend Van Spriel [this message]
2016-03-03 15:03 ` Johannes Berg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56CEFA9C.1080201@broadcom.com \
--to=arend.vanspriel@broadcom.com \
--cc=arend@broadcom.com \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.