]> git.sur5r.net Git - groeck-nct6775/blobdiff - nct6775.c
Clean up probe function: Use devm_request_region
[groeck-nct6775] / nct6775.c
index 9cf99f506691bc9ec3ca74f6e59c0c8453bcfae0..f32346d36bf62f09578a8056dc5fc89a8b8b8c73 100644 (file)
--- a/nct6775.c
+++ b/nct6775.c
@@ -153,14 +153,14 @@ superio_exit(int ioreg)
  * REG_CHIP_ID is at port 0x58
  */
 
-static const u16 NCT6775_REG_FAN_MIN[] = { 0x3b, 0x3c, 0x3d };
-
 /* Voltage min/max registers for nr=7..14 are in bank 5 */
 
 static const u16 NCT6775_REG_IN_MAX[] = {
-       0x2b, 0x2d, 0x2f, 0x31, 0x33, 0x35, 0x37, 0x554, 0x556 };
+       0x2b, 0x2d, 0x2f, 0x31, 0x33, 0x35, 0x37, 0x554, 0x556, 0x558, 0x55a,
+       0x55c, 0x55e, 0x560, 0x562 };
 static const u16 NCT6775_REG_IN_MIN[] = {
-       0x2c, 0x2e, 0x30, 0x32, 0x34, 0x36, 0x38, 0x555, 0x557 };
+       0x2c, 0x2e, 0x30, 0x32, 0x34, 0x36, 0x38, 0x555, 0x557, 0x559, 0x55b,
+       0x55d, 0x55f, 0x561, 0x563 };
 static const u16 NCT6775_REG_IN[] = {
        0x20, 0x21, 0x22, 0x23, 0x24, 0x25, 0x26, 0x550, 0x551, 0x552
 };
@@ -250,14 +250,15 @@ static const u16 NCT6775_REG_PWM[] = { 0x109, 0x209, 0x309, 0x809, 0x909 };
 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 NCT6776_REG_FAN_MIN[] = { 0x63a, 0x63c, 0x63e, 0x640, 0x642};
 
+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_FAN[] = { 0x4c0, 0x4c2, 0x4c4, 0x4c6, 0x4c8 };
-
 static const u16 NCT6775_REG_TEMP[]
        = { 0x27, 0x150, 0x250, 0x62b, 0x62c, 0x62d };
 static const u16 NCT6779_REG_TEMP[] = { 0x27, 0x150 };
@@ -269,6 +270,11 @@ static const u16 NCT6775_REG_TEMP_HYST[]
 static const u16 NCT6775_REG_TEMP_OVER[]
        = { 0x39, 0x155, 0x255, 0x672, 0x677, 0x67C };
 
+static const u16 NCT6779_REG_TEMP_HYST[]
+       = { 0x3a, 0x153, 0, 0, 0, 0 };
+static const u16 NCT6779_REG_TEMP_OVER[]
+       = { 0x39, 0x155, 0, 0, 0, 0 };
+
 static const u16 NCT6775_REG_TEMP_SOURCE[]
        = { 0x621, 0x622, 0x623, 0x624, 0x625, 0x626 };
 
@@ -297,6 +303,8 @@ static const u16 NCT6779_REG_TEMP_OFFSET[]
 static const u16 NCT6776_REG_TEMP_CONFIG[11]
        = { 0x18, 0x152, 0x252, 0x628, 0x629, 0x62A };
 
+static const u16 NCT6779_REG_TEMP_CONFIG[11] = { 0x18, 0x152 };
+
 static const u16 NCT6775_REG_TEMP_MON[] = { 0x73, 0x75, 0x77, 0x79, 0x7b };
 
 static const u16 NCT6775_REG_AUTO_TEMP[]
@@ -399,22 +407,31 @@ static const char *const nct6779_temp_label[] = {
        "BYTE_TEMP"
 };
 
-static const u16 NCT6775_REG_TEMP_ALTERNATE[ARRAY_SIZE(nct6775_temp_label)]
+static const u16 NCT6775_REG_TEMP_ALTERNATE[ARRAY_SIZE(nct6775_temp_label) - 1]
        = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0x661, 0x662, 0x664 };
 
-static const u16 NCT6776_REG_TEMP_ALTERNATE[ARRAY_SIZE(nct6776_temp_label)]
+static const u16 NCT6776_REG_TEMP_ALTERNATE[ARRAY_SIZE(nct6776_temp_label) - 1]
        = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0x401, 0x402, 0x404 };
 
