* HP Compaq nc6320 SSDT
@ 2006-10-10 20:51 Peter Clifton
0 siblings, 0 replies; 2+ messages in thread
From: Peter Clifton @ 2006-10-10 20:51 UTC (permalink / raw)
To: linux-acpi
Hi,
Just for larks, I decided to iasl -d my SSDT, and found the following
horridness:
Intel ACPI Component Architecture
ASL Optimizing Compiler version 20060608 [Jun 29 2006]
Copyright (C) 2000 - 2006 Intel Corporation
Supports ACPI Specification Revision 3.0a
ssdt.orig.dsl 117: Store (C0E0, 0x00)
Error 4094 - ^ syntax error,
unexpected PARSEOP_INTEGER
ssdt.orig.dsl 119: If (And (C33D, 0x01))
Error 4094 - syntax error, unexpected ')' ^
ssdt.orig.dsl 123: Else
Error 4094 - ^ syntax error, unexpected
PARSEOP_ELSE
ssdt.orig.dsl 137: Store (C0DD (0x00, 0x01), C34C)
Error 4062 - Object does not exist ^
(C34C)
ssdt.orig.dsl 138: If (And (C33D, 0x04))
Error 4062 - Object does not exist ^ (C33D)
ssdt.orig.dsl 140: Store (C0DF (0x00, 0x01),
C34D)
Error 4062 - Object does not
exist ^ (C34D)
ssdt.orig.dsl 144: Store (C34C, C34D)
Error 4062 - Object does not exist ^ (C34C)
ssdt.orig.dsl 144: Store (C34C, C34D)
Error 4062 - Object does not exist ^ (C34D)
ssdt.orig.dsl 148: Return
Error 4094 - ^ syntax error, unexpected
PARSEOP_RETURN, expecting $end
ASL Input: ssdt.orig.dsl - 149 lines, 4718 bytes, 72 keywords
Compilation complete. 9 Errors, 0 Warnings, 0 Remarks, 7 Optimizations
To make it compile nicely, I looked at the DSDT and fixed up the
external method calls (Much of the typing in the External definitions
was wrong), checked the number of arguments, and noted the missing
parenthesis around the arguments which should have been passed to these
calls. (Possibly a decompile bug due to the erronous External()
definitions?)
Diff is:
--- ssdt.orig.dsl 2006-10-10 21:46:26.000000000 +0100
+++ ssdt.dsl 2006-10-10 21:38:55.000000000 +0100
@@ -2,7 +2,7 @@
* Intel ACPI Component Architecture
* AML Disassembler version 20060608
*
- * Disassembly of ssdt.orig.aml, Tue Oct 10 21:46:26 2006
+ * Disassembly of ssdt.asl, Tue Oct 10 16:21:27 2006
*
*
* Original Table Header:
@@ -15,19 +15,19 @@
* Creator ID "MSFT"
* Creator Revision 0x0100000E (16777230)
*/
-DefinitionBlock ("ssdt.orig.aml", "SSDT", 1, "HP ", "HPQSAT",
0x00000001)
+DefinitionBlock ("ssdt.aml", "SSDT", 1, "HP ", "HPQSAT", 0x00000001)
{
External (C0E5, MethodObj) // 6 Arguments
External (C0E1, MethodObj) // 5 Arguments
- External (C236)
+ External (C236, BuffObj)
External (C0DF, MethodObj) // 2 Arguments
- External (C0E0, IntObj)
+ External (C0E0, MethodObj) // 1 Arguments
External (C0DD, MethodObj) // 2 Arguments
- External (C239, IntObj)
- External (C238)
- External (\_SB_.C240, IntObj)
+ External (C239, BuffObj)
+ External (C238, IntObj)
+ External (\_SB_.C240, MethodObj) // 2 Arguments
External (\_SB_.C23A, MethodObj) // 4 Arguments
- External (\_SB_.C143)
+ External (\_SB_.C143, MethodObj) // 0 Arguments
External (\_SB_.C002, DeviceObj)
External (\_SB_.C002.C0D5, DeviceObj)
@@ -114,8 +114,7 @@
CreateDWordField (Local0, 0x0C, C34D)
CreateDWordField (Local0, 0x10, C33D)
Store (C0DD (0x00, 0x00), C33B)
- Store (C0E0, 0x00)
- C33D
+ Store (C0E0 (0x00), C33D )
If (And (C33D, 0x01))
{
Store (C0DF (0x00, 0x00), C33C)
@@ -174,11 +173,7 @@
}
Store (C0E1 (0x00, 0x00, C33B, C33D, Local1),
Local2)
- Store (C0E5 (0x00, 0x00, C33C, C33D, Local1,
Local3), \_SB.C23A)
- Local2
- Local3
- Local1
- C33A
+ Store (C0E5 (0x00, 0x00, C33C, C33D, Local1,
Local3), \_SB.C23A (Local2, Local3, Local1, C33A ))
}
}
}
@@ -188,9 +183,7 @@
Name (_ADR, 0x00)
Method (_GTF, 0, NotSerialized)
{
- Return (\_SB.C240)
- C33A
- 0x00
+ Return (\_SB.C240 (C33A, 0x00))
}
}
}
Does this fix seem reasonable to anyone.. how can I override the SSDT in
linux (I've got the DSDT over-ridden, no problem).
Clearly to verify this properly, I need to post the DSDT and SSDT in
their entirety, please let me know if I should post them to this list,
or privately email me and I'll send them.
Are these problems "real", or are they simply the symptom of decompiling
the SSDT in the absence of context from the DSDT?
Regards
Peter Clifton
^ permalink raw reply [flat|nested] 2+ messages in thread* RE: HP Compaq nc6320 SSDT
@ 2006-10-10 20:59 Moore, Robert
0 siblings, 0 replies; 2+ messages in thread
From: Moore, Robert @ 2006-10-10 20:59 UTC (permalink / raw)
To: Peter Clifton, linux-acpi
There is no real way for the disassembler to figure out how many
arguments a method has, in the absence of an actual declaration or an
external.
Basically, it tries to guess, and often fails.
Perhaps it may be better for the disassembler to just give up with an
error message.
> -----Original Message-----
> From: linux-acpi-owner@vger.kernel.org [mailto:linux-acpi-
> owner@vger.kernel.org] On Behalf Of Peter Clifton
> Sent: Tuesday, October 10, 2006 1:52 PM
> To: linux-acpi@vger.kernel.org
> Subject: HP Compaq nc6320 SSDT
>
> Hi,
>
> Just for larks, I decided to iasl -d my SSDT, and found the following
> horridness:
>
> Intel ACPI Component Architecture
> ASL Optimizing Compiler version 20060608 [Jun 29 2006]
> Copyright (C) 2000 - 2006 Intel Corporation
> Supports ACPI Specification Revision 3.0a
>
> ssdt.orig.dsl 117: Store (C0E0, 0x00)
> Error 4094 - ^ syntax error,
> unexpected PARSEOP_INTEGER
>
> ssdt.orig.dsl 119: If (And (C33D, 0x01))
> Error 4094 - syntax error, unexpected ')' ^
>
> ssdt.orig.dsl 123: Else
> Error 4094 - ^ syntax error, unexpected
> PARSEOP_ELSE
>
> ssdt.orig.dsl 137: Store (C0DD (0x00, 0x01),
C34C)
> Error 4062 - Object does not exist
^
> (C34C)
>
> ssdt.orig.dsl 138: If (And (C33D, 0x04))
> Error 4062 - Object does not exist ^ (C33D)
>
> ssdt.orig.dsl 140: Store (C0DF (0x00, 0x01),
> C34D)
> Error 4062 - Object does not
> exist ^ (C34D)
>
> ssdt.orig.dsl 144: Store (C34C, C34D)
> Error 4062 - Object does not exist ^ (C34C)
>
> ssdt.orig.dsl 144: Store (C34C, C34D)
> Error 4062 - Object does not exist ^ (C34D)
>
> ssdt.orig.dsl 148: Return
> Error 4094 - ^ syntax error, unexpected
> PARSEOP_RETURN, expecting $end
>
> ASL Input: ssdt.orig.dsl - 149 lines, 4718 bytes, 72 keywords
> Compilation complete. 9 Errors, 0 Warnings, 0 Remarks, 7 Optimizations
>
> To make it compile nicely, I looked at the DSDT and fixed up the
> external method calls (Much of the typing in the External definitions
> was wrong), checked the number of arguments, and noted the missing
> parenthesis around the arguments which should have been passed to
these
> calls. (Possibly a decompile bug due to the erronous External()
> definitions?)
>
> Diff is:
>
> --- ssdt.orig.dsl 2006-10-10 21:46:26.000000000 +0100
> +++ ssdt.dsl 2006-10-10 21:38:55.000000000 +0100
> @@ -2,7 +2,7 @@
> * Intel ACPI Component Architecture
> * AML Disassembler version 20060608
> *
> - * Disassembly of ssdt.orig.aml, Tue Oct 10 21:46:26 2006
> + * Disassembly of ssdt.asl, Tue Oct 10 16:21:27 2006
> *
> *
> * Original Table Header:
> @@ -15,19 +15,19 @@
> * Creator ID "MSFT"
> * Creator Revision 0x0100000E (16777230)
> */
> -DefinitionBlock ("ssdt.orig.aml", "SSDT", 1, "HP ", "HPQSAT",
> 0x00000001)
> +DefinitionBlock ("ssdt.aml", "SSDT", 1, "HP ", "HPQSAT",
0x00000001)
> {
> External (C0E5, MethodObj) // 6 Arguments
> External (C0E1, MethodObj) // 5 Arguments
> - External (C236)
> + External (C236, BuffObj)
> External (C0DF, MethodObj) // 2 Arguments
> - External (C0E0, IntObj)
> + External (C0E0, MethodObj) // 1 Arguments
> External (C0DD, MethodObj) // 2 Arguments
> - External (C239, IntObj)
> - External (C238)
> - External (\_SB_.C240, IntObj)
> + External (C239, BuffObj)
> + External (C238, IntObj)
> + External (\_SB_.C240, MethodObj) // 2 Arguments
> External (\_SB_.C23A, MethodObj) // 4 Arguments
> - External (\_SB_.C143)
> + External (\_SB_.C143, MethodObj) // 0 Arguments
> External (\_SB_.C002, DeviceObj)
> External (\_SB_.C002.C0D5, DeviceObj)
>
> @@ -114,8 +114,7 @@
> CreateDWordField (Local0, 0x0C, C34D)
> CreateDWordField (Local0, 0x10, C33D)
> Store (C0DD (0x00, 0x00), C33B)
> - Store (C0E0, 0x00)
> - C33D
> + Store (C0E0 (0x00), C33D )
> If (And (C33D, 0x01))
> {
> Store (C0DF (0x00, 0x00), C33C)
> @@ -174,11 +173,7 @@
> }
>
> Store (C0E1 (0x00, 0x00, C33B, C33D, Local1),
> Local2)
> - Store (C0E5 (0x00, 0x00, C33C, C33D, Local1,
> Local3), \_SB.C23A)
> - Local2
> - Local3
> - Local1
> - C33A
> + Store (C0E5 (0x00, 0x00, C33C, C33D, Local1,
> Local3), \_SB.C23A (Local2, Local3, Local1, C33A ))
> }
> }
> }
> @@ -188,9 +183,7 @@
> Name (_ADR, 0x00)
> Method (_GTF, 0, NotSerialized)
> {
> - Return (\_SB.C240)
> - C33A
> - 0x00
> + Return (\_SB.C240 (C33A, 0x00))
> }
> }
> }
>
>
>
> Does this fix seem reasonable to anyone.. how can I override the SSDT
in
> linux (I've got the DSDT over-ridden, no problem).
>
> Clearly to verify this properly, I need to post the DSDT and SSDT in
> their entirety, please let me know if I should post them to this list,
> or privately email me and I'll send them.
>
> Are these problems "real", or are they simply the symptom of
decompiling
> the SSDT in the absence of context from the DSDT?
>
> Regards
>
> Peter Clifton
>
>
> -
> 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
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2006-10-10 20:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-10 20:51 HP Compaq nc6320 SSDT Peter Clifton
-- strict thread matches above, loose matches on Subject: below --
2006-10-10 20:59 Moore, Robert
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox