From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alan Jenkins Subject: Re: [PATCH 09/19] eeepc-laptop: support for super hybrid engine (SHE) Date: Mon, 25 May 2009 10:43:58 +0100 Message-ID: <4A1A685E.1070004@tuffmail.co.uk> References: <45cb50e646d1560eff01c5ad0f0df3c7fd6148dd.1242453625.git.len.brown@intel.com> <71cd59b00905241023h6ecfa13axc2dc82863053cdc1@mail.gmail.com> <4A198401.3040003@tuffmail.co.uk> <200905251012.31526.corentin.chary@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ew0-f176.google.com ([209.85.219.176]:57668 "EHLO mail-ew0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751649AbZEYJoA (ORCPT ); Mon, 25 May 2009 05:44:00 -0400 Received: by ewy24 with SMTP id 24so2948342ewy.37 for ; Mon, 25 May 2009 02:44:01 -0700 (PDT) In-Reply-To: <200905251012.31526.corentin.chary@gmail.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Corentin Chary Cc: Grigori Goronzy , linux-acpi@vger.kernel.org, Len Brown Corentin Chary wrote: > On Sunday 24 May 2009 19:29:37 Alan Jenkins wrote: > >> Corentin Chary wrote: >> >>> On Sun, May 24, 2009 at 11:28 AM, Alan Jenkins >>> >>> wrote: >>> >>>> On 5/16/09, Len Brown wrote: >>>> >>>>> From: Grigori Goronzy >>>>> >>>>> The older eeepc-acpi driver allowed to control the SHE performance >>>>> preset through a ACPI function for just this purpose. SHE underclocks >>>>> and undervolts the FSB and undervolts the CPU (at preset 2, >>>>> "powersave"), or slightly overclocks the CPU (at preset 0, >>>>> "performance"). Preset 1 is the default setting with default clocks and >>>>> voltage. >>>>> >>>>> The new eeepc-laptop driver doesn't support it anymore. >>>>> The attached patch adds support for it to eeepc-laptop. It's very >>>>> straight-forward and almost trivial. >>>>> >>>>> Signed-off-by: Grigori Goronzy >>>>> Signed-off-by: Corentin Chary >>>>> Signed-off-by: Len Brown >>>>> --- >>>>> >>>> Hi, out of curiosity I tried this on my EeePC 701. I upgraded the >>>> BIOS to the latest version available a few months ago. >>>> >>>> I find that the file is present and can successfully be read from. >>>> The file returns the value "513". If I write "1" to it, nothing >>>> happens. If I write "0" to it, the speakers start hissing and the >>>> file then returns the value "512". Writing "1" again gets it back to >>>> normal. There is no apparent effect on performance. >>>> >>>> This is stupid, because we _do_ appear to check the BIOS supported >>>> features bitmask, but that's Asus firmware for you. Can you please >>>> add an extra test, so this file only allows reads or writes if the >>>> current value is 0 or 1? If you're quick you might slip it into -rc8 >>>> > > Hi, Can you try this patch ? It seems to works for me. > Thanks, it does make the interface less confusing. The behaviour (no performance change, hissing speakers) is the same. Do any of the models this actually works on, have only 2 different modes? I thought SHE included at least three, including an "overclocked" mode. Perhaps the cpufv file could be disabled for the 701 by checking the number of modes. I'm thinking about distributions designed for all EeePC models, which will want to include an SHE applet, but will want an easy way to avoid running it on the 701, where it can't work. BTW, is there a convention for sysfs files like this? You've used "*" to indicate the current value. /sys/power/disk uses square brackets instead. I see the "*" could be clearer though, when the options are single digits. Regards Alan