Hi!

Merge viapm and viaenv

Pluse:
- Add support SMBus for VT82C596, VT82C596B, VT82C686A, VT8231
- Add support ACPI timer for all VIA South Bridges

Tested on:
- VT82C596B (Gigabyte GA-6VXE7+)
- VT82C686A (Neoware CA2)
- VT8231 (HP T5700)
- VT8235 (Neoware CA10)

OK?

-- 
Alexandr Shadchin

Index: share/man/man4/Makefile
===================================================================
RCS file: /cvs/src/share/man/man4/Makefile,v
retrieving revision 1.520
diff -u -p -r1.520 Makefile
--- share/man/man4/Makefile     28 Mar 2011 15:25:37 -0000      1.520
+++ share/man/man4/Makefile     6 Apr 2011 16:34:10 -0000
@@ -62,7 +62,7 @@ MAN=  aac.4 ac97.4 acphy.4 \
        urndis.4 urtw.4 urtwn.4 usb.4  usbf.4 uscanner.4 uslcom.4 uthum.4 \
        uticom.4 utrh.4 uts.4 uvideo.4 uvisor.4 uvscom.4 uyap.4 utwitch.4 \
        vether.4 vga.4 vgafb.4 vge.4 \
-       viaenv.4 viapm.4 viasio.4 vic.4 video.4 vlan.4 vmt.4 vnd.4 vr.4 \
+       viapm.4 viasio.4 vic.4 video.4 vlan.4 vmt.4 vnd.4 vr.4 \
        vscsi.4 vte.4 \
        watchdog.4 wb.4 wbenv.4 wbng.4 wbsd.4 wbsio.4 wd.4 wdc.4 we.4 \
        wi.4tbl wpi.4 wscons.4 wsdisplay.4 wskbd.4 wsmouse.4 wsmux.4 \
Index: share/man/man4/pci.4
===================================================================
RCS file: /cvs/src/share/man/man4/pci.4,v
retrieving revision 1.301
diff -u -p -r1.301 pci.4
--- share/man/man4/pci.4        15 Jan 2011 04:35:34 -0000      1.301
+++ share/man/man4/pci.4        6 Apr 2011 16:34:10 -0000
@@ -417,7 +417,7 @@ AMD K8 temperature sensor
 AMD K10 temperature sensor
 .It Xr pwdog 4
 Quancom PWDOG1 watchdog timer device
-.It Xr viaenv 4
+.It Xr viapm 4
 VIA VT82C686A/VT8231 hardware monitor
 .It Xr wdt 4
 ICS PCI-WDT500/501 watchdog timer device
Index: share/man/man4/viaenv.4
===================================================================
RCS file: share/man/man4/viaenv.4
diff -N share/man/man4/viaenv.4
--- share/man/man4/viaenv.4     31 May 2007 19:19:53 -0000      1.11
+++ /dev/null   1 Jan 1970 00:00:00 -0000
@@ -1,90 +0,0 @@
-.\"    $OpenBSD: viaenv.4,v 1.11 2007/05/31 19:19:53 jmc Exp $
-.\"    $NetBSD: viaenv.4,v 1.5 2001/09/11 23:18:55 wiz Exp $
-.\"
-.\" Copyright (c) 2000 Johan Danielsson
-.\" All rights reserved.
-.\"
-.\" Redistribution and use in source and binary forms, with or without
-.\" modification, are permitted provided that the following conditions
-.\" are met:
-.\"
-.\" 1. Redistributions of source code must retain the above copyright
-.\"    notice, this list of conditions and the following disclaimer.
-.\"
-.\" 2. Redistributions in binary form must reproduce the above copyright
-.\"    notice, this list of conditions and the following disclaimer in the
-.\"    documentation and/or other materials provided with the distribution.
-.\"
-.\" 3. Neither the name of author nor the names of any contributors may
-.\"    be used to endorse or promote products derived from this
-.\"    software without specific prior written permission.
-.\"
-.\" THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS
-.\" ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
-.\" TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
-.\" PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
-.\" BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
-.\" CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
-.\" SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-.\" INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
-.\" CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
-.\" ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
-.\" POSSIBILITY OF SUCH DAMAGE.
-.Dd $Mdocdate: May 31 2007 $
-.Dt VIAENV 4
-.Os
-.Sh NAME
-.Nm viaenv
-.Nd VIA VT82C686A/VT8231 hardware monitor
-.Sh SYNOPSIS
-.Cd "viaenv* at pci?"
-.Sh DESCRIPTION
-The
-.Nm
-driver provides support for the hardware monitor and power management
-timer in the
-.Tn VIA
-VT82C686A and VT8231 South Bridges.
-The hardware monitor is used with the
-.Xr sysctl 8
-interface.
-.Pp
-The device has 10 sensors:
-.Bl -column "Sensor" "Units" "Typical" -offset indent
-.It Sy "Sensor" Ta Sy "Units" Ta Sy "Typical Use"
-.It Li "TSENS1" Ta "uK" Ta "CPU temperature"
-.It Li "TSENS2" Ta "uK" Ta "System temperature"
-.It Li "TSENS3" Ta "uK" Ta "Unknown"
-.It Li "FAN1" Ta "RPM" Ta "CPU fan"
-.It Li "FAN2" Ta "RPM" Ta "System fan"
-.It Li "VSENS1" Ta "uV DC" Ta "CPU core voltage (2.0V)"
-.It Li "VSENS2" Ta "uV DC" Ta "North Bridge core voltage (2.5V)"
-.It Li "Vcore" Ta "uV DC" Ta "Internal core voltage (3.3V)"
-.It Li "VSENS3" Ta "uV DC" Ta "+5V"
-.It Li "VSENS4" Ta "uV DC" Ta "+12V"
-.El
-.Pp
-Sensor data is updated every 1.5 seconds.
-.Sh SEE ALSO
-.Xr sensorsd 8 ,
-.Xr sysctl 8
-.Sh HISTORY
-The
-.Nm
-driver first appeared in
-.Nx 1.5 .
-.Ox
-support was added in
-.Ox 3.4 .
-.Sh AUTHORS
-.An -nosplit
-The
-.Nm
-driver was written by
-.An Johan Danielsson
-and ported to
-.Ox 3.4
-by
-.An Alexander Yurchenko Aq gra...@openbsd.org .
-.Sh BUGS
-Interrupt support is unimplemented, as is support for setting values.
Index: share/man/man4/viapm.4
===================================================================
RCS file: /cvs/src/share/man/man4/viapm.4,v
retrieving revision 1.8
diff -u -p -r1.8 viapm.4
--- share/man/man4/viapm.4      26 Jun 2009 02:46:27 -0000      1.8
+++ share/man/man4/viapm.4      6 Apr 2011 16:34:10 -0000
@@ -19,7 +19,7 @@
 .Os
 .Sh NAME
 .Nm viapm
-.Nd VIA SMBus controller
+.Nd VIA SMBus controller and VIA VT82C686A/VT8231 hardware monitor
 .Sh SYNOPSIS
 .Cd "viapm* at pci?"
 .Cd "iic* at viapm?"
@@ -27,25 +27,79 @@
 The
 .Nm
 driver provides support for the bus control and power management
-component of the VIA VT8233, VT8235, VT8237, VT8237A, VT8237S, VT8251,
-CX700, CX800 and VX855 South Bridges.
+component of the VIA VT82C596, VT82C596B, VT82C686A, VT8231, VT8233,
+VT8233A, VT8235, VT8237, VT8237A, VT8237S, VT8251, CX700, VX800 and
+VX855 South Bridges.
 Only the SMBus host interface is supported and can be used with the
 .Xr iic 4
 framework.
+.Pp
+The
+.Nm
+driver provides support for the hardware monitor in the VIA VT82C686A
+and VT8231 South Bridges.
+The hardware monitor is used with the
+.Xr sysctl 8
+interface.
+.Pp
+The device has 10 sensors:
+.Bl -column "Sensor" "Units" "Typical" -offset indent
+.It Sy "Sensor" Ta Sy "Units" Ta Sy "Typical Use"
+.It Li "TSENS1" Ta "uK" Ta "CPU temperature"
+.It Li "TSENS2" Ta "uK" Ta "System temperature"
+.It Li "TSENS3" Ta "uK" Ta "Unknown"
+.It Li "FAN1" Ta "RPM" Ta "CPU fan"
+.It Li "FAN2" Ta "RPM" Ta "System fan"
+.It Li "VSENS1" Ta "uV DC" Ta "CPU core voltage (2.0V)"
+.It Li "VSENS2" Ta "uV DC" Ta "North Bridge core voltage (2.5V)"
+.It Li "Vcore" Ta "uV DC" Ta "Internal core voltage (3.3V)"
+.It Li "VSENS3" Ta "uV DC" Ta "+5V"
+.It Li "VSENS4" Ta "uV DC" Ta "+12V"
+.El
+.Pp
+Sensor data is updated every 1.5 seconds.
 .Sh SEE ALSO
 .Xr iic 4 ,
 .Xr intro 4 ,
