hwmon: (pmbus/stpddc60) Add ST STPDDC60 pmbus driver
authorErik Rosen <erik.rosen@metormote.com>
Thu, 18 Feb 2021 11:52:49 +0000 (12:52 +0100)
committerGuenter Roeck <linux@roeck-us.net>
Tue, 20 Apr 2021 13:50:14 +0000 (06:50 -0700)
Add hardware monitoring support for ST STPDDC60 Unversal Digital
Multicell Controller.

Signed-off-by: Erik Rosen <erik.rosen@metormote.com>
Link: https://lore.kernel.org/r/20210218115249.28513-3-erik.rosen@metormote.com
[groeck: Fixed whitespace error in Makefile]
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Documentation/hwmon/index.rst
Documentation/hwmon/stpddc60.rst [new file with mode: 0644]
MAINTAINERS
drivers/hwmon/pmbus/Kconfig
drivers/hwmon/pmbus/Makefile
drivers/hwmon/pmbus/stpddc60.c [new file with mode: 0644]

index b34894403c2beb9f8a2bea58cd2e4851b4e02ff0..d4b422edbe3a9a2a01963b80bd98fa0f40bde4a2 100644 (file)
@@ -169,6 +169,7 @@ Hardware Monitoring Kernel Drivers
    smsc47m192
    smsc47m1
    sparx5-temp
+   stpddc60
    tc654
    tc74
    thmc50
diff --git a/Documentation/hwmon/stpddc60.rst b/Documentation/hwmon/stpddc60.rst
new file mode 100644 (file)
index 0000000..7f7ce7f
--- /dev/null
@@ -0,0 +1,90 @@
+.. SPDX-License-Identifier: GPL-2.0
+
+Kernel driver stpddc60
+======================
+
+Supported chips:
+
+  * ST STPDDC60
+
+    Prefix: 'stpddc60', 'bmr481'
+
+    Addresses scanned: -
+
+    Datasheet: https://flexpowermodules.com/documents/fpm-techspec-bmr481
+
+Author: Erik Rosen <erik.rosen@metormote.com>
+
+
+Description
+-----------
+
+This driver supports hardware monitoring for ST STPDDC60 controller chip and
+compatible modules.
+
+The driver is a client driver to the core PMBus driver. Please see
+Documentation/hwmon/pmbus.rst and Documentation.hwmon/pmbus-core for details
+on PMBus client drivers.
+
+
+Usage Notes
+-----------
+
+This driver does not auto-detect devices. You will have to instantiate the
+devices explicitly. Please see Documentation/i2c/instantiating-devices.rst for
+details.
+
+The vout under- and over-voltage limits are set in relation to the commanded
+output voltage as a positive or negative offset in the interval 50mV to 400mV
+in 50mV steps. This means that the absolute values of the limits will change
+when the commanded output voltage changes. Also, care should be taken when
+writing to those limits since in the worst case the commanded output voltage
+could change at the same time as the limit is written to, wich will lead to
+unpredictable results.
+
+
+Platform data support
+---------------------
+
+The driver supports standard PMBus driver platform data.
+
+
+Sysfs entries
+-------------
+
+The following attributes are supported. Vin, iout, pout and temp limits
+are read-write; all other attributes are read-only.
+
+======================= ========================================================
+in1_label              "vin"
+in1_input              Measured input voltage.
+in1_lcrit              Critical minimum input voltage.
+in1_crit               Critical maximum input voltage.
+in1_lcrit_alarm                Input voltage critical low alarm.
+in1_crit_alarm         Input voltage critical high alarm.
+
+in2_label              "vout1"
+in2_input              Measured output voltage.
+in2_lcrit              Critical minimum output voltage.
+in2_crit               Critical maximum output voltage.
+in2_lcrit_alarm                Critical output voltage critical low alarm.
+in2_crit_alarm         Critical output voltage critical high alarm.
+
+curr1_label            "iout1"
+curr1_input            Measured output current.
+curr1_max              Maximum output current.
+curr1_max_alarm                Output current high alarm.
+curr1_crit             Critical maximum output current.
+curr1_crit_alarm       Output current critical high alarm.
+
+power1_label           "pout1"
+power1_input           Measured output power.
+power1_crit            Critical maximum output power.
+power1_crit_alarm      Output power critical high alarm.
+
+temp1_input            Measured maximum temperature of all phases.
+temp1_max              Maximum temperature limit.
+temp1_max_alarm                High temperature alarm.
+temp1_crit             Critical maximum temperature limit.
+temp1_crit_alarm       Critical maximum temperature alarm.
+======================= ========================================================
index aa84121c56117ff16eefb996ade37e586183e985..3aaeab8c6b15dffa6dfec8b52270b8f6c73e5e31 100644 (file)
@@ -16947,6 +16947,13 @@ L:     linux-i2c@vger.kernel.org
 S:     Maintained
 F:     drivers/i2c/busses/i2c-stm32*
 
