Search Linux Wireless

Re: [PATCH 2/2] iw: remove ifdefs for mcs mask

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

 



Hey Johannes,

On Mon, Jan 09, 2012 at 02:05:39PM +0100, Johannes Berg wrote:
> On Thu, 2012-01-05 at 20:58 +0100, Simon Wunderlich wrote:
> > These are enums, not defines. Therefore the ifdef check can never be
> > true.
> 
> The original idea was that in nl80211.h you do
> 
> #define FOO FOO

Oh, ok, sorry, that's right, I haven't thought about this.
> 
> (like we already have for a few things) to trigger the ifdef, but I
> don't really care.

I could as well add it into nl80211.h, and we skip this patch, or we remove
these ifdefs - your choice. :) 

However, I don't think we really need the ifdefs as a copy of the nl80211.h
seems to be delivered with the iw source code anyway ...?

Cheers,
	Simon

Attachment: signature.asc
Description: Digital signature


[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