-.Xr pci 4
+.Xr pci 4 ,
+.Xr sensorsd 8 ,
+.Xr sysctl 8
 .Sh HISTORY
 The
 .Nm
 driver first appeared in
 .Ox 3.9 .
+.Pp
+The
+.Nm viaenv
+driver first appeared in
+.Nx 1.5 .
+.Ox
+support was added in
+.Ox 3.4 .
 .Sh AUTHORS
+.An -nosplit
 The
 .Nm
 driver was written by
 .An Mark Kettenis Aq kette...@openbsd.org .
+.Pp
+The
+.Nm viaenv
+driver was written by
+.An Johan Danielsson
+and ported to
+.Ox 3.4
+by
+.An Alexander Yurchenko Aq gra...@openbsd.org .
+The
+.Nm viaenv
+was merged with
+.Nm
+in
+.Ox 5.0 .
 .Sh BUGS
 The driver doesn't support commands with a data buffer size of more
 than 2 bytes.
+.Pp
+Interrupt support is unimplemented, as is support for setting values.
Index: share/man/man9/tc_init.9
===================================================================
RCS file: /cvs/src/share/man/man9/tc_init.9,v
retrieving revision 1.4
diff -u -p -r1.4 tc_init.9
--- share/man/man9/tc_init.9    31 May 2007 19:20:01 -0000      1.4
+++ share/man/man9/tc_init.9    6 Apr 2011 16:34:12 -0000
@@ -103,7 +103,7 @@ The timecounter framework is implemented
 .Xr elansc 4 ,
 .Xr gscpm 4 ,
 .Xr ichpcib 4 ,
-.Xr viaenv 4 ,
+.Xr viapm 4 ,
 .Xr hz 9 ,
 .Xr microtime 9
 .Rs
Index: sys/arch/amd64/conf/GENERIC
===================================================================
RCS file: /cvs/src/sys/arch/amd64/conf/GENERIC,v
retrieving revision 1.315
diff -u -p -r1.315 GENERIC
--- sys/arch/amd64/conf/GENERIC 3 Apr 2011 12:32:05 -0000       1.315
+++ sys/arch/amd64/conf/GENERIC 6 Apr 2011 16:34:13 -0000
@@ -90,7 +90,6 @@ schsio*       at isa? port 0x4e
 schsio*        at isa? port 0x162e
 schsio*        at isa? port 0x164e
 
-viaenv*        at pci?                 # VIA VT82C686A hardware monitor
 #viasio* at isa? port 0x2e flags 0x0000        # VIA VT1211 LPC Super I/O
 #viasio* at isa? port 0x4e flags 0x0000
 
@@ -105,7 +104,7 @@ piixpm*     at pci?                 # Intel PIIX PM
 iic*   at piixpm?
 ichiic*        at pci?                 # Intel ICH SMBus controller
 iic*   at ichiic?
-viapm* at pci?                 # VIA VT8237 power management
+viapm* at pci?                 # VIA SMBus controller
 iic*   at viapm?
 amdiic* at pci?                        # AMD-8111 SMBus controller
 iic*   at amdiic?
Index: sys/arch/i386/conf/GENERIC
===================================================================
RCS file: /cvs/src/sys/arch/i386/conf/GENERIC,v
retrieving revision 1.712
diff -u -p -r1.712 GENERIC
--- sys/arch/i386/conf/GENERIC  5 Apr 2011 12:50:15 -0000       1.712
+++ sys/arch/i386/conf/GENERIC  6 Apr 2011 16:34:15 -0000
@@ -123,7 +123,7 @@ alipm*      at pci?                         # Acer Labs 
M7101 powe
 iic*   at alipm?
 ichiic*        at pci?                         # Intel ICH SMBus controller
 iic*   at ichiic?
-viapm* at pci?                         # VIA VT8237 power management
+viapm* at pci?                         # VIA SMBus controller
 iic*   at viapm?
 amdiic*        at pci?                         # AMD-8111 SMBus controller
 iic*   at amdiic?
@@ -141,7 +141,6 @@ schsio*     at isa? port 0x4e
 schsio*        at isa? port 0x162e
 schsio*        at isa? port 0x164e
 
-viaenv*        at pci?                         # VIA VT82C686A hardware monitor
 viasio*        at isa? port 0x2e flags 0x0000  # VIA VT1211 LPC Super I/O
 viasio*        at isa? port 0x4e flags 0x0000
 wbsio* at isa? port 0x2e               # Winbond LPC Super I/O
Index: sys/arch/loongson/conf/GENERIC
===================================================================
RCS file: /cvs/src/sys/arch/loongson/conf/GENERIC,v
retrieving revision 1.32
diff -u -p -r1.32 GENERIC
--- sys/arch/loongson/conf/GENERIC      17 Feb 2011 20:14:30 -0000      1.32
+++ sys/arch/loongson/conf/GENERIC      6 Apr 2011 16:34:15 -0000
@@ -79,7 +79,7 @@ vga*          at pci?
 wsdisplay*     at vga?
 auvia*         at pci?
 audio*         at auvia?
-viaenv*                at pci?
+viapm*         at pci?
 
 # USB Controllers
 ohci*          at pci?
Index: sys/dev/pci/files.pci
===================================================================
RCS file: /cvs/src/sys/dev/pci/files.pci,v
retrieving revision 1.277
diff -u -p -r1.277 files.pci
--- sys/dev/pci/files.pci       15 Jan 2011 04:35:34 -0000      1.277
+++ sys/dev/pci/files.pci       6 Apr 2011 16:34:21 -0000
@@ -658,11 +658,6 @@ device     amdpm: i2cbus
 attach amdpm at pci
 file   dev/pci/amdpm.c                 amdpm
 
-# VIA VT82C686A hardware monitor
-device viaenv
-attach viaenv at pci
-file   dev/pci/viaenv.c                viaenv
-
 # Broadcom BCM4401 gigabit ethernet
 device bce: ether, ifnet, mii, ifmedia, mii_phy
 attach bce at pci
