[PATCH] lm83: Add LM82 support

Add LM82 temperature sensor support (similar to the LM83,
but less featureful).

Signed-off-by: Jordan Crouse <jordan.crouse@amd.com>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

2 files changed, 39 insertions, 15 deletions

diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
index 99cdc612d2c6..a6a8d66129b3 100644
--- a/drivers/hwmon/Kconfig
+++ b/drivers/hwmon/Kconfig
@@ -236,11 +236,11 @@ config SENSORS_LM80
           will be called lm80.
 
 config SENSORS_LM83
-        tristate "National Semiconductor LM83"
+        tristate "National Semiconductor LM83 and compatibles"
         depends on HWMON && I2C
         help
           If you say yes here you get support for National Semiconductor
-          LM83 sensor chips.
+          LM82 and LM83 sensor chips.
 
           This driver can also be built as a module.  If so, the module
           will be called lm83.
diff --git a/drivers/hwmon/lm83.c b/drivers/hwmon/lm83.c
index aac4ec2bf694..2137d7879df6 100644
--- a/drivers/hwmon/lm83.c
+++ b/drivers/hwmon/lm83.c
@@ -12,6 +12,10 @@
  * Since the datasheet omits to give the chip stepping code, I give it
  * here: 0x03 (at register 0xff).
  *
+ * Also supports the LM82 temp sensor, which is basically a stripped down
+ * model of the LM83.  Datasheet is here:
+ * http://www.national.com/pf/LM/LM82.html
+ *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
@@ -52,7 +56,7 @@ static unsigned short normal_i2c[] = { 0x18, 0x19, 0x1a,
  * Insmod parameters
  */
 
-I2C_CLIENT_INSMOD_1(lm83);
+I2C_CLIENT_INSMOD_2(lm83, lm82);
 
 /*
  * The LM83 registers
@@ -283,6 +287,9 @@ static int lm83_detect(struct i2c_adapter *adapter, int address, int kind)
                 if (man_id == 0x01) { /* National Semiconductor */
                         if (chip_id == 0x03) {
                                 kind = lm83;
+                        } else
+                        if (chip_id == 0x01) {
+                                kind = lm82;
                         }
                 }
 
@@ -296,6 +303,9 @@ static int lm83_detect(struct i2c_adapter *adapter, int address, int kind)
 
         if (kind == lm83) {
                 name = "lm83";
+        } else
+        if (kind == lm82) {
+                name = "lm82";
         }
 
         /* We can fill in the remaining client fields */
@@ -319,32 +329,46 @@ static int lm83_detect(struct i2c_adapter *adapter, int address, int kind)
                 goto exit_detach;
         }
 
+        /*
+         * The LM82 can only monitor one external diode which is
+         * at the same register as the LM83 temp3 entry - so we
+         * declare 1 and 3 common, and then 2 and 4 only for the LM83.
+         */
+
         device_create_file(&new_client->dev,
                            &sensor_dev_attr_temp1_input.dev_attr);
         device_create_file(&new_client->dev,
-                           &sensor_dev_attr_temp2_input.dev_attr);
-        device_create_file(&new_client->dev,
                            &sensor_dev_attr_temp3_input.dev_attr);
-        device_create_file(&new_client->dev,
-                           &sensor_dev_attr_temp4_input.dev_attr);
+
         device_create_file(&new_client->dev,
                            &sensor_dev_attr_temp1_max.dev_attr);
         device_create_file(&new_client->dev,
-                           &sensor_dev_attr_temp2_max.dev_attr);
-        device_create_file(&new_client->dev,
                            &sensor_dev_attr_temp3_max.dev_attr);
-        device_create_file(&new_client->dev,
-                           &sensor_dev_attr_temp4_max.dev_attr);
+
         device_create_file(&new_client->dev,
                            &sensor_dev_attr_temp1_crit.dev_attr);
         device_create_file(&new_client->dev,
-                           &sensor_dev_attr_temp2_crit.dev_attr);
-        device_create_file(&new_client->dev,
                            &sensor_dev_attr_temp3_crit.dev_attr);
-        device_create_file(&new_client->dev,
-                           &sensor_dev_attr_temp4_crit.dev_attr);
+
         device_create_file(&new_client->dev, &dev_attr_alarms);
 
+        if (kind == lm83) {
+                device_create_file(&new_client->dev,
+                                   &sensor_dev_attr_temp2_input.dev_attr);
+                device_create_file(&new_client->dev,
+                                   &sensor_dev_attr_temp4_input.dev_attr);
+
+                device_create_file(&new_client->dev,
+                                   &sensor_dev_attr_temp2_max.dev_attr);
+                device_create_file(&new_client->dev,
+                                   &sensor_dev_attr_temp4_max.dev_attr);
+
+                device_create_file(&new_client->dev,
+                                   &sensor_dev_attr_temp2_crit.dev_attr);
+                device_create_file(&new_client->dev,
+                                   &sensor_dev_attr_temp4_crit.dev_attr);
+        }
+
         return 0;
 
 exit_detach: