On Wed, May 16, 2012 at 02:53:24PM -0600, Marc Jones wrote:
I think that this is a good idea. I think that I am hitting a problem with incompatible .hex and iasl versions. I am having problem with commit 15952a : Revert "Fix 64bit PCI issues on Windows" and I am using the coreboot crosstools iasl.
Intel ACPI Component Architecture ASL Optimizing Compiler version 20110922-64 [Mar 2 2012] Copyright (c) 2000 - 2011 Intel Corporation
ASL Input: /home/marc/devel/coreboot/build/seabios/out/ssdt-proc.dsl.i
- 36 lines, 765 bytes, 8 keywords
AML Output: /home/marc/devel/coreboot/build/seabios/out/ssdt-proc.aml
- 120 bytes, 6 named objects, 2 executable opcodes
Listing File: /home/marc/devel/coreboot/build/seabios/out/ssdt-proc.lst
- 2956 bytes
Hex Dump: /home/marc/devel/coreboot/build/seabios/out/ssdt-proc.hex
- 1492 bytes
Compilation complete. 0 Errors, 0 Warnings, 0 Remarks, 0 Optimizations /home/marc/devel/coreboot/build/seabios/out/ssdt-pcihp.dsl.i 21: Method (_EJ0, 1) { Return(PCEJ(0x01)) } Name (_SUN, 0x01) } Warning 1099 - Reserved method should not return a value ^ (_EJ0)
/home/marc/devel/coreboot/build/seabios/out/ssdt-pcihp.dsl.i 26: Method (_EJ0, 1) { Return(PCEJ(0x02)) } Name (_SUN, 0x02) } Warning 1099 - Reserved method should not return a value ^ (_EJ0)
...
The build passes with my distro toolchain version: ASL Optimizing Compiler version 20100528 [Oct 15 2010]
It seems that this might be a problem with the .hex files being generated by a different version of IASL. Just building the .hex file seems to fix this.
The hex file is just C code that defines an array, so it can't be the cause of any iasl errors. I'm not sure what would cause these errors, but can you confirm the error is still there after a "make clean". If so, you can email me a tar of your tree (including the out/ directory) and I'll try to take a look.
-Kevin