-static const u16 NCT6779_REG_TEMP_ALTERNATE[ARRAY_SIZE(nct6779_temp_label)]
+static const u16 NCT6779_REG_TEMP_ALTERNATE[ARRAY_SIZE(nct6779_temp_label) - 1]
        = { 0x490, 0x491, 0x492, 0x493, 0x494, 0x495, 0, 0,
            0, 0, 0, 0, 0, 0, 0, 0,
-           0, 0, 0x401, 0x402, 0x404 };
+           0, 0x400, 0x401, 0x402, 0x404, 0x405, 0x406, 0x407,
+           0x408, 0 };
+
+static const u16 NCT6775_REG_TEMP_CRIT[ARRAY_SIZE(nct6775_temp_label) - 1]
+       = { 0, 0, 0, 0, 0xa00, 0xa01, 0xa02, 0xa03, 0xa04, 0xa05, 0xa06,
+           0xa07 };
+
+static const u16 NCT6776_REG_TEMP_CRIT[ARRAY_SIZE(nct6776_temp_label) - 1]
+       = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0x709, 0x70a };
+
+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 };
 
 #define NUM_TEMP       10      /* Max number of temp attribute sets w/ limits*/
 #define NUM_TEMP_FIXED 6       /* Max number of fixed temp attribute sets */
 
