Re: [PATCH 09/19] eeepc-laptop: support for super hybrid engine (SHE)

[Corentin Chary <corentin.chary@gmail.com>]

On Sun, May 24, 2009 at 11:28 AM, Alan Jenkins
<sourcejedi.lkml@googlemail.com> wrote:
> On 5/16/09, Len Brown <lenb@kernel.org> wrote:
>> From: Grigori Goronzy <greg@chown.ath.cx>
>>
>> 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 <greg@chown.ath.cx>
>> Signed-off-by: Corentin Chary <corentincj@iksaif.net>
>> Signed-off-by: Len Brown <len.brown@intel.com>
>> ---
>
> 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
> :-).
>
> I think this is the only attribute which needs this treatment.  It
> 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

-- 
Corentin Chary
http://xf.iksaif.net - http://uffs.org
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html