Index: sys/dev/pci/viaenv.c
===================================================================
RCS file: sys/dev/pci/viaenv.c
diff -N sys/dev/pci/viaenv.c
--- sys/dev/pci/viaenv.c        15 Feb 2011 21:46:21 -0000      1.15
+++ /dev/null   1 Jan 1970 00:00:00 -0000
@@ -1,384 +0,0 @@
-/*     $OpenBSD: viaenv.c,v 1.15 2011/02/15 21:46:21 miod Exp $        */
-/*     $NetBSD: viaenv.c,v 1.9 2002/10/02 16:51:59 thorpej Exp $       */
-
-/*
- * Copyright (c) 2000 Johan Danielsson
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- *
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- *
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * 3. Neither the name of author nor the names of any contributors may
- *    be used to endorse or promote products derived from this
- *    software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS
- * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
- * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
- * POSSIBILITY OF SUCH DAMAGE.
- */
-
-/* driver for the hardware monitoring part of the VIA VT82C686A */
-
-#include <sys/param.h>
-#include <sys/systm.h>
-#include <sys/device.h>
-#include <sys/kernel.h>
-#include <sys/queue.h>
-#include <sys/sensors.h>
-#include <sys/timeout.h>
-#ifdef __HAVE_TIMECOUNTER
-#include <sys/timetc.h>
-#endif
-
-#include <machine/bus.h>
-
-#include <dev/pci/pcivar.h>
-#include <dev/pci/pcireg.h>
-#include <dev/pci/pcidevs.h>
-
-#ifdef VIAENV_DEBUG
-unsigned int viaenv_debug = 0;
-#define DPRINTF(X) do { if(viaenv_debug) printf X ; } while(0)
-#else
-#define DPRINTF(X)
-#endif
-
-#define VIANUMSENSORS 10       /* three temp, two fan, five voltage */
-
-struct viaenv_softc {
-       struct device sc_dev;
-       bus_space_tag_t sc_iot;
-       bus_space_handle_t sc_ioh;
-       bus_space_handle_t sc_pm_ioh;
-
-       int     sc_fan_div[2];  /* fan RPM divisor */
-
-       struct ksensor sc_data[VIANUMSENSORS];
-       struct ksensordev sc_sensordev;
-};
-
-int  viaenv_match(struct device *, void *, void *);
-void viaenv_attach(struct device *, struct device *, void *);
-
-int  val_to_uK(unsigned int);
-int  val_to_rpm(unsigned int, int);
-long val_to_uV(unsigned int, int);
-void viaenv_refresh_sensor_data(struct viaenv_softc *);
-void viaenv_refresh(void *);
-
-#ifdef __HAVE_TIMECOUNTER
-u_int viaenv_get_timecount(struct timecounter *tc);
-
-struct timecounter viaenv_timecounter = {
-       viaenv_get_timecount,   /* get_timecount */
-       0,                      /* no poll_pps */
-       0xffffff,               /* counter_mask */
-       3579545,                /* frequency */
-       "VIAPM",                /* name */
-       1000                    /* quality */
-};
-#endif /* __HAVE_TIMECOUNTER */
-
-struct cfattach viaenv_ca = {
-       sizeof(struct viaenv_softc),
-       viaenv_match,
-       viaenv_attach
-};
-
-struct cfdriver viaenv_cd = {
-       NULL, "viaenv", DV_DULL
-};
-
-struct timeout viaenv_timeout;
-
-const struct pci_matchid viaenv_devices[] = {
-       { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT82C686A_SMB },
-       { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT8231_PWR }
-};
-
-int
-viaenv_match(struct device *parent, void *match, void *aux)
-{
-       return (pci_matchbyid((struct pci_attach_args *)aux, viaenv_devices,
-           sizeof(viaenv_devices) / sizeof(viaenv_devices[0])));
-}
-
-/*
- * XXX there doesn't seem to exist much hard documentation on how to
- * convert the raw values to usable units, this code is more or less
- * stolen from the Linux driver, but changed to suit our conditions
- */
-
-/*
- * lookup-table to translate raw values to uK, this is the same table
- * used by the Linux driver (modulo units); there is a fifth degree
- * polynomial that supposedly been used to generate this table, but I
- * haven't been able to figure out how -- it doesn't give the same values
- */
-
-static const long val_to_temp[] = {
-       20225, 20435, 20645, 20855, 21045, 21245, 21425, 21615, 21785, 21955,
-       22125, 22285, 22445, 22605, 22755, 22895, 23035, 23175, 23315, 23445,
-       23565, 23695, 23815, 23925, 24045, 24155, 24265, 24365, 24465, 24565,
-       24665, 24765, 24855, 24945, 25025, 25115, 25195, 25275, 25355, 25435,
-       25515, 25585, 25655, 25725, 25795, 25865, 25925, 25995, 26055, 26115,
-       26175, 26235, 26295, 26355, 26405, 26465, 26515, 26575, 26625, 26675,
-       26725, 26775, 26825, 26875, 26925, 26975, 27025, 27065, 27115, 27165,
-       27205, 27255, 27295, 27345, 27385, 27435, 27475, 27515, 27565, 27605,
-       27645, 27685, 27735, 27775, 27815, 27855, 27905, 27945, 27985, 28025,
-       28065, 28105, 28155, 28195, 28235, 28275, 28315, 28355, 28405, 28445,
-       28485, 28525, 28565, 28615, 28655, 28695, 28735, 28775, 28825, 28865,
-       28905, 28945, 28995, 29035, 29075, 29125, 29165, 29205, 29245, 29295,
-       29335, 29375, 29425, 29465, 29505, 29555, 29595, 29635, 29685, 29725,
-       29765, 29815, 29855, 29905, 29945, 29985, 30035, 30075, 30125, 30165,
-       30215, 30255, 30305, 30345, 30385, 30435, 30475, 30525, 30565, 30615,
-       30655, 30705, 30755, 30795, 30845, 30885, 30935, 30975, 31025, 31075,
-       31115, 31165, 31215, 31265, 31305, 31355, 31405, 31455, 31505, 31545,
-       31595, 31645, 31695, 31745, 31805, 31855, 31905, 31955, 32005, 32065,
-       32115, 32175, 32225, 32285, 32335, 32395, 32455, 32515, 32575, 32635,
-       32695, 32755, 32825, 32885, 32955, 33025, 33095, 33155, 33235, 33305,
-       33375, 33455, 33525, 33605, 33685, 33765, 33855, 33935, 34025, 34115,
-       34205, 34295, 34395, 34495, 34595, 34695, 34805, 34905, 35015, 35135,
-       35245, 35365, 35495, 35615, 35745, 35875, 36015, 36145, 36295, 36435,
-       36585, 36745, 36895, 37065, 37225, 37395, 37575, 37755, 37935, 38125,
-       38325, 38525, 38725, 38935, 39155, 39375, 39605, 39835, 40075, 40325,
-       40575, 40835, 41095, 41375, 41655, 41935,
-};
-
-/* use above table to convert values to temperatures in micro-Kelvins */
-int
-val_to_uK(unsigned int val)
-{
-       int     i = val / 4;
-       int     j = val % 4;
-
-       KASSERT(i >= 0 && i <= 255);
-
-       if (j == 0 || i == 255)
-               return val_to_temp[i] * 10000;
-
-       /* is linear interpolation ok? */
-       return (val_to_temp[i] * (4 - j) +
-           val_to_temp[i + 1] * j) * 2500 /* really: / 4 * 10000 */ ;
-}
-
-int
-val_to_rpm(unsigned int val, int div)
-{
-
-       if (val == 0)
-               return 0;
-
-       return 1350000 / val / div;
-}
-
-long
-val_to_uV(unsigned int val, int index)
-{
-       static const long mult[] =
-           {1250000, 1250000, 1670000, 2600000, 6300000};
-
-       KASSERT(index >= 0 && index <= 4);
-
-       return (25LL * val + 133) * mult[index] / 2628;
-}
-
-#define VIAENV_TSENS3  0x1f
-#define VIAENV_TSENS1  0x20
-#define VIAENV_TSENS2  0x21
-#define VIAENV_VSENS1  0x22
-#define VIAENV_VSENS2  0x23
-#define VIAENV_VCORE   0x24
-#define VIAENV_VSENS3  0x25
-#define VIAENV_VSENS4  0x26
-#define VIAENV_FAN1    0x29
-#define VIAENV_FAN2    0x2a
-#define VIAENV_FANCONF 0x47    /* fan configuration */
-#define VIAENV_TLOW    0x49    /* temperature low order value */
-#define VIAENV_TIRQ    0x4b    /* temperature interrupt configuration */
-
-#define VIAENV_GENCFG  0x40    /* general configuration */
-#define VIAENV_GENCFG_TMR32    (1 << 11)       /* 32-bit PM timer */
-#define VIAENV_GENCFG_PMEN     (1 << 15)       /* enable PM I/O space */
-#define VIAENV_PMBASE  0x48    /* power management I/O space base */
-#define VIAENV_PMSIZE  128     /* power management I/O space size */
-#define VIAENV_PM_TMR  0x08    /* PM timer */
-
-void
-viaenv_refresh_sensor_data(struct viaenv_softc *sc)
-{
-       int i;
-       u_int8_t v, v2;
-
-       /* temperature */
-       v = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAENV_TIRQ);
-       v2 = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAENV_TSENS1);
-       DPRINTF(("TSENS1 = %d\n", (v2 << 2) | (v >> 6)));
-       sc->sc_data[0].value = val_to_uK((v2 << 2) | (v >> 6));
-
-       v = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAENV_TLOW);
-       v2 = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAENV_TSENS2);
-       DPRINTF(("TSENS2 = %d\n", (v2 << 2) | ((v >> 4) & 0x3)));
-       sc->sc_data[1].value = val_to_uK((v2 << 2) | ((v >> 4) & 0x3));
-
-       v2 = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAENV_TSENS3);
-       DPRINTF(("TSENS3 = %d\n", (v2 << 2) | (v >> 6)));
-       sc->sc_data[2].value = val_to_uK((v2 << 2) | (v >> 6));
-
-       v = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAENV_FANCONF);
-
-       sc->sc_fan_div[0] = 1 << ((v >> 4) & 0x3);
-       sc->sc_fan_div[1] = 1 << ((v >> 6) & 0x3);
-
-       /* fan */
-       for (i = 3; i <= 4; i++) {
-               v = bus_space_read_1(sc->sc_iot, sc->sc_ioh,
-                   VIAENV_FAN1 + i - 3);
-               DPRINTF(("FAN%d = %d / %d\n", i - 3, v,
-                   sc->sc_fan_div[i - 3]));
-               sc->sc_data[i].value = val_to_rpm(v, sc->sc_fan_div[i - 3]);
-       }
-
-       /* voltage */
-       for (i = 5; i <= 9; i++) {
-               v = bus_space_read_1(sc->sc_iot, sc->sc_ioh,
-                   VIAENV_VSENS1 + i - 5);
-               DPRINTF(("V%d = %d\n", i - 5, v));
-               sc->sc_data[i].value = val_to_uV(v, i - 5);
-       }
-}
-
-void
-viaenv_attach(struct device * parent, struct device * self, void *aux)
-{
-       struct viaenv_softc *sc = (struct viaenv_softc *) self;
-       struct pci_attach_args *pa = aux;
-       pcireg_t iobase, control;
-       int i;
-
-       sc->sc_iot = pa->pa_iot;
-       iobase = pci_conf_read(pa->pa_pc, pa->pa_tag, 0x70);
-       control = pci_conf_read(pa->pa_pc, pa->pa_tag, 0x74);
-       if ((iobase & 0xff80) == 0 || (control & 1) == 0) {
-               printf(": HWM disabled");
-               goto nohwm;
-       }
-       if (bus_space_map(sc->sc_iot, iobase & 0xff80, 128, 0, &sc->sc_ioh)) {
-               printf(": can't map HWM i/o space");
-               goto nohwm;
-       }
-
-       for (i = 0; i <= 2; i++) {
-               sc->sc_data[i].type = SENSOR_TEMP;
-       }
-
-       for (i = 3; i <= 4; i++) {
-               sc->sc_data[i].type = SENSOR_FANRPM;
-       }
-
-       for (i = 5; i <= 9; ++i) {
-               sc->sc_data[i].type = SENSOR_VOLTS_DC;
-       }
-       strlcpy(sc->sc_data[5].desc, "VSENS1",
-           sizeof(sc->sc_data[5].desc));       /* CPU core (2V) */
-       strlcpy(sc->sc_data[6].desc, "VSENS2",
-           sizeof(sc->sc_data[6].desc));       /* NB core? (2.5V) */
-       strlcpy(sc->sc_data[7].desc, "Vcore",
-           sizeof(sc->sc_data[7].desc));       /* Vcore (3.3V) */
-       strlcpy(sc->sc_data[8].desc, "VSENS3",
-           sizeof(sc->sc_data[8].desc));       /* VSENS3 (5V) */
-       strlcpy(sc->sc_data[9].desc, "VSENS4",
-           sizeof(sc->sc_data[9].desc));       /* VSENS4 (12V) */
-
-       /* Get initial set of sensor values. */
-       viaenv_refresh_sensor_data(sc);
-
-       /* Register sensors with sysctl */
-       strlcpy(sc->sc_sensordev.xname, sc->sc_dev.dv_xname,
-           sizeof(sc->sc_sensordev.xname));
-       for (i = 0; i < VIANUMSENSORS; ++i)
-               sensor_attach(&sc->sc_sensordev, &sc->sc_data[i]);
-       sensordev_install(&sc->sc_sensordev);
-
-       /* Refresh sensors data every 1.5 seconds */
-       timeout_set(&viaenv_timeout, viaenv_refresh, sc);
-       timeout_add_msec(&viaenv_timeout, 1500);
-
-nohwm:
-#ifdef __HAVE_TIMECOUNTER
-       /* Check if power management I/O space is enabled */
-       control = pci_conf_read(pa->pa_pc, pa->pa_tag, VIAENV_GENCFG);
-       if ((control & VIAENV_GENCFG_PMEN) == 0) {
-               printf(": PM disabled");
-               goto nopm;
-       }
-
-       /* Map power management I/O space */
-       iobase = pci_conf_read(pa->pa_pc, pa->pa_tag, VIAENV_PMBASE);
-       if (bus_space_map(sc->sc_iot, PCI_MAPREG_IO_ADDR(iobase),
-           VIAENV_PMSIZE, 0, &sc->sc_pm_ioh)) {
-               printf(": can't map PM i/o space");
-               goto nopm;
-       }
-
-       /* Check for 32-bit PM timer */
-       if (control & VIAENV_GENCFG_TMR32)
-               viaenv_timecounter.tc_counter_mask = 0xffffffff;
-
-       /* Register new timecounter */
-       viaenv_timecounter.tc_priv = sc;
-       tc_init(&viaenv_timecounter);
-
-       printf(": %s-bit timer at %lluHz",
-           (viaenv_timecounter.tc_counter_mask == 0xffffffff ? "32" : "24"),
-           (unsigned long long)viaenv_timecounter.tc_frequency);
-
-nopm:
-#endif /* __HAVE_TIMECOUNTER */
-       printf("\n");
-}
-
-void
-viaenv_refresh(void *arg)
-{
-       struct viaenv_softc *sc = (struct viaenv_softc *)arg;
-
-       viaenv_refresh_sensor_data(sc);
-       timeout_add_msec(&viaenv_timeout, 1500);
-}
-
-#ifdef __HAVE_TIMECOUNTER
-u_int
-viaenv_get_timecount(struct timecounter *tc)
-{
-       struct viaenv_softc *sc = tc->tc_priv;
-       u_int u1, u2, u3;
-
-       u2 = bus_space_read_4(sc->sc_iot, sc->sc_pm_ioh, VIAENV_PM_TMR);
-       u3 = bus_space_read_4(sc->sc_iot, sc->sc_pm_ioh, VIAENV_PM_TMR);
-       do {
-               u1 = u2;
-               u2 = u3;
-               u3 = bus_space_read_4(sc->sc_iot, sc->sc_pm_ioh,
-                   VIAENV_PM_TMR);
-       } while (u1 > u2 || u2 > u3);
-
-       return (u2);
-}
-#endif /* __HAVE_TIMECOUNTER */
Index: sys/dev/pci/viapm.c
===================================================================
RCS file: /cvs/src/sys/dev/pci/viapm.c,v
retrieving revision 1.13
diff -u -p -r1.13 viapm.c
--- sys/dev/pci/viapm.c 8 Apr 2010 00:23:54 -0000       1.13
+++ sys/dev/pci/viapm.c 6 Apr 2011 16:34:21 -0000
@@ -16,8 +16,45 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
+/*     $NetBSD: viaenv.c,v 1.9 2002/10/02 16:51:59 thorpej Exp $       */
+
+/*
+ * Copyright (c) 2000 Johan Danielsson
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ *
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * 3. Neither the name of author nor the names of any contributors may
+ *    be used to endorse or promote products derived from this
+ *    software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+ * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ */
+
 /*
- * VIA VT8237 SMBus controller driver.
+ * Driver for the SMBus controller and power management timer
+ * in the VIA VT82C596[B], VT82C686A, VT8231, VT8233[A], VT8235, VT8237[A,S],
+ * VT8251, CX700, VX800 and VX855 South Bridges.
+ * Also for the hardware monitoring part of the VIA VT82C686A and VT8231.
  */
 
 #include <sys/param.h>
