Return-path: Received: from senator.holtmann.net ([87.106.208.187]:39931 "EHLO mail.holtmann.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752332AbYIHFWA (ORCPT ); Mon, 8 Sep 2008 01:22:00 -0400 Subject: Re: [PATCH] Ath9k: Add RF kill support. From: Marcel Holtmann To: Vasanthakumar Thiagarajan Cc: Vasanth Thiagarajan , "linville@tuxdriver.com" , "linux-wireless@vger.kernel.org" , Jouni Malinen , Luis Rodriguez In-Reply-To: <20080908045926.GA28149@vasanth-lnx.users.atheros.com> References: <20080906123842.GA27504@vasanth-lnx.users.atheros.com> <1220721347.6714.114.camel@californication> <20080908045926.GA28149@vasanth-lnx.users.atheros.com> Content-Type: text/plain Date: Mon, 08 Sep 2008 07:22:20 +0200 Message-Id: <1220851340.6714.132.camel@californication> (sfid-20080908_072205_891833_FD49730F) Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: Hi Vasanthakumar, > > > CONFIG_ATH9K_RFKILL needs to be set > > > to enable this support. > > > > why do we have to introduce yet another config option for this? What is > > the advantage of disabling RFKILL support? I really don't see it and all > > these extra config options are rather confusing than useful. > Normally home users may not hit a situation where they have to > disable the radio, so they can compile out the rf kill support > and essentially disable the periodic run of the work queue > (rfkill_poll ,runs for every 2 secs).If this does not make > any sense, I will remove this config option. I don't know about the actual implementation, but if you use a polling mechanism then it is wrong. Can't you make this interrupt driven. The desktop use case needs RFKILL support, because that will be used for switching off the radio. Just think about the flight mode case. So the only case where disabling RFKILL support makes sense would be an embedded system with specific constraints. However even in the embedded case, I don't see that usefulness. Regards Marcel