-#define NUM_REG_TEMP   ARRAY_SIZE(NCT6775_REG_TEMP)
-
 static inline int reg_to_pwm_enable(int pwm, int mode)
 {
        if (mode == 0 && pwm == 255)
@@ -515,7 +532,9 @@ struct nct6775_data {
        struct device *hwmon_dev;
        struct mutex lock;
 
-       u16 reg_temp[3][NUM_TEMP]; /* 0=temp, 1=temp_over, 2=temp_hyst */
+       u16 reg_temp[4][NUM_TEMP]; /* 0=temp, 1=temp_over, 2=temp_hyst,
+                                   * 3=temp_crit
+                                   */
        u8 temp_src[NUM_TEMP];
        u16 reg_temp_config[NUM_TEMP];
        const char * const *temp_label;
@@ -545,7 +564,6 @@ struct nct6775_data {
                                 */
        const u16 *REG_PWM_READ;
 
-       const u16 *REG_TEMP_ALTERNATE;
        const u16 *REG_TEMP_MON;
        const u16 *REG_AUTO_TEMP;
        const u16 *REG_AUTO_PWM;
@@ -661,7 +679,7 @@ static bool is_word_sized(struct nct6775_data *data, u16 reg)
                  reg == 0x73 || reg == 0x75 || reg == 0x77;
        case nct6779:
                return reg == 0x150 || reg == 0x153 || reg == 0x155 ||
-                 ((reg & 0xfff0) == 0x4c0 && (reg & 0x000f) < 0x09) ||
+                 ((reg & 0xfff0) == 0x4b0 && (reg & 0x000f) < 0x09) ||
                  reg == 0x402 ||
                  reg == 0x63a || reg == 0x63c || reg == 0x63e ||
                  reg == 0x640 || reg == 0x642 ||
@@ -962,9 +980,7 @@ static struct nct6775_data *nct6775_update_device(struct device *dev)
                         * divider can be increased, let's try that for next
                         * time
                         */
-                       if (data->has_fan_div
-                           && (reg >= 0xff || (data->kind == nct6775
-                                               && reg == 0x00))
+                       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",
@@ -989,7 +1005,7 @@ static struct nct6775_data *nct6775_update_device(struct device *dev)
                for (i = 0; i < NUM_TEMP; i++) {
                        if (!(data->have_temp & (1 << i)))
                                continue;
-                       for (j = 0; j < 3; j++) {
+                       for (j = 0; j < 4; j++) {
                                if (data->reg_temp[j][i])
                                        data->temp[j][i]
                                          = nct6775_read_temp(data,
@@ -1328,10 +1344,7 @@ store_fan_min(struct device *dev, struct device_attribute *attr,
 
        mutex_lock(&data->update_lock);
        if (!data->has_fan_div) {
-               /*
-                * Only NCT6776F for now, so we know that this is a 13 bit
-                * register
-                */
+               /* NCT6776F or NCT6779D; we know this is a 13 bit register */
                if (!val) {
                        val = 0xff1f;
                } else {
@@ -1401,8 +1414,7 @@ write_div:
        }
 
 write_min:
-       nct6775_write_value(data, data->REG_FAN_MIN[nr],
-                             data->fan_min[nr]);
+       nct6775_write_value(data, data->REG_FAN_MIN[nr], data->fan_min[nr]);
        mutex_unlock(&data->update_lock);
 
        return count;
@@ -1644,6 +1656,29 @@ static struct sensor_device_attribute_2 sda_temp_max_hyst[] = {
                      9, 2),
 };
 
+static struct sensor_device_attribute_2 sda_temp_crit[] = {
+       SENSOR_ATTR_2(temp1_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     0, 3),
+       SENSOR_ATTR_2(temp2_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     1, 3),
+       SENSOR_ATTR_2(temp3_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     2, 3),
+       SENSOR_ATTR_2(temp4_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     3, 3),
+       SENSOR_ATTR_2(temp5_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     4, 3),
+       SENSOR_ATTR_2(temp6_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     5, 3),
+       SENSOR_ATTR_2(temp7_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     6, 3),
+       SENSOR_ATTR_2(temp8_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     7, 3),
+       SENSOR_ATTR_2(temp9_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     8, 3),
+       SENSOR_ATTR_2(temp10_crit, S_IRUGO | S_IWUSR, show_temp, store_temp,
+                     9, 3),
+};
+
 static struct sensor_device_attribute sda_temp_offset[] = {
        SENSOR_ATTR(temp1_offset, S_IRUGO | S_IWUSR, show_temp_offset,
                    store_temp_offset, 0),
@@ -1860,24 +1895,25 @@ store_pwm_enable(struct device *dev, struct device_attribute *attr,
        return count;
 }
 
-static ssize_t
-show_pwm_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);
-       int nr = sattr->index;
-
-       return sprintf(buf, "%d\n", LM75_TEMP_FROM_REG(data->pwm_temp[nr]));
-}
-
 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_2 *sattr = to_sensor_dev_attr_2(attr);
+       int i, src, sel = 0;
 
-       return sprintf(buf, "%d\n",
-                      data->pwm_temp_sel[sattr->index][sattr->nr]);
+       src = data->pwm_temp_sel[sattr->index][sattr->nr];
+
+       for (i = 0; i < NUM_TEMP; i++) {
+               if (!(data->have_temp & (1 << i)))
+                       continue;
+               if (src == data->temp_src[i]) {
+                       sel = i + 1;
+                       break;
+               }
+       }
+
+       return sprintf(buf, "%d\n", sel);
 }
 
 static ssize_t
@@ -1889,25 +1925,22 @@ store_pwm_temp_sel(struct device *dev, struct device_attribute *attr,
        int nr = sattr->nr;
        int index = sattr->index;
        unsigned long val;
-       int err;
-       int reg;
+       int err, reg, src;
 
        err = kstrtoul(buf, 10, &val);
        if (err < 0)
                return err;
-       if (val == 0 || val > 0x1f)
+       if (val == 0 || val > NUM_TEMP)
                return -EINVAL;
-
-       val = SENSORS_LIMIT(val, 1, data->temp_label_num - 1);
-
-       if (!strlen(data->temp_label[val]))
+       if (!(data->have_temp & (1 << (val - 1))) || !data->temp_src[val - 1])
                return -EINVAL;
 
        mutex_lock(&data->update_lock);
-       data->pwm_temp_sel[index][nr] = val;
+       src = data->temp_src[val - 1];
+       data->pwm_temp_sel[index][nr] = src;
        reg = nct6775_read_value(data, data->REG_TEMP_SEL[index][nr]);
        reg &= 0xe0;
-       reg |= val;
+       reg |= src;
        nct6775_write_value(data, data->REG_TEMP_SEL[index][nr], reg);
        mutex_unlock(&data->update_lock);
 
@@ -2059,13 +2092,6 @@ static SENSOR_DEVICE_ATTR(pwm4_target, S_IWUSR | S_IRUGO, show_target_temp,
 static SENSOR_DEVICE_ATTR(pwm5_target, S_IWUSR | S_IRUGO, show_target_temp,
                          store_target_temp, 4);
 
-/* Monitored pwm temperatures */
-static SENSOR_DEVICE_ATTR(temp11_input, S_IRUGO, show_pwm_temp, NULL, 0);
-static SENSOR_DEVICE_ATTR(temp12_input, S_IRUGO, show_pwm_temp, NULL, 1);
-static SENSOR_DEVICE_ATTR(temp13_input, S_IRUGO, show_pwm_temp, NULL, 2);
-static SENSOR_DEVICE_ATTR(temp14_input, S_IRUGO, show_pwm_temp, NULL, 3);
-static SENSOR_DEVICE_ATTR(temp15_input, S_IRUGO, show_pwm_temp, NULL, 4);
-
 /* Smart Fan registers */
 
 static ssize_t
@@ -2364,9 +2390,8 @@ static struct sensor_device_attribute_2 sda_step_output[] = {
                      4, 4),
 };
 
-static struct attribute *nct6775_attributes_pwm[5][20] = {
+static struct attribute *nct6775_attributes_pwm[5][19] = {
        {
-               &sensor_dev_attr_temp11_input.dev_attr.attr,
                &sensor_dev_attr_pwm1.dev_attr.attr,
                &sensor_dev_attr_pwm1_mode.dev_attr.attr,
                &sensor_dev_attr_pwm1_enable.dev_attr.attr,
@@ -2387,7 +2412,6 @@ static struct attribute *nct6775_attributes_pwm[5][20] = {
                NULL
        },
        {
-               &sensor_dev_attr_temp12_input.dev_attr.attr,
                &sensor_dev_attr_pwm2.dev_attr.attr,
                &sensor_dev_attr_pwm2_mode.dev_attr.attr,
                &sensor_dev_attr_pwm2_enable.dev_attr.attr,
@@ -2408,7 +2432,6 @@ static struct attribute *nct6775_attributes_pwm[5][20] = {
                NULL
        },
        {
-               &sensor_dev_attr_temp13_input.dev_attr.attr,
                &sensor_dev_attr_pwm3.dev_attr.attr,
                &sensor_dev_attr_pwm3_mode.dev_attr.attr,
                &sensor_dev_attr_pwm3_enable.dev_attr.attr,
@@ -2429,7 +2452,6 @@ static struct attribute *nct6775_attributes_pwm[5][20] = {
                NULL
        },
        {
-               &sensor_dev_attr_temp14_input.dev_attr.attr,
                &sensor_dev_attr_pwm4.dev_attr.attr,
                &sensor_dev_attr_pwm4_mode.dev_attr.attr,
                &sensor_dev_attr_pwm4_enable.dev_attr.attr,
@@ -2450,7 +2472,6 @@ static struct attribute *nct6775_attributes_pwm[5][20] = {
                NULL
        },
        {
-               &sensor_dev_attr_temp15_input.dev_attr.attr,
                &sensor_dev_attr_pwm5.dev_attr.attr,
                &sensor_dev_attr_pwm5_mode.dev_attr.attr,
                &sensor_dev_attr_pwm5_enable.dev_attr.attr,
@@ -2919,6 +2940,7 @@ static void nct6775_device_remove_files(struct device *dev)
                device_remove_file(dev, &sda_temp_label[i].dev_attr);
                device_remove_file(dev, &sda_temp_max[i].dev_attr);
                device_remove_file(dev, &sda_temp_max_hyst[i].dev_attr);
+               device_remove_file(dev, &sda_temp_crit[i].dev_attr);
                if (!(data->have_temp_fixed & (1 << i)))
                        continue;
                device_remove_file(dev, &sda_temp_type[i].dev_attr);
@@ -3038,8 +3060,8 @@ nct6775_check_fan_inputs(const struct nct6775_sio_data *sio_data,
                pwm4pin = !(regval & (1 << 1));
                pwm5pin = !(regval & (1 << 2));
 
-               fan3min = 0;
-               fan4min = 0;
+               fan3min = fan3pin;
+               fan4min = fan4pin;
        }
 
        superio_exit(sio_data->sioreg);
@@ -3063,22 +3085,18 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
        int i, s, err = 0;
        int src, mask, available;
        const u16 *reg_temp, *reg_temp_over, *reg_temp_hyst, *reg_temp_config;
+       const u16 *reg_temp_alternate, *reg_temp_crit;
+       int num_reg_temp;
 
        res = platform_get_resource(pdev, IORESOURCE_IO, 0);
-       if (!request_region(res->start, IOREGION_LENGTH, DRVNAME)) {
-               err = -EBUSY;
-               dev_err(dev, "Failed to request region 0x%lx-0x%lx\n",
-                       (unsigned long)res->start,
-                       (unsigned long)res->start + IOREGION_LENGTH - 1);
-               goto exit;
-       }
+       if (!devm_request_region(&pdev->dev, res->start, IOREGION_LENGTH,
+                                DRVNAME))
+               return -EBUSY;
 
        data = devm_kzalloc(&pdev->dev, sizeof(struct nct6775_data),
                            GFP_KERNEL);
-       if (!data) {
-               err = -ENOMEM;
-               goto exit_release;
-       }
+       if (!data)
+               return -ENOMEM;
 
        data->kind = sio_data->kind;
        data->addr = res->start;
@@ -3090,7 +3108,6 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
        switch (data->kind) {
        case nct6775:
                data->in_num = 9;
-               data->have_in = 0x1ff;
                data->pwm_num = 3;
                data->auto_pwm_num = 6;
                data->has_fan_div = true;
@@ -3126,7 +3143,6 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                data->REG_PWM_READ = NCT6775_REG_PWM_READ;
                data->REG_PWM_MODE = NCT6775_REG_PWM_MODE;
                data->PWM_MODE_MASK = NCT6775_PWM_MODE_MASK;
-               data->REG_TEMP_ALTERNATE = NCT6775_REG_TEMP_ALTERNATE;
                data->REG_TEMP_MON = NCT6775_REG_TEMP_MON;
                data->REG_AUTO_TEMP = NCT6775_REG_AUTO_TEMP;
                data->REG_AUTO_PWM = NCT6775_REG_AUTO_PWM;
@@ -3143,14 +3159,16 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                data->REG_ALARM = NCT6775_REG_ALARM;
 
                reg_temp = NCT6775_REG_TEMP;
+               num_reg_temp = ARRAY_SIZE(NCT6775_REG_TEMP);
                reg_temp_over = NCT6775_REG_TEMP_OVER;
                reg_temp_hyst = NCT6775_REG_TEMP_HYST;
                reg_temp_config = NCT6775_REG_TEMP_CONFIG;
+               reg_temp_alternate = NCT6775_REG_TEMP_ALTERNATE;
+               reg_temp_crit = NCT6775_REG_TEMP_CRIT;
 
                break;
        case nct6776:
                data->in_num = 9;
-               data->have_in = 0x1ff;
                data->pwm_num = 3;
                data->auto_pwm_num = 4;
                data->has_fan_div = false;
@@ -3185,7 +3203,6 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                data->REG_PWM_READ = NCT6775_REG_PWM_READ;
                data->REG_PWM_MODE = NCT6776_REG_PWM_MODE;
                data->PWM_MODE_MASK = NCT6776_PWM_MODE_MASK;
-               data->REG_TEMP_ALTERNATE = NCT6776_REG_TEMP_ALTERNATE;
                data->REG_TEMP_MON = NCT6775_REG_TEMP_MON;
                data->REG_AUTO_TEMP = NCT6775_REG_AUTO_TEMP;
                data->REG_AUTO_PWM = NCT6775_REG_AUTO_PWM;
@@ -3202,14 +3219,16 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                data->REG_ALARM = NCT6775_REG_ALARM;
 
                reg_temp = NCT6775_REG_TEMP;
+               num_reg_temp = ARRAY_SIZE(NCT6775_REG_TEMP);
                reg_temp_over = NCT6775_REG_TEMP_OVER;
                reg_temp_hyst = NCT6775_REG_TEMP_HYST;
                reg_temp_config = NCT6776_REG_TEMP_CONFIG;
+               reg_temp_alternate = NCT6776_REG_TEMP_ALTERNATE;
+               reg_temp_crit = NCT6776_REG_TEMP_CRIT;
 
                break;
        case nct6779:
                data->in_num = 15;
-               data->have_in = 0x7fff;
                data->pwm_num = 5;
                data->auto_pwm_num = 4;
                data->has_fan_div = false;
@@ -3244,7 +3263,6 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                data->REG_PWM_READ = NCT6775_REG_PWM_READ;
                data->REG_PWM_MODE = NCT6776_REG_PWM_MODE;
                data->PWM_MODE_MASK = NCT6776_PWM_MODE_MASK;
-               data->REG_TEMP_ALTERNATE = NCT6779_REG_TEMP_ALTERNATE;
                data->REG_TEMP_MON = NCT6775_REG_TEMP_MON;
                data->REG_AUTO_TEMP = NCT6775_REG_AUTO_TEMP;
                data->REG_AUTO_PWM = NCT6775_REG_AUTO_PWM;
@@ -3261,16 +3279,18 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                data->REG_ALARM = NCT6779_REG_ALARM;
 
                reg_temp = NCT6779_REG_TEMP;
-               reg_temp_over = NCT6775_REG_TEMP_OVER;
-               reg_temp_hyst = NCT6775_REG_TEMP_HYST;
-               reg_temp_config = NCT6776_REG_TEMP_CONFIG;
+               num_reg_temp = ARRAY_SIZE(NCT6779_REG_TEMP);
+               reg_temp_over = NCT6779_REG_TEMP_OVER;
+               reg_temp_hyst = NCT6779_REG_TEMP_HYST;
+               reg_temp_config = NCT6779_REG_TEMP_CONFIG;
+               reg_temp_alternate = NCT6779_REG_TEMP_ALTERNATE;
+               reg_temp_crit = NCT6779_REG_TEMP_CRIT;
 
                break;
        default:
-               err = -ENODEV;
-               goto exit_release;
+               return -ENODEV;
        }
-
+       data->have_in = (1 << data->in_num) - 1;
        data->have_temp = 0;
 
        /*
@@ -3283,7 +3303,7 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
         */
        mask = 0;
        available = 0;
-       for (i = 0; i < NUM_REG_TEMP; i++) {
+       for (i = 0; i < num_reg_temp; i++) {
                if (reg_temp[i] == 0)
                        continue;
 
@@ -3313,6 +3333,13 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                        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,
@@ -3325,7 +3352,7 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
 
        mask = 0;
        s = NUM_TEMP_FIXED;     /* First dynamic temperature attribute */
-       for (i = 0; i < NUM_REG_TEMP; i++) {
+       for (i = 0; i < num_reg_temp; i++) {
                if (reg_temp[i] == 0)
                        continue;
 
@@ -3333,6 +3360,14 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                if (!src || (mask & (1 << src)))
                        continue;
 
+               if (src >= data->temp_label_num ||
+                   !strlen(data->temp_label[src])) {
+                       dev_info(dev,
+                                "Invalid temperature source %d at index %d, source register 0x%x, temp register 0x%x\n",
+                                src, i, data->REG_TEMP_SOURCE[i], reg_temp[i]);
+                       continue;
+               }
+
                mask |= 1 << src;
 
                /* Use fixed index for SYSTIN(1), CPUTIN(2), AUXTIN(3) */
@@ -3356,6 +3391,8 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                data->reg_temp[1][s] = reg_temp_over[i];
                data->reg_temp[2][s] = reg_temp_hyst[i];
                data->reg_temp_config[s] = reg_temp_config[i];
+               if (reg_temp_crit[src - 1])
+                       data->reg_temp[3][s] = reg_temp_crit[src - 1];
 
                data->temp_src[s] = src;
                s++;
@@ -3363,35 +3400,35 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
 
 #ifdef TESTING
        /*
-        * We may have alternate registers for some sensors.
-        * Go through the list and enable if possible.
+        * Go through the list of alternate temp registers and enable
+        * if possible.
         * The temperature is already monitored if the respective bit in <mask>
         * is set.
         */
-       if (data->REG_TEMP_ALTERNATE) {
-               for (i = 0; i < data->temp_label_num; i++) {
-                       if (!data->REG_TEMP_ALTERNATE[i])
-                               continue;
-                       if (mask & (1 << (i + 1)))
-                               continue;
-                       if (i < data->temp_fixed_num) {
-                               if (data->have_temp & (1 << i))
-                                       continue;
-                               data->have_temp |= 1 << i;
-                               data->reg_temp[0][i]
-                                 = data->REG_TEMP_ALTERNATE[i];
-                               data->temp_src[i] = i + 1;
+       for (i = 0; i < data->temp_label_num - 1; i++) {
+               if (!reg_temp_alternate[i])
+                       continue;
+               if (mask & (1 << (i + 1)))
+                       continue;
+               if (i < data->temp_fixed_num) {
+                       if (data->have_temp & (1 << i))
                                continue;
-                       }
+                       data->have_temp |= 1 << i;
+                       data->have_temp_fixed |= 1 << i;
+                       data->reg_temp[0][i] = reg_temp_alternate[i];
+                       data->reg_temp[1][i] = reg_temp_over[i];
+                       data->reg_temp[2][i] = reg_temp_hyst[i];
+                       data->temp_src[i] = i + 1;
+                       continue;
+               }
 
-                       if (s >= NUM_TEMP)      /* Abort if no more space */
-                               break;
+               if (s >= NUM_TEMP)      /* Abort if no more space */
+                       break;
 
-                       data->have_temp |= 1 << s;
-                       data->reg_temp[0][s] = data->REG_TEMP_ALTERNATE[i];
-                       data->temp_src[s] = i + 1;
-                       s++;
-               }
+               data->have_temp |= 1 << s;
+               data->reg_temp[0][s] = reg_temp_alternate[i];
+               data->temp_src[s] = i + 1;
+               s++;
        }
 #endif /* TESTING */
 
@@ -3424,21 +3461,11 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                 *      VIN5 / AUXTIN1
                 *      VIN6 / AUXTIN2
                 *      VIN7 / AUXTIN3
-                * Assume voltage is disabled if the respective temperature is
-                * used as temperature source.
+                *
+                * There does not seem to be a clean way to detect if VINx or
+                * AUXTINx is active, so for keep both sensor types enabled
+                * for now.
                 */
-               for (i = 0; i < ARRAY_SIZE(NCT6779_REG_TEMP); i++) {
-                       if (!(data->have_temp & (1 << i)))
-                               continue;
-                       if (i == 2)                             /* AUXTIN0 */
-                               data->have_in &= ~(1 << 6);     /* no VIN4 */
-                       if (i == 3)                             /* AUXTIN1 */
-                               data->have_in &= ~(1 << 10);    /* no VIN5 */
-                       if (i == 4)                             /* AUXTIN2 */
-                               data->have_in &= ~(1 << 11);    /* no VIN6 */
-                       if (i == 5)                             /* AUXTIN3 */
-                               data->have_in &= ~(1 << 14);    /* no VIN7 */
-               }
                break;
        }
 
@@ -3453,9 +3480,6 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
         */
        superio_select(sio_data->sioreg, NCT6775_LD_VID);
        data->vid = superio_inb(sio_data->sioreg, 0xe3);
-       err = device_create_file(dev, &dev_attr_cpu0_vid);
-       if (err)
-               goto exit_release;
 
        if (fan_debounce) {
                u8 tmp;
@@ -3474,11 +3498,16 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                }
                superio_outb(sio_data->sioreg, NCT6775_REG_CR_FAN_DEBOUNCE,
                             tmp);
-               pr_info("Enabled fan debounce for chip %s\n", data->name);
+               dev_info(&pdev->dev, "Enabled fan debounce for chip %s\n",
+                        data->name);
        }
 
        superio_exit(sio_data->sioreg);
 
+       err = device_create_file(dev, &dev_attr_cpu0_vid);
+       if (err)
+               return err;
+
        nct6775_check_fan_inputs(sio_data, data);
 
        /* Read fan clock dividers immediately */
@@ -3583,13 +3612,18 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
                        if (err)
                                goto exit_remove;
                }
+               if (data->reg_temp[3][i]) {
+                       err = device_create_file(dev,
+                                                &sda_temp_crit[i].dev_attr);
+                       if (err)
+                               goto exit_remove;
+               }
                if (!(data->have_temp_fixed & (1 << i)))
                        continue;
                err = device_create_file(dev, &sda_temp_type[i].dev_attr);
                if (err)
                        goto exit_remove;
-               err = device_create_file(dev,
-                                        &sda_temp_offset[i].dev_attr);
+               err = device_create_file(dev, &sda_temp_offset[i].dev_attr);
                if (err)
                        goto exit_remove;
                if (i >= NUM_TEMP_ALARM ||
@@ -3622,9 +3656,6 @@ static int __devinit nct6775_probe(struct platform_device *pdev)
 
 exit_remove:
        nct6775_device_remove_files(dev);
-exit_release:
-       release_region(res->start, IOREGION_LENGTH);
-exit:
        return err;
 }
 
@@ -3634,7 +3665,6 @@ static int __devexit nct6775_remove(struct platform_device *pdev)
 
        hwmon_device_unregister(data->hwmon_dev);
        nct6775_device_remove_files(&pdev->dev);
-       release_region(data->addr, IOREGION_LENGTH);
 
        return 0;
 }