@@ -25,6 +62,10 @@
 #include <sys/device.h>
 #include <sys/kernel.h>
 #include <sys/rwlock.h>
+#include <sys/timeout.h>
+#ifdef __HAVE_TIMECOUNTER
+#include <sys/timetc.h>
+#endif
 
 #include <machine/bus.h>
 
@@ -35,18 +76,57 @@
 #include <dev/i2c/i2cvar.h>
 
 /*
- * VIA VT8237 ISA register definitions.
+ * Register definitions.
  */
 
 /* PCI configuration registers */
-#define VIAPM_SMB_BASE 0xd0            /* SMBus base address */
-#define VIAPM_SMB_HOSTC        0xd2            /* host configuration */
-#define VIAPM_SMB_HOSTC_HSTEN  (1 << 0)        /* enable host controller */
-#define VIAPM_SMB_HOSTC_INTEN  (1 << 1)        /* enable SCI/SMI */
-#define VIAPM_SMB_HOSTC_SCIEN  (1 << 3)        /* interrupt type (SCI/SMI) */
+#define VIAPM_PM_CFG1          0x40    /* general configuration */
+#define VIAPM_PM_CFG2          0x80
+#define VIAPM_PM_CFG_TMR32     (1 << 11)       /* 32-bit PM timer */
+#define VIAPM_PM_CFG_PMEN      (1 << 15)       /* enable PM I/O space */
+#define VIAPM_PM_BASE1         0x48    /* power management I/O base address */
+#define VIAPM_PM_BASE2         0x88
+#define VIAPM_PM_BASE_MASK     0xff80
+
+#define VIAPM_HWMON_BASE       0x70    /* HWMon I/O base address */
+#define VIAPM_HWMON_BASE_MASK  0xff80
+#define VIAPM_HWMON_CFG                0x74    /* HWMon control register */
+#define VIAPM_HWMON_CFG_HWEN   (1 << 0)        /* enable HWMon I/O space */
+
+#define VIAPM_SMB_BASE1                0x90    /* SMBus I/O base address */
+#define VIAPM_SMB_BASE2                0x80
+#define VIAPM_SMB_BASE3                0xd0
+#define VIAPM_SMB_BASE_MASK    0xfff0
+#define VIAPM_SMB_CFG1         0xd2    /* host configuration */
+#define VIAPM_SMB_CFG2         0x84
+#define VIAPM_SMB_CFG_HSTEN    (1 << 0)        /* enable SMBus I/O space */
+#define VIAPM_SMB_CFG_INTEN    (1 << 1)        /* enable SCI/SMI */
+#define VIAPM_SMB_CFG_SCIEN    (1 << 3)        /* interrupt type (SCI/SMI) */
+
+#define VIAPM_PM_SIZE          256     /* Power management I/O space size */
+#define VIAPM_HWMON_SIZE       128     /* HWMon I/O space size */
+#define VIAPM_SMB_SIZE         16      /* SMBus I/O space size */
+
+/* HWMon I/O registers */
+#define VIAPM_HWMON_TSENS3     0x1f
+#define VIAPM_HWMON_TSENS1     0x20
+#define VIAPM_HWMON_TSENS2     0x21
+#define VIAPM_HWMON_VSENS1     0x22
+#define VIAPM_HWMON_VSENS2     0x23
+#define VIAPM_HWMON_VCORE      0x24
+#define VIAPM_HWMON_VSENS3     0x25
+#define VIAPM_HWMON_VSENS4     0x26
+#define VIAPM_HWMON_FAN1       0x29
+#define VIAPM_HWMON_FAN2       0x2a
+#define VIAPM_HWMON_FANCONF    0x47    /* fan configuration */
+#define VIAPM_HWMON_TLOW       0x49    /* temperature low order value */
+#define VIAPM_HWMON_TIRQ       0x4b    /* temperature interrupt configuration 
*/
+
+/* ACPI I/O registers */
+#define VIAPM_PM_TMR           0x08    /* PM timer */
 
 /* SMBus I/O registers */