+ST STPDDC60 DRIVER
+M:     Daniel Nilsson <daniel.nilsson@flex.com>
+L:     linux-hwmon@vger.kernel.org
+S:     Maintained
+F:     Documentation/hwmon/stpddc60.rst
+F:     drivers/hwmon/pmbus/stpddc60.c
+
 ST VL53L0X ToF RANGER(I2C) IIO DRIVER
 M:     Song Qiang <songqiang1304521@gmail.com>
 L:     linux-iio@vger.kernel.org
index ee8c27b3b83d3183df514b59b077b6ab86032331..28658b9f09ff6eca72ca4338de3ec1a054e9c679 100644 (file)
@@ -256,6 +256,16 @@ config SENSORS_Q54SJ108A2
          This driver can also be built as a module. If so, the module will
          be called q54sj108a2.
 
+config SENSORS_STPDDC60
+       tristate "ST STPDDC60"
+       help
+         If you say yes here you get hardware monitoring support for ST
+         STPDDC60 Universal Digital Multicell Controller, as well as for
+         Flex BMR481.
+
+         This driver can also be built as a module. If so, the module will
+         be called stpddc60.
+
 config SENSORS_TPS40422
        tristate "TI TPS40422"
        help
index 685a6bc2b15fb5a03db8604dcf9ffde8b0181e18..ed1266f6fb01d6929e1d547a3e99ccb781644f62 100644 (file)
@@ -29,6 +29,7 @@ obj-$(CONFIG_SENSORS_MP2975)  += mp2975.o
 obj-$(CONFIG_SENSORS_PM6764TR) += pm6764tr.o
 obj-$(CONFIG_SENSORS_PXE1610)  += pxe1610.o
 obj-$(CONFIG_SENSORS_Q54SJ108A2)       += q54sj108a2.o
+obj-$(CONFIG_SENSORS_STPDDC60) += stpddc60.o
 obj-$(CONFIG_SENSORS_TPS40422) += tps40422.o
 obj-$(CONFIG_SENSORS_TPS53679) += tps53679.o
 obj-$(CONFIG_SENSORS_UCD9000)  += ucd9000.o
