From mboxrd@z Thu Jan 1 00:00:00 1970 From: Corentin Chary Subject: Re: [PATCH 09/19] eeepc-laptop: support for super hybrid engine (SHE) Date: Sun, 24 May 2009 19:23:08 +0200 Message-ID: <71cd59b00905241023h6ecfa13axc2dc82863053cdc1@mail.gmail.com> References: <45cb50e646d1560eff01c5ad0f0df3c7fd6148dd.1242453625.git.len.brown@intel.com> <1242453736-20763-1-git-send-email-lenb@kernel.org> <158ca1d75dd0d6223f3b1dd741d30777da62ab80.1242453625.git.len.brown@intel.com> <9b2b86520905240228y5cf7111dsa6d5d34e03360bd9@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mail-fx0-f168.google.com ([209.85.220.168]:47965 "EHLO mail-fx0-f168.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751401AbZEXRXH convert rfc822-to-8bit (ORCPT ); Sun, 24 May 2009 13:23:07 -0400 Received: by fxm12 with SMTP id 12so860674fxm.37 for ; Sun, 24 May 2009 10:23:08 -0700 (PDT) In-Reply-To: <9b2b86520905240228y5cf7111dsa6d5d34e03360bd9@mail.gmail.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: alan-jenkins@tuffmail.co.uk Cc: Grigori Goronzy , linux-acpi@vger.kernel.org, Len Brown 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 underclock= s >> 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. =A0I 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". =A0If I write "1" to it, nothing > happens. =A0If I write "0" to it, the speakers start hissing and the > file then returns the value "512". =A0Writing "1" again gets it back = to > normal. =A0There 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. =A0Can you please > add an extra test, so this file only allows =A0reads or writes if the > current value is 0 or 1? =A0If you're quick you might slip it into -r= c8 > :-). > > I think this is the only attribute which needs this treatment. =A0It > looks like the SHE feature may have been attempted for the 701 model, > but not completed on time. > > Thanks! > Alan > Hi, Thanks for the repport. Can you send me your dsdt ? I can't upgrade my bios right now. Thanks --=20 Corentin Chary http://xf.iksaif.net - http://uffs.org -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html