-#define VIAPM_SMB_HS   0x00            /* host status */
+#define VIAPM_SMB_HS           0x00    /* host status */
 #define VIAPM_SMB_HS_BUSY      (1 << 0)        /* running a command */
 #define VIAPM_SMB_HS_INTR      (1 << 1)        /* command completed */
 #define VIAPM_SMB_HS_DEVERR    (1 << 2)        /* command error */
@@ -55,7 +135,7 @@
 #define VIAPM_SMB_HS_INUSE     (1 << 6)        /* bus semaphore */
 #define VIAPM_SMB_HS_BITS      \
   "\020\001BUSY\002INTR\003DEVERR\004BUSERR\005FAILED\007INUSE"
-#define VIAPM_SMB_HC   0x02            /* host control */
+#define VIAPM_SMB_HC           0x02    /* host control */
 #define VIAPM_SMB_HC_INTREN    (1 << 0)        /* enable interrupts */
 #define VIAPM_SMB_HC_KILL      (1 << 1)        /* kill current transaction */
 #define VIAPM_SMB_HC_CMD_QUICK (0 << 2)        /* QUICK command */
@@ -65,33 +145,61 @@
 #define VIAPM_SMB_HC_CMD_PCALL (4 << 2)        /* PROCESS CALL command */
 #define VIAPM_SMB_HC_CMD_BLOCK (5 << 2)        /* BLOCK command */
 #define VIAPM_SMB_HC_START     (1 << 6)        /* start transaction */
-#define VIAPM_SMB_HCMD 0x03            /* host command */
-#define VIAPM_SMB_TXSLVA       0x04            /* transmit slave address */
+#define VIAPM_SMB_HCMD         0x03    /* host command */
+#define VIAPM_SMB_TXSLVA       0x04    /* transmit slave address */
 #define VIAPM_SMB_TXSLVA_READ  (1 << 0)        /* read direction */
 #define VIAPM_SMB_TXSLVA_ADDR(x) (((x) & 0x7f) << 1) /* 7-bit address */
-#define VIAPM_SMB_HD0  0x05            /* host data 0 */
-#define VIAPM_SMB_HD1  0x06            /* host data 1 */
-#define VIAPM_SMB_HBDB 0x07            /* host block data byte */
-
-#define VIAPM_SMB_SIZE 16
+#define VIAPM_SMB_HD0          0x05    /* host data 0 */
+#define VIAPM_SMB_HD1          0x06    /* host data 1 */
+#define VIAPM_SMB_HBDB         0x07    /* host block data byte */
 
 #ifdef VIAPM_DEBUG
-#define DPRINTF(x) printf x
+#define DPRINTF(x...) printf(x)
 #else
-#define DPRINTF(x)
+#define DPRINTF(x...)
 #endif
 
-#define VIAPM_DELAY    100
-#define VIAPM_TIMEOUT  1
+#define DEVNAME(sc) ((sc)->sc_dev.dv_xname)
+
+#define VIAPM_SMBUS_DELAY      100
+#define VIAPM_SMBUS_TIMEOUT    1
+
+#define VIAPM_NUM_SENSORS      10      /* three temp, two fan, five voltage */
+
+#ifdef __HAVE_TIMECOUNTER
+u_int  viapm_get_timecount(struct timecounter *tc);
+
+#ifndef VIAPM_FREQUENCY
+#define VIAPM_FREQUENCY 3579545
+#endif
+
+static struct timecounter viapm_timecounter = {
+       viapm_get_timecount,    /* get_timecount */
+       0,                      /* no poll_pps */
+       0xffffff,               /* counter_mask */
+       VIAPM_FREQUENCY,        /* frequency */
+       "VIAPM",                /* name */
+       1000                    /* quality */
+};
+#endif /* __HAVE_TIMECOUNTER */
+
+struct timeout viapm_timeout;
 
 struct viapm_softc {
        struct device           sc_dev;
 
        bus_space_tag_t         sc_iot;
-       bus_space_handle_t      sc_ioh;
+       bus_space_handle_t      sc_pm_ioh;
+       bus_space_handle_t      sc_smbus_ioh;
+       bus_space_handle_t      sc_hwmon_ioh;
        void *                  sc_ih;
        int                     sc_poll;
 
+       int                     sc_fan_div[2];  /* fan RPM divisor */
+
+       struct ksensor          sc_data[VIAPM_NUM_SENSORS];
+       struct ksensordev       sc_sensordev;
+
        struct i2c_controller   sc_i2c_tag;
        struct rwlock           sc_i2c_lock;
        struct {
@@ -113,10 +221,14 @@ int       viapm_i2c_exec(void *, i2c_op_t, i2c
 
 int    viapm_intr(void *);
 
+int    val_to_uK(unsigned int);
+int    val_to_rpm(unsigned int, int);
+long   val_to_uV(unsigned int, int);
+void   viapm_refresh_sensor_data(struct viapm_softc *);
+void   viapm_refresh(void *);
+
 struct cfattach viapm_ca = {
-       sizeof(struct viapm_softc),
-       viapm_match,
-       viapm_attach
+       sizeof(struct viapm_softc), viapm_match, viapm_attach
 };
 
 struct cfdriver viapm_cd = {
@@ -124,6 +236,10 @@ struct cfdriver viapm_cd = {
 };
 
 const struct pci_matchid viapm_ids[] = {
+       { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT82C596 },
+       { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT82C596B_PM },
+       { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT82C686A_SMB },
+       { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT8231_PWR },
        { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT8233_ISA },
        { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT8233A_ISA },
        { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VT8235_ISA },
@@ -136,11 +252,52 @@ const struct pci_matchid viapm_ids[] = {
        { PCI_VENDOR_VIATECH, PCI_PRODUCT_VIATECH_VX855_ISA }
 };
 
+/*
+ * XXX there doesn't seem to exist much hard documentation on how to
+ * convert the raw values to usable units, this code is more or less
+ * stolen from the Linux driver, but changed to suit our conditions
+ */
+
+/*
+ * lookup-table to translate raw values to uK, this is the same table
+ * used by the Linux driver (modulo units); there is a fifth degree
+ * polynomial that supposedly been used to generate this table, but I
+ * haven't been able to figure out how -- it doesn't give the same values
+ */
+
+static const long val_to_temp[] = {
+       20225, 20435, 20645, 20855, 21045, 21245, 21425, 21615, 21785, 21955,
+       22125, 22285, 22445, 22605, 22755, 22895, 23035, 23175, 23315, 23445,
+       23565, 23695, 23815, 23925, 24045, 24155, 24265, 24365, 24465, 24565,
+       24665, 24765, 24855, 24945, 25025, 25115, 25195, 25275, 25355, 25435,
+       25515, 25585, 25655, 25725, 25795, 25865, 25925, 25995, 26055, 26115,
+       26175, 26235, 26295, 26355, 26405, 26465, 26515, 26575, 26625, 26675,
+       26725, 26775, 26825, 26875, 26925, 26975, 27025, 27065, 27115, 27165,
+       27205, 27255, 27295, 27345, 27385, 27435, 27475, 27515, 27565, 27605,
+       27645, 27685, 27735, 27775, 27815, 27855, 27905, 27945, 27985, 28025,
+       28065, 28105, 28155, 28195, 28235, 28275, 28315, 28355, 28405, 28445,
+       28485, 28525, 28565, 28615, 28655, 28695, 28735, 28775, 28825, 28865,
+       28905, 28945, 28995, 29035, 29075, 29125, 29165, 29205, 29245, 29295,
+       29335, 29375, 29425, 29465, 29505, 29555, 29595, 29635, 29685, 29725,
+       29765, 29815, 29855, 29905, 29945, 29985, 30035, 30075, 30125, 30165,
+       30215, 30255, 30305, 30345, 30385, 30435, 30475, 30525, 30565, 30615,
+       30655, 30705, 30755, 30795, 30845, 30885, 30935, 30975, 31025, 31075,
+       31115, 31165, 31215, 31265, 31305, 31355, 31405, 31455, 31505, 31545,
+       31595, 31645, 31695, 31745, 31805, 31855, 31905, 31955, 32005, 32065,
+       32115, 32175, 32225, 32285, 32335, 32395, 32455, 32515, 32575, 32635,
+       32695, 32755, 32825, 32885, 32955, 33025, 33095, 33155, 33235, 33305,
+       33375, 33455, 33525, 33605, 33685, 33765, 33855, 33935, 34025, 34115,
+       34205, 34295, 34395, 34495, 34595, 34695, 34805, 34905, 35015, 35135,
+       35245, 35365, 35495, 35615, 35745, 35875, 36015, 36145, 36295, 36435,
+       36585, 36745, 36895, 37065, 37225, 37395, 37575, 37755, 37935, 38125,
+       38325, 38525, 38725, 38935, 39155, 39375, 39605, 39835, 40075, 40325,
+       40575, 40835, 41095, 41375, 41655, 41935,
+};
+
 int
 viapm_match(struct device *parent, void *match, void *aux)
 {
-       return (pci_matchbyid(aux, viapm_ids,
-           sizeof(viapm_ids) / sizeof(viapm_ids[0])));
+       return (pci_matchbyid(aux, viapm_ids, nitems(viapm_ids)));
 }
 
 void
@@ -150,56 +307,71 @@ viapm_attach(struct device *parent, stru
        struct pci_attach_args *pa = aux;
        struct i2cbus_attach_args iba;
        pcireg_t conf, iobase;
-#if 0
        pci_intr_handle_t ih;
        const char *intrstr = NULL;
-#endif
+       int basereg, cfgreg;
+       int i, v;
 
-       /* Map I/O space */
        sc->sc_iot = pa->pa_iot;
-       iobase = pci_conf_read(pa->pa_pc, pa->pa_tag, VIAPM_SMB_BASE);
-       if (iobase == 0 ||
-           bus_space_map(sc->sc_iot, iobase & 0xfffe,
-           VIAPM_SMB_SIZE, 0, &sc->sc_ioh)) {
-               printf(": can't map i/o space\n");
-               return;
+
+       /* SMBus */
+       switch (PCI_PRODUCT(pa->pa_id)) {
+       case PCI_PRODUCT_VIATECH_VT82C596:
+       case PCI_PRODUCT_VIATECH_VT82C596B_PM:
+       case PCI_PRODUCT_VIATECH_VT82C686A_SMB:
+       case PCI_PRODUCT_VIATECH_VT8231_PWR:
+               basereg = VIAPM_SMB_BASE1;
+               break;
+       default:
+               basereg = VIAPM_SMB_BASE3;
+       }
+
+       cfgreg = (VIAPM_SMB_CFG1 & (~0x03));    /* XXX 4-byte aligned */
+
+       /* Check 2nd address for VT82C596 */
+       iobase = pci_conf_read(pa->pa_pc, pa->pa_tag, basereg);
+       if ((PCI_PRODUCT(pa->pa_id) == PCI_PRODUCT_VIATECH_VT82C596) &&
+           ((iobase & 0x0001) == 0)) {
+               iobase = pci_conf_read(pa->pa_pc, pa->pa_tag, VIAPM_SMB_BASE2);
+               cfgreg = VIAPM_SMB_CFG2;
+       }
+
+       /* Check if SMBus I/O space is enabled */
+       conf = pci_conf_read(pa->pa_pc, pa->pa_tag, cfgreg);
+       if (cfgreg != VIAPM_SMB_CFG2)
+               conf >>= 16;
+       DPRINTF(": conf 0x%02x", conf & 0xff);
+
+       if ((conf & VIAPM_SMB_CFG_HSTEN) == 0) {
+               printf(": SMBus disabled\n");
+               goto nosmb;
        }
 
-       /* Read configuration */
-       conf = (iobase >> 16);
-       DPRINTF((": conf 0x%x", conf));
-
-       if ((conf & VIAPM_SMB_HOSTC_HSTEN) == 0) {
-               printf(": SMBus host disabled\n");
-               goto fail;
+       /* Map SMBus I/O space */
+       iobase &= VIAPM_SMB_BASE_MASK;
+       if (iobase == 0 || bus_space_map(sc->sc_iot, iobase,
+           VIAPM_SMB_SIZE, 0, &sc->sc_smbus_ioh)) {
+               printf(": can't map SMBus i/o space\n");
+               goto nosmb;
        }
 
-       if (conf & VIAPM_SMB_HOSTC_INTEN) {
-               if (conf & VIAPM_SMB_HOSTC_SCIEN)
-                       printf(": SCI");
-               else
-                       printf(": SMI");
-               sc->sc_poll = 1;
+       sc->sc_poll = 1;
+       if ((conf & VIAPM_SMB_CFG_SCIEN) == 0) {
+               /* No PCI IRQ */
+               printf(": SMI");
        } else {
-#if 0
                /* Install interrupt handler */
-               if (pci_intr_map(pa, &ih)) {
-                       printf(": can't map interrupt\n");
-                       goto fail;
+               if (pci_intr_map(pa, &ih) == 0) {
+                       intrstr = pci_intr_string(pa->pa_pc, ih);
+                       sc->sc_ih = pci_intr_establish(pa->pa_pc, ih, IPL_BIO,
+                           viapm_intr, sc, DEVNAME(sc));
+                       if (sc->sc_ih != NULL) {
+                               printf(": %s", intrstr);
+                               sc->sc_poll = 0;
+                       }
                }
-               intrstr = pci_intr_string(pa->pa_pc, ih);
-               sc->sc_ih = pci_intr_establish(pa->pa_pc, ih, IPL_BIO,
-                   viapm_intr, sc, sc->sc_dev.dv_xname);
-               if (sc->sc_ih == NULL) {
-                       printf(": can't establish interrupt");
-                       if (intrstr != NULL)
-                               printf(" at %s", intrstr);
-                       printf("\n");
-                       goto fail;
-               }
-               printf(": %s", intrstr);
-#endif
-               sc->sc_poll = 1;
+               if (sc->sc_poll)
+                       printf(": polling");
        }
 
        printf("\n");
@@ -216,10 +388,116 @@ viapm_attach(struct device *parent, stru
        iba.iba_tag = &sc->sc_i2c_tag;
        config_found(self, &iba, iicbus_print);
 
-       return;
+nosmb:
+
+#ifdef __HAVE_TIMECOUNTER
+       /* Power management */
+       switch (PCI_PRODUCT(pa->pa_id)) {
+       case PCI_PRODUCT_VIATECH_VT82C596:
+       case PCI_PRODUCT_VIATECH_VT82C596B_PM:
+       case PCI_PRODUCT_VIATECH_VT82C686A_SMB:
+       case PCI_PRODUCT_VIATECH_VT8231_PWR:
+               basereg = VIAPM_PM_BASE1;
+               cfgreg = VIAPM_PM_CFG1;
+               break;
+       default:
+               basereg = VIAPM_PM_BASE2;
+               cfgreg = VIAPM_PM_CFG2;
+       }
+
+       /* Check if power management I/O space is enabled */
+       conf = pci_conf_read(pa->pa_pc, pa->pa_tag, cfgreg);
+       if ((conf & VIAPM_PM_CFG_PMEN) == 0) {
+               printf("%s: PM disabled\n", DEVNAME(sc));
+               goto nopm;
+       }
+
+       /* Map power management I/O space */
+       iobase = pci_conf_read(pa->pa_pc, pa->pa_tag, basereg);
+       iobase &= VIAPM_PM_BASE_MASK;
+       if (iobase == 0 || bus_space_map(sc->sc_iot, iobase,
+           VIAPM_PM_SIZE, 0, &sc->sc_pm_ioh)) {
+               /* XXX can't map PM i/o space if ACPI mode */
+               DPRINTF("%s: can't map PM i/o space\n", DEVNAME(sc));
+               goto nopm;
+       }
+
+       /* Check for 32-bit PM timer */
+       if (conf & VIAPM_PM_CFG_TMR32)
+               viapm_timecounter.tc_counter_mask = 0xffffffff;
+
+       /* Register new timecounter */
+       viapm_timecounter.tc_priv = sc;
+       tc_init(&viapm_timecounter);
+
+       printf("%s: %s-bit timer at %lluHz\n", DEVNAME(sc),
+           (viapm_timecounter.tc_counter_mask == 0xffffffff ? "32" : "24"),
+           (unsigned long long)viapm_timecounter.tc_frequency);
+
+nopm:
+#endif /* __HAVE_TIMECOUNTER */
+
+       /* HWMon */
+       switch (PCI_PRODUCT(pa->pa_id)) {
+       case PCI_PRODUCT_VIATECH_VT82C686A_SMB:
+       case PCI_PRODUCT_VIATECH_VT8231_PWR:
+               break;
+       default:
+               return;
+       }
+
+       /* Check if HWMon I/O space is enabled */
+       conf = pci_conf_read(pa->pa_pc, pa->pa_tag, VIAPM_HWMON_CFG);
+       if ((conf & VIAPM_HWMON_CFG_HWEN) == 0) {
+               printf("%s: HWM disabled\n", DEVNAME(sc));
+               return;
+       }
+
+       /* Map HWMon I/O space */
+       iobase = pci_conf_read(pa->pa_pc, pa->pa_tag, VIAPM_HWMON_BASE);
+       iobase &= VIAPM_HWMON_BASE_MASK;
+       if (iobase == 0 || bus_space_map(sc->sc_iot, iobase,
+           VIAPM_HWMON_SIZE, 0, &sc->sc_hwmon_ioh)) {
+               printf("%s: can't map HWM i/o space\n", DEVNAME(sc));
+               return;
+       }
+
+       v = bus_space_read_1(sc->sc_iot, sc->sc_hwmon_ioh, VIAPM_HWMON_FANCONF);
+
+       sc->sc_fan_div[0] = 1 << ((v >> 4) & 0x3);
+       sc->sc_fan_div[1] = 1 << ((v >> 6) & 0x3);
 
-fail:
-       bus_space_unmap(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_SIZE);
+       for (i = 0; i <= 2; i++)
+               sc->sc_data[i].type = SENSOR_TEMP;
+       for (i = 3; i <= 4; i++)
+               sc->sc_data[i].type = SENSOR_FANRPM;
+       for (i = 5; i <= 9; ++i)
+               sc->sc_data[i].type = SENSOR_VOLTS_DC;
+
+       strlcpy(sc->sc_data[5].desc, "VSENS1",
+           sizeof(sc->sc_data[5].desc));       /* CPU core (2V) */
+       strlcpy(sc->sc_data[6].desc, "VSENS2",
+           sizeof(sc->sc_data[6].desc));       /* NB core? (2.5V) */
+       strlcpy(sc->sc_data[7].desc, "Vcore",
+           sizeof(sc->sc_data[7].desc));       /* Vcore (3.3V) */
+       strlcpy(sc->sc_data[8].desc, "VSENS3",
+           sizeof(sc->sc_data[8].desc));       /* VSENS3 (5V) */
+       strlcpy(sc->sc_data[9].desc, "VSENS4",
+           sizeof(sc->sc_data[9].desc));       /* VSENS4 (12V) */
+
+       /* Get initial set of sensor values. */
+       viapm_refresh_sensor_data(sc);
+
+       /* Register sensors with sysctl */
+       strlcpy(sc->sc_sensordev.xname, DEVNAME(sc),
+           sizeof(sc->sc_sensordev.xname));
+       for (i = 0; i < VIAPM_NUM_SENSORS; ++i)
+               sensor_attach(&sc->sc_sensordev, &sc->sc_data[i]);
+       sensordev_install(&sc->sc_sensordev);
+
+       /* Refresh sensors data every 1.5 seconds */
+       timeout_set(&viapm_timeout, viapm_refresh, sc);
+       timeout_add_msec(&viapm_timeout, 1500);
 }
 
 int
@@ -253,15 +531,11 @@ viapm_i2c_exec(void *cookie, i2c_op_t op
        u_int8_t ctl, st;
        int retries;
 
-       DPRINTF(("%s: exec op %d, addr 0x%x, cmdlen %d, len %d, "
-           "flags 0x%x, status 0x%b\n", sc->sc_dev.dv_xname, op, addr,
-           cmdlen, len, flags, bus_space_read_1(sc->sc_iot, sc->sc_ioh,
-           VIAPM_SMB_HS), VIAPM_SMB_HS_BITS));
-
        /* Check if there's a transfer already running */
-       st = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HS);
-       DPRINTF(("%s: exec: st 0x%b\n", sc->sc_dev.dv_xname, st,
-           VIAPM_SMB_HS_BITS));
+       st = bus_space_read_1(sc->sc_iot, sc->sc_smbus_ioh, VIAPM_SMB_HS);
+       DPRINTF("%s: exec op %d, addr 0x%x, cmdlen %d, len %d, "
+           "flags 0x%x, status 0x%b\n", DEVNAME(sc), op, addr,
+           cmdlen, len, flags, st, VIAPM_SMB_HS_BITS);
        if (st & VIAPM_SMB_HS_BUSY)
                return (1);
 
@@ -279,24 +553,24 @@ viapm_i2c_exec(void *cookie, i2c_op_t op
        sc->sc_i2c_xfer.error = 0;
 
        /* Set slave address and transfer direction */
-       bus_space_write_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_TXSLVA,
+       bus_space_write_1(sc->sc_iot, sc->sc_smbus_ioh, VIAPM_SMB_TXSLVA,
            VIAPM_SMB_TXSLVA_ADDR(addr) |
            (I2C_OP_READ_P(op) ? VIAPM_SMB_TXSLVA_READ : 0));
 
        b = (void *)cmdbuf;
        if (cmdlen > 0)
                /* Set command byte */
-               bus_space_write_1(sc->sc_iot, sc->sc_ioh,
+               bus_space_write_1(sc->sc_iot, sc->sc_smbus_ioh,
                    VIAPM_SMB_HCMD, b[0]);
 
        if (I2C_OP_WRITE_P(op)) {
                /* Write data */
                b = buf;
                if (len > 0)
-                       bus_space_write_1(sc->sc_iot, sc->sc_ioh,
+                       bus_space_write_1(sc->sc_iot, sc->sc_smbus_ioh,
                            VIAPM_SMB_HD0, b[0]);
                if (len > 1)
-                       bus_space_write_1(sc->sc_iot, sc->sc_ioh,
+                       bus_space_write_1(sc->sc_iot, sc->sc_smbus_ioh,
                            VIAPM_SMB_HD1, b[1]);
        }
 
@@ -313,26 +587,26 @@ viapm_i2c_exec(void *cookie, i2c_op_t op
 
        /* Start transaction */
        ctl |= VIAPM_SMB_HC_START;
-       bus_space_write_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HC, ctl);
+       bus_space_write_1(sc->sc_iot, sc->sc_smbus_ioh, VIAPM_SMB_HC, ctl);
 
        if (flags & I2C_F_POLL) {
                /* Poll for completion */
-               DELAY(VIAPM_DELAY);
+               DELAY(VIAPM_SMBUS_DELAY);
                for (retries = 1000; retries > 0; retries--) {
-                       st = bus_space_read_1(sc->sc_iot, sc->sc_ioh,
+                       st = bus_space_read_1(sc->sc_iot, sc->sc_smbus_ioh,
                            VIAPM_SMB_HS);
                        if ((st & VIAPM_SMB_HS_BUSY) == 0)
                                break;
-                       DELAY(VIAPM_DELAY);
+                       DELAY(VIAPM_SMBUS_DELAY);
                }
                if (st & VIAPM_SMB_HS_BUSY)
                        goto timeout;
                viapm_intr(sc);
        } else {
                /* Wait for interrupt */
-               if (tsleep(sc, PRIBIO, "iicexec", VIAPM_TIMEOUT * hz))
+               if (tsleep(sc, PRIBIO, "iicexec", VIAPM_SMBUS_TIMEOUT * hz))
                        goto timeout;
-       }       
+       }
 
        if (sc->sc_i2c_xfer.error)
                return (1);
@@ -343,16 +617,16 @@ timeout:
        /*
         * Transfer timeout. Kill the transaction and clear status bits.
         */
-       printf("%s: timeout, status 0x%b\n", sc->sc_dev.dv_xname, st,
+       printf("%s: timeout, status 0x%b\n", DEVNAME(sc), st,
            VIAPM_SMB_HS_BITS);
-       bus_space_write_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HC,
+       bus_space_write_1(sc->sc_iot, sc->sc_smbus_ioh, VIAPM_SMB_HC,
            VIAPM_SMB_HC_KILL);
-       DELAY(VIAPM_DELAY);
-       st = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HS);
+       DELAY(VIAPM_SMBUS_DELAY);
+       st = bus_space_read_1(sc->sc_iot, sc->sc_smbus_ioh, VIAPM_SMB_HS);
        if ((st & VIAPM_SMB_HS_FAILED) == 0)
                printf("%s: transaction abort failed, status 0x%b\n",
-                   sc->sc_dev.dv_xname, st, VIAPM_SMB_HS_BITS);
-       bus_space_write_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HS, st);
+                   DEVNAME(sc), st, VIAPM_SMB_HS_BITS);
+       bus_space_write_1(sc->sc_iot, sc->sc_smbus_ioh, VIAPM_SMB_HS, st);
        return (1);
 }
 
@@ -365,18 +639,17 @@ viapm_intr(void *arg)
        size_t len;
 
        /* Read status */
-       st = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HS);
+       st = bus_space_read_1(sc->sc_iot, sc->sc_smbus_ioh, VIAPM_SMB_HS);
        if ((st & VIAPM_SMB_HS_BUSY) != 0 || (st & (VIAPM_SMB_HS_INTR |
            VIAPM_SMB_HS_DEVERR | VIAPM_SMB_HS_BUSERR |
            VIAPM_SMB_HS_FAILED)) == 0)
                /* Interrupt was not for us */
                return (0);
 
-       DPRINTF(("%s: intr st 0x%b\n", sc->sc_dev.dv_xname, st,
-           VIAPM_SMB_HS_BITS));
+       DPRINTF("%s: intr st 0x%b\n", DEVNAME(sc), st, VIAPM_SMB_HS_BITS);
 
        /* Clear status bits */
-       bus_space_write_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HS, st);
+       bus_space_write_1(sc->sc_iot, sc->sc_smbus_ioh, VIAPM_SMB_HS, st);
 
        /* Check for errors */
        if (st & (VIAPM_SMB_HS_DEVERR | VIAPM_SMB_HS_BUSERR |
@@ -393,10 +666,10 @@ viapm_intr(void *arg)
                b = sc->sc_i2c_xfer.buf;
                len = sc->sc_i2c_xfer.len;
                if (len > 0)
-                       b[0] = bus_space_read_1(sc->sc_iot, sc->sc_ioh,
+                       b[0] = bus_space_read_1(sc->sc_iot, sc->sc_smbus_ioh,
                            VIAPM_SMB_HD0);
                if (len > 1)
-                       b[1] = bus_space_read_1(sc->sc_iot, sc->sc_ioh,
+                       b[1] = bus_space_read_1(sc->sc_iot, sc->sc_smbus_ioh,
                            VIAPM_SMB_HD1);
        }
 
@@ -405,3 +678,106 @@ done:
                wakeup(sc);
        return (1);
 }
+
+int
+val_to_uK(unsigned int val)
+{
+       int i = val / 4;
+       int j = val % 4;
+
+       KASSERT(i >= 0 && i <= 255);
+
+       if (j == 0 || i == 255)
+               return val_to_temp[i] * 10000;
+
+       /* is linear interpolation ok? */
+       return (val_to_temp[i] * (4 - j) +
+           val_to_temp[i + 1] * j) * 2500 /* really: / 4 * 10000 */ ;
+}
+
+int
+val_to_rpm(unsigned int val, int div)
+{
+       if (val == 0)
+               return 0;
+
+       return 1350000 / val / div;
+}
+
+long
+val_to_uV(unsigned int val, int index)
+{
+       static const long mult[] =
+           {1250000, 1250000, 1670000, 2600000, 6300000};
+
+       KASSERT(index >= 0 && index <= 4);
+
+       return (25LL * val + 133) * mult[index] / 2628;
+}
+
+void
+viapm_refresh_sensor_data(struct viapm_softc *sc)
+{
+       int i;
+       u_int8_t v, v2;
+
+       /* temperature */
+       v = bus_space_read_1(sc->sc_iot, sc->sc_hwmon_ioh, VIAPM_HWMON_TIRQ);
+       v2 = bus_space_read_1(sc->sc_iot, sc->sc_hwmon_ioh, VIAPM_HWMON_TSENS1);
+       DPRINTF("%s: TSENS1 = %d\n", DEVNAME(sc), (v2 << 2) | (v >> 6));
+       sc->sc_data[0].value = val_to_uK((v2 << 2) | (v >> 6));
+
+       v = bus_space_read_1(sc->sc_iot, sc->sc_hwmon_ioh, VIAPM_HWMON_TLOW);
+       v2 = bus_space_read_1(sc->sc_iot, sc->sc_hwmon_ioh, VIAPM_HWMON_TSENS2);
+       DPRINTF("%s: TSENS2 = %d\n", DEVNAME(sc), (v2 << 2) | ((v >> 4) & 0x3));
+       sc->sc_data[1].value = val_to_uK((v2 << 2) | ((v >> 4) & 0x3));
+
+       v2 = bus_space_read_1(sc->sc_iot, sc->sc_hwmon_ioh, VIAPM_HWMON_TSENS3);
+       DPRINTF("%s: TSENS3 = %d\n", DEVNAME(sc), (v2 << 2) | (v >> 6));
+       sc->sc_data[2].value = val_to_uK((v2 << 2) | (v >> 6));
+
+       /* fan */
+       for (i = 3; i <= 4; i++) {
+               v = bus_space_read_1(sc->sc_iot, sc->sc_hwmon_ioh,
+                   VIAPM_HWMON_FAN1 + i - 3);
+               DPRINTF("%s: FAN%d = %d / %d\n", DEVNAME(sc), i - 3, v,
+                   sc->sc_fan_div[i - 3]);
+               sc->sc_data[i].value = val_to_rpm(v, sc->sc_fan_div[i - 3]);
+       }
+
+       /* voltage */
+       for (i = 5; i <= 9; i++) {
+               v = bus_space_read_1(sc->sc_iot, sc->sc_hwmon_ioh,
+                   VIAPM_HWMON_VSENS1 + i - 5);
+               DPRINTF("%s: V%d = %d\n", DEVNAME(sc), i - 5, v);
+               sc->sc_data[i].value = val_to_uV(v, i - 5);
+       }
+}
+
+void
+viapm_refresh(void *arg)
+{
+       struct viapm_softc *sc = (struct viapm_softc *)arg;
+
+       viapm_refresh_sensor_data(sc);
+       timeout_add_msec(&viapm_timeout, 1500);
+}
+
+#ifdef __HAVE_TIMECOUNTER
+u_int
+viapm_get_timecount(struct timecounter *tc)
+{
+       struct viapm_softc *sc = tc->tc_priv;
+       u_int u1, u2, u3;
+
+       u2 = bus_space_read_4(sc->sc_iot, sc->sc_pm_ioh, VIAPM_PM_TMR);
+       u3 = bus_space_read_4(sc->sc_iot, sc->sc_pm_ioh, VIAPM_PM_TMR);
+       do {
+               u1 = u2;
+               u2 = u3;
+               u3 = bus_space_read_4(sc->sc_iot, sc->sc_pm_ioh, VIAPM_PM_TMR);
+       } while (u1 > u2 || u2 > u3);
+
+       return (u2);
+}
+#endif /* __HAVE_TIMECOUNTER */

Reply via email to