Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752120Ab0FLCDR (ORCPT ); Fri, 11 Jun 2010 22:03:17 -0400 Received: from mail-wy0-f174.google.com ([74.125.82.174]:58788 "EHLO mail-wy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751069Ab0FLCDP (ORCPT ); Fri, 11 Jun 2010 22:03:15 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type; b=KX4NfETsHS+YY5qIRYMF+N6/KpBt7JHpDAUmemjVukfe4gfJH/6ATxkBwTRT6GPSAi UBNwZDtLlxY97kl1kBi1sAh6GajMl/YW4K/+c/LsYq9pOPFc5WOeWe/vz778y089fOf5 QtDqbeoJbiwOOirajrRR1R3sWbzzwHOjZhHms= MIME-Version: 1.0 In-Reply-To: <1276288163.3918.1.camel@jlt3.sipsolutions.net> References: <4c0f4326.0a41df0a.7bde.ffffeef2@mx.google.com> <1276286993.3918.0.camel@jlt3.sipsolutions.net> <1276288163.3918.1.camel@jlt3.sipsolutions.net> From: Dmytro Milinevskyy Date: Sat, 12 Jun 2010 05:02:53 +0300 X-Google-Sender-Auth: Mv4siuVxmW2LM_EpF03SPug_vlI Message-ID: Subject: Re: [ath5k-devel] [PATCH] [ath5k][leds] Ability to disable leds support. If leds support enabled do not force mac802.11 leds layer selection. To: Johannes Berg Cc: Bob Copeland , ath5k-devel@venema.h4ckr.net, Kalle Valo , linux-wireless@vger.kernel.org, GeunSik Lim , Jiri Slaby , Greg Kroah-Hartman , "John W. Linville" , Keng-Yu Lin , netdev@vger.kernel.org, Jiri Kosina , Shahar Or , linux-kernel@vger.kernel.org, Luca Verdesca Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 785 Lines: 24 In this case no way to control LEDs. If you turn LEDs support you can't tune them out, no? I believe that user should be able to chose whether LEDs support is needed or not. This is how done in intel wireless drivers. The driver should be tolerant. -- Dima On Fri, Jun 11, 2010 at 11:29 PM, Johannes Berg wrote: > On Fri, 2010-06-11 at 23:26 +0300, Dmytro Milinevskyy wrote: >> Generic LED class is not disabled so it's possible to control leds in sysfs. > > But if you turn off ATH5K_LEDS?? > > johannes > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/