Search Linux Wireless

Re: [PATCH v2 00/27] mac80211: create mac80211_ prefix

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

 



"John W. Linville" <linville@xxxxxxxxxxxxx> writes:

> On Fri, Jun 17, 2011 at 02:11:40PM +0300, Kalle Valo wrote:
>
>> What do you suggest here? So that we should rename everything (both
>> mac80211 and 19 drivers) in one go? If we do that, we would have to do
>> everything in one patch to not break bisect, and it would be huge.
>> 
>> With mac80211-compat.h it's possible to break down the patches into
>> small pieces. This makes it easier to create the patches and, most
>> importantly, review them.
>> 
>> I'm willing to work on that huge patch if we so decide, but I'm not
>> sure if that's the correct way.
>> 
>> Ah, while writing this I realised that I could instead rename symbols
>> in mac80211.h one at a time and that way the patches would be smaller.
>> Did you mean that?
>
> Is there going to be another spin of this?

Yes, based on Johannes' feedback I'll drop this patchset and will
rework it. Instead I will send patches which rename mac80211.h symbols
one at a time (both from mac80211 and drivers at the same). Again this
won't happen all in once, instead I will send few patchsets to make it
easier for me. But this way we get to avoid the ugly mac80211-compat.h
file.

> I had trouble applying patch 5 -- maybe this is something that is
> best handled with a pull request?

Sure, if it's easier for you I can do that.

-- 
Kalle Valo
--
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 Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux