Re: ASUS M6800N: processor has only C1

Luca Capello
2004-03-04T13:53:44Z

[-- Attachment #1: Type: text/plain, Size: 1579 bytes --]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hello,

on 03/04/04 12:12, Bruno Ducrot wrote:
> Could you please try:
>
> --- linux-2.6-bk/drivers/acpi/processor.c 2004/03/04 11:05:47 1.2
> +++ linux-2.6-bk/drivers/acpi/processor.c 2004/03/04 11:09:21
> @@ -649,8 +649,8 @@ acpi_processor_get_power_info_cst (
>
> count = cst->package.elements[0].integer.value;
>
> - /* There must be at least 2 power states. */
> - if (count <= 1 || count != cst->package.count - 1) {
> + /* There must be at least one power state (some bios forgot C1), and
validate count */
> + if (count < 1 || count != cst->package.count - 1) {
> ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "Invalid _CST data\n"));
> status = -EFAULT;
> goto end;
no problem, I forgot to advise you that I can make all the necessary tests to
bring my new laptop to a completely ACPI working system :-)

> I think also that the bios will only allow C2, not C3 nor C4 if you are on AC.
You're right, but even on battery I get only C1 & C2 (as from the attached).

> Could you then try on battery also? Note that I have not implemented
> yet the reevaluation of _CST in case of AC/battery switch...
As above, it doesn't matter if I'm on battery or on AC: it's the same, but
anyway I have no more the _CST error message, but only the PBLK length one.

Thx, bye,
Gismo / Luca
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)
Comment: Using GnuPG with Debian - http://enigmail.mozdev.org

iD8DBQFARzTnVAp7Xm10JmkRAo/2AJ93Hek8RsyZvF2hlrVjgAdVAsWSsACeILws
mLV3QeCgx3/wbW6onD3ZX8k=
=Mhm+
-----END PGP SIGNATURE-----

[-- Attachment #2: 2.6.4-rc1.processor-cst.20040304.battery.script.gz --]
[-- Type: application/x-gzip, Size: 5605 bytes --]