diff --git a/drivers/hwmon/pmbus/stpddc60.c b/drivers/hwmon/pmbus/stpddc60.c
new file mode 100644 (file)
index 0000000..3e67095
--- /dev/null
@@ -0,0 +1,248 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Hardware monitoring driver for the STPDDC60 controller
+ *
+ * Copyright (c) 2021 Flextronics International Sweden AB.
+ */
+
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/init.h>
+#include <linux/err.h>
+#include <linux/i2c.h>
+#include <linux/pmbus.h>
+#include "pmbus.h"
+
+#define STPDDC60_MFR_READ_VOUT         0xd2
+#define STPDDC60_MFR_OV_LIMIT_OFFSET   0xe5
+#define STPDDC60_MFR_UV_LIMIT_OFFSET   0xe6
+
+static const struct i2c_device_id stpddc60_id[] = {
+       {"stpddc60", 0},
+       {"bmr481", 0},
+       {}
+};
+MODULE_DEVICE_TABLE(i2c, stpddc60_id);
+
+static struct pmbus_driver_info stpddc60_info = {
+       .pages = 1,
+       .func[0] = PMBUS_HAVE_VOUT | PMBUS_HAVE_STATUS_VOUT
+               | PMBUS_HAVE_VIN | PMBUS_HAVE_STATUS_INPUT
+               | PMBUS_HAVE_TEMP | PMBUS_HAVE_STATUS_TEMP
+               | PMBUS_HAVE_IOUT | PMBUS_HAVE_STATUS_IOUT
+               | PMBUS_HAVE_POUT,
+};
+
+/*
+ * Calculate the closest absolute offset between commanded vout value
+ * and limit value in steps of 50mv in the range 0 (50mv) to 7 (400mv).
+ * Return 0 if the upper limit is lower than vout or if the lower limit
+ * is higher than vout.
+ */
+static u8 stpddc60_get_offset(int vout, u16 limit, bool over)
+{
+       int offset;
+       long v, l;
+
+       v = 250 + (vout - 1) * 5; /* Convert VID to mv */
+       l = (limit * 1000L) >> 8; /* Convert LINEAR to mv */
+
+       if (over == (l < v))
+               return 0;
+
+       offset = DIV_ROUND_CLOSEST(abs(l - v), 50);
+
+       if (offset > 0)
+               offset--;
+
+       return clamp_val(offset, 0, 7);
+}
+
+/*
+ * Adjust the linear format word to use the given fixed exponent.
+ */
+static u16 stpddc60_adjust_linear(u16 word, s16 fixed)
+{
+       s16 e, m, d;
+
+       e = ((s16)word) >> 11;
+       m = ((s16)((word & 0x7ff) << 5)) >> 5;
+       d = e - fixed;
+
+       if (d >= 0)
+               m <<= d;
+       else
+               m >>= -d;
+
+       return clamp_val(m, 0, 0x3ff) | ((fixed << 11) & 0xf800);
+}
+
+/*
+ * The VOUT_COMMAND register uses the VID format but the vout alarm limit
+ * registers use the LINEAR format so we override VOUT_MODE here to force
+ * LINEAR format for all registers.
+ */
+static int stpddc60_read_byte_data(struct i2c_client *client, int page, int reg)
+{
+       int ret;
+
+       if (page > 0)
+               return -ENXIO;
+
+       switch (reg) {
+       case PMBUS_VOUT_MODE:
+               ret = 0x18;
+               break;
+       default:
+               ret = -ENODATA;
+               break;
+       }
+
+       return ret;
+}
+
+/*
+ * The vout related registers return values in LINEAR11 format when LINEAR16
+ * is expected. Clear the top 5 bits to set the exponent part to zero to
+ * convert the value to LINEAR16 format.
+ */
+static int stpddc60_read_word_data(struct i2c_client *client, int page,
+                                  int phase, int reg)
+{
+       int ret;
+
+       if (page > 0)
+               return -ENXIO;
+
+       switch (reg) {
+       case PMBUS_READ_VOUT:
+               ret = pmbus_read_word_data(client, page, phase,
+                                          STPDDC60_MFR_READ_VOUT);
+               if (ret < 0)
+                       return ret;
+               ret &= 0x7ff;
+               break;
+       case PMBUS_VOUT_OV_FAULT_LIMIT:
+       case PMBUS_VOUT_UV_FAULT_LIMIT:
+               ret = pmbus_read_word_data(client, page, phase, reg);
+               if (ret < 0)
+                       return ret;
+               ret &= 0x7ff;
+               break;
+       default:
+               ret = -ENODATA;
+               break;
+       }
+
+       return ret;
+}
+
+/*
+ * The vout under- and over-voltage limits are set as an offset relative to
+ * the commanded vout voltage. The vin, iout, pout and temp limits must use
+ * the same fixed exponent the chip uses to encode the data when read.
+ */
+static int stpddc60_write_word_data(struct i2c_client *client, int page,
+                                   int reg, u16 word)
+{
+       int ret;
+       u8 offset;
+
+       if (page > 0)
+               return -ENXIO;
+
+       switch (reg) {
+       case PMBUS_VOUT_OV_FAULT_LIMIT:
+               ret = pmbus_read_word_data(client, page, 0xff,
+                                          PMBUS_VOUT_COMMAND);
+               if (ret < 0)
+                       return ret;
+               offset = stpddc60_get_offset(ret, word, true);
+               ret = pmbus_write_byte_data(client, page,
+                                           STPDDC60_MFR_OV_LIMIT_OFFSET,
+                                           offset);
+               break;
+       case PMBUS_VOUT_UV_FAULT_LIMIT:
+               ret = pmbus_read_word_data(client, page, 0xff,
+                                          PMBUS_VOUT_COMMAND);
+               if (ret < 0)
+                       return ret;
+               offset = stpddc60_get_offset(ret, word, false);
+               ret = pmbus_write_byte_data(client, page,
+                                           STPDDC60_MFR_UV_LIMIT_OFFSET,
+                                           offset);
+               break;
+       case PMBUS_VIN_OV_FAULT_LIMIT:
+       case PMBUS_VIN_UV_FAULT_LIMIT:
+       case PMBUS_OT_FAULT_LIMIT:
+       case PMBUS_OT_WARN_LIMIT:
+       case PMBUS_IOUT_OC_FAULT_LIMIT:
+       case PMBUS_IOUT_OC_WARN_LIMIT:
+       case PMBUS_POUT_OP_FAULT_LIMIT:
+               ret = pmbus_read_word_data(client, page, 0xff, reg);
+               if (ret < 0)
+                       return ret;
+               word = stpddc60_adjust_linear(word, ret >> 11);
+               ret = pmbus_write_word_data(client, page, reg, word);
+               break;
+       default:
+               ret = -ENODATA;
+               break;
+       }
+
+       return ret;
+}
+
+static int stpddc60_probe(struct i2c_client *client)
+{
+       int status;
+       u8 device_id[I2C_SMBUS_BLOCK_MAX + 1];
+       const struct i2c_device_id *mid;
+       struct pmbus_driver_info *info = &stpddc60_info;
+
+       if (!i2c_check_functionality(client->adapter,
+                                    I2C_FUNC_SMBUS_READ_BYTE_DATA
+                                    | I2C_FUNC_SMBUS_BLOCK_DATA))
+               return -ENODEV;
+
+       status = i2c_smbus_read_block_data(client, PMBUS_MFR_MODEL, device_id);
+       if (status < 0) {
+               dev_err(&client->dev, "Failed to read Manufacturer Model\n");
+               return status;
+       }
+       for (mid = stpddc60_id; mid->name[0]; mid++) {
+               if (!strncasecmp(mid->name, device_id, strlen(mid->name)))
+                       break;
+       }
+       if (!mid->name[0]) {
+               dev_err(&client->dev, "Unsupported device\n");
+               return -ENODEV;
+       }
+
+       info->read_byte_data = stpddc60_read_byte_data;
+       info->read_word_data = stpddc60_read_word_data;
+       info->write_word_data = stpddc60_write_word_data;
+
+       status = pmbus_do_probe(client, info);
+       if (status < 0)
+               return status;
+
+       pmbus_set_update(client, PMBUS_VOUT_OV_FAULT_LIMIT, true);
+       pmbus_set_update(client, PMBUS_VOUT_UV_FAULT_LIMIT, true);
+
+       return 0;
+}
+
+static struct i2c_driver stpddc60_driver = {
+       .driver = {
+                  .name = "stpddc60",
+                  },
+       .probe_new = stpddc60_probe,
+       .id_table = stpddc60_id,
+};
+
+module_i2c_driver(stpddc60_driver);
+
+MODULE_AUTHOR("Erik Rosen <erik.rosen@metormote.com>");
+MODULE_DESCRIPTION("PMBus driver for ST STPDDC60");
+MODULE_LICENSE("GPL");