CVS commit: src/sys/dev/acpi

[Jukka Ruohonen]

Module Name:    src
Committed By:   jruoho
Date:           Wed Aug 11 11:48:21 UTC 2010

Modified Files:
        src/sys/dev/acpi: acpi_cpu.h acpi_cpu_pstate.c

Log Message:
Use a define instead of a magic constant for the arbitrary P-state limit.


To generate a diff of this commit:
cvs rdiff -u -r1.12 -r1.13 src/sys/dev/acpi/acpi_cpu.h
cvs rdiff -u -r1.8 -r1.9 src/sys/dev/acpi/acpi_cpu_pstate.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/dev/acpi/acpi_cpu.h
diff -u src/sys/dev/acpi/acpi_cpu.h:1.12 src/sys/dev/acpi/acpi_cpu.h:1.13
--- src/sys/dev/acpi/acpi_cpu.h:1.12	Tue Aug 10 02:42:05 2010
+++ src/sys/dev/acpi/acpi_cpu.h	Wed Aug 11 11:48:21 2010
@@ -1,4 +1,4 @@
-/* $NetBSD: acpi_cpu.h,v 1.12 2010/08/10 02:42:05 jruoho Exp $ */
+/* $NetBSD: acpi_cpu.h,v 1.13 2010/08/11 11:48:21 jruoho Exp $ */
 
 /*-
  * Copyright (c) 2010 Jukka Ruohonen <jruoho...@iki.fi>
@@ -84,6 +84,7 @@
 /*
  * P-states.
  */
+#define ACPICPU_P_STATE_MAX	 255		/* Arbitrary upper limit     */
 #define ACPICPU_P_STATE_RETRY	 100
 #define ACPICPU_P_STATE_UNKNOWN	 0x0
 

Index: src/sys/dev/acpi/acpi_cpu_pstate.c
diff -u src/sys/dev/acpi/acpi_cpu_pstate.c:1.8 src/sys/dev/acpi/acpi_cpu_pstate.c:1.9
--- src/sys/dev/acpi/acpi_cpu_pstate.c:1.8	Wed Aug 11 10:44:07 2010
+++ src/sys/dev/acpi/acpi_cpu_pstate.c	Wed Aug 11 11:48:21 2010
@@ -1,4 +1,4 @@
-/* $NetBSD: acpi_cpu_pstate.c,v 1.8 2010/08/11 10:44:07 jruoho Exp $ */
+/* $NetBSD: acpi_cpu_pstate.c,v 1.9 2010/08/11 11:48:21 jruoho Exp $ */
 
 /*-
  * Copyright (c) 2010 Jukka Ruohonen <jruoho...@iki.fi>
@@ -27,7 +27,7 @@
  * SUCH DAMAGE.
  */
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: acpi_cpu_pstate.c,v 1.8 2010/08/11 10:44:07 jruoho Exp $");
+__KERNEL_RCSID(0, "$NetBSD: acpi_cpu_pstate.c,v 1.9 2010/08/11 11:48:21 jruoho Exp $");
 
 #include <sys/param.h>
 #include <sys/evcnt.h>
@@ -281,7 +281,7 @@
 		goto out;
 	}
 
-	if (sc->sc_pstate_count > 0xFF) {
+	if (sc->sc_pstate_count > ACPICPU_P_STATE_MAX) {
 		rv = AE_LIMIT;
 		goto out;
 	}