[src/trunk]: src/sys/dev/acpi Don't update the value_{min, max} - these fields...

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

details:   https://anonhg.NetBSD.org/src/rev/78c422388d46
branches:  trunk
changeset: 765810:78c422388d46
user:      pgoyette <pgoyette%NetBSD.org@localhost>
date:      Mon Jun 06 17:53:01 2011 +0000

description:
Don't update the value_{min,max} - these fields are not {low,high}-water
marks, and setting them without also updating the ENVSYS_FVALID_{MIN,MAX}
flags is quite pointless.

diffstat:

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

diffs (29 lines):

diff -r c9eeed279c43 -r 78c422388d46 sys/dev/acpi/thinkpad_acpi.c
--- a/sys/dev/acpi/thinkpad_acpi.c      Mon Jun 06 17:49:41 2011 +0000
+++ b/sys/dev/acpi/thinkpad_acpi.c      Mon Jun 06 17:53:01 2011 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: thinkpad_acpi.c,v 1.37 2011/04/14 07:06:52 jruoho Exp $ */
+/* $NetBSD: thinkpad_acpi.c,v 1.38 2011/06/06 17:53:01 pgoyette Exp $ */
 
 /*-
  * Copyright (c) 2007 Jared D. McNeill <jmcneill%invisible.ca@localhost>
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: thinkpad_acpi.c,v 1.37 2011/04/14 07:06:52 jruoho Exp $");
+__KERNEL_RCSID(0, "$NetBSD: thinkpad_acpi.c,v 1.38 2011/06/06 17:53:01 pgoyette Exp $");
 
 #include <sys/param.h>
 #include <sys/device.h>
@@ -577,10 +577,6 @@
        }
 
        edata->value_cur = rpm;
-       if (rpm < edata->value_min || edata->value_min == -1)
-               edata->value_min = rpm;
-       if (rpm > edata->value_max || edata->value_max == -1)
-               edata->value_max = rpm;
        edata->state = ENVSYS_SVALID;
 }