nagai/kernel
3
0
Fork 0
forked from mirrors/linux
Code Pull requests Activity

hwmon: (vt8231) Fix multi-line comments

Browse source 
Cc: Roger Lucas <vt8231@hiddenengine.co.uk>
Acked-by: Roger Lucas <vt8231@hiddenengine.co.uk>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
This commit is contained in:
Guenter Roeck 2012-01-19 11:02:27 -08:00 committed by Guenter Roeck
parent 9ed5bc24fa
commit 61ba031841
1 changed files with 86 additions and 76 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

146
drivers/hwmon/vt8231.c
View file

@ -1,27 +1,28 @@
/* /*
vt8231.c - Part of lm_sensors, Linux kernel modules * vt8231.c - Part of lm_sensors, Linux kernel modules
for hardware monitoring * for hardware monitoring
*
Copyright (c) 2005 Roger Lucas <vt8231@hiddenengine.co.uk> * Copyright (c) 2005 Roger Lucas <vt8231@hiddenengine.co.uk>
Copyright (c) 2002 Mark D. Studebaker <mdsxyz123@yahoo.com> * Copyright (c) 2002 Mark D. Studebaker <mdsxyz123@yahoo.com>
Aaron M. Marsh <amarsh@sdf.lonestar.org> * Aaron M. Marsh <amarsh@sdf.lonestar.org>
*
This program is free software; you can redistribute it and/or modify * This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by * it under the terms of the GNU General Public License as published by
the Free Software Foundation; either version 2 of the License, or * the Free Software Foundation; either version 2 of the License, or
(at your option) any later version. * (at your option) any later version.
*
This program is distributed in the hope that it will be useful, * This program is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of * but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details. * GNU General Public License for more details.
*
You should have received a copy of the GNU General Public License * You should have received a copy of the GNU General Public License
along with this program; if not, write to the Free Software * along with this program; if not, write to the Free Software
Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*/ */
/* Supports VIA VT8231 South Bridge embedded sensors /*
* Supports VIA VT8231 South Bridge embedded sensors
*/ */
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
@ -50,25 +51,26 @@ static struct platform_device *pdev;
#define VT8231_BASE_REG 0x70 #define VT8231_BASE_REG 0x70
#define VT8231_ENABLE_REG 0x74 #define VT8231_ENABLE_REG 0x74
/* The VT8231 registers /*
* The VT8231 registers
The reset value for the input channel configuration is used (Reg 0x4A=0x07) *
which sets the selected inputs marked with '*' below if multiple options are * The reset value for the input channel configuration is used (Reg 0x4A=0x07)
possible: * which sets the selected inputs marked with '*' below if multiple options are
* possible:
Voltage Mode Temperature Mode *
Sensor Linux Id Linux Id VIA Id * Voltage Mode Temperature Mode
-------- -------- -------- ------ * Sensor Linux Id Linux Id VIA Id
CPU Diode N/A temp1 0 * -------- -------- -------- ------
UIC1 in0 temp2 * 1 * CPU Diode N/A temp1 0
UIC2 in1 * temp3 2 * UIC1 in0 temp2 * 1
UIC3 in2 * temp4 3 * UIC2 in1 * temp3 2
UIC4 in3 * temp5 4 * UIC3 in2 * temp4 3
UIC5 in4 * temp6 5 * UIC4 in3 * temp5 4
3.3V in5 N/A * UIC5 in4 * temp6 5
* 3.3V in5 N/A
Note that the BIOS may set the configuration register to a different value *
to match the motherboard configuration. * Note that the BIOS may set the configuration register to a different value
* to match the motherboard configuration.
*/ */
/* fans numbered 0-1 */ /* fans numbered 0-1 */
@ -81,12 +83,13 @@ static const u8 regvolt[] = { 0x21, 0x22, 0x23, 0x24, 0x25, 0x26 };
static const u8 regvoltmax[] = { 0x3d, 0x2b, 0x2d, 0x2f, 0x31, 0x33 }; static const u8 regvoltmax[] = { 0x3d, 0x2b, 0x2d, 0x2f, 0x31, 0x33 };
static const u8 regvoltmin[] = { 0x3e, 0x2c, 0x2e, 0x30, 0x32, 0x34 }; static const u8 regvoltmin[] = { 0x3e, 0x2c, 0x2e, 0x30, 0x32, 0x34 };
/* Temperatures are numbered 1-6 according to the Linux kernel specification. /*
** * Temperatures are numbered 1-6 according to the Linux kernel specification.
** In the VIA datasheet, however, the temperatures are numbered from zero. *
** Since it is important that this driver can easily be compared to the VIA * In the VIA datasheet, however, the temperatures are numbered from zero.
** datasheet, we will use the VIA numbering within this driver and map the * Since it is important that this driver can easily be compared to the VIA
** kernel sysfs device name to the VIA number in the sysfs callback. * datasheet, we will use the VIA numbering within this driver and map the
* kernel sysfs device name to the VIA number in the sysfs callback.
*/ */
#define VT8231_REG_TEMP_LOW01 0x49 #define VT8231_REG_TEMP_LOW01 0x49
@ -108,8 +111,9 @@ static const u8 regtempmin[] = { 0x3a, 0x3e, 0x2c, 0x2e, 0x30, 0x32 };
#define VT8231_REG_TEMP1_CONFIG 0x4b #define VT8231_REG_TEMP1_CONFIG 0x4b
#define VT8231_REG_TEMP2_CONFIG 0x4c #define VT8231_REG_TEMP2_CONFIG 0x4c
/* temps 0-5 as numbered in VIA datasheet - see later for mapping to Linux /*
** numbering * temps 0-5 as numbered in VIA datasheet - see later for mapping to Linux
* numbering
*/ */
#define ISTEMP(i, ch_config) ((i) == 0 ? 1 : \ #define ISTEMP(i, ch_config) ((i) == 0 ? 1 : \
((ch_config) >> ((i)+1)) & 0x01) ((ch_config) >> ((i)+1)) & 0x01)
@ -119,23 +123,25 @@ static const u8 regtempmin[] = { 0x3a, 0x3e, 0x2c, 0x2e, 0x30, 0x32 };
#define DIV_FROM_REG(val) (1 << (val)) #define DIV_FROM_REG(val) (1 << (val))
/* NB The values returned here are NOT temperatures. The calibration curves /*
** for the thermistor curves are board-specific and must go in the * NB The values returned here are NOT temperatures. The calibration curves
** sensors.conf file. Temperature sensors are actually ten bits, but the * for the thermistor curves are board-specific and must go in the
** VIA datasheet only considers the 8 MSBs obtained from the regtemp[] * sensors.conf file. Temperature sensors are actually ten bits, but the
** register. The temperature value returned should have a magnitude of 3, * VIA datasheet only considers the 8 MSBs obtained from the regtemp[]
** so we use the VIA scaling as the "true" scaling and use the remaining 2 * register. The temperature value returned should have a magnitude of 3,
** LSBs as fractional precision. * so we use the VIA scaling as the "true" scaling and use the remaining 2
** * LSBs as fractional precision.
** All the on-chip hardware temperature comparisons for the alarms are only *
** 8-bits wide, and compare against the 8 MSBs of the temperature. The bits * All the on-chip hardware temperature comparisons for the alarms are only
** in the registers VT8231_REG_TEMP_LOW01 and VT8231_REG_TEMP_LOW25 are * 8-bits wide, and compare against the 8 MSBs of the temperature. The bits
** ignored. * in the registers VT8231_REG_TEMP_LOW01 and VT8231_REG_TEMP_LOW25 are
* ignored.
*/ */
/******** FAN RPM CONVERSIONS ******** /*
** This chip saturates back at 0, not at 255 like many the other chips. ****** FAN RPM CONVERSIONS ********
** So, 0 means 0 RPM * This chip saturates back at 0, not at 255 like many the other chips.
* So, 0 means 0 RPM
*/ */
static inline u8 FAN_TO_REG(long rpm, int div) static inline u8 FAN_TO_REG(long rpm, int div)
{ {
@ -463,8 +469,9 @@ static ssize_t set_temp_min(struct device *dev, struct device_attribute *attr,
return count; return count;
} }
/* Note that these map the Linux temperature sensor numbering (1-6) to the VIA /*
** temperature sensor numbering (0-5) * Note that these map the Linux temperature sensor numbering (1-6) to the VIA
* temperature sensor numbering (0-5)
*/ */
#define define_temperature_sysfs(offset) \ #define define_temperature_sysfs(offset) \
static SENSOR_DEVICE_ATTR(temp##offset##_input, S_IRUGO, \ static SENSOR_DEVICE_ATTR(temp##offset##_input, S_IRUGO, \
@ -1032,13 +1039,16 @@ static int __devinit vt8231_pci_probe(struct pci_dev *dev,
if (vt8231_device_add(address)) if (vt8231_device_add(address))
goto exit_unregister; goto exit_unregister;
/* Always return failure here. This is to allow other drivers to bind /*
* Always return failure here. This is to allow other drivers to bind
* to this pci device. We don't really want to have control over the * to this pci device. We don't really want to have control over the
* pci device, we only wanted to read as few register values from it. * pci device, we only wanted to read as few register values from it.
*/ */
/* We do, however, mark ourselves as using the PCI device to stop it /*
getting unloaded. */ * We do, however, mark ourselves as using the PCI device to stop it
* getting unloaded.
*/
s_bridge = pci_dev_get(dev); s_bridge = pci_dev_get(dev);
return -ENODEV; return -ENODEV;