From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751923Ab0IIHlk (ORCPT ); Thu, 9 Sep 2010 03:41:40 -0400 Received: from adelie.canonical.com ([91.189.90.139]:47659 "EHLO adelie.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751071Ab0IIHlf (ORCPT ); Thu, 9 Sep 2010 03:41:35 -0400 Message-ID: <4C888FA7.9090402@canonical.com> Date: Thu, 09 Sep 2010 15:41:27 +0800 From: Ike Panhc User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.9) Gecko/20100903 Thunderbird/3.1.3 MIME-Version: 1.0 To: acpi4asus-user@lists.sourceforge.net, "platform-driver-x86@vger.kernel.org" , "linux-kernel@vger.kernel.org" CC: Corentin Chary , Matthew Garrett Subject: asus-laptop kill wifi when initital Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, I have a report[1] that user has to blacklist the asus-laptop otherwise wifi can not work. I copied asus-laptop.c from 2.6.32 kernel and remove two lines[2] that set WLED/BLED on. The modified driver works fine. User uses 2.6.32 based kernel and when 2.6.34, a module parameter "wifi_status" is added[3] to let user choose what he want to do on WLED/BLED when initial. So I guess there are some reports tells the parameter for WLED/BLED maybe changed. Since the default action is still on because some machine need to enable WLED/BLED when booting. Any idea for the future change? For example telling user add an options on /etc/modprobe.d, or try to have a list of machine that not to set WLED/BLED to 1 when booting? [1] http://launchpad.net/bugs/612454 [2] Patch is shown below: @@ -1273,8 +1273,10 @@ static int asus_hotk_add(struct acpi_dev asus_hotk_found = 1; /* WLED and BLED are on by default */ +#if 0 /* Remove by Ike Panhc */ write_status(bt_switch_handle, 1, BT_ON); write_status(wl_switch_handle, 1, WL_ON); +#endif /* If the h/w switch is off, we need to check the real status */ write_status(NULL, read_status(BT_ON), BT_ON); [3] https://patchwork.kernel.org/patch/76283/