[src/trunk]: src/sys/dev/acpi Don't set sensor state to SWARNOVER here. Doin...

[pgoyette <pgoyette%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/1de80b9edcc7
branches:  trunk
changeset: 750462:1de80b9edcc7
user:      pgoyette <pgoyette%NetBSD.org@localhost>
date:      Fri Jan 01 15:55:30 2010 +0000

description:
Don't set sensor state to SWARNOVER here.  Doing so based only on the
device's active cooling levels prevents the user from overriding these
limits with his own values.

XXX We should probably also change setting SCRITOVER state to SCRITICAL,
XXX but right now that just confuses things.  We currently don't have a
XXX way for a sensor to be both SCRITICAL and SCRITOVER.

diffstat:

 sys/dev/acpi/acpi_tz.c |  6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diffs (27 lines):

diff -r 702b02bbc627 -r 1de80b9edcc7 sys/dev/acpi/acpi_tz.c
--- a/sys/dev/acpi/acpi_tz.c    Fri Jan 01 15:43:49 2010 +0000
+++ b/sys/dev/acpi/acpi_tz.c    Fri Jan 01 15:55:30 2010 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: acpi_tz.c,v 1.52 2009/12/24 00:26:28 pgoyette Exp $ */
+/* $NetBSD: acpi_tz.c,v 1.53 2010/01/01 15:55:30 pgoyette Exp $ */
 
 /*
  * Copyright (c) 2003 Jared D. McNeill <jmcneill%invisible.ca@localhost>
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: acpi_tz.c,v 1.52 2009/12/24 00:26:28 pgoyette Exp $");
+__KERNEL_RCSID(0, "$NetBSD: acpi_tz.c,v 1.53 2010/01/01 15:55:30 pgoyette Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -303,8 +303,6 @@
                        if (sc->sc_zone.ac[i] <= tmp)
                                active = i;
                }
-               if (active != ATZ_ACTIVE_NONE)
-                       sc->sc_temp_sensor.state = ENVSYS_SWARNOVER;
 
                flags = sc->sc_flags &
                    ~(ATZ_F_CRITICAL|ATZ_F_HOT|ATZ_F_PASSIVE);