Module Name:    src
Committed By:   jruoho
Date:           Tue Jan  5 13:57:18 UTC 2010

Modified Files:
        src/sys/arch/i386/acpi: vald_acpi.c
        src/sys/arch/x86/x86: mpacpi.c

Log Message:
Use acpi_eval_set_integer() to simplify code. No functional change intended.

ok pgoyette@, jmcneill@


To generate a diff of this commit:
cvs rdiff -u -r1.30 -r1.31 src/sys/arch/i386/acpi/vald_acpi.c
cvs rdiff -u -r1.81 -r1.82 src/sys/arch/x86/x86/mpacpi.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/arch/i386/acpi/vald_acpi.c
diff -u src/sys/arch/i386/acpi/vald_acpi.c:1.30 src/sys/arch/i386/acpi/vald_acpi.c:1.31
--- src/sys/arch/i386/acpi/vald_acpi.c:1.30	Wed Sep 16 10:47:54 2009
+++ src/sys/arch/i386/acpi/vald_acpi.c	Tue Jan  5 13:57:18 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: vald_acpi.c,v 1.30 2009/09/16 10:47:54 mlelstv Exp $	*/
+/*	$NetBSD: vald_acpi.c,v 1.31 2010/01/05 13:57:18 jruoho Exp $	*/
 
 /*-
  * Copyright (c) 2002 The NetBSD Foundation, Inc.
@@ -76,7 +76,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: vald_acpi.c,v 1.30 2009/09/16 10:47:54 mlelstv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: vald_acpi.c,v 1.31 2010/01/05 13:57:18 jruoho Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -671,20 +671,7 @@
 static ACPI_STATUS
 vald_acpi_dssx_set(UINT32 value)
 {
-	ACPI_STATUS rv;
-	ACPI_OBJECT Arg;
-	ACPI_OBJECT_LIST ArgList;
-
-	ArgList.Count = 1;
-	ArgList.Pointer = &Arg;
-
-	Arg.Type = ACPI_TYPE_INTEGER;
-	Arg.Integer.Value = value;
-
-	rv = AcpiEvaluateObject(ACPI_ROOT_OBJECT, "\\_SB_.VALX.DSSX",
-	    &ArgList, NULL);
-
-	return (rv);
+	return acpi_eval_set_integer(NULL, "\\_SB_.VALX.DSSX", value);
 }
 
 /*

Index: src/sys/arch/x86/x86/mpacpi.c
diff -u src/sys/arch/x86/x86/mpacpi.c:1.81 src/sys/arch/x86/x86/mpacpi.c:1.82
--- src/sys/arch/x86/x86/mpacpi.c:1.81	Tue Jan  5 13:32:49 2010
+++ src/sys/arch/x86/x86/mpacpi.c	Tue Jan  5 13:57:18 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: mpacpi.c,v 1.81 2010/01/05 13:32:49 jruoho Exp $	*/
+/*	$NetBSD: mpacpi.c,v 1.82 2010/01/05 13:57:18 jruoho Exp $	*/
 
 /*
  * Copyright (c) 2003 Wasabi Systems, Inc.
@@ -36,7 +36,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: mpacpi.c,v 1.81 2010/01/05 13:32:49 jruoho Exp $");
+__KERNEL_RCSID(0, "$NetBSD: mpacpi.c,v 1.82 2010/01/05 13:57:18 jruoho Exp $");
 
 #include "acpica.h"
 #include "opt_acpi.h"
@@ -1050,8 +1050,6 @@
 mpacpi_find_interrupts(void *self)
 {
 #if NIOAPIC > 0
-	ACPI_OBJECT_LIST arglist;
-	ACPI_OBJECT arg;
 	ACPI_STATUS rv;
 #endif
 	struct acpi_softc *acpi = self;
@@ -1073,12 +1071,11 @@
 		 * Switch us into APIC mode by evaluating _PIC(1).
 		 * Needs to be done now, since it has an effect on
 		 * the interrupt information we're about to retrieve.
+		 *
+		 * ACPI 3.0 (section 5.8.1):
+		 *   0 = PIC mode, 1 = APIC mode, 2 = SAPIC mode.
 		 */
-		arglist.Count = 1;
-		arglist.Pointer = &arg;
-		arg.Type = ACPI_TYPE_INTEGER;
-		arg.Integer.Value = 1;	/* I/O APIC (0 = PIC, 2 = IOSAPIC) */
-		rv = AcpiEvaluateObject(NULL, "\\_PIC", &arglist, NULL);
+		rv = acpi_eval_set_integer(NULL, "\\_PIC", 1);
 		if (ACPI_FAILURE(rv) && rv != AE_NOT_FOUND) {
 			if (mp_verbose)
 				printf("mpacpi: switch to APIC mode failed\n");

Reply via email to