On Mon, Dec 04, 2006 at 04:38:26PM +0100, Andreas Henriksson wrote: > On Sun, Dec 03, 2006 at 08:58:31PM +0200, Guillem Jover wrote: > [...] > > The attached patch fixes the segfault, and corrects the CFLAGS > > handling for upstream and Debian, it also adds alpha to the list of > > 64 bit arches. The fix for compiler/aslopcodes.c is needed because > > the macros ACPI_UINT32_MAX: and ACPI_INTEGER_MAX are the same if > > ACPI_INTEGER is defined to be 32 bits. > > > > regards, > > guillem > > I can confirm that the patch provided by Guillem Jover seems to fix > the testcase in the original bug-report for me on Debian Testing > PowerPC. > > $ iasl -tc ../acpi-dsdt.dsl > > Intel ACPI Component Architecture > ASL Optimizing Compiler version 20060912 [Dec 4 2006] > Copyright (C) 2000 - 2006 Intel Corporation > Supports ACPI Specification Revision 3.0a > > ASL Input: ../acpi-dsdt.dsl - 561 lines, 18338 bytes, 209 keywords > AML Output: acpi-dsdt.aml - 2098 bytes 104 named objects 105 executable > opcodes > > Compilation complete. 0 Errors, 0 Warnings, 0 Remarks, 68 Optimizations > $
Ok, I'll see if it doesn't introduce evident regressions (eg by compiling some of the DSDT's available on acpi.sf.net + my own here) and eventually include it in the package. Robert: does the patch looks correct to you? Thanks -- mattia :wq! -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
