Search Linux Wireless

Re: [Fwd: please don't regress ath5k.h]

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

 



Jouni Malinen wrote:
On Mon, Nov 24, 2008 at 10:29:52AM -0500, Richard Farina wrote:

I actually don't have a problem with removing chan_debug, I was merely requesting that the size hack it enables not be removed.

More specifically in base.h I believe the code I specifically require is:

#if CHAN_DEBUG
#define ATH_CHAN_MAX    (26+26+26+200+200)
#else
#define ATH_CHAN_MAX    (14+14+14+252+20)
#endif


When removing chan_debug just please leave the higher max.

I would actually prefer to reduce this number _way_ down to something
reasonable in the upstream kernel. ath5k should not really register more
than channels 1-14 in 2.4 GHz and 5 GHz channels that are really used
(i.e., only every fourth and only at ISM bands). The maximum number
would be somewhere closer to 40 than 500..

If you have a license to use other frequencies, you can take care of
that with your own private patches, but the upstream kernel should not
do that. Channels in 2.3 GHz or 2.5 GHz or many of the 5 GHz areas that
are currently registered do not really belong here in the upstream
kernel no matter what the hardware might be capable of doing.

Your response to my request seems a bit flawed in my eyes. Drivers are supposed to run the hardware, we have crda to handle limiting the device back to to what is legal in your area. I'm sure there are places where it is legal to use 2192MHz and even if not, look at the driver, it has a max of 2732MHz set in the driver (for the .11b/g radio), clearly the person who wrote that agrees that drivers should support the hardware.

Honestly if you want to argue the legality I purposely posted an RFC for that, this change (and this email thread) is to stop a regression that would cause the hardware to no longer be fully and properly supported.

Yes, I can maintain my own personal patches to all of the drivers in the kernel (as a matter of fact I likely do) but this kind of thing belongs in the kernel, we are discussing hardware support, nothing more. Limiting the driver to enforce some random regulatory domain is pointless and against kernel policy as reg domain enforcement is supposed to be handled by crda or oldreg.

Thanks,
Rick
--
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