From owner-acpi-jp@jp.freebsd.org  Sat Jun 30 03:11:51 2001
Received: (from daemon@localhost)
	by castle.jp.freebsd.org (8.9.3+3.2W/8.7.3) id DAA81188;
	Sat, 30 Jun 2001 03:11:51 +0900 (JST)
	(envelope-from owner-acpi-jp@jp.FreeBSD.org)
Received: from tasogare.imasy.or.jp (root@tasogare.imasy.or.jp [202.227.24.5])
	by castle.jp.freebsd.org (8.9.3+3.2W/8.7.3) with ESMTP id DAA81183
	for <acpi-jp@jp.freebsd.org>; Sat, 30 Jun 2001 03:11:50 +0900 (JST)
	(envelope-from iwasaki@jp.FreeBSD.org)
Received: from localhost (iwasaki.imasy.or.jp [202.227.24.92])
	by tasogare.imasy.or.jp (8.11.3+3.4W/8.11.3/tasogare) with ESMTP/inet id f5TIBmI74623;
	Sat, 30 Jun 2001 03:11:48 +0900 (JST)
	(envelope-from iwasaki@jp.FreeBSD.org)
To: acpi-jp@jp.freebsd.org, msmith@freebsd.org
Cc: current@freebsd.org
In-Reply-To: <200106280635.f5S6Z7w07241@mass.dis.org>
References: <200106280635.f5S6Z7w07241@mass.dis.org>
X-Mailer: Mew version 1.94.1 on Emacs 19.34 / Mule 2.3 (SUETSUMUHANA)
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-Id: <20010630031147Q.iwasaki@jp.FreeBSD.org>
Date: Sat, 30 Jun 2001 03:11:47 +0900
From: Mitsuru IWASAKI <iwasaki@jp.freebsd.org>
X-Dispatcher: imput version 20000228(IM140)
Lines: 35
Reply-To: acpi-jp@jp.freebsd.org
Precedence: list
X-Distribute: distribute version 2.1 (Alpha) patchlevel 24e+010328
X-Sequence: acpi-jp 1152
Subject: [acpi-jp 1152] Re: HEADS UP: ACPI update - thermal management
Errors-To: owner-acpi-jp@jp.freebsd.org
Sender: owner-acpi-jp@jp.freebsd.org
X-Originator: iwasaki@jp.freebsd.org

Hi, mike.

> This is just a heads-up to let folks know that I've committed some early 
> code to handle thermal management under ACPI.  This should DTRT with 
> active cooling (fans, etc.).  It won't help with passive cooling yet (we 
> need to sort out the processor device control first), and it may well 
> have problems (there are places where the specification is vague about 
> what should be in the namespace and my ability to test these options is 
> limited).

I tested this a bit and noticed following messages.

	unknown: error fetching current temperature

I think that notify handler got wrong parameter, device_t not softc.
Here is the fix for it.

Index: acpi_thermal.c
===================================================================
RCS file: /home/ncvs/src/sys/dev/acpica/acpi_thermal.c,v
retrieving revision 1.8
diff -u -r1.8 acpi_thermal.c
--- acpi_thermal.c      2001/06/28 06:17:16     1.8
+++ acpi_thermal.c      2001/06/29 17:48:14
@@ -151,7 +151,7 @@
      * Register for any Notify events sent to this zone.
      */
     AcpiInstallNotifyHandler(sc->tz_handle, ACPI_DEVICE_NOTIFY, 
-			      acpi_tz_notify_handler, dev);
+			      acpi_tz_notify_handler, sc);

BTW, how about sysctl interface for ACPI thermal zone?
At least the representative temperature would be useful for users.

Thanks
