hwmon: (lm90) Add support for ON Semiconductor NCT218

NCT218 is compatible to NCT72 and NCT214. It also supports PEC (packet
error checking). Similar to NCT72 and NCT214, PEC support is undocumented.

Unlike NCT214 and NCT72, NCT218 does not support the undocumented secondary
chip and manufacturer ID registers at 0x3e and 0x3f and returns 0x00 when
reading those registers. The value for the chip revision register is not
documented but was observed to be 0xca. Use that information to improve
chip detection accuracy.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

diff --git a/Documentation/hwmon/lm90.rst b/Documentation/hwmon/lm90.rst
index 91947825e7342ab3fdd1133bc5508d070b251be3..ef8b55c17118c4e37d62cbee1e013a8e76f18be1 100644 (file)
--- a/Documentation/hwmon/lm90.rst
+++ b/Documentation/hwmon/lm90.rst
@@ -157,6 +157,16 @@ Supported chips:
 
               https://www.onsemi.com/PowerSolutions/product.do?id=NCT214
 
+  * ON Semiconductor NCT218
+
+    Prefix: 'nct218'
+
+    Addresses scanned: I2C 0x4c - 0x4d
+
+    Datasheet: Publicly available at the ON Semiconductor website
+
+              https://www.onsemi.com/PowerSolutions/product.do?id=NCT218
+
   * ON Semiconductor NCT72
 
     Prefix: 'nct72'

diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
index 9353d207f254a6d092798844301597738d267c25..32c605eaec7e828426ada0b3260c7ecdd1cdd176 100644 (file)
--- a/drivers/hwmon/Kconfig
+++ b/drivers/hwmon/Kconfig
@@ -1365,7 +1365,7 @@ config SENSORS_LM90
          Maxim MAX1617, MAX6642, MAX6646, MAX6647, MAX6648, MAX6649, MAX6654,
          MAX6657, MAX6658, MAX6659, MAX6680, MAX6681, MAX6692, MAX6695,
          MAX6696,
-         ON Semiconductor NCT1008, NCT210, NCT72, NCT214,
+         ON Semiconductor NCT1008, NCT210, NCT72, NCT214, NCT218,
          Winbond/Nuvoton W83L771W/G/AWG/ASG,
          Philips SA56004, GMT G781, Texas Instruments TMP451 and TMP461
          sensor chips.

diff --git a/drivers/hwmon/lm90.c b/drivers/hwmon/lm90.c
index 63bd7c3dcc6be0eaa437917b10a667b958958ef8..a553c611624b61831ce97e90087a5b7891fc7816 100644 (file)
--- a/drivers/hwmon/lm90.c
+++ b/drivers/hwmon/lm90.c
@@ -69,8 +69,9 @@
  * / ON Semiconductor. The chips are similar to ADT7461 but support two external
  * temperature sensors.
  *
- * This driver also supports NCT72 and NCT214 from ON Semiconductor. The chips
- * are similar to ADT7461/ADT7461A but have full PEC support (undocumented).
+ * This driver also supports NCT72, NCT214, and NCT218 from ON Semiconductor.
+ * The chips are similar to ADT7461/ADT7461A but have full PEC support
+ * (undocumented).
  *
  * This driver also supports the SA56004 from Philips. This device is
  * pin-compatible with the LM86, the ED/EDP parts are also address-compatible.
@@ -262,6 +263,7 @@ static const struct i2c_device_id lm90_id[] = {
        { "nct1008", adt7461a },
        { "nct210", nct210 },
        { "nct214", nct72 },
+       { "nct218", nct72 },
        { "nct72", nct72 },
        { "w83l771", w83l771 },
        { "sa56004", sa56004 },
@@ -357,6 +359,10 @@ static const struct of_device_id __maybe_unused lm90_of_match[] = {
                .compatible = "onnn,nct214",
                .data = (void *)nct72
        },
+       {
+               .compatible = "onnn,nct218",
+               .data = (void *)nct72
+       },
        {
                .compatible = "onnn,nct72",
                .data = (void *)nct72
@@ -1780,6 +1786,24 @@ static const char *lm90_detect_national(struct i2c_client *client, int chip_id,
        return name;
 }
 
+static const char *lm90_detect_on(struct i2c_client *client, int chip_id, int config1,
+                                 int convrate)
+{
+       int address = client->addr;
+       const char *name = NULL;
+
+       switch (chip_id) {
+       case 0xca:              /* NCT218 */
+               if ((address == 0x4c || address == 0x4d) && !(config1 & 0x1b) &&
+                   convrate <= 0x0a)
+                       name = "nct218";
+               break;
+       default:
+               break;
+       }
+       return name;
+}
+
 static const char *lm90_detect_analog(struct i2c_client *client, bool common_address,
                                      int chip_id, int config1, int convrate)
 {
@@ -2265,6 +2289,9 @@ static int lm90_detect(struct i2c_client *client, struct i2c_board_info *info)
        case 0x01:      /* National Semiconductor */
                name = lm90_detect_national(client, chip_id, config1, convrate);
                break;
+       case 0x1a:      /* ON */
+               name = lm90_detect_on(client, chip_id, config1, convrate);
+               break;
        case 0x23:      /* Genesys Logic */
                if (common_address && !(config1 & 0x3f) && !(convrate & 0xf8))
                        name = "gl523sm";