drivers/hwmon/max6650.c

   1 /*
   2  * max6650.c - Part of lm_sensors, Linux kernel modules for hardware
   3  *             monitoring.
   4  *
   5  * (C) 2007 by Hans J. Koch <hjk@hansjkoch.de>
   6  *
   7  * based on code written by John Morris <john.morris@spirentcom.com>
   8  * Copyright (c) 2003 Spirent Communications
   9  * and Claus Gindhart <claus.gindhart@kontron.com>
  10  *
  11  * This module has only been tested with the MAX6650 chip. It should
  12  * also work with the MAX6651. It does not distinguish max6650 and max6651
  13  * chips.
  14  *
  15  * The datasheet was last seen at:
  16  *
  17  *        http://pdfserv.maxim-ic.com/en/ds/MAX6650-MAX6651.pdf
  18  *
  19  * This program is free software; you can redistribute it and/or modify
  20  * it under the terms of the GNU General Public License as published by
  21  * the Free Software Foundation; either version 2 of the License, or
  22  * (at your option) any later version.
  23  *
  24  * This program is distributed in the hope that it will be useful,
  25  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  26  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  27  * GNU General Public License for more details.
  28  *
  29  * You should have received a copy of the GNU General Public License
  30  * along with this program; if not, write to the Free Software
  31  * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  32  */
  33
  34 #include <linux/module.h>
  35 #include <linux/init.h>
  36 #include <linux/slab.h>
  37 #include <linux/jiffies.h>
  38 #include <linux/i2c.h>
  39 #include <linux/hwmon.h>
  40 #include <linux/hwmon-sysfs.h>
  41 #include <linux/err.h>
  42
  43 /*
  44  * Insmod parameters
  45  */
  46
  47 /* fan_voltage: 5=5V fan, 12=12V fan, 0=don't change */
  48 static int fan_voltage;
  49 /* prescaler: Possible values are 1, 2, 4, 8, 16 or 0 for don't change */
  50 static int prescaler;
  51 /* clock: The clock frequency of the chip the driver should assume */
  52 static int clock = 254000;
  53
  54 module_param(fan_voltage, int, S_IRUGO);
  55 module_param(prescaler, int, S_IRUGO);
  56 module_param(clock, int, S_IRUGO);
  57
  58 /*
  59  * MAX 6650/6651 registers
  60  */
  61
  62 #define MAX6650_REG_SPEED       0x00
  63 #define MAX6650_REG_CONFIG      0x02
  64 #define MAX6650_REG_GPIO_DEF    0x04
  65 #define MAX6650_REG_DAC         0x06
  66 #define MAX6650_REG_ALARM_EN    0x08
  67 #define MAX6650_REG_ALARM       0x0A
  68 #define MAX6650_REG_TACH0       0x0C
  69 #define MAX6650_REG_TACH1       0x0E
  70 #define MAX6650_REG_TACH2       0x10
  71 #define MAX6650_REG_TACH3       0x12
  72 #define MAX6650_REG_GPIO_STAT   0x14
  73 #define MAX6650_REG_COUNT       0x16
  74
  75 /*
  76  * Config register bits
  77  */
  78
  79 #define MAX6650_CFG_V12                 0x08
  80 #define MAX6650_CFG_PRESCALER_MASK      0x07
  81 #define MAX6650_CFG_PRESCALER_2         0x01
  82 #define MAX6650_CFG_PRESCALER_4         0x02
  83 #define MAX6650_CFG_PRESCALER_8         0x03
  84 #define MAX6650_CFG_PRESCALER_16        0x04
  85 #define MAX6650_CFG_MODE_MASK           0x30
  86 #define MAX6650_CFG_MODE_ON             0x00
  87 #define MAX6650_CFG_MODE_OFF            0x10
  88 #define MAX6650_CFG_MODE_CLOSED_LOOP    0x20
  89 #define MAX6650_CFG_MODE_OPEN_LOOP      0x30
  90 #define MAX6650_COUNT_MASK              0x03
  91
  92 /*
  93  * Alarm status register bits
  94  */
  95
  96 #define MAX6650_ALRM_MAX        0x01
  97 #define MAX6650_ALRM_MIN        0x02
  98 #define MAX6650_ALRM_TACH       0x04
  99 #define MAX6650_ALRM_GPIO1      0x08
 100 #define MAX6650_ALRM_GPIO2      0x10
 101
 102 /* Minimum and maximum values of the FAN-RPM */
 103 #define FAN_RPM_MIN 240
 104 #define FAN_RPM_MAX 30000
 105
 106 #define DIV_FROM_REG(reg) (1 << (reg & 7))
 107
 108 /*
 109  * Client data (each client gets its own)
 110  */
 111
 112 struct max6650_data {
 113         struct device *hwmon_dev;
 114         struct mutex update_lock;
 115         int nr_fans;
 116         char valid; /* zero until following fields are valid */
 117         unsigned long last_updated; /* in jiffies */
 118
 119         /* register values */
 120         u8 speed;
 121         u8 config;
 122         u8 tach[4];
 123         u8 count;
 124         u8 dac;
 125         u8 alarm;
 126 };
 127
 128 static const u8 tach_reg[] = {
 129         MAX6650_REG_TACH0,
 130         MAX6650_REG_TACH1,
 131         MAX6650_REG_TACH2,
 132         MAX6650_REG_TACH3,
 133 };
 134
 135 static struct max6650_data *max6650_update_device(struct device *dev)
 136 {
 137         int i;
 138         struct i2c_client *client = to_i2c_client(dev);
 139         struct max6650_data *data = i2c_get_clientdata(client);
 140
 141         mutex_lock(&data->update_lock);
 142
 143         if (time_after(jiffies, data->last_updated + HZ) || !data->valid) {
 144                 data->speed = i2c_smbus_read_byte_data(client,
 145                                                        MAX6650_REG_SPEED);
 146                 data->config = i2c_smbus_read_byte_data(client,
 147                                                         MAX6650_REG_CONFIG);
 148                 for (i = 0; i < data->nr_fans; i++) {
 149                         data->tach[i] = i2c_smbus_read_byte_data(client,
 150                                                                  tach_reg[i]);
 151                 }
 152                 data->count = i2c_smbus_read_byte_data(client,
 153                                                         MAX6650_REG_COUNT);
 154                 data->dac = i2c_smbus_read_byte_data(client, MAX6650_REG_DAC);
 155
 156                 /*
 157                  * Alarms are cleared on read in case the condition that
 158                  * caused the alarm is removed. Keep the value latched here
 159                  * for providing the register through different alarm files.
 160                  */
 161                 data->alarm |= i2c_smbus_read_byte_data(client,
 162                                                         MAX6650_REG_ALARM);
 163
 164                 data->last_updated = jiffies;
 165                 data->valid = 1;
 166         }
 167
 168         mutex_unlock(&data->update_lock);
 169
 170         return data;
 171 }
 172
 173 static ssize_t get_fan(struct device *dev, struct device_attribute *devattr,
 174                        char *buf)
 175 {
 176         struct sensor_device_attribute *attr = to_sensor_dev_attr(devattr);
 177         struct max6650_data *data = max6650_update_device(dev);
 178         int rpm;
 179
 180         /*
 181          * Calculation details:
 182          *
 183          * Each tachometer counts over an interval given by the "count"
 184          * register (0.25, 0.5, 1 or 2 seconds). This module assumes
 185          * that the fans produce two pulses per revolution (this seems
 186          * to be the most common).
 187          */
 188
 189         rpm = ((data->tach[attr->index] * 120) / DIV_FROM_REG(data->count));
 190         return sprintf(buf, "%d\n", rpm);
 191 }
 192
 193 /*
 194  * Set the fan speed to the specified RPM (or read back the RPM setting).
 195  * This works in closed loop mode only. Use pwm1 for open loop speed setting.
 196  *
 197  * The MAX6650/1 will automatically control fan speed when in closed loop
 198  * mode.
 199  *
 200  * Assumptions:
 201  *
 202  * 1) The MAX6650/1 internal 254kHz clock frequency is set correctly. Use
 203  *    the clock module parameter if you need to fine tune this.
 204  *
 205  * 2) The prescaler (low three bits of the config register) has already
 206  *    been set to an appropriate value. Use the prescaler module parameter
 207  *    if your BIOS doesn't initialize the chip properly.
 208  *
 209  * The relevant equations are given on pages 21 and 22 of the datasheet.
 210  *
 211  * From the datasheet, the relevant equation when in regulation is:
 212  *
 213  *    [fCLK / (128 x (KTACH + 1))] = 2 x FanSpeed / KSCALE
 214  *
 215  * where:
 216  *
 217  *    fCLK is the oscillator frequency (either the 254kHz internal
 218  *         oscillator or the externally applied clock)
 219  *
 220  *    KTACH is the value in the speed register
 221  *
 222  *    FanSpeed is the speed of the fan in rps
 223  *
 224  *    KSCALE is the prescaler value (1, 2, 4, 8, or 16)
 225  *
 226  * When reading, we need to solve for FanSpeed. When writing, we need to
 227  * solve for KTACH.
 228  *
 229  * Note: this tachometer is completely separate from the tachometers
 230  * used to measure the fan speeds. Only one fan's speed (fan1) is
 231  * controlled.
 232  */
 233
 234 static ssize_t get_target(struct device *dev, struct device_attribute *devattr,
 235                          char *buf)
 236 {
 237         struct max6650_data *data = max6650_update_device(dev);
 238         int kscale, ktach, rpm;
 239
 240         /*
 241          * Use the datasheet equation:
 242          *
 243          *    FanSpeed = KSCALE x fCLK / [256 x (KTACH + 1)]
 244          *
 245          * then multiply by 60 to give rpm.
 246          */
 247
 248         kscale = DIV_FROM_REG(data->config);
 249         ktach = data->speed;
 250         rpm = 60 * kscale * clock / (256 * (ktach + 1));
 251         return sprintf(buf, "%d\n", rpm);
 252 }
 253
 254 static ssize_t set_target(struct device *dev, struct device_attribute *devattr,
 255                          const char *buf, size_t count)
 256 {
 257         struct i2c_client *client = to_i2c_client(dev);
 258         struct max6650_data *data = i2c_get_clientdata(client);
 259         int kscale, ktach;
 260         unsigned long rpm;
 261         int err;
 262
 263         err = kstrtoul(buf, 10, &rpm);
 264         if (err)
 265                 return err;
 266
 267         rpm = clamp_val(rpm, FAN_RPM_MIN, FAN_RPM_MAX);
 268
 269         /*
 270          * Divide the required speed by 60 to get from rpm to rps, then
 271          * use the datasheet equation:
 272          *
 273          *     KTACH = [(fCLK x KSCALE) / (256 x FanSpeed)] - 1
 274          */
 275
 276         mutex_lock(&data->update_lock);
 277
 278         kscale = DIV_FROM_REG(data->config);
 279         ktach = ((clock * kscale) / (256 * rpm / 60)) - 1;
 280         if (ktach < 0)
 281                 ktach = 0;
 282         if (ktach > 255)
 283                 ktach = 255;
 284         data->speed = ktach;
 285
 286         i2c_smbus_write_byte_data(client, MAX6650_REG_SPEED, data->speed);
 287
 288         mutex_unlock(&data->update_lock);
 289
 290         return count;
 291 }
 292
 293 /*
 294  * Get/set the fan speed in open loop mode using pwm1 sysfs file.
 295  * Speed is given as a relative value from 0 to 255, where 255 is maximum
 296  * speed. Note that this is done by writing directly to the chip's DAC,
 297  * it won't change the closed loop speed set by fan1_target.
 298  * Also note that due to rounding errors it is possible that you don't read
 299  * back exactly the value you have set.
 300  */
 301
 302 static ssize_t get_pwm(struct device *dev, struct device_attribute *devattr,
 303                        char *buf)
 304 {
 305         int pwm;
 306         struct max6650_data *data = max6650_update_device(dev);
 307
 308         /*
 309          * Useful range for dac is 0-180 for 12V fans and 0-76 for 5V fans.
 310          * Lower DAC values mean higher speeds.
 311          */
 312         if (data->config & MAX6650_CFG_V12)
 313                 pwm = 255 - (255 * (int)data->dac)/180;
 314         else
 315                 pwm = 255 - (255 * (int)data->dac)/76;
 316
 317         if (pwm < 0)
 318                 pwm = 0;
 319
 320         return sprintf(buf, "%d\n", pwm);
 321 }
 322
 323 static ssize_t set_pwm(struct device *dev, struct device_attribute *devattr,
 324                         const char *buf, size_t count)
 325 {
 326         struct i2c_client *client = to_i2c_client(dev);
 327         struct max6650_data *data = i2c_get_clientdata(client);
 328         unsigned long pwm;
 329         int err;
 330
 331         err = kstrtoul(buf, 10, &pwm);
 332         if (err)
 333                 return err;
 334
 335         pwm = clamp_val(pwm, 0, 255);
 336
 337         mutex_lock(&data->update_lock);
 338
 339         if (data->config & MAX6650_CFG_V12)
 340                 data->dac = 180 - (180 * pwm)/255;
 341         else
 342                 data->dac = 76 - (76 * pwm)/255;
 343
 344         i2c_smbus_write_byte_data(client, MAX6650_REG_DAC, data->dac);
 345
 346         mutex_unlock(&data->update_lock);
 347
 348         return count;
 349 }
 350
 351 /*
 352  * Get/Set controller mode:
 353  * Possible values:
 354  * 0 = Fan always on
 355  * 1 = Open loop, Voltage is set according to speed, not regulated.
 356  * 2 = Closed loop, RPM for all fans regulated by fan1 tachometer
 357  */
 358
 359 static ssize_t get_enable(struct device *dev, struct device_attribute *devattr,
 360                           char *buf)
 361 {
 362         struct max6650_data *data = max6650_update_device(dev);
 363         int mode = (data->config & MAX6650_CFG_MODE_MASK) >> 4;
 364         int sysfs_modes[4] = {0, 1, 2, 1};
 365
 366         return sprintf(buf, "%d\n", sysfs_modes[mode]);
 367 }
 368
 369 static ssize_t set_enable(struct device *dev, struct device_attribute *devattr,
 370                           const char *buf, size_t count)
 371 {
 372         struct i2c_client *client = to_i2c_client(dev);
 373         struct max6650_data *data = i2c_get_clientdata(client);
 374         int max6650_modes[3] = {0, 3, 2};
 375         unsigned long mode;
 376         int err;
 377
 378         err = kstrtoul(buf, 10, &mode);
 379         if (err)
 380                 return err;
 381
 382         if (mode > 2)
 383                 return -EINVAL;
 384
 385         mutex_lock(&data->update_lock);
 386
 387         data->config = i2c_smbus_read_byte_data(client, MAX6650_REG_CONFIG);
 388         data->config = (data->config & ~MAX6650_CFG_MODE_MASK)
 389                        | (max6650_modes[mode] << 4);
 390
 391         i2c_smbus_write_byte_data(client, MAX6650_REG_CONFIG, data->config);
 392
 393         mutex_unlock(&data->update_lock);
 394
 395         return count;
 396 }
 397
 398 /*
 399  * Read/write functions for fan1_div sysfs file. The MAX6650 has no such
 400  * divider. We handle this by converting between divider and counttime:
 401  *
 402  * (counttime == k) <==> (divider == 2^k), k = 0, 1, 2, or 3
 403  *
 404  * Lower values of k allow to connect a faster fan without the risk of
 405  * counter overflow. The price is lower resolution. You can also set counttime
 406  * using the module parameter. Note that the module parameter "prescaler" also
 407  * influences the behaviour. Unfortunately, there's no sysfs attribute
 408  * defined for that. See the data sheet for details.
 409  */
 410
 411 static ssize_t get_div(struct device *dev, struct device_attribute *devattr,
 412                        char *buf)
 413 {
 414         struct max6650_data *data = max6650_update_device(dev);
 415
 416         return sprintf(buf, "%d\n", DIV_FROM_REG(data->count));
 417 }
 418
 419 static ssize_t set_div(struct device *dev, struct device_attribute *devattr,
 420                        const char *buf, size_t count)
 421 {
 422         struct i2c_client *client = to_i2c_client(dev);
 423         struct max6650_data *data = i2c_get_clientdata(client);
 424         unsigned long div;
 425         int err;
 426
 427         err = kstrtoul(buf, 10, &div);
 428         if (err)
 429                 return err;
 430
 431         mutex_lock(&data->update_lock);
 432         switch (div) {
 433         case 1:
 434                 data->count = 0;
 435                 break;
 436         case 2:
 437                 data->count = 1;
 438                 break;
 439         case 4:
 440                 data->count = 2;
 441                 break;
 442         case 8:
 443                 data->count = 3;
 444                 break;
 445         default:
 446                 mutex_unlock(&data->update_lock);
 447                 return -EINVAL;
 448         }
 449
 450         i2c_smbus_write_byte_data(client, MAX6650_REG_COUNT, data->count);
 451         mutex_unlock(&data->update_lock);
 452
 453         return count;
 454 }
 455
 456 /*
 457  * Get alarm stati:
 458  * Possible values:
 459  * 0 = no alarm
 460  * 1 = alarm
 461  */
 462
 463 static ssize_t get_alarm(struct device *dev, struct device_attribute *devattr,
 464                          char *buf)
 465 {
 466         struct sensor_device_attribute *attr = to_sensor_dev_attr(devattr);
 467         struct max6650_data *data = max6650_update_device(dev);
 468         struct i2c_client *client = to_i2c_client(dev);
 469         int alarm = 0;
 470
 471         if (data->alarm & attr->index) {
 472                 mutex_lock(&data->update_lock);
 473                 alarm = 1;
 474                 data->alarm &= ~attr->index;
 475                 data->alarm |= i2c_smbus_read_byte_data(client,
 476                                                         MAX6650_REG_ALARM);
 477                 mutex_unlock(&data->update_lock);
 478         }
 479
 480         return sprintf(buf, "%d\n", alarm);
 481 }
 482
 483 static SENSOR_DEVICE_ATTR(fan1_input, S_IRUGO, get_fan, NULL, 0);
 484 static SENSOR_DEVICE_ATTR(fan2_input, S_IRUGO, get_fan, NULL, 1);
 485 static SENSOR_DEVICE_ATTR(fan3_input, S_IRUGO, get_fan, NULL, 2);
 486 static SENSOR_DEVICE_ATTR(fan4_input, S_IRUGO, get_fan, NULL, 3);
 487 static DEVICE_ATTR(fan1_target, S_IWUSR | S_IRUGO, get_target, set_target);
 488 static DEVICE_ATTR(fan1_div, S_IWUSR | S_IRUGO, get_div, set_div);
 489 static DEVICE_ATTR(pwm1_enable, S_IWUSR | S_IRUGO, get_enable, set_enable);
 490 static DEVICE_ATTR(pwm1, S_IWUSR | S_IRUGO, get_pwm, set_pwm);
 491 static SENSOR_DEVICE_ATTR(fan1_max_alarm, S_IRUGO, get_alarm, NULL,
 492                           MAX6650_ALRM_MAX);
 493 static SENSOR_DEVICE_ATTR(fan1_min_alarm, S_IRUGO, get_alarm, NULL,
 494                           MAX6650_ALRM_MIN);
 495 static SENSOR_DEVICE_ATTR(fan1_fault, S_IRUGO, get_alarm, NULL,
 496                           MAX6650_ALRM_TACH);
 497 static SENSOR_DEVICE_ATTR(gpio1_alarm, S_IRUGO, get_alarm, NULL,
 498                           MAX6650_ALRM_GPIO1);
 499 static SENSOR_DEVICE_ATTR(gpio2_alarm, S_IRUGO, get_alarm, NULL,
 500                           MAX6650_ALRM_GPIO2);
 501
 502 static umode_t max6650_attrs_visible(struct kobject *kobj, struct attribute *a,
 503                                     int n)
 504 {
 505         struct device *dev = container_of(kobj, struct device, kobj);
 506         struct i2c_client *client = to_i2c_client(dev);
 507         u8 alarm_en = i2c_smbus_read_byte_data(client, MAX6650_REG_ALARM_EN);
 508         struct device_attribute *devattr;
 509
 510         /*
 511          * Hide the alarms that have not been enabled by the firmware
 512          */
 513
 514         devattr = container_of(a, struct device_attribute, attr);
 515         if (devattr == &sensor_dev_attr_fan1_max_alarm.dev_attr
 516          || devattr == &sensor_dev_attr_fan1_min_alarm.dev_attr
 517          || devattr == &sensor_dev_attr_fan1_fault.dev_attr
 518          || devattr == &sensor_dev_attr_gpio1_alarm.dev_attr
 519          || devattr == &sensor_dev_attr_gpio2_alarm.dev_attr) {
 520                 if (!(alarm_en & to_sensor_dev_attr(devattr)->index))
 521                         return 0;
 522         }
 523
 524         return a->mode;
 525 }
 526
 527 static struct attribute *max6650_attrs[] = {
 528         &sensor_dev_attr_fan1_input.dev_attr.attr,
 529         &dev_attr_fan1_target.attr,
 530         &dev_attr_fan1_div.attr,
 531         &dev_attr_pwm1_enable.attr,
 532         &dev_attr_pwm1.attr,
 533         &sensor_dev_attr_fan1_max_alarm.dev_attr.attr,
 534         &sensor_dev_attr_fan1_min_alarm.dev_attr.attr,
 535         &sensor_dev_attr_fan1_fault.dev_attr.attr,
 536         &sensor_dev_attr_gpio1_alarm.dev_attr.attr,
 537         &sensor_dev_attr_gpio2_alarm.dev_attr.attr,
 538         NULL
 539 };
 540
 541 static struct attribute_group max6650_attr_grp = {
 542         .attrs = max6650_attrs,
 543         .is_visible = max6650_attrs_visible,
 544 };
 545
 546 static struct attribute *max6651_attrs[] = {
 547         &sensor_dev_attr_fan2_input.dev_attr.attr,
 548         &sensor_dev_attr_fan3_input.dev_attr.attr,
 549         &sensor_dev_attr_fan4_input.dev_attr.attr,
 550         NULL
 551 };
 552
 553 static const struct attribute_group max6651_attr_grp = {
 554         .attrs = max6651_attrs,
 555 };
 556
 557 /*
 558  * Real code
 559  */
 560
 561 static int max6650_init_client(struct i2c_client *client)
 562 {
 563         struct max6650_data *data = i2c_get_clientdata(client);
 564         int config;
 565         int err = -EIO;
 566
 567         config = i2c_smbus_read_byte_data(client, MAX6650_REG_CONFIG);
 568
 569         if (config < 0) {
 570                 dev_err(&client->dev, "Error reading config, aborting.\n");
 571                 return err;
 572         }
 573
 574         switch (fan_voltage) {
 575         case 0:
 576                 break;
 577         case 5:
 578                 config &= ~MAX6650_CFG_V12;
 579                 break;
 580         case 12:
 581                 config |= MAX6650_CFG_V12;
 582                 break;
 583         default:
 584                 dev_err(&client->dev, "illegal value for fan_voltage (%d)\n",
 585                         fan_voltage);
 586         }
 587
 588         dev_info(&client->dev, "Fan voltage is set to %dV.\n",
 589                  (config & MAX6650_CFG_V12) ? 12 : 5);
 590
 591         switch (prescaler) {
 592         case 0:
 593                 break;
 594         case 1:
 595                 config &= ~MAX6650_CFG_PRESCALER_MASK;
 596                 break;
 597         case 2:
 598                 config = (config & ~MAX6650_CFG_PRESCALER_MASK)
 599                          | MAX6650_CFG_PRESCALER_2;
 600                 break;
 601         case  4:
 602                 config = (config & ~MAX6650_CFG_PRESCALER_MASK)
 603                          | MAX6650_CFG_PRESCALER_4;
 604                 break;
 605         case  8:
 606                 config = (config & ~MAX6650_CFG_PRESCALER_MASK)
 607                          | MAX6650_CFG_PRESCALER_8;
 608                 break;
 609         case 16:
 610                 config = (config & ~MAX6650_CFG_PRESCALER_MASK)
 611                          | MAX6650_CFG_PRESCALER_16;
 612                 break;
 613         default:
 614                 dev_err(&client->dev, "illegal value for prescaler (%d)\n",
 615                         prescaler);
 616         }
 617
 618         dev_info(&client->dev, "Prescaler is set to %d.\n",
 619                  1 << (config & MAX6650_CFG_PRESCALER_MASK));
 620
 621         /*
 622          * If mode is set to "full off", we change it to "open loop" and
 623          * set DAC to 255, which has the same effect. We do this because
 624          * there's no "full off" mode defined in hwmon specifications.
 625          */
 626
 627         if ((config & MAX6650_CFG_MODE_MASK) == MAX6650_CFG_MODE_OFF) {
 628                 dev_dbg(&client->dev, "Change mode to open loop, full off.\n");
 629                 config = (config & ~MAX6650_CFG_MODE_MASK)
 630                          | MAX6650_CFG_MODE_OPEN_LOOP;
 631                 if (i2c_smbus_write_byte_data(client, MAX6650_REG_DAC, 255)) {
 632                         dev_err(&client->dev, "DAC write error, aborting.\n");
 633                         return err;
 634                 }
 635         }
 636
 637         if (i2c_smbus_write_byte_data(client, MAX6650_REG_CONFIG, config)) {
 638                 dev_err(&client->dev, "Config write error, aborting.\n");
 639                 return err;
 640         }
 641
 642         data->config = config;
 643         data->count = i2c_smbus_read_byte_data(client, MAX6650_REG_COUNT);
 644
 645         return 0;
 646 }
 647
 648 static int max6650_probe(struct i2c_client *client,
 649                          const struct i2c_device_id *id)
 650 {
 651         struct max6650_data *data;
 652         int err;
 653
 654         data = devm_kzalloc(&client->dev, sizeof(struct max6650_data),
 655                             GFP_KERNEL);
 656         if (!data) {
 657                 dev_err(&client->dev, "out of memory.\n");
 658                 return -ENOMEM;
 659         }
 660
 661         i2c_set_clientdata(client, data);
 662         mutex_init(&data->update_lock);
 663         data->nr_fans = id->driver_data;
 664
 665         /*
 666          * Initialize the max6650 chip
 667          */
 668         err = max6650_init_client(client);
 669         if (err)
 670                 return err;
 671
 672         err = sysfs_create_group(&client->dev.kobj, &max6650_attr_grp);
 673         if (err)
 674                 return err;
 675         /* 3 additional fan inputs for the MAX6651 */
 676         if (data->nr_fans == 4) {
 677                 err = sysfs_create_group(&client->dev.kobj, &max6651_attr_grp);
 678                 if (err)
 679                         goto err_remove;
 680         }
 681
 682         data->hwmon_dev = hwmon_device_register(&client->dev);
 683         if (!IS_ERR(data->hwmon_dev))
 684                 return 0;
 685
 686         err = PTR_ERR(data->hwmon_dev);
 687         dev_err(&client->dev, "error registering hwmon device.\n");
 688         if (data->nr_fans == 4)
 689                 sysfs_remove_group(&client->dev.kobj, &max6651_attr_grp);
 690 err_remove:
 691         sysfs_remove_group(&client->dev.kobj, &max6650_attr_grp);
 692         return err;
 693 }
 694
 695 static int max6650_remove(struct i2c_client *client)
 696 {
 697         struct max6650_data *data = i2c_get_clientdata(client);
 698
 699         hwmon_device_unregister(data->hwmon_dev);
 700         if (data->nr_fans == 4)
 701                 sysfs_remove_group(&client->dev.kobj, &max6651_attr_grp);
 702         sysfs_remove_group(&client->dev.kobj, &max6650_attr_grp);
 703         return 0;
 704 }
 705
 706 static const struct i2c_device_id max6650_id[] = {
 707         { "max6650", 1 },
 708         { "max6651", 4 },
 709         { }
 710 };
 711 MODULE_DEVICE_TABLE(i2c, max6650_id);
 712
 713 static struct i2c_driver max6650_driver = {
 714         .driver = {
 715                 .name   = "max6650",
 716         },
 717         .probe          = max6650_probe,
 718         .remove         = max6650_remove,
 719         .id_table       = max6650_id,
 720 };
 721
 722 module_i2c_driver(max6650_driver);
 723
 724 MODULE_AUTHOR("Hans J. Koch");
 725 MODULE_DESCRIPTION("MAX6650 sensor driver");
 726 MODULE_LICENSE("GPL");