From: Guenter Roeck Date: Tue, 11 Dec 2012 20:56:55 +0000 (-0800) Subject: Latest code; sync with proposed kernel version X-Git-Tag: nct6775-v0.9~4 X-Git-Url: https://git.sur5r.net/?p=groeck-nct6775;a=commitdiff_plain;h=bd0a8ec4f0a8daf0365062c79952afd0a13b3488 Latest code; sync with proposed kernel version Many attribute changes. Signed-off-by: Guenter Roeck --- diff --git a/nct6775.c b/nct6775.c index 2320e0b..b47ec0b 100644 --- a/nct6775.c +++ b/nct6775.c @@ -5,7 +5,7 @@ * Copyright (C) 2012 Guenter Roeck * * Derived from w83627ehf driver - * Copyright (C) 2005-2011 Jean Delvare + * Copyright (C) 2005-2012 Jean Delvare * Copyright (C) 2006 Yuan Mu (Winbond), * Rudolf Marek * David Hubbard @@ -35,7 +35,10 @@ * Chip #vin #fan #pwm #temp chip IDs man ID * nct6775f 9 4 3 6+3 0xb470 0xc1 0x5ca3 * nct6776f 9 5 3 6+3 0xc330 0xc1 0x5ca3 - * nct6779d 15 5 5 2+5 0xc560 0xc1 0x5ca3 + * nct6779d 15 5 5 2+6 0xc560 0xc1 0x5ca3 + * + * #temp lists the number of monitored temperature sources (first value) plus + * the number of directly connectable temperature sensors (second value). */ #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt @@ -53,9 +56,8 @@ #include #include #include "lm75.h" -#include "compat.h" -#define TESTING +#define USE_ALTERNATE enum kinds { nct6775, nct6776, nct6779 }; @@ -86,17 +88,16 @@ MODULE_PARM_DESC(fan_debounce, "Enable debouncing for fan RPM signal"); #define SIO_REG_LDSEL 0x07 /* Logical device select */ #define SIO_REG_DEVID 0x20 /* Device ID (2 bytes) */ -#define SIO_REG_EN_VRM10 0x2C /* GPIO3, GPIO4 selection */ #define SIO_REG_ENABLE 0x30 /* Logical device enable */ #define SIO_REG_ADDR 0x60 /* Logical device address (2 bytes) */ -#define SIO_REG_VID_CTRL 0xF0 /* VID control */ -#define SIO_REG_VID_DATA 0xF1 /* VID data */ #define SIO_NCT6775_ID 0xb470 #define SIO_NCT6776_ID 0xc330 #define SIO_NCT6779_ID 0xc560 #define SIO_ID_MASK 0xFFF0 +enum pwm_enable { off, manual, thermal_cruise, speed_cruise, sf3, sf4 }; + static inline void superio_outb(int ioreg, int reg, int val) { @@ -216,8 +217,6 @@ static const u8 NCT6775_PWM_MODE_MASK[] = { 0x01, 0x02, 0x01 }; /* Advanced Fan control, some values are common for all fans */ -static const u16 NCT6775_REG_FAN_PULSES[] = { 0x641, 0x642, 0x643, 0x644, 0 }; - static const u16 NCT6775_REG_TARGET[] = { 0x101, 0x201, 0x301, 0x801, 0x901 }; static const u16 NCT6775_REG_FAN_MODE[] = { 0x102, 0x202, 0x302, 0x802, 0x902 }; static const u16 NCT6775_REG_FAN_STEP_DOWN_TIME[] = { @@ -228,8 +227,6 @@ static const u16 NCT6775_REG_FAN_STOP_OUTPUT[] = { 0x105, 0x205, 0x305, 0x805, 0x905 }; static const u16 NCT6775_REG_FAN_START_OUTPUT[] = { 0x106, 0x206, 0x306, 0x806, 0x906 }; -static const u16 NCT6775_REG_FAN_STEP_ENABLE[] = { - 0x120, 0x220, 0x320, 0x820, 0x920 }; static const u16 NCT6775_REG_FAN_MAX_OUTPUT[] = { 0x10a, 0x20a, 0x30a }; static const u16 NCT6775_REG_FAN_STEP_OUTPUT[] = { 0x10b, 0x20b, 0x30b }; @@ -240,6 +237,7 @@ static const u16 NCT6775_REG_PWM_READ[] = { 0x01, 0x03, 0x11, 0x13, 0x15 }; static const u16 NCT6775_REG_FAN[] = { 0x630, 0x632, 0x634, 0x636, 0x638 }; static const u16 NCT6775_REG_FAN_MIN[] = { 0x3b, 0x3c, 0x3d }; +static const u16 NCT6775_REG_FAN_PULSES[] = { 0x641, 0x642, 0x643, 0x644, 0 }; static const u16 NCT6775_REG_TEMP[] = { 0x27, 0x150, 0x250, 0x62b, 0x62c, 0x62d }; @@ -327,11 +325,14 @@ static const s8 NCT6776_ALARM_BITS[] = { 4, 5, 13, -1, -1, -1, /* temp1..temp6 */ 12, 9 }; /* intrusion0, intrusion1 */ +static const u16 NCT6776_REG_TOLERANCE_H[] = { + 0x10c, 0x20c, 0x30c, 0x80c, 0x90c }; + static const u8 NCT6776_REG_PWM_MODE[] = { 0x04, 0, 0 }; static const u8 NCT6776_PWM_MODE_MASK[] = { 0x01, 0, 0 }; -static const u16 NCT6776_REG_FAN_PULSES[] = { 0x644, 0x645, 0x646, 0, 0 }; static const u16 NCT6776_REG_FAN_MIN[] = { 0x63a, 0x63c, 0x63e, 0x640, 0x642 }; +static const u16 NCT6776_REG_FAN_PULSES[] = { 0x644, 0x645, 0x646, 0, 0 }; static const u16 NCT6776_REG_WEIGHT_DUTY_BASE[] = { 0x13e, 0x23e, 0x33e, 0x83e, 0x93e }; @@ -389,14 +390,10 @@ static const s8 NCT6779_ALARM_BITS[] = { 4, 5, 13, -1, -1, -1, /* temp1..temp6 */ 12, 9 }; /* intrusion0, intrusion1 */ +static const u16 NCT6779_REG_FAN[] = { 0x4b0, 0x4b2, 0x4b4, 0x4b6, 0x4b8 }; static const u16 NCT6779_REG_FAN_PULSES[] = { 0x644, 0x645, 0x646, 0x647, 0x648 }; -static const u16 NCT6779_REG_FAN[] = { 0x4b0, 0x4b2, 0x4b4, 0x4b6, 0x4b8 }; - -static const u16 NCT6779_REG_TOLERANCE_H[] = { - 0x10c, 0x20c, 0x30c, 0x80c, 0x90c }; - static const u16 NCT6779_REG_CRITICAL_PWM_ENABLE[] = { 0x136, 0x236, 0x336, 0x836, 0x936 }; static const u16 NCT6779_REG_CRITICAL_PWM[] = { @@ -449,16 +446,16 @@ static const u16 NCT6779_REG_TEMP_ALTERNATE[ARRAY_SIZE(nct6779_temp_label) - 1] static const u16 NCT6779_REG_TEMP_CRIT[ARRAY_SIZE(nct6779_temp_label) - 1] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0x709, 0x70a }; -static inline int reg_to_pwm_enable(int pwm, int mode) +static enum pwm_enable reg_to_pwm_enable(int pwm, int mode) { if (mode == 0 && pwm == 255) - return 0; /* off */ + return off; return mode + 1; } -static inline int pwm_enable_to_reg(int mode) +static int pwm_enable_to_reg(enum pwm_enable mode) { - if (mode == 0) + if (mode == off) return 0; return mode - 1; } @@ -468,12 +465,12 @@ static inline int pwm_enable_to_reg(int mode) */ /* 1 is DC mode, output in ms */ -static inline unsigned int step_time_from_reg(u8 reg, u8 mode) +static unsigned int step_time_from_reg(u8 reg, u8 mode) { return mode ? 400 * reg : 100 * reg; } -static inline u8 step_time_to_reg(unsigned int msec, u8 mode) +static u8 step_time_to_reg(unsigned int msec, u8 mode) { return SENSORS_LIMIT((mode ? (msec + 200) / 400 : (msec + 50) / 100), 1, 255); @@ -511,6 +508,14 @@ static unsigned int fan_from_reg16(u16 reg, unsigned int divreg) return 1350000U / (reg << divreg); } +static u16 fan_to_reg(u32 fan, unsigned int divreg) +{ + if (!fan) + return 0; + + return (1350000U / fan) >> divreg; +} + static inline unsigned int div_from_reg(u8 reg) { @@ -570,13 +575,16 @@ struct nct6775_data { const u16 *REG_FAN; const u16 *REG_FAN_MODE; const u16 *REG_FAN_MIN; + const u16 *REG_FAN_PULSES; const u16 *REG_FAN_TIME[3]; + const u16 *REG_TOLERANCE_H; + const u8 *REG_PWM_MODE; const u8 *PWM_MODE_MASK; - const u16 *REG_PWM[7]; /* [0]=pwm, [1]=start_output, [2]=stop_output, - * [3]=max_output, [4]=step_output, + const u16 *REG_PWM[7]; /* [0]=pwm, [1]=pwm_start, [2]=pwm_floor, + * [3]=pwm_max, [4]=pwm_step, * [5]=weight_duty_step, [6]=weight_duty_base */ const u16 *REG_PWM_READ; @@ -588,11 +596,9 @@ struct nct6775_data { const u16 *REG_CRITICAL_TEMP_TOLERANCE; const u16 *REG_TEMP_SOURCE; /* temp register sources */ - const u16 *REG_TEMP_SEL[3]; /* pwm temp: - * 0=base, 1=weight, 2=step enable - */ - - const u16 *REG_WEIGHT_TEMP[3]; /* 0=base, 1=hyst, 2=step */ + const u16 *REG_TEMP_SEL; + const u16 *REG_WEIGHT_TEMP_SEL; + const u16 *REG_WEIGHT_TEMP[3]; /* 0=base, 1=tolerance, 2=step */ const u16 *REG_TEMP_OFFSET; @@ -602,7 +608,7 @@ struct nct6775_data { unsigned int (*fan_from_reg_min)(u16 reg, unsigned int divreg); struct mutex update_lock; - char valid; /* !=0 if following fields are valid */ + bool valid; /* true if following fields are valid */ unsigned long last_updated; /* In jiffies */ /* Register values */ @@ -611,6 +617,7 @@ struct nct6775_data { u8 in[15][3]; /* [0]=in, [1]=in_max, [2]=in_min */ unsigned int rpm[5]; u16 fan_min[5]; + u8 fan_pulses[5]; u8 fan_div[5]; u8 has_pwm; u8 has_fan; /* some fan inputs can be disabled */ @@ -625,21 +632,28 @@ struct nct6775_data { u64 alarms; u8 pwm_num; /* number of pwm */ - u8 pwm_mode[5]; /* 1->DC variable voltage, 0->PWM variable duty cycle */ - u8 pwm_enable[5]; /* 0->off - * 1->manual - * 2->thermal cruise mode (also called SmartFan I) - * 3->fan speed cruise mode - * 4->SmartFan III - * 5->enhanced variable thermal cruise (SmartFan IV) - */ - u8 pwm[7][5]; /* [0]=pwm, [1]=start_output, [2]=stop_output, - * [3]=max_output, [4]=step_output, + enum pwm_enable pwm_enable[5]; + /* 0->off + * 1->manual + * 2->thermal cruise mode (also called SmartFan I) + * 3->fan speed cruise mode + * 4->SmartFan III + * 5->enhanced variable thermal cruise (SmartFan IV) + */ + u8 pwm[7][5]; /* [0]=pwm, [1]=pwm_start, [2]=pwm_floor, + * [3]=pwm_max, [4]=pwm_step, * [5]=weight_duty_step, [6]=weight_duty_base */ + u8 target_temp[5]; - u8 tolerance[5][2]; + u8 target_temp_mask; + u32 target_speed[5]; + u32 target_speed_tolerance[5]; + u8 speed_tolerance_limit; + + u8 temp_tolerance[2][5]; + u8 tolerance_mask; u8 fan_time[3][5]; /* 0 = stop_time, 1 = step_up, 2 = step_down */ @@ -647,12 +661,8 @@ struct nct6775_data { int auto_pwm_num; u8 auto_pwm[5][7]; u8 auto_temp[5][7]; - - u8 pwm_temp_sel[2][5]; - - bool pwm_sel_enable[3][5];/* 0->stop_val, 1->weight, 2->step; - * false->off, true->on - */ + u8 pwm_temp_sel[5]; + u8 pwm_weight_temp_sel[5]; u8 weight_temp[3][5]; /* 0->temp_step, 1->temp_step_tol, * 2->temp_base */ @@ -663,6 +673,12 @@ struct nct6775_data { u16 have_temp; u16 have_temp_fixed; u16 have_in; +#ifdef CONFIG_PM + /* Remember extra register values over suspend/resume */ + u8 vbat; + u8 fandiv1; + u8 fandiv2; +#endif }; struct nct6775_sio_data { @@ -809,8 +825,7 @@ static void nct6775_write_fan_div(struct nct6775_data *data, int nr) } } -static void nct6775_write_fan_div_common(struct device *dev, - struct nct6775_data *data, int nr) +static void nct6775_write_fan_div_common(struct nct6775_data *data, int nr) { if (data->kind == nct6775) nct6775_write_fan_div(data, nr); @@ -829,13 +844,104 @@ static void nct6775_update_fan_div(struct nct6775_data *data) data->fan_div[3] = (i & 0x70) >> 4; } -static void nct6775_update_fan_div_common(struct device *dev, - struct nct6775_data *data) +static void nct6775_update_fan_div_common(struct nct6775_data *data) { if (data->kind == nct6775) nct6775_update_fan_div(data); } +static void nct6775_init_fan_div(struct nct6775_data *data) +{ + int i; + + nct6775_update_fan_div_common(data); + /* + * For all fans, start with highest divider value if the divider + * register is not innitialized. This ensures that we get a + * reading from the fan count register, even if it is not optimal. + * We'll compute a better divider later on. + */ + for (i = 0; i < 3; i++) { + if (!(data->has_fan & (1 << i))) + continue; + if (data->fan_div[i] == 0) { + data->fan_div[i] = 7; + nct6775_write_fan_div_common(data, i); + } + } +} + +static void nct6775_init_fan_common(struct device *dev, + struct nct6775_data *data) +{ + int i; + u8 reg; + + if (data->has_fan_div) + nct6775_init_fan_div(data); + + /* + * If fan_min is not set (0), set it to 0xff to disable it. This + * prevents the unnecessary warning when fanX_min is reported as 0. + */ + for (i = 0; i < 5; i++) { + if (data->has_fan_min & (1 << i)) { + reg = nct6775_read_value(data, data->REG_FAN_MIN[i]); + if (!reg) + nct6775_write_value(data, data->REG_FAN_MIN[i], + 0xff); + } + } +} + +static void nct6775_select_fan_div(struct device *dev, + struct nct6775_data *data, int nr, u16 reg) +{ + u8 fan_div = data->fan_div[nr]; + u16 fan_min; + + if (!data->has_fan_div) + return; + + /* + * If we failed to measure the fan speed, or the reported value is not + * in the optimal range, and the clock divider can be modified, + * let's try that for next time. + */ + if (reg == 0x00 && fan_div < 0x07) + fan_div++; + else if (reg != 0x00 && reg < 0x30 && fan_div > 0) + fan_div--; + + if (fan_div != data->fan_div[nr]) { + dev_dbg(dev, "Modifying fan%d clock divider from %u to %u\n", + nr + 1, div_from_reg(data->fan_div[nr]), + div_from_reg(fan_div)); + + /* Preserve min limit if possible */ + if (data->has_fan_min & (1 << nr)) { + fan_min = data->fan_min[nr]; + if (fan_div > data->fan_div[nr]) { + if (fan_min != 255 && fan_min > 1) + fan_min >>= 1; + } else { + if (fan_min != 255) { + fan_min <<= 1; + if (fan_min > 254) + fan_min = 254; + } + } + if (fan_min != data->fan_min[nr]) { + data->fan_min[nr] = fan_min; + nct6775_write_value(data, data->REG_FAN_MIN[nr], + fan_min); + } + } + data->fan_div[nr] = fan_div; + nct6775_write_fan_div_common(data, nr); + } +} + static void nct6775_update_pwm(struct device *dev) { struct nct6775_data *data = dev_get_drvdata(dev); @@ -864,32 +970,35 @@ static void nct6775_update_pwm(struct device *dev) data->pwm_enable[i] = reg_to_pwm_enable(data->pwm[0][i], (fanmodecfg >> 4) & 7); - data->tolerance[i][0] = fanmodecfg & 0x0f; - if (data->kind == nct6779) { - reg = nct6775_read_value(data, - NCT6779_REG_TOLERANCE_H[i]); - data->tolerance[i][0] |= (reg & 0x70) >> 1; + if (!data->temp_tolerance[0][i] || + data->pwm_enable[i] != speed_cruise) + data->temp_tolerance[0][i] = fanmodecfg & 0x0f; + if (!data->target_speed_tolerance[i] || + data->pwm_enable[i] == speed_cruise) { + u8 t = fanmodecfg & 0x0f; + if (data->REG_TOLERANCE_H) { + t |= (nct6775_read_value(data, + data->REG_TOLERANCE_H[i]) & 0x70) >> 1; + } + data->target_speed_tolerance[i] = t; } - data->tolerance[i][1] = + + data->temp_tolerance[1][i] = nct6775_read_value(data, data->REG_CRITICAL_TEMP_TOLERANCE[i]); - data->pwm_sel_enable[0][i] = - nct6775_read_value(data, data->REG_TEMP_SEL[0][i]) - & 0x80; - data->pwm_sel_enable[1][i] = - nct6775_read_value(data, data->REG_TEMP_SEL[1][i]) - & 0x80; - data->pwm_sel_enable[2][i] = - nct6775_read_value(data, data->REG_TEMP_SEL[2][i]) - & 0x01; - - /* Weight data */ - for (j = 0; j < 2; j++) { - reg = nct6775_read_value(data, - data->REG_TEMP_SEL[j][i]); - data->pwm_temp_sel[j][i] = reg & 0x1f; - } + reg = nct6775_read_value(data, data->REG_TEMP_SEL[i]); + data->pwm_temp_sel[i] = reg & 0x1f; + /* If fan can stop, report floor as 0 */ + if (reg & 0x80) + data->pwm[2][i] = 0; + + reg = nct6775_read_value(data, data->REG_WEIGHT_TEMP_SEL[i]); + data->pwm_weight_temp_sel[i] = reg & 0x1f; + /* If weight is disabled, report weight source as 0 */ + if (j == 1 && !(reg & 0x80)) + data->pwm_weight_temp_sel[i] = 0; + /* Weight temp data */ for (j = 0; j < 3; j++) { data->weight_temp[j][i] @@ -904,6 +1013,7 @@ static void nct6775_update_pwm_limits(struct device *dev) struct nct6775_data *data = dev_get_drvdata(dev); int i, j; u8 reg; + u16 reg_t; for (i = 0; i < data->pwm_num; i++) { if (!(data->has_pwm & (1 << i))) @@ -914,9 +1024,19 @@ static void nct6775_update_pwm_limits(struct device *dev) nct6775_read_value(data, data->REG_FAN_TIME[j][i]); } - data->target_temp[i] = - nct6775_read_value(data, data->REG_TARGET[i]) & - (data->pwm_mode[i] ? 0xff : 0x7f); + reg_t = nct6775_read_value(data, data->REG_TARGET[i]); + /* Update only in matching mode or if never updated */ + if (!data->target_temp[i] || + data->pwm_enable[i] == thermal_cruise) + data->target_temp[i] = reg_t & data->target_temp_mask; + if (!data->target_speed[i] || + data->pwm_enable[i] == speed_cruise) { + if (data->REG_TOLERANCE_H) { + reg_t |= (nct6775_read_value(data, + data->REG_TOLERANCE_H[i]) & 0x0f) << 8; + } + data->target_speed[i] = reg_t; + } for (j = 0; j < data->auto_pwm_num; j++) { data->auto_pwm[i][j] = @@ -965,7 +1085,7 @@ static struct nct6775_data *nct6775_update_device(struct device *dev) if (time_after(jiffies, data->last_updated + HZ + HZ/2) || !data->valid) { /* Fan clock dividers */ - nct6775_update_fan_div_common(dev, data); + nct6775_update_fan_div_common(data); /* Measured voltages and limits */ for (i = 0; i < data->in_num; i++) { @@ -994,28 +1114,10 @@ static struct nct6775_data *nct6775_update_device(struct device *dev) if (data->has_fan_min & (1 << i)) data->fan_min[i] = nct6775_read_value(data, data->REG_FAN_MIN[i]); + data->fan_pulses[i] = + nct6775_read_value(data, data->REG_FAN_PULSES[i]); - /* - * If we failed to measure the fan speed and clock - * divider can be increased, let's try that for next - * time - */ - if (data->has_fan_div && (reg >= 0xff || reg == 0x00) - && data->fan_div[i] < 0x07) { - dev_dbg(dev, - "Increasing fan%d clock divider from %u to %u\n", - i + 1, div_from_reg(data->fan_div[i]), - div_from_reg(data->fan_div[i] + 1)); - data->fan_div[i]++; - nct6775_write_fan_div_common(dev, data, i); - /* Preserve min limit if possible */ - if ((data->has_fan_min & (1 << i)) - && data->fan_min[i] >= 2 - && data->fan_min[i] != 255) - nct6775_write_value(data, - data->REG_FAN_MIN[i], - (data->fan_min[i] /= 2)); - } + nct6775_select_fan_div(dev, data, i, reg); } nct6775_update_pwm(dev); @@ -1047,7 +1149,7 @@ static struct nct6775_data *nct6775_update_device(struct device *dev) } data->last_updated = jiffies; - data->valid = 1; + data->valid = true; } mutex_unlock(&data->update_lock); @@ -1428,7 +1530,7 @@ write_div: nr + 1, div_from_reg(data->fan_div[nr]), div_from_reg(new_div)); data->fan_div[nr] = new_div; - nct6775_write_fan_div_common(dev, data, nr); + nct6775_write_fan_div_common(data, nr); /* Give the chip time to sample a new speed value */ data->last_updated = jiffies; } @@ -1440,6 +1542,41 @@ write_min: return count; } +static ssize_t +show_fan_pulses(struct device *dev, struct device_attribute *attr, char *buf) +{ + struct nct6775_data *data = nct6775_update_device(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int p = data->fan_pulses[sattr->index]; + + return sprintf(buf, "%d\n", p ? : 4); +} + +static ssize_t +store_fan_pulses(struct device *dev, struct device_attribute *attr, + const char *buf, size_t count) +{ + struct nct6775_data *data = dev_get_drvdata(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int nr = sattr->index; + unsigned long val; + int err; + + err = kstrtoul(buf, 10, &val); + if (err < 0) + return err; + + if (val > 4) + return -EINVAL; + + mutex_lock(&data->update_lock); + data->fan_pulses[nr] = val & 3; + nct6775_write_value(data, data->REG_FAN_PULSES[nr], val & 3); + mutex_unlock(&data->update_lock); + + return count; +} + static struct sensor_device_attribute sda_fan_input[] = { SENSOR_ATTR(fan1_input, S_IRUGO, show_fan, NULL, 0), SENSOR_ATTR(fan2_input, S_IRUGO, show_fan, NULL, 1), @@ -1469,6 +1606,19 @@ static struct sensor_device_attribute sda_fan_min[] = { store_fan_min, 4), }; +static struct sensor_device_attribute sda_fan_pulses[] = { + SENSOR_ATTR(fan1_pulses, S_IWUSR | S_IRUGO, show_fan_pulses, + store_fan_pulses, 0), + SENSOR_ATTR(fan2_pulses, S_IWUSR | S_IRUGO, show_fan_pulses, + store_fan_pulses, 1), + SENSOR_ATTR(fan3_pulses, S_IWUSR | S_IRUGO, show_fan_pulses, + store_fan_pulses, 2), + SENSOR_ATTR(fan4_pulses, S_IWUSR | S_IRUGO, show_fan_pulses, + store_fan_pulses, 3), + SENSOR_ATTR(fan5_pulses, S_IWUSR | S_IRUGO, show_fan_pulses, + store_fan_pulses, 4), +}; + static struct sensor_device_attribute sda_fan_div[] = { SENSOR_ATTR(fan1_div, S_IRUGO, show_fan_div, NULL, 0), SENSOR_ATTR(fan2_div, S_IRUGO, show_fan_div, NULL, 1), @@ -1791,15 +1941,6 @@ store_pwm_mode(struct device *dev, struct device_attribute *attr, return count; } -static ssize_t -show_pwm_enable(struct device *dev, struct device_attribute *attr, char *buf) -{ - struct nct6775_data *data = nct6775_update_device(dev); - struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); - - return sprintf(buf, "%d\n", data->pwm_enable[sattr->index]); -} - static ssize_t show_pwm(struct device *dev, struct device_attribute *attr, char *buf) { @@ -1813,7 +1954,7 @@ show_pwm(struct device *dev, struct device_attribute *attr, char *buf) * For automatic fan control modes, show current pwm readings. * Otherwise, show the configured value. */ - if (index == 0 && data->pwm_enable[nr] > 1) + if (index == 0 && data->pwm_enable[nr] > manual) pwm = nct6775_read_value(data, data->REG_PWM_READ[nr]); else pwm = data->pwm[index][nr]; @@ -1834,6 +1975,7 @@ store_pwm(struct device *dev, struct device_attribute *attr, const char *buf, int maxval[7] = { 255, 255, data->pwm[3][nr] ? : 255, 255, 255, 255, 255 }; int err; + u8 reg; err = kstrtoul(buf, 10, &val); if (err < 0) @@ -1843,6 +1985,13 @@ store_pwm(struct device *dev, struct device_attribute *attr, const char *buf, mutex_lock(&data->update_lock); data->pwm[index][nr] = val; nct6775_write_value(data, data->REG_PWM[index][nr], val); + if (index == 2) { /* floor: disable if val == 0 */ + reg = nct6775_read_value(data, data->REG_TEMP_SEL[nr]); + reg &= 0x7f; + if (val) + reg |= 0x80; + nct6775_write_value(data, data->REG_TEMP_SEL[nr], reg); + } mutex_unlock(&data->update_lock); return count; } @@ -1871,6 +2020,51 @@ static int check_trip_points(struct nct6775_data *data, int nr) return 0; } +static void pwm_update_registers(struct nct6775_data *data, int nr) +{ + u8 reg; + + switch (data->pwm_enable[nr]) { + case off: + case manual: + break; + case speed_cruise: + reg = nct6775_read_value(data, data->REG_FAN_MODE[nr]); + reg = (reg & ~data->tolerance_mask) | + (data->target_speed_tolerance[nr] & data->tolerance_mask); + nct6775_write_value(data, data->REG_FAN_MODE[nr], reg); + nct6775_write_value(data, data->REG_TARGET[nr], + data->target_speed[nr] & 0xff); + if (data->REG_TOLERANCE_H) { + reg = (data->target_speed[nr] >> 8) & 0x0f; + reg |= (data->target_speed_tolerance[nr] & 0x38) << 1; + nct6775_write_value(data, + data->REG_TOLERANCE_H[nr], + reg); + } + break; + case thermal_cruise: + nct6775_write_value(data, data->REG_TARGET[nr], + data->target_temp[nr]); + /* intentional */ + default: + reg = nct6775_read_value(data, data->REG_FAN_MODE[nr]); + reg = (reg & ~data->tolerance_mask) | + data->temp_tolerance[0][nr]; + nct6775_write_value(data, data->REG_FAN_MODE[nr], reg); + break; + } +} + +static ssize_t +show_pwm_enable(struct device *dev, struct device_attribute *attr, char *buf) +{ + struct nct6775_data *data = nct6775_update_device(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + + return sprintf(buf, "%d\n", data->pwm_enable[sattr->index]); +} + static ssize_t store_pwm_enable(struct device *dev, struct device_attribute *attr, const char *buf, size_t count) @@ -1886,13 +2080,13 @@ store_pwm_enable(struct device *dev, struct device_attribute *attr, if (err < 0) return err; - if (val > 5) + if (val > sf4) return -EINVAL; - if (val == 4 && data->kind != nct6775) + if (val == sf3 && data->kind != nct6775) return -EINVAL; - if (val == 5 && check_trip_points(data, nr)) { + if (val == sf4 && check_trip_points(data, nr)) { dev_err(dev, "Inconsistent trip points, not switching to SmartFan IV mode\n"); dev_err(dev, "Adjust trip points and try again\n"); return -EINVAL; @@ -1900,13 +2094,14 @@ store_pwm_enable(struct device *dev, struct device_attribute *attr, mutex_lock(&data->update_lock); data->pwm_enable[nr] = val; - if (!val) { + if (val == off) { /* * turn off pwm control: select manual mode, set pwm to maximum */ data->pwm[0][nr] = 255; nct6775_write_value(data, data->REG_PWM[0][nr], 255); } + pwm_update_registers(data, nr); reg = nct6775_read_value(data, data->REG_FAN_MODE[nr]); reg &= 0x0f; reg |= pwm_enable_to_reg(val) << 4; @@ -1916,13 +2111,9 @@ store_pwm_enable(struct device *dev, struct device_attribute *attr, } static ssize_t -show_pwm_temp_sel(struct device *dev, struct device_attribute *attr, char *buf) +show_pwm_temp_sel_common(struct nct6775_data *data, char *buf, int src) { - struct nct6775_data *data = nct6775_update_device(dev); - struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - int i, src, sel = 0; - - src = data->pwm_temp_sel[sattr->index][sattr->nr]; + int i, sel = 0; for (i = 0; i < NUM_TEMP; i++) { if (!(data->have_temp & (1 << i))) @@ -1936,14 +2127,23 @@ show_pwm_temp_sel(struct device *dev, struct device_attribute *attr, char *buf) return sprintf(buf, "%d\n", sel); } +static ssize_t +show_pwm_temp_sel(struct device *dev, struct device_attribute *attr, char *buf) +{ + struct nct6775_data *data = nct6775_update_device(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int index = sattr->index; + + return show_pwm_temp_sel_common(data, buf, data->pwm_temp_sel[index]); +} + static ssize_t store_pwm_temp_sel(struct device *dev, struct device_attribute *attr, const char *buf, size_t count) { struct nct6775_data *data = nct6775_update_device(dev); - struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - int nr = sattr->nr; - int index = sattr->index; + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int nr = sattr->index; unsigned long val; int err, reg, src; @@ -1957,11 +2157,61 @@ store_pwm_temp_sel(struct device *dev, struct device_attribute *attr, mutex_lock(&data->update_lock); src = data->temp_src[val - 1]; - data->pwm_temp_sel[index][nr] = src; - reg = nct6775_read_value(data, data->REG_TEMP_SEL[index][nr]); + data->pwm_temp_sel[nr] = src; + reg = nct6775_read_value(data, data->REG_TEMP_SEL[nr]); reg &= 0xe0; reg |= src; - nct6775_write_value(data, data->REG_TEMP_SEL[index][nr], reg); + nct6775_write_value(data, data->REG_TEMP_SEL[nr], reg); + mutex_unlock(&data->update_lock); + + return count; +} + +static ssize_t +show_pwm_weight_temp_sel(struct device *dev, struct device_attribute *attr, + char *buf) +{ + struct nct6775_data *data = nct6775_update_device(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int index = sattr->index; + + return show_pwm_temp_sel_common(data, buf, + data->pwm_weight_temp_sel[index]); +} + +static ssize_t +store_pwm_weight_temp_sel(struct device *dev, struct device_attribute *attr, + const char *buf, size_t count) +{ + struct nct6775_data *data = nct6775_update_device(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int nr = sattr->index; + unsigned long val; + int err, reg, src; + + err = kstrtoul(buf, 10, &val); + if (err < 0) + return err; + if (val > NUM_TEMP) + return -EINVAL; + if (val && (!(data->have_temp & (1 << (val - 1))) || + !data->temp_src[val - 1])) + return -EINVAL; + + mutex_lock(&data->update_lock); + if (val) { + src = data->temp_src[val - 1]; + data->pwm_weight_temp_sel[nr] = src; + reg = nct6775_read_value(data, data->REG_WEIGHT_TEMP_SEL[nr]); + reg &= 0xe0; + reg |= (src | 0x80); + nct6775_write_value(data, data->REG_WEIGHT_TEMP_SEL[nr], reg); + } else { + data->pwm_weight_temp_sel[nr] = 0; + reg = nct6775_read_value(data, data->REG_WEIGHT_TEMP_SEL[nr]); + reg &= 0x7f; + nct6775_write_value(data, data->REG_WEIGHT_TEMP_SEL[nr], reg); + } mutex_unlock(&data->update_lock); return count; @@ -1973,8 +2223,7 @@ show_target_temp(struct device *dev, struct device_attribute *attr, char *buf) struct nct6775_data *data = nct6775_update_device(dev); struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); - return sprintf(buf, "%d\n", - data->target_temp[sattr->index] * 1000); + return sprintf(buf, "%d\n", data->target_temp[sattr->index] * 1000); } static ssize_t @@ -1984,80 +2233,165 @@ store_target_temp(struct device *dev, struct device_attribute *attr, struct nct6775_data *data = dev_get_drvdata(dev); struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); int nr = sattr->index; - long val; + unsigned long val; int err; - err = kstrtol(buf, 10, &val); + err = kstrtoul(buf, 10, &val); if (err < 0) return err; - val = SENSORS_LIMIT(DIV_ROUND_CLOSEST(val, 1000), 0, 127); + val = SENSORS_LIMIT(DIV_ROUND_CLOSEST(val, 1000), 0, + data->target_temp_mask); mutex_lock(&data->update_lock); data->target_temp[nr] = val; - nct6775_write_value(data, data->REG_TARGET[nr], val); + pwm_update_registers(data, nr); mutex_unlock(&data->update_lock); return count; } static ssize_t -show_auto_temp_hyst(struct device *dev, struct device_attribute *attr, +show_target_speed(struct device *dev, struct device_attribute *attr, char *buf) +{ + struct nct6775_data *data = nct6775_update_device(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int nr = sattr->index; + + return sprintf(buf, "%d\n", + fan_from_reg16(data->target_speed[nr], + data->fan_div[nr])); +} + +static ssize_t +store_target_speed(struct device *dev, struct device_attribute *attr, + const char *buf, size_t count) +{ + struct nct6775_data *data = dev_get_drvdata(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int nr = sattr->index; + unsigned long val; + int err; + u16 speed; + + err = kstrtoul(buf, 10, &val); + if (err < 0) + return err; + + val = SENSORS_LIMIT(val, 0, 1350000U); + speed = fan_to_reg(val, data->fan_div[nr]); + + mutex_lock(&data->update_lock); + data->target_speed[nr] = speed; + pwm_update_registers(data, nr); + mutex_unlock(&data->update_lock); + return count; +} + +static ssize_t +show_temp_tolerance(struct device *dev, struct device_attribute *attr, char *buf) { struct nct6775_data *data = nct6775_update_device(dev); struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); int nr = sattr->nr; - int point = sattr->index >= data->auto_pwm_num ? 1 : 0; - int tolerance = data->tolerance[nr][point]; - int temp = data->auto_temp[nr][sattr->index]; + int index = sattr->index; - return sprintf(buf, "%d\n", (temp - tolerance) * 1000); + return sprintf(buf, "%d\n", data->temp_tolerance[index][nr] * 1000); } static ssize_t -store_auto_temp_hyst(struct device *dev, struct device_attribute *attr, +store_temp_tolerance(struct device *dev, struct device_attribute *attr, const char *buf, size_t count) { struct nct6775_data *data = dev_get_drvdata(dev); struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); int nr = sattr->nr; - int point = sattr->index >= data->auto_pwm_num ? 1 : 0; - u16 reg; - long val; + int index = sattr->index; + unsigned long val; int err; - int maxlimit[2][3] = { { 15, 7, 63 }, { 15, 7, 7 } }; - int mask[] = { 0x0f, 0x07, 0x07 }; - int temp; - err = kstrtol(buf, 10, &val); + err = kstrtoul(buf, 10, &val); if (err < 0) return err; - temp = data->auto_temp[nr][sattr->index]; - val = temp - DIV_ROUND_CLOSEST(val, 1000); - /* Limit tolerance as needed */ - val = SENSORS_LIMIT(val, 0, maxlimit[point][data->kind]); + val = SENSORS_LIMIT(DIV_ROUND_CLOSEST(val, 1000), 0, + data->tolerance_mask); mutex_lock(&data->update_lock); - if (point) { + data->temp_tolerance[index][nr] = val; + if (index) + pwm_update_registers(data, nr); + else nct6775_write_value(data, data->REG_CRITICAL_TEMP_TOLERANCE[nr], val); - } else { - reg = nct6775_read_value(data, data->REG_FAN_MODE[nr]); - reg = (reg & ~mask[nr]) | (val & mask[nr]); - nct6775_write_value(data, data->REG_FAN_MODE[nr], reg); - if (data->kind == nct6779) { - reg = nct6775_read_value(data, - NCT6779_REG_TOLERANCE_H[nr]); - reg = (reg & 0x70) | ((val & 0x38) << 1); - nct6775_write_value(data, - NCT6779_REG_TOLERANCE_H[nr], reg); - } - } + mutex_unlock(&data->update_lock); + return count; +} + +/* + * Fan speed tolerance is a tricky beast, since the associated register is + * a tick counter, but the value is reported and configured as rpm. + * Compute resulting low and high rpm values and report the difference. + */ +static ssize_t +show_speed_tolerance(struct device *dev, struct device_attribute *attr, + char *buf) +{ + struct nct6775_data *data = nct6775_update_device(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int nr = sattr->index; + int low = data->target_speed[nr] - data->target_speed_tolerance[nr]; + int high = data->target_speed[nr] + data->target_speed_tolerance[nr]; + int tolerance; + + if (low <= 0) + low = 1; + if (high > 0xffff) + high = 0xffff; + if (high < low) + high = low; + + tolerance = (fan_from_reg16(low, data->fan_div[nr]) + - fan_from_reg16(high, data->fan_div[nr])) / 2; + + return sprintf(buf, "%d\n", tolerance); +} - data->tolerance[nr][point] = val; +static ssize_t +store_speed_tolerance(struct device *dev, struct device_attribute *attr, + const char *buf, size_t count) +{ + struct nct6775_data *data = dev_get_drvdata(dev); + struct sensor_device_attribute *sattr = to_sensor_dev_attr(attr); + int nr = sattr->index; + unsigned long val; + int err; + int low, high; + + err = kstrtoul(buf, 10, &val); + if (err < 0) + return err; + + high = fan_from_reg16(data->target_speed[nr], + data->fan_div[nr]) + val; + low = fan_from_reg16(data->target_speed[nr], + data->fan_div[nr]) - val; + if (low <= 0) + low = 1; + if (high < low) + high = low; + + val = (fan_to_reg(low, data->fan_div[nr]) - + fan_to_reg(high, data->fan_div[nr])) / 2; + + /* Limit tolerance as needed */ + val = SENSORS_LIMIT(val, 0, data->speed_tolerance_limit); + + mutex_lock(&data->update_lock); + data->target_speed_tolerance[nr] = val; + pwm_update_registers(data, nr); mutex_unlock(&data->update_lock); return count; } @@ -2090,72 +2424,51 @@ static SENSOR_DEVICE_ATTR(pwm4_enable, S_IWUSR | S_IRUGO, show_pwm_enable, static SENSOR_DEVICE_ATTR(pwm5_enable, S_IWUSR | S_IRUGO, show_pwm_enable, store_pwm_enable, 4); -static SENSOR_DEVICE_ATTR_2(pwm1_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 0, 0); -static SENSOR_DEVICE_ATTR_2(pwm2_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 1, 0); -static SENSOR_DEVICE_ATTR_2(pwm3_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 2, 0); -static SENSOR_DEVICE_ATTR_2(pwm4_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 3, 0); -static SENSOR_DEVICE_ATTR_2(pwm5_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 4, 0); - -static SENSOR_DEVICE_ATTR(pwm1_target, S_IWUSR | S_IRUGO, show_target_temp, +static SENSOR_DEVICE_ATTR(pwm1_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_temp_sel, store_pwm_temp_sel, 0); +static SENSOR_DEVICE_ATTR(pwm2_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_temp_sel, store_pwm_temp_sel, 1); +static SENSOR_DEVICE_ATTR(pwm3_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_temp_sel, store_pwm_temp_sel, 2); +static SENSOR_DEVICE_ATTR(pwm4_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_temp_sel, store_pwm_temp_sel, 3); +static SENSOR_DEVICE_ATTR(pwm5_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_temp_sel, store_pwm_temp_sel, 4); + +static SENSOR_DEVICE_ATTR(pwm1_target_temp, S_IWUSR | S_IRUGO, show_target_temp, store_target_temp, 0); -static SENSOR_DEVICE_ATTR(pwm2_target, S_IWUSR | S_IRUGO, show_target_temp, +static SENSOR_DEVICE_ATTR(pwm2_target_temp, S_IWUSR | S_IRUGO, show_target_temp, store_target_temp, 1); -static SENSOR_DEVICE_ATTR(pwm3_target, S_IWUSR | S_IRUGO, show_target_temp, +static SENSOR_DEVICE_ATTR(pwm3_target_temp, S_IWUSR | S_IRUGO, show_target_temp, store_target_temp, 2); -static SENSOR_DEVICE_ATTR(pwm4_target, S_IWUSR | S_IRUGO, show_target_temp, +static SENSOR_DEVICE_ATTR(pwm4_target_temp, S_IWUSR | S_IRUGO, show_target_temp, store_target_temp, 3); -static SENSOR_DEVICE_ATTR(pwm5_target, S_IWUSR | S_IRUGO, show_target_temp, +static SENSOR_DEVICE_ATTR(pwm5_target_temp, S_IWUSR | S_IRUGO, show_target_temp, store_target_temp, 4); -/* Smart Fan registers */ - -static ssize_t -show_pwm_sel_enable(struct device *dev, struct device_attribute *attr, - char *buf) -{ - struct nct6775_data *data = nct6775_update_device(dev); - struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - - return sprintf(buf, "%d\n", - data->pwm_sel_enable[sattr->index][sattr->nr]); -} - -static ssize_t -store_pwm_sel_enable(struct device *dev, struct device_attribute *attr, - const char *buf, size_t count) -{ - struct nct6775_data *data = dev_get_drvdata(dev); - struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - int nr = sattr->nr; - int index = sattr->index; - unsigned long val; - int err; - u8 reg; - static const u8 bit[] = { 0x80, 0x80, 0x01 }; - - - err = kstrtoul(buf, 10, &val); - if (err < 0) - return err; - - if (val > 1) - return -EINVAL; +static SENSOR_DEVICE_ATTR(fan1_target, S_IWUSR | S_IRUGO, show_target_speed, + store_target_speed, 0); +static SENSOR_DEVICE_ATTR(fan2_target, S_IWUSR | S_IRUGO, show_target_speed, + store_target_speed, 1); +static SENSOR_DEVICE_ATTR(fan3_target, S_IWUSR | S_IRUGO, show_target_speed, + store_target_speed, 2); +static SENSOR_DEVICE_ATTR(fan4_target, S_IWUSR | S_IRUGO, show_target_speed, + store_target_speed, 3); +static SENSOR_DEVICE_ATTR(fan5_target, S_IWUSR | S_IRUGO, show_target_speed, + store_target_speed, 4); + +static SENSOR_DEVICE_ATTR(fan1_tolerance, S_IWUSR | S_IRUGO, + show_speed_tolerance, store_speed_tolerance, 0); +static SENSOR_DEVICE_ATTR(fan2_tolerance, S_IWUSR | S_IRUGO, + show_speed_tolerance, store_speed_tolerance, 1); +static SENSOR_DEVICE_ATTR(fan3_tolerance, S_IWUSR | S_IRUGO, + show_speed_tolerance, store_speed_tolerance, 2); +static SENSOR_DEVICE_ATTR(fan4_tolerance, S_IWUSR | S_IRUGO, + show_speed_tolerance, store_speed_tolerance, 3); +static SENSOR_DEVICE_ATTR(fan5_tolerance, S_IWUSR | S_IRUGO, + show_speed_tolerance, store_speed_tolerance, 4); - mutex_lock(&data->update_lock); - data->pwm_sel_enable[index][nr] = val; - reg = nct6775_read_value(data, data->REG_TEMP_SEL[index][nr]); - reg &= ~bit[index]; - if (val) - reg |= bit[index]; - nct6775_write_value(data, data->REG_TEMP_SEL[index][nr], reg); - mutex_unlock(&data->update_lock); - return count; -} +/* Smart Fan registers */ static ssize_t show_weight_temp(struct device *dev, struct device_attribute *attr, char *buf) @@ -2192,49 +2505,21 @@ store_weight_temp(struct device *dev, struct device_attribute *attr, return count; } -static SENSOR_DEVICE_ATTR_2(pwm1_stop_output_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 0, 0); -static SENSOR_DEVICE_ATTR_2(pwm2_stop_output_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 1, 0); -static SENSOR_DEVICE_ATTR_2(pwm3_stop_output_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 2, 0); -static SENSOR_DEVICE_ATTR_2(pwm4_stop_output_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 3, 0); -static SENSOR_DEVICE_ATTR_2(pwm5_stop_output_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 4, 0); - -static SENSOR_DEVICE_ATTR_2(pwm1_weight_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 0, 1); -static SENSOR_DEVICE_ATTR_2(pwm2_weight_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 1, 1); -static SENSOR_DEVICE_ATTR_2(pwm3_weight_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 2, 1); -static SENSOR_DEVICE_ATTR_2(pwm4_weight_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 3, 1); -static SENSOR_DEVICE_ATTR_2(pwm5_weight_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 4, 1); - -static SENSOR_DEVICE_ATTR_2(pwm1_step_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 0, 2); -static SENSOR_DEVICE_ATTR_2(pwm2_step_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 1, 2); -static SENSOR_DEVICE_ATTR_2(pwm3_step_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 2, 2); -static SENSOR_DEVICE_ATTR_2(pwm4_step_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 3, 2); -static SENSOR_DEVICE_ATTR_2(pwm5_step_enable, S_IWUSR | S_IRUGO, - show_pwm_sel_enable, store_pwm_sel_enable, 4, 2); - -static SENSOR_DEVICE_ATTR_2(pwm1_weight_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 0, 1); -static SENSOR_DEVICE_ATTR_2(pwm2_weight_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 1, 1); -static SENSOR_DEVICE_ATTR_2(pwm3_weight_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 2, 1); -static SENSOR_DEVICE_ATTR_2(pwm4_weight_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 3, 1); -static SENSOR_DEVICE_ATTR_2(pwm5_weight_temp_sel, S_IWUSR | S_IRUGO, - show_pwm_temp_sel, store_pwm_temp_sel, 4, 1); +static SENSOR_DEVICE_ATTR(pwm1_weight_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_weight_temp_sel, store_pwm_weight_temp_sel, + 0); +static SENSOR_DEVICE_ATTR(pwm2_weight_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_weight_temp_sel, store_pwm_weight_temp_sel, + 1); +static SENSOR_DEVICE_ATTR(pwm3_weight_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_weight_temp_sel, store_pwm_weight_temp_sel, + 2); +static SENSOR_DEVICE_ATTR(pwm4_weight_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_weight_temp_sel, store_pwm_weight_temp_sel, + 3); +static SENSOR_DEVICE_ATTR(pwm5_weight_temp_sel, S_IWUSR | S_IRUGO, + show_pwm_weight_temp_sel, store_pwm_weight_temp_sel, + 4); static SENSOR_DEVICE_ATTR_2(pwm1_weight_temp_step, S_IWUSR | S_IRUGO, show_weight_temp, store_weight_temp, 0, 0); @@ -2373,54 +2658,71 @@ static SENSOR_DEVICE_ATTR_2(pwm4_step_down_time, S_IWUSR | S_IRUGO, static SENSOR_DEVICE_ATTR_2(pwm5_step_down_time, S_IWUSR | S_IRUGO, show_fan_time, store_fan_time, 4, 2); -static SENSOR_DEVICE_ATTR_2(pwm1_start_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm1_start, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 0, 1); -static SENSOR_DEVICE_ATTR_2(pwm2_start_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm2_start, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 1, 1); -static SENSOR_DEVICE_ATTR_2(pwm3_start_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm3_start, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 2, 1); -static SENSOR_DEVICE_ATTR_2(pwm4_start_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm4_start, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 3, 1); -static SENSOR_DEVICE_ATTR_2(pwm5_start_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm5_start, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 4, 1); -static SENSOR_DEVICE_ATTR_2(pwm1_stop_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm1_floor, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 0, 2); -static SENSOR_DEVICE_ATTR_2(pwm2_stop_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm2_floor, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 1, 2); -static SENSOR_DEVICE_ATTR_2(pwm3_stop_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm3_floor, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 2, 2); -static SENSOR_DEVICE_ATTR_2(pwm4_stop_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm4_floor, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 3, 2); -static SENSOR_DEVICE_ATTR_2(pwm5_stop_output, S_IWUSR | S_IRUGO, show_pwm, +static SENSOR_DEVICE_ATTR_2(pwm5_floor, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 4, 2); -/* max_output is not supported on all chips */ -static struct sensor_device_attribute_2 sda_max_output[] = { - SENSOR_ATTR_2(pwm1_max_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, +static SENSOR_DEVICE_ATTR_2(pwm1_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 0, 0); +static SENSOR_DEVICE_ATTR_2(pwm2_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 1, 0); +static SENSOR_DEVICE_ATTR_2(pwm3_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 2, 0); +static SENSOR_DEVICE_ATTR_2(pwm4_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 3, 0); +static SENSOR_DEVICE_ATTR_2(pwm5_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 4, 0); + +static SENSOR_DEVICE_ATTR_2(pwm1_crit_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 0, 1); +static SENSOR_DEVICE_ATTR_2(pwm2_crit_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 1, 1); +static SENSOR_DEVICE_ATTR_2(pwm3_crit_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 2, 1); +static SENSOR_DEVICE_ATTR_2(pwm4_crit_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 3, 1); +static SENSOR_DEVICE_ATTR_2(pwm5_crit_temp_tolerance, S_IWUSR | S_IRUGO, + show_temp_tolerance, store_temp_tolerance, 4, 1); + +/* pwm_max is not supported on all chips */ +static struct sensor_device_attribute_2 sda_pwm_max[] = { + SENSOR_ATTR_2(pwm1_max, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 0, 3), - SENSOR_ATTR_2(pwm2_max_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, + SENSOR_ATTR_2(pwm2_max, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 1, 3), - SENSOR_ATTR_2(pwm3_max_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, + SENSOR_ATTR_2(pwm3_max, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 2, 3), - SENSOR_ATTR_2(pwm4_max_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, + SENSOR_ATTR_2(pwm4_max, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 3, 3), - SENSOR_ATTR_2(pwm5_max_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, + SENSOR_ATTR_2(pwm5_max, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 4, 3), }; -/* step_output is not supported on all chips */ -static struct sensor_device_attribute_2 sda_step_output[] = { - SENSOR_ATTR_2(pwm1_step_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, - 0, 4), - SENSOR_ATTR_2(pwm2_step_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, - 1, 4), - SENSOR_ATTR_2(pwm3_step_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, - 2, 4), - SENSOR_ATTR_2(pwm4_step_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, - 3, 4), - SENSOR_ATTR_2(pwm5_step_output, S_IWUSR | S_IRUGO, show_pwm, store_pwm, - 4, 4), +/* pwm_step is not supported on all chips */ +static struct sensor_device_attribute_2 sda_pwm_step[] = { + SENSOR_ATTR_2(pwm1_step, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 0, 4), + SENSOR_ATTR_2(pwm2_step, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 1, 4), + SENSOR_ATTR_2(pwm3_step, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 2, 4), + SENSOR_ATTR_2(pwm4_step, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 3, 4), + SENSOR_ATTR_2(pwm5_step, S_IWUSR | S_IRUGO, show_pwm, store_pwm, 4, 4), }; static struct attribute *nct6775_attributes_pwm[5][20] = { @@ -2428,16 +2730,17 @@ static struct attribute *nct6775_attributes_pwm[5][20] = { &sensor_dev_attr_pwm1.dev_attr.attr, &sensor_dev_attr_pwm1_mode.dev_attr.attr, &sensor_dev_attr_pwm1_enable.dev_attr.attr, - &sensor_dev_attr_pwm1_stop_output_enable.dev_attr.attr, - &sensor_dev_attr_pwm1_weight_enable.dev_attr.attr, - &sensor_dev_attr_pwm1_step_enable.dev_attr.attr, &sensor_dev_attr_pwm1_temp_sel.dev_attr.attr, - &sensor_dev_attr_pwm1_target.dev_attr.attr, + &sensor_dev_attr_pwm1_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm1_crit_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm1_target_temp.dev_attr.attr, + &sensor_dev_attr_fan1_target.dev_attr.attr, + &sensor_dev_attr_fan1_tolerance.dev_attr.attr, &sensor_dev_attr_pwm1_stop_time.dev_attr.attr, &sensor_dev_attr_pwm1_step_up_time.dev_attr.attr, &sensor_dev_attr_pwm1_step_down_time.dev_attr.attr, - &sensor_dev_attr_pwm1_start_output.dev_attr.attr, - &sensor_dev_attr_pwm1_stop_output.dev_attr.attr, + &sensor_dev_attr_pwm1_start.dev_attr.attr, + &sensor_dev_attr_pwm1_floor.dev_attr.attr, &sensor_dev_attr_pwm1_weight_temp_sel.dev_attr.attr, &sensor_dev_attr_pwm1_weight_temp_step.dev_attr.attr, &sensor_dev_attr_pwm1_weight_temp_step_tol.dev_attr.attr, @@ -2449,16 +2752,17 @@ static struct attribute *nct6775_attributes_pwm[5][20] = { &sensor_dev_attr_pwm2.dev_attr.attr, &sensor_dev_attr_pwm2_mode.dev_attr.attr, &sensor_dev_attr_pwm2_enable.dev_attr.attr, - &sensor_dev_attr_pwm2_stop_output_enable.dev_attr.attr, - &sensor_dev_attr_pwm2_weight_enable.dev_attr.attr, - &sensor_dev_attr_pwm2_step_enable.dev_attr.attr, &sensor_dev_attr_pwm2_temp_sel.dev_attr.attr, - &sensor_dev_attr_pwm2_target.dev_attr.attr, + &sensor_dev_attr_pwm2_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm2_crit_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm2_target_temp.dev_attr.attr, + &sensor_dev_attr_fan2_target.dev_attr.attr, + &sensor_dev_attr_fan2_tolerance.dev_attr.attr, &sensor_dev_attr_pwm2_stop_time.dev_attr.attr, &sensor_dev_attr_pwm2_step_up_time.dev_attr.attr, &sensor_dev_attr_pwm2_step_down_time.dev_attr.attr, - &sensor_dev_attr_pwm2_start_output.dev_attr.attr, - &sensor_dev_attr_pwm2_stop_output.dev_attr.attr, + &sensor_dev_attr_pwm2_start.dev_attr.attr, + &sensor_dev_attr_pwm2_floor.dev_attr.attr, &sensor_dev_attr_pwm2_weight_temp_sel.dev_attr.attr, &sensor_dev_attr_pwm2_weight_temp_step.dev_attr.attr, &sensor_dev_attr_pwm2_weight_temp_step_tol.dev_attr.attr, @@ -2470,16 +2774,17 @@ static struct attribute *nct6775_attributes_pwm[5][20] = { &sensor_dev_attr_pwm3.dev_attr.attr, &sensor_dev_attr_pwm3_mode.dev_attr.attr, &sensor_dev_attr_pwm3_enable.dev_attr.attr, - &sensor_dev_attr_pwm3_stop_output_enable.dev_attr.attr, - &sensor_dev_attr_pwm3_weight_enable.dev_attr.attr, - &sensor_dev_attr_pwm3_step_enable.dev_attr.attr, &sensor_dev_attr_pwm3_temp_sel.dev_attr.attr, - &sensor_dev_attr_pwm3_target.dev_attr.attr, + &sensor_dev_attr_pwm3_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm3_crit_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm3_target_temp.dev_attr.attr, + &sensor_dev_attr_fan3_target.dev_attr.attr, + &sensor_dev_attr_fan3_tolerance.dev_attr.attr, &sensor_dev_attr_pwm3_stop_time.dev_attr.attr, &sensor_dev_attr_pwm3_step_up_time.dev_attr.attr, &sensor_dev_attr_pwm3_step_down_time.dev_attr.attr, - &sensor_dev_attr_pwm3_start_output.dev_attr.attr, - &sensor_dev_attr_pwm3_stop_output.dev_attr.attr, + &sensor_dev_attr_pwm3_start.dev_attr.attr, + &sensor_dev_attr_pwm3_floor.dev_attr.attr, &sensor_dev_attr_pwm3_weight_temp_sel.dev_attr.attr, &sensor_dev_attr_pwm3_weight_temp_step.dev_attr.attr, &sensor_dev_attr_pwm3_weight_temp_step_tol.dev_attr.attr, @@ -2491,16 +2796,17 @@ static struct attribute *nct6775_attributes_pwm[5][20] = { &sensor_dev_attr_pwm4.dev_attr.attr, &sensor_dev_attr_pwm4_mode.dev_attr.attr, &sensor_dev_attr_pwm4_enable.dev_attr.attr, - &sensor_dev_attr_pwm4_stop_output_enable.dev_attr.attr, - &sensor_dev_attr_pwm4_weight_enable.dev_attr.attr, - &sensor_dev_attr_pwm4_step_enable.dev_attr.attr, &sensor_dev_attr_pwm4_temp_sel.dev_attr.attr, - &sensor_dev_attr_pwm4_target.dev_attr.attr, + &sensor_dev_attr_pwm4_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm4_crit_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm4_target_temp.dev_attr.attr, + &sensor_dev_attr_fan4_target.dev_attr.attr, + &sensor_dev_attr_fan4_tolerance.dev_attr.attr, &sensor_dev_attr_pwm4_stop_time.dev_attr.attr, &sensor_dev_attr_pwm4_step_up_time.dev_attr.attr, &sensor_dev_attr_pwm4_step_down_time.dev_attr.attr, - &sensor_dev_attr_pwm4_start_output.dev_attr.attr, - &sensor_dev_attr_pwm4_stop_output.dev_attr.attr, + &sensor_dev_attr_pwm4_start.dev_attr.attr, + &sensor_dev_attr_pwm4_floor.dev_attr.attr, &sensor_dev_attr_pwm4_weight_temp_sel.dev_attr.attr, &sensor_dev_attr_pwm4_weight_temp_step.dev_attr.attr, &sensor_dev_attr_pwm4_weight_temp_step_tol.dev_attr.attr, @@ -2512,16 +2818,17 @@ static struct attribute *nct6775_attributes_pwm[5][20] = { &sensor_dev_attr_pwm5.dev_attr.attr, &sensor_dev_attr_pwm5_mode.dev_attr.attr, &sensor_dev_attr_pwm5_enable.dev_attr.attr, - &sensor_dev_attr_pwm5_stop_output_enable.dev_attr.attr, - &sensor_dev_attr_pwm5_weight_enable.dev_attr.attr, - &sensor_dev_attr_pwm5_step_enable.dev_attr.attr, &sensor_dev_attr_pwm5_temp_sel.dev_attr.attr, - &sensor_dev_attr_pwm5_target.dev_attr.attr, + &sensor_dev_attr_pwm5_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm5_crit_temp_tolerance.dev_attr.attr, + &sensor_dev_attr_pwm5_target_temp.dev_attr.attr, + &sensor_dev_attr_fan5_target.dev_attr.attr, + &sensor_dev_attr_fan5_tolerance.dev_attr.attr, &sensor_dev_attr_pwm5_stop_time.dev_attr.attr, &sensor_dev_attr_pwm5_step_up_time.dev_attr.attr, &sensor_dev_attr_pwm5_step_down_time.dev_attr.attr, - &sensor_dev_attr_pwm5_start_output.dev_attr.attr, - &sensor_dev_attr_pwm5_stop_output.dev_attr.attr, + &sensor_dev_attr_pwm5_start.dev_attr.attr, + &sensor_dev_attr_pwm5_floor.dev_attr.attr, &sensor_dev_attr_pwm5_weight_temp_sel.dev_attr.attr, &sensor_dev_attr_pwm5_weight_temp_step.dev_attr.attr, &sensor_dev_attr_pwm5_weight_temp_step_tol.dev_attr.attr, @@ -2668,216 +2975,146 @@ static struct sensor_device_attribute_2 sda_auto_pwm_arrays[] = { show_auto_pwm, store_auto_pwm, 0, 0), SENSOR_ATTR_2(pwm1_auto_point1_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 0, 0), - SENSOR_ATTR_2(pwm1_auto_point1_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 0, 0), SENSOR_ATTR_2(pwm1_auto_point2_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 0, 1), SENSOR_ATTR_2(pwm1_auto_point2_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 0, 1), - SENSOR_ATTR_2(pwm1_auto_point2_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 0, 1), SENSOR_ATTR_2(pwm1_auto_point3_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 0, 2), SENSOR_ATTR_2(pwm1_auto_point3_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 0, 2), - SENSOR_ATTR_2(pwm1_auto_point3_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 0, 2), SENSOR_ATTR_2(pwm1_auto_point4_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 0, 3), SENSOR_ATTR_2(pwm1_auto_point4_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 0, 3), - SENSOR_ATTR_2(pwm1_auto_point4_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 0, 3), SENSOR_ATTR_2(pwm1_auto_point5_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 0, 4), SENSOR_ATTR_2(pwm1_auto_point5_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 0, 4), - SENSOR_ATTR_2(pwm1_auto_point5_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 0, 4), SENSOR_ATTR_2(pwm1_auto_point6_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 0, 5), SENSOR_ATTR_2(pwm1_auto_point6_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 0, 5), - SENSOR_ATTR_2(pwm1_auto_point6_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 0, 5), SENSOR_ATTR_2(pwm1_auto_point7_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 0, 6), SENSOR_ATTR_2(pwm1_auto_point7_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 0, 6), - SENSOR_ATTR_2(pwm1_auto_point7_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 0, 6), SENSOR_ATTR_2(pwm2_auto_point1_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 1, 0), SENSOR_ATTR_2(pwm2_auto_point1_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 1, 0), - SENSOR_ATTR_2(pwm2_auto_point1_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 1, 0), SENSOR_ATTR_2(pwm2_auto_point2_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 1, 1), SENSOR_ATTR_2(pwm2_auto_point2_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 1, 1), - SENSOR_ATTR_2(pwm2_auto_point2_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 1, 1), SENSOR_ATTR_2(pwm2_auto_point3_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 1, 2), SENSOR_ATTR_2(pwm2_auto_point3_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 1, 2), - SENSOR_ATTR_2(pwm2_auto_point3_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 1, 2), SENSOR_ATTR_2(pwm2_auto_point4_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 1, 3), SENSOR_ATTR_2(pwm2_auto_point4_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 1, 3), - SENSOR_ATTR_2(pwm2_auto_point4_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 1, 3), SENSOR_ATTR_2(pwm2_auto_point5_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 1, 4), SENSOR_ATTR_2(pwm2_auto_point5_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 1, 4), - SENSOR_ATTR_2(pwm2_auto_point5_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 1, 4), SENSOR_ATTR_2(pwm2_auto_point6_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 1, 5), SENSOR_ATTR_2(pwm2_auto_point6_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 1, 5), - SENSOR_ATTR_2(pwm2_auto_point6_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 1, 5), SENSOR_ATTR_2(pwm2_auto_point7_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 1, 6), SENSOR_ATTR_2(pwm2_auto_point7_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 1, 6), - SENSOR_ATTR_2(pwm2_auto_point7_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 1, 6), SENSOR_ATTR_2(pwm3_auto_point1_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 2, 0), SENSOR_ATTR_2(pwm3_auto_point1_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 2, 0), - SENSOR_ATTR_2(pwm3_auto_point1_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 2, 0), SENSOR_ATTR_2(pwm3_auto_point2_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 2, 1), SENSOR_ATTR_2(pwm3_auto_point2_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 2, 1), - SENSOR_ATTR_2(pwm3_auto_point2_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 2, 1), SENSOR_ATTR_2(pwm3_auto_point3_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 2, 2), SENSOR_ATTR_2(pwm3_auto_point3_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 2, 2), - SENSOR_ATTR_2(pwm3_auto_point3_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 2, 2), SENSOR_ATTR_2(pwm3_auto_point4_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 2, 3), SENSOR_ATTR_2(pwm3_auto_point4_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 2, 3), - SENSOR_ATTR_2(pwm3_auto_point4_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 2, 3), SENSOR_ATTR_2(pwm3_auto_point5_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 2, 4), SENSOR_ATTR_2(pwm3_auto_point5_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 2, 4), - SENSOR_ATTR_2(pwm3_auto_point5_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 2, 4), SENSOR_ATTR_2(pwm3_auto_point6_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 2, 5), SENSOR_ATTR_2(pwm3_auto_point6_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 2, 5), - SENSOR_ATTR_2(pwm3_auto_point6_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 2, 5), SENSOR_ATTR_2(pwm3_auto_point7_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 2, 6), SENSOR_ATTR_2(pwm3_auto_point7_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 2, 6), - SENSOR_ATTR_2(pwm3_auto_point7_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 2, 6), SENSOR_ATTR_2(pwm4_auto_point1_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 3, 0), SENSOR_ATTR_2(pwm4_auto_point1_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 3, 0), - SENSOR_ATTR_2(pwm4_auto_point1_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 3, 0), SENSOR_ATTR_2(pwm4_auto_point2_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 3, 1), SENSOR_ATTR_2(pwm4_auto_point2_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 3, 1), - SENSOR_ATTR_2(pwm4_auto_point2_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 3, 1), SENSOR_ATTR_2(pwm4_auto_point3_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 3, 2), SENSOR_ATTR_2(pwm4_auto_point3_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 3, 2), - SENSOR_ATTR_2(pwm4_auto_point3_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 3, 2), SENSOR_ATTR_2(pwm4_auto_point4_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 3, 3), SENSOR_ATTR_2(pwm4_auto_point4_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 3, 3), - SENSOR_ATTR_2(pwm4_auto_point4_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 3, 3), SENSOR_ATTR_2(pwm4_auto_point5_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 3, 4), SENSOR_ATTR_2(pwm4_auto_point5_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 3, 4), - SENSOR_ATTR_2(pwm4_auto_point5_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 3, 4), SENSOR_ATTR_2(pwm4_auto_point6_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 3, 5), SENSOR_ATTR_2(pwm4_auto_point6_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 3, 5), - SENSOR_ATTR_2(pwm4_auto_point6_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 3, 5), SENSOR_ATTR_2(pwm4_auto_point7_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 3, 6), SENSOR_ATTR_2(pwm4_auto_point7_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 3, 6), - SENSOR_ATTR_2(pwm4_auto_point7_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 3, 6), SENSOR_ATTR_2(pwm5_auto_point1_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 4, 0), SENSOR_ATTR_2(pwm5_auto_point1_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 4, 0), - SENSOR_ATTR_2(pwm5_auto_point1_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 4, 0), SENSOR_ATTR_2(pwm5_auto_point2_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 4, 1), SENSOR_ATTR_2(pwm5_auto_point2_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 4, 1), - SENSOR_ATTR_2(pwm5_auto_point2_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 4, 1), SENSOR_ATTR_2(pwm5_auto_point3_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 4, 2), SENSOR_ATTR_2(pwm5_auto_point3_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 4, 2), - SENSOR_ATTR_2(pwm5_auto_point3_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 4, 2), SENSOR_ATTR_2(pwm5_auto_point4_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 4, 3), SENSOR_ATTR_2(pwm5_auto_point4_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 4, 3), - SENSOR_ATTR_2(pwm5_auto_point4_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 4, 3), SENSOR_ATTR_2(pwm5_auto_point5_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 4, 4), SENSOR_ATTR_2(pwm5_auto_point5_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 4, 4), - SENSOR_ATTR_2(pwm5_auto_point5_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 4, 4), SENSOR_ATTR_2(pwm5_auto_point6_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 4, 5), SENSOR_ATTR_2(pwm5_auto_point6_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 4, 5), - SENSOR_ATTR_2(pwm5_auto_point6_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 4, 5), SENSOR_ATTR_2(pwm5_auto_point7_pwm, S_IWUSR | S_IRUGO, show_auto_pwm, store_auto_pwm, 4, 6), SENSOR_ATTR_2(pwm5_auto_point7_temp, S_IWUSR | S_IRUGO, show_auto_temp, store_auto_temp, 4, 6), - SENSOR_ATTR_2(pwm5_auto_point7_temp_hyst, S_IWUSR | S_IRUGO, - show_auto_temp_hyst, store_auto_temp_hyst, 4, 6), }; static ssize_t @@ -2926,7 +3163,7 @@ clear_caseopen(struct device *dev, struct device_attribute *attr, superio_outb(sio_data->sioreg, NCT6775_REG_CR_CASEOPEN_CLR[nr], reg); superio_exit(sio_data->sioreg); - data->valid = 0; /* Force cache refresh */ + data->valid = false; /* Force cache refresh */ error: mutex_unlock(&data->update_lock); return count; @@ -2955,11 +3192,11 @@ static void nct6775_device_remove_files(struct device *dev) for (i = 0; i < data->pwm_num; i++) sysfs_remove_group(&dev->kobj, &nct6775_group_pwm[i]); - for (i = 0; i < ARRAY_SIZE(sda_max_output); i++) - device_remove_file(dev, &sda_max_output[i].dev_attr); + for (i = 0; i < ARRAY_SIZE(sda_pwm_max); i++) + device_remove_file(dev, &sda_pwm_max[i].dev_attr); - for (i = 0; i < ARRAY_SIZE(sda_step_output); i++) - device_remove_file(dev, &sda_max_output[i].dev_attr); + for (i = 0; i < ARRAY_SIZE(sda_pwm_step); i++) + device_remove_file(dev, &sda_pwm_step[i].dev_attr); for (i = 0; i < ARRAY_SIZE(sda_weight_duty_base); i++) device_remove_file(dev, &sda_weight_duty_base[i].dev_attr); @@ -2975,6 +3212,7 @@ static void nct6775_device_remove_files(struct device *dev) device_remove_file(dev, &sda_fan_alarm[i].dev_attr); device_remove_file(dev, &sda_fan_div[i].dev_attr); device_remove_file(dev, &sda_fan_min[i].dev_attr); + device_remove_file(dev, &sda_fan_pulses[i].dev_attr); } for (i = 0; i < NUM_TEMP; i++) { if (!(data->have_temp & (1 << i))) @@ -3001,7 +3239,7 @@ static void nct6775_device_remove_files(struct device *dev) } /* Get the monitoring functions started */ -static inline void __devinit nct6775_init_device(struct nct6775_data *data) +static inline void nct6775_init_device(struct nct6775_data *data) { int i; u8 tmp, diode; @@ -3042,7 +3280,7 @@ static inline void __devinit nct6775_init_device(struct nct6775_data *data) } } -static int __devinit +static int nct6775_check_fan_inputs(const struct nct6775_sio_data *sio_data, struct nct6775_data *data) { @@ -3124,7 +3362,33 @@ nct6775_check_fan_inputs(const struct nct6775_sio_data *sio_data, return 0; } -static int __devinit nct6775_probe(struct platform_device *pdev) +static void add_temp_sensors(struct nct6775_data *data, const u16 *regp, + int *available, int *mask) +{ + int i; + u8 src; + + for (i = 0; i < data->pwm_num && *available; i++) { + int index; + + if (!regp[i]) + continue; + src = nct6775_read_value(data, regp[i]); + src &= 0x1f; + if (!src || (*mask & (1 << src))) + continue; + if (src >= data->temp_label_num || + !strlen(data->temp_label[src])) + continue; + + index = __ffs(*available); + nct6775_write_value(data, data->REG_TEMP_SOURCE[index], src); + *available &= ~(1 << index); + *mask |= 1 << src; + } +} + +static int nct6775_probe(struct platform_device *pdev) { struct device *dev = &pdev->dev; struct nct6775_sio_data *sio_data = dev->platform_data; @@ -3151,6 +3415,7 @@ static int __devinit nct6775_probe(struct platform_device *pdev) mutex_init(&data->lock); mutex_init(&data->update_lock); data->name = nct6775_device_names[data->kind]; + data->bank = 0xff; /* Force initial bank selection */ platform_set_drvdata(pdev, data); switch (data->kind) { @@ -3165,6 +3430,9 @@ static int __devinit nct6775_probe(struct platform_device *pdev) data->fan_from_reg = fan_from_reg16; data->fan_from_reg_min = fan_from_reg8; + data->target_temp_mask = 0x7f; + data->tolerance_mask = 0x0f; + data->speed_tolerance_limit = 15; data->temp_label = nct6775_temp_label; data->temp_label_num = ARRAY_SIZE(nct6775_temp_label); @@ -3179,6 +3447,7 @@ static int __devinit nct6775_probe(struct platform_device *pdev) data->REG_FAN = NCT6775_REG_FAN; data->REG_FAN_MODE = NCT6775_REG_FAN_MODE; data->REG_FAN_MIN = NCT6775_REG_FAN_MIN; + data->REG_FAN_PULSES = NCT6775_REG_FAN_PULSES; data->REG_FAN_TIME[0] = NCT6775_REG_FAN_STOP_TIME; data->REG_FAN_TIME[1] = NCT6775_REG_FAN_STEP_UP_TIME; data->REG_FAN_TIME[2] = NCT6775_REG_FAN_STEP_DOWN_TIME; @@ -3198,9 +3467,8 @@ static int __devinit nct6775_probe(struct platform_device *pdev) = NCT6775_REG_CRITICAL_TEMP_TOLERANCE; data->REG_TEMP_OFFSET = NCT6775_REG_TEMP_OFFSET; data->REG_TEMP_SOURCE = NCT6775_REG_TEMP_SOURCE; - data->REG_TEMP_SEL[0] = NCT6775_REG_TEMP_SEL; - data->REG_TEMP_SEL[1] = NCT6775_REG_WEIGHT_TEMP_SEL; - data->REG_TEMP_SEL[2] = NCT6775_REG_FAN_STEP_ENABLE; + data->REG_TEMP_SEL = NCT6775_REG_TEMP_SEL; + data->REG_WEIGHT_TEMP_SEL = NCT6775_REG_WEIGHT_TEMP_SEL; data->REG_WEIGHT_TEMP[0] = NCT6775_REG_WEIGHT_TEMP_STEP; data->REG_WEIGHT_TEMP[1] = NCT6775_REG_WEIGHT_TEMP_STEP_TOL; data->REG_WEIGHT_TEMP[2] = NCT6775_REG_WEIGHT_TEMP_BASE; @@ -3226,6 +3494,9 @@ static int __devinit nct6775_probe(struct platform_device *pdev) data->fan_from_reg = fan_from_reg13; data->fan_from_reg_min = fan_from_reg13; + data->target_temp_mask = 0xff; + data->tolerance_mask = 0x07; + data->speed_tolerance_limit = 63; data->temp_label = nct6776_temp_label; data->temp_label_num = ARRAY_SIZE(nct6776_temp_label); @@ -3240,9 +3511,11 @@ static int __devinit nct6775_probe(struct platform_device *pdev) data->REG_FAN = NCT6775_REG_FAN; data->REG_FAN_MODE = NCT6775_REG_FAN_MODE; data->REG_FAN_MIN = NCT6776_REG_FAN_MIN; + data->REG_FAN_PULSES = NCT6776_REG_FAN_PULSES; data->REG_FAN_TIME[0] = NCT6775_REG_FAN_STOP_TIME; data->REG_FAN_TIME[1] = NCT6775_REG_FAN_STEP_UP_TIME; data->REG_FAN_TIME[2] = NCT6775_REG_FAN_STEP_DOWN_TIME; + data->REG_TOLERANCE_H = NCT6776_REG_TOLERANCE_H; data->REG_PWM[0] = NCT6775_REG_PWM; data->REG_PWM[1] = NCT6775_REG_FAN_START_OUTPUT; data->REG_PWM[2] = NCT6775_REG_FAN_STOP_OUTPUT; @@ -3258,9 +3531,8 @@ static int __devinit nct6775_probe(struct platform_device *pdev) = NCT6775_REG_CRITICAL_TEMP_TOLERANCE; data->REG_TEMP_OFFSET = NCT6775_REG_TEMP_OFFSET; data->REG_TEMP_SOURCE = NCT6775_REG_TEMP_SOURCE; - data->REG_TEMP_SEL[0] = NCT6775_REG_TEMP_SEL; - data->REG_TEMP_SEL[1] = NCT6775_REG_WEIGHT_TEMP_SEL; - data->REG_TEMP_SEL[2] = NCT6775_REG_FAN_STEP_ENABLE; + data->REG_TEMP_SEL = NCT6775_REG_TEMP_SEL; + data->REG_WEIGHT_TEMP_SEL = NCT6775_REG_WEIGHT_TEMP_SEL; data->REG_WEIGHT_TEMP[0] = NCT6775_REG_WEIGHT_TEMP_STEP; data->REG_WEIGHT_TEMP[1] = NCT6775_REG_WEIGHT_TEMP_STEP_TOL; data->REG_WEIGHT_TEMP[2] = NCT6775_REG_WEIGHT_TEMP_BASE; @@ -3286,6 +3558,9 @@ static int __devinit nct6775_probe(struct platform_device *pdev) data->fan_from_reg = fan_from_reg13; data->fan_from_reg_min = fan_from_reg13; + data->target_temp_mask = 0xff; + data->tolerance_mask = 0x07; + data->speed_tolerance_limit = 63; data->temp_label = nct6779_temp_label; data->temp_label_num = ARRAY_SIZE(nct6779_temp_label); @@ -3300,9 +3575,11 @@ static int __devinit nct6775_probe(struct platform_device *pdev) data->REG_FAN = NCT6779_REG_FAN; data->REG_FAN_MODE = NCT6775_REG_FAN_MODE; data->REG_FAN_MIN = NCT6776_REG_FAN_MIN; + data->REG_FAN_PULSES = NCT6779_REG_FAN_PULSES; data->REG_FAN_TIME[0] = NCT6775_REG_FAN_STOP_TIME; data->REG_FAN_TIME[1] = NCT6775_REG_FAN_STEP_UP_TIME; data->REG_FAN_TIME[2] = NCT6775_REG_FAN_STEP_DOWN_TIME; + data->REG_TOLERANCE_H = NCT6776_REG_TOLERANCE_H; data->REG_PWM[0] = NCT6775_REG_PWM; data->REG_PWM[1] = NCT6775_REG_FAN_START_OUTPUT; data->REG_PWM[2] = NCT6775_REG_FAN_STOP_OUTPUT; @@ -3318,9 +3595,8 @@ static int __devinit nct6775_probe(struct platform_device *pdev) = NCT6775_REG_CRITICAL_TEMP_TOLERANCE; data->REG_TEMP_OFFSET = NCT6779_REG_TEMP_OFFSET; data->REG_TEMP_SOURCE = NCT6775_REG_TEMP_SOURCE; - data->REG_TEMP_SEL[0] = NCT6775_REG_TEMP_SEL; - data->REG_TEMP_SEL[1] = NCT6775_REG_WEIGHT_TEMP_SEL; - data->REG_TEMP_SEL[2] = NCT6775_REG_FAN_STEP_ENABLE; + data->REG_TEMP_SEL = NCT6775_REG_TEMP_SEL; + data->REG_WEIGHT_TEMP_SEL = NCT6775_REG_WEIGHT_TEMP_SEL; data->REG_WEIGHT_TEMP[0] = NCT6775_REG_WEIGHT_TEMP_STEP; data->REG_WEIGHT_TEMP[1] = NCT6775_REG_WEIGHT_TEMP_STEP_TOL; data->REG_WEIGHT_TEMP[2] = NCT6775_REG_WEIGHT_TEMP_BASE; @@ -3366,37 +3642,8 @@ static int __devinit nct6775_probe(struct platform_device *pdev) * Now find unmonitored temperature registers and enable monitoring * if additional monitoring registers are available. */ - for (i = 0; i < ARRAY_SIZE(data->REG_TEMP_SEL) && available; i++) { - int j; - - if (!data->REG_TEMP_SEL[i]) - continue; - for (j = 0; j < data->pwm_num && available; j++) { - int index; - - if (!data->REG_TEMP_SEL[i][j]) - continue; - src = nct6775_read_value(data, - data->REG_TEMP_SEL[i][j]); - src &= 0x1f; - if (!src || (mask & (1 << src))) - continue; - if (src >= data->temp_label_num || - !strlen(data->temp_label[src])) { - dev_info(dev, - "Select source %d:%d reg 0x%x invalid (%d)\n", - i, j, data->REG_TEMP_SEL[i][j], src); - continue; - } - - index = __ffs(available); - nct6775_write_value(data, - data->REG_TEMP_SOURCE[index], - src); - available &= ~(1 << index); - mask |= 1 << src; - } - } + add_temp_sensors(data, data->REG_TEMP_SEL, &available, &mask); + add_temp_sensors(data, data->REG_WEIGHT_TEMP_SEL, &available, &mask); mask = 0; s = NUM_TEMP_FIXED; /* First dynamic temperature attribute */ @@ -3446,7 +3693,7 @@ static int __devinit nct6775_probe(struct platform_device *pdev) s++; } -#ifdef TESTING +#ifdef USE_ALTERNATE /* * Go through the list of alternate temp registers and enable * if possible. @@ -3478,7 +3725,7 @@ static int __devinit nct6775_probe(struct platform_device *pdev) data->temp_src[s] = i + 1; s++; } -#endif /* TESTING */ +#endif /* USE_ALTERNATE */ switch (data->kind) { case nct6775: @@ -3564,7 +3811,7 @@ static int __devinit nct6775_probe(struct platform_device *pdev) goto exit_remove; /* Read fan clock dividers immediately */ - nct6775_update_fan_div_common(dev, data); + nct6775_init_fan_common(dev, data); /* Register sysfs hooks */ for (i = 0; i < data->pwm_num; i++) { @@ -3577,13 +3824,13 @@ static int __devinit nct6775_probe(struct platform_device *pdev) if (data->REG_PWM[3]) { err = device_create_file(dev, - &sda_max_output[i].dev_attr); + &sda_pwm_max[i].dev_attr); if (err) goto exit_remove; } if (data->REG_PWM[4]) { err = device_create_file(dev, - &sda_step_output[i].dev_attr); + &sda_pwm_step[i].dev_attr); if (err) goto exit_remove; } @@ -3638,6 +3885,10 @@ static int __devinit nct6775_probe(struct platform_device *pdev) if (err) goto exit_remove; } + err = device_create_file(dev, + &sda_fan_pulses[i].dev_attr); + if (err) + goto exit_remove; } } @@ -3712,7 +3963,7 @@ exit_remove: return err; } -static int __devexit nct6775_remove(struct platform_device *pdev) +static int nct6775_remove(struct platform_device *pdev) { struct nct6775_data *data = platform_get_drvdata(pdev); @@ -3722,13 +3973,93 @@ static int __devexit nct6775_remove(struct platform_device *pdev) return 0; } +#ifdef CONFIG_PM +static int nct6775_suspend(struct device *dev) +{ + struct nct6775_data *data = nct6775_update_device(dev); + struct nct6775_sio_data *sio_data = dev->platform_data; + + mutex_lock(&data->update_lock); + data->vbat = nct6775_read_value(data, data->REG_VBAT); + if (sio_data->kind == nct6775) { + data->fandiv1 = nct6775_read_value(data, NCT6775_REG_FANDIV1); + data->fandiv2 = nct6775_read_value(data, NCT6775_REG_FANDIV2); + } + mutex_unlock(&data->update_lock); + + return 0; +} + +static int nct6775_resume(struct device *dev) +{ + struct nct6775_data *data = dev_get_drvdata(dev); + struct nct6775_sio_data *sio_data = dev->platform_data; + int i, j; + + mutex_lock(&data->update_lock); + data->bank = 0xff; /* Force initial bank selection */ + + /* Restore limits */ + for (i = 0; i < data->in_num; i++) { + if (!(data->have_in & (1 << i))) + continue; + + nct6775_write_value(data, data->REG_IN_MINMAX[0][i], + data->in[i][1]); + nct6775_write_value(data, data->REG_IN_MINMAX[1][i], + data->in[i][2]); + } + + for (i = 0; i < 5; i++) { + if (!(data->has_fan_min & (1 << i))) + continue; + + nct6775_write_value(data, data->REG_FAN_MIN[i], + data->fan_min[i]); + } + + for (i = 0; i < NUM_TEMP; i++) { + if (!(data->have_temp & (1 << i))) + continue; + + for (j = 1; j < 4; j++) + if (data->reg_temp[j][i]) + nct6775_write_temp(data, data->reg_temp[j][i], + data->temp[j][i]); + } + + /* Restore other settings */ + nct6775_write_value(data, data->REG_VBAT, data->vbat); + if (sio_data->kind == nct6775) { + nct6775_write_value(data, NCT6775_REG_FANDIV1, data->fandiv1); + nct6775_write_value(data, NCT6775_REG_FANDIV2, data->fandiv2); + } + + /* Force re-reading all values */ + data->valid = false; + mutex_unlock(&data->update_lock); + + return 0; +} + +static const struct dev_pm_ops nct6775_dev_pm_ops = { + .suspend = nct6775_suspend, + .resume = nct6775_resume, +}; + +#define NCT6775_DEV_PM_OPS (&nct6775_dev_pm_ops) +#else +#define NCT6775_DEV_PM_OPS NULL +#endif /* CONFIG_PM */ + static struct platform_driver nct6775_driver = { .driver = { .owner = THIS_MODULE, .name = DRVNAME, + .pm = NCT6775_DEV_PM_OPS, }, .probe = nct6775_probe, - .remove = __devexit_p(nct6775_remove), + .remove = nct6775_remove, }; /* nct6775_find() looks for a '627 in the Super-I/O config space */