Search Linux Wireless

Re: Can we make ieee80211_iface_combination.limits non-const?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2014-06-26 at 13:23 -0700, Ben Greear wrote:
> On 06/26/2014 01:06 PM, Johannes Berg wrote:
> > On Thu, 2014-06-26 at 11:20 -0700, Ben Greear wrote:
> >> Depending on firmware loaded (and/or module options), a different
> >> number of vdev interfaces may be supported.
> >>
> >> Can we remove the const from the "struct ieee80211_iface_limit *limits"
> >> member of ieee80211_iface_combination?
> > 
> > No, it would break drivers who actually have a static const for the
> > pointer target.
> 
> That could be easily fixed, eh?

Yeah but it'd be stupid :)

> > You don't have to assign something that's const, but you also can't be
> > changing it on the fly after wiphy registration anyway.
> 
> This would be done before registration.  I like static init
> for defaults, but wanted to poke a few variables at run-time
> based on firmware, etc.

I don't really get it anyway. If you're going to do anything dynamic,
you don't want to be changing a global static value anyway. You want to
be making a copy or so and building it in a dynamic data structure, and
then point to it.

johannes

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux