Re: [RFC] rt2x00: advertise interface combinations

[Ivo Van Doorn <ivdoorn@gmail.com>]

Hi,

On Fri, May 13, 2011 at 11:09 AM, Johannes Berg
<johannes@sipsolutions.net> wrote:
> From: Johannes Berg <johannes.berg@intel.com>
>
> Make rt2x00 advertise that it potentially supports
> more than one AP interface.
>
> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
> ---
> I don't have any rt2x00 HW so this is completely untested. It compiles
> though ;-)
>
>  drivers/net/wireless/rt2x00/rt2x00.h    |    6 ++++++
>  drivers/net/wireless/rt2x00/rt2x00dev.c |   19 +++++++++++++++++++
>  2 files changed, 25 insertions(+)
>
> --- a/drivers/net/wireless/rt2x00/rt2x00.h      2011-05-13 11:02:38.000000000 +0200
> +++ b/drivers/net/wireless/rt2x00/rt2x00.h      2011-05-13 11:03:28.000000000 +0200
> @@ -960,6 +960,12 @@ struct rt2x00_dev {
>         * Protect the interrupt mask register.
>         */
>        spinlock_t irqmask_lock;
> +
> +       /*
> +        * Interface combinations
> +        */
> +       struct ieee80211_iface_limit ap_limit;
> +       struct ieee80211_iface_combination ap_combination;
>  };
>
>  /*
> --- a/drivers/net/wireless/rt2x00/rt2x00dev.c   2011-05-13 11:00:28.000000000 +0200
> +++ b/drivers/net/wireless/rt2x00/rt2x00dev.c   2011-05-13 11:08:04.000000000 +0200
> @@ -942,6 +942,25 @@ static int rt2x00lib_probe_hw(struct rt2
>  #undef RT2X00_TASKLET_INIT
>
>        /*
> +        * Initialise supported virtual interface combinations
> +        *
> +        * Since currently, max_sta_intf is always 1, ignore it.
> +        */
> +       WARN_ON(rt2x00dev->ops->max_sta_intf != 1);

I don't think this WARN_ON will be needed, but with this entire patch,
I think that our own code for checking the interface limitations are no
longer needed. But I will clean that up later in a separate patch.
So for now:

Acked-by: Ivo van Doorn <IvDoorn@gmail.com>