* [PATCH 0/2] hwmon: Add support for MPS mp2845 chip
@ 2026-02-25 8:55 wenswang
2026-02-25 8:56 ` [PATCH 1/2] dt-bindings: hwmon: Add MPS mp2845 wenswang
0 siblings, 1 reply; 6+ messages in thread
From: wenswang @ 2026-02-25 8:55 UTC (permalink / raw)
To: robh, krzk+dt, conor+dt, linux, corbet, skhan
Cc: devicetree, linux-kernel, linux-hwmon, linux-doc, Wensheng Wang
From: Wensheng Wang <wenswang@yeah.net>
Add mp2845 driver in hwmon and add dt-bindings for it.
Wensheng Wang (2):
dt-bindings: hwmon: Add MPS mp2845
hwmon: add MP2845 driver
.../devicetree/bindings/trivial-devices.yaml | 2 +
Documentation/hwmon/index.rst | 1 +
Documentation/hwmon/mp2845.rst | 143 +++++
MAINTAINERS | 7 +
drivers/hwmon/Kconfig | 10 +
drivers/hwmon/Makefile | 1 +
drivers/hwmon/mp2845.c | 493 ++++++++++++++++++
7 files changed, 657 insertions(+)
create mode 100644 Documentation/hwmon/mp2845.rst
create mode 100644 drivers/hwmon/mp2845.c
--
2.25.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] dt-bindings: hwmon: Add MPS mp2845
2026-02-25 8:55 [PATCH 0/2] hwmon: Add support for MPS mp2845 chip wenswang
@ 2026-02-25 8:56 ` wenswang
2026-02-25 8:56 ` [PATCH 2/2] hwmon: add MP2845 driver wenswang
2026-02-26 7:45 ` [PATCH 1/2] dt-bindings: hwmon: Add MPS mp2845 Krzysztof Kozlowski
0 siblings, 2 replies; 6+ messages in thread
From: wenswang @ 2026-02-25 8:56 UTC (permalink / raw)
To: robh, krzk+dt, conor+dt, linux, corbet, skhan
Cc: devicetree, linux-kernel, linux-hwmon, linux-doc, Wensheng Wang
From: Wensheng Wang <wenswang@yeah.net>
Add support for MPS mp2845 controller.
Signed-off-by: Wensheng Wang <wenswang@yeah.net>
---
Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index a482aeadcd44..2c6c84185bb3 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -303,6 +303,8 @@ properties:
- miramems,da280
# MiraMEMS DA311 3-axis 12-bit digital accelerometer
- miramems,da311
+ # Monolithic Power Systems Inc. multi-phase controller mp2845
+ - mps,mp2845
# Monolithic Power Systems Inc. multi-phase controller mp2856
- mps,mp2856
# Monolithic Power Systems Inc. multi-phase controller mp2857
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] hwmon: add MP2845 driver
2026-02-25 8:56 ` [PATCH 1/2] dt-bindings: hwmon: Add MPS mp2845 wenswang
@ 2026-02-25 8:56 ` wenswang
2026-02-26 7:48 ` Krzysztof Kozlowski
2026-02-26 14:22 ` Guenter Roeck
2026-02-26 7:45 ` [PATCH 1/2] dt-bindings: hwmon: Add MPS mp2845 Krzysztof Kozlowski
1 sibling, 2 replies; 6+ messages in thread
From: wenswang @ 2026-02-25 8:56 UTC (permalink / raw)
To: robh, krzk+dt, conor+dt, linux, corbet, skhan
Cc: devicetree, linux-kernel, linux-hwmon, linux-doc, Wensheng Wang
From: Wensheng Wang <wenswang@yeah.net>
Add support for MPS VR controller mp2845. This driver exposes
telemetry and limit value readings.
Signed-off-by: Wensheng Wang <wenswang@yeah.net>
---
Documentation/hwmon/index.rst | 1 +
Documentation/hwmon/mp2845.rst | 143 ++++++++++
MAINTAINERS | 7 +
drivers/hwmon/Kconfig | 10 +
drivers/hwmon/Makefile | 1 +
drivers/hwmon/mp2845.c | 493 +++++++++++++++++++++++++++++++++
6 files changed, 655 insertions(+)
create mode 100644 Documentation/hwmon/mp2845.rst
create mode 100644 drivers/hwmon/mp2845.c
diff --git a/Documentation/hwmon/index.rst b/Documentation/hwmon/index.rst
index d91dbb20c7dc..0a2176e5b694 100644
--- a/Documentation/hwmon/index.rst
+++ b/Documentation/hwmon/index.rst
@@ -176,6 +176,7 @@ Hardware Monitoring Kernel Drivers
mcp3021
menf21bmc
mlxreg-fan
+ mp2845
mp2856
mp2869
mp2888
diff --git a/Documentation/hwmon/mp2845.rst b/Documentation/hwmon/mp2845.rst
new file mode 100644
index 000000000000..dc6328855e27
--- /dev/null
+++ b/Documentation/hwmon/mp2845.rst
@@ -0,0 +1,143 @@
+.. SPDX-License-Identifier: GPL-2.0
+
+Kernel driver mp2845
+====================
+
+Supported chips:
+
+ * MPS mp2845
+
+ Prefix: 'mp2845'
+
+ * MPS mp2845
+
+ Prefix: 'mp2845'
+
+Author:
+
+ Wensheng Wang <wenswang@yeah.net>
+
+Description
+-----------
+
+This driver implements support for Monolithic Power Systems, Inc. (MPS)
+MP2845 Dual Loop Digital Multi-phase Controller.
+
+Device compliant with:
+
+- Smbus interface.
+
+The driver exports the following attributes via the 'sysfs' files
+for input voltage:
+
+**in0_input**
+
+**in0_label**
+
+**in0_crit**
+
+**in0_crit_alarm**
+
+**in0_lcrit_alarm**
+
+**in0_min**
+
+The driver provides the following attributes for output voltage:
+
+**in1_input**
+
+**in1_label**
+
+**in1_crit_alarm**
+
+**in1_lcrit_alarm**
+
+**in2_input**
+
+**in2_label**
+
+**in2_crit_alarm**
+
+**in2_lcrit_alarm**
+
+**in3_input**
+
+**in3_label**
+
+**in3_crit_alarm**
+
+**in3_lcrit_alarm**
+
+**in4_input**
+
+**in4_label**
+
+**in4_crit_alarm**
+
+**in4_lcrit_alarm**
+
+The driver provides the following attributes for input current:
+
+**curr1_input**
+
+**curr1_label**
+
+The driver provides the following attributes for output current:
+
+**curr1_input**
+
+**curr1_label**
+
+**curr1_crit**
+
+**curr1_crit_alarm**
+
+**curr1_max**
+
+**curr2_input**
+
+**curr2_label**
+
+**curr2_crit**
+
+**curr2_crit_alarm**
+
+**curr2_max**
+
+**curr3_input**
+
+**curr3_label**
+
+**curr3_crit**
+
+**curr3_crit_alarm**
+
+**curr3_max**
+
+**curr4_input**
+
+**curr4_label**
+
+**curr4_crit**
+
+**curr4_crit_alarm**
+
+**curr4_max**
+
+The driver provides the following attributes for temperature:
+
+**temp1_input**
+
+**temp1_crit_alarm**
+
+**temp2_input**
+
+**temp2_crit_alarm**
+
+**temp3_input**
+
+**temp3_crit_alarm**
+
+**temp4_input**
+
+**temp4_crit_alarm**
diff --git a/MAINTAINERS b/MAINTAINERS
index 55af015174a5..f1539307de5f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17905,6 +17905,13 @@ F: drivers/resctrl/mpam_*
F: drivers/resctrl/test_mpam_*
F: include/linux/arm_mpam.h
+MPS MP2845 DRIVER
+M: Wensheng Wang <wenswang@yeah.net>
+L: linux-hwmon@vger.kernel.org
+S: Maintained
+F: Documentation/hwmon/mp2845.rst
+F: drivers/hwmon/mp2845.c
+
MPS MP2869 DRIVER
M: Wensheng Wang <wenswang@yeah.net>
L: linux-hwmon@vger.kernel.org
diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
index 41c381764c2b..3f77982e50ea 100644
--- a/drivers/hwmon/Kconfig
+++ b/drivers/hwmon/Kconfig
@@ -1432,6 +1432,16 @@ config SENSORS_MENF21BMC_HWMON
This driver can also be built as a module. If so the module
will be called menf21bmc_hwmon.
+config SENSORS_MP2845
+ tristate "MPS MP2845"
+ depends on I2C
+ help
+ If you say yes here you get hardware monitoring support for
+ MPS MP2845 Dual Loop Digital Multi-Phase Controller.
+
+ This driver can also be built as a module. If so, the module
+ will be called mp2845.
+
config SENSORS_MR75203
tristate "Moortec Semiconductor MR75203 PVT Controller"
select REGMAP_MMIO
diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile
index eade8e3b1bde..4372759f2e60 100644
--- a/drivers/hwmon/Makefile
+++ b/drivers/hwmon/Makefile
@@ -174,6 +174,7 @@ obj-$(CONFIG_SENSORS_TC654) += tc654.o
obj-$(CONFIG_SENSORS_TPS23861) += tps23861.o
obj-$(CONFIG_SENSORS_MLXREG_FAN) += mlxreg-fan.o
obj-$(CONFIG_SENSORS_MENF21BMC_HWMON) += menf21bmc_hwmon.o
+obj-$(CONFIG_SENSORS_MP2845) += mp2845.o
obj-$(CONFIG_SENSORS_MR75203) += mr75203.o
obj-$(CONFIG_SENSORS_NCT6683) += nct6683.o
obj-$(CONFIG_SENSORS_NCT6694) += nct6694-hwmon.o
diff --git a/drivers/hwmon/mp2845.c b/drivers/hwmon/mp2845.c
new file mode 100644
index 000000000000..adecde7c9b71
--- /dev/null
+++ b/drivers/hwmon/mp2845.c
@@ -0,0 +1,493 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Hardware monitoring driver for MPS Digital Controller(MP2845)
+ */
+#include <linux/bitfield.h>
+#include <linux/delay.h>
+#include <linux/err.h>
+#include <linux/hwmon.h>
+#include <linux/hwmon-sysfs.h>
+#include <linux/i2c.h>
+#include <linux/init.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+
+#define MP2845_IOUT_SCALE_MASK1 GENMASK(5, 3)
+#define MP2845_IOUT_SCALE_MASK2 GENMASK(13, 11)
+
+#define MFR_VIN_OV_UV_SET 0x71
+#define MFR_OVUV_OCWARN_THRES 0x75
+#define MFR_TOTAL_OCP_SET 0x76
+#define MFR_PROTECT_STATUS1 0x80
+#define MFR_PROTECT_STATUS2 0x81
+
+#define MP2845_VIN_LIMIT_UINT 125
+#define MP2845_READ_VIN_UINT 3125
+#define MP2845_READ_VIN_DIV 100
+#define MP2845_READ_IOUT_UINT 3125
+#define MP2845_READ_IOUT_DIV 100
+#define MP2845_READ_VOUT_UINT 5
+#define MP2845_TEMP_UINT 1000
+
+#define MFR_READ_VIN 0xA6
+#define MFR_READ_VOUT 0xA7
+#define MFR_READ_IOUT 0xA8
+#define MFR_READ_TEMP 0xA9
+#define MFR_MFG_ID_SCALE_VI1 0x77
+#define MFR_MFG_ID_SCALE_VI2 0x78
+
+struct mp2845_data {
+ struct i2c_client *client;
+ int iout_gain[4];
+ /* lock for preventing concurrency issue */
+ struct mutex lock;
+};
+
+static umode_t mp2845_is_visible(const void *data, enum hwmon_sensor_types type,
+ u32 attr, int channel)
+{
+ switch (type) {
+ case hwmon_temp:
+ case hwmon_in:
+ case hwmon_curr:
+ return 0444;
+ default:
+ break;
+ }
+
+ return 0;
+}
+
+static int mp2845_read_string(struct device *dev, enum hwmon_sensor_types type,
+ u32 attr, int channel, const char **str)
+{
+ struct mp2845_data *data;
+ int ret = 0;
+
+ data = dev_get_drvdata(dev);
+
+ mutex_lock(&data->lock);
+
+ switch (type) {
+ case hwmon_in:
+ if (channel == 0)
+ *str = "vin";
+ else if (channel == 1)
+ *str = "vout1";
+ else if (channel == 2)
+ *str = "vout2";
+ else if (channel == 3)
+ *str = "vout3";
+ else
+ *str = "vout4";
+ break;
+ case hwmon_curr:
+ if (channel == 0)
+ *str = "iout1";
+ else if (channel == 1)
+ *str = "iout2";
+ else if (channel == 2)
+ *str = "iout3";
+ else
+ *str = "iout4";
+ break;
+ default:
+ ret = -EINVAL;
+ break;
+ }
+
+ mutex_unlock(&data->lock);
+
+ return ret;
+}
+
+static int mp2845_read(struct device *dev, enum hwmon_sensor_types type,
+ u32 attr, int channel, long *val)
+{
+ int ret;
+ struct mp2845_data *data;
+
+ data = dev_get_drvdata(dev);
+ if (!data)
+ return -ENOMEM;
+
+ mutex_lock(&data->lock);
+
+ switch (type) {
+ case hwmon_in:
+ if (channel == 0) {
+ if (attr == hwmon_in_input) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, 0);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_READ_VIN);
+ if (ret < 0)
+ break;
+
+ *val = DIV_ROUND_CLOSEST((ret & GENMASK(9, 0)) *
+ MP2845_READ_VIN_UINT,
+ MP2845_READ_VIN_DIV);
+ } else if (attr == hwmon_in_crit) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, 0);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_VIN_OV_UV_SET);
+ if (ret < 0)
+ break;
+
+ *val = FIELD_GET(GENMASK(15, 8), ret) * MP2845_VIN_LIMIT_UINT;
+ } else if (attr == hwmon_in_min) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, 0);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_VIN_OV_UV_SET);
+ if (ret < 0)
+ break;
+
+ *val = FIELD_GET(GENMASK(7, 0), ret) * MP2845_VIN_LIMIT_UINT;
+ } else if (attr == hwmon_in_crit_alarm) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, 0);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS2);
+ if (ret < 0)
+ break;
+
+ *val = FIELD_GET(GENMASK(5, 5), ret);
+ } else {
+ ret = i2c_smbus_write_byte_data(data->client, 0, 0);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS2);
+ if (ret < 0)
+ break;
+
+ *val = FIELD_GET(GENMASK(4, 4), ret);
+ }
+ } else {
+ if (attr == hwmon_in_input) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, channel - 1);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_READ_VOUT);
+ if (ret < 0)
+ break;
+
+ *val = (ret & GENMASK(9, 0)) * MP2845_READ_VOUT_UINT;
+ } else if (attr == hwmon_in_crit_alarm) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, 0);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS1);
+ if (ret < 0)
+ break;
+
+ if (channel == 1)
+ *val = FIELD_GET(GENMASK(10, 10), ret);
+ else if (channel == 2)
+ *val = FIELD_GET(GENMASK(7, 7), ret);
+ else if (channel == 3)
+ *val = FIELD_GET(GENMASK(4, 4), ret);
+ else
+ *val = FIELD_GET(GENMASK(1, 1), ret);
+ } else {
+ ret = i2c_smbus_write_byte_data(data->client, 0, 0);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS1);
+ if (ret < 0)
+ break;
+
+ if (channel == 1)
+ *val = FIELD_GET(GENMASK(11, 11), ret);
+ else if (channel == 2)
+ *val = FIELD_GET(GENMASK(8, 8), ret);
+ else if (channel == 3)
+ *val = FIELD_GET(GENMASK(5, 5), ret);
+ else
+ *val = FIELD_GET(GENMASK(2, 2), ret);
+ }
+ }
+ break;
+ case hwmon_temp:
+ if (attr == hwmon_temp_input) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, channel);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_READ_TEMP);
+ if (ret < 0)
+ break;
+
+ *val = ((ret & GENMASK(7, 0)) - 40) * MP2845_TEMP_UINT;
+ } else {
+ ret = i2c_smbus_write_byte_data(data->client, 0, 0);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS1);
+ if (ret < 0)
+ break;
+
+ if (channel == 0) {
+ *val = FIELD_GET(GENMASK(12, 12), ret);
+ } else if (channel == 1) {
+ *val = FIELD_GET(GENMASK(14, 14), ret);
+ } else if (channel == 2) {
+ ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS2);
+ if (ret < 0)
+ break;
+
+ *val = FIELD_GET(GENMASK(0, 0), ret);
+ } else {
+ ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS2);
+ if (ret < 0)
+ break;
+
+ *val = FIELD_GET(GENMASK(2, 2), ret);
+ }
+ }
+ break;
+ case hwmon_curr:
+ if (attr == hwmon_curr_input) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, channel);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_READ_IOUT);
+ if (ret < 0)
+ break;
+
+ *val = DIV_ROUND_CLOSEST((ret & GENMASK(9, 0)) * data->iout_gain[channel] *
+ MP2845_READ_IOUT_UINT, MP2845_READ_IOUT_DIV);
+ } else if (attr == hwmon_curr_max) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, channel);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_OVUV_OCWARN_THRES);
+ if (ret < 0)
+ break;
+
+ *val = DIV_ROUND_CLOSEST(FIELD_GET(GENMASK(15, 8), ret) *
+ data->iout_gain[channel] * MP2845_READ_IOUT_UINT *
+ 4 * 2, MP2845_READ_IOUT_DIV);
+ } else if (attr == hwmon_curr_crit) {
+ ret = i2c_smbus_write_byte_data(data->client, 0, channel);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_TOTAL_OCP_SET);
+ if (ret < 0)
+ break;
+
+ *val = DIV_ROUND_CLOSEST(FIELD_GET(GENMASK(15, 8), ret) *
+ data->iout_gain[channel] * MP2845_READ_IOUT_UINT *
+ 4 * 2, MP2845_READ_IOUT_DIV);
+ } else {
+ ret = i2c_smbus_write_byte_data(data->client, 0, 0);
+ if (ret < 0)
+ break;
+
+ ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS1);
+ if (ret < 0)
+ break;
+
+ if (channel == 0)
+ *val = FIELD_GET(GENMASK(9, 9), ret);
+ else if (channel == 2)
+ *val = FIELD_GET(GENMASK(6, 6), ret);
+ else if (channel == 3)
+ *val = FIELD_GET(GENMASK(3, 3), ret);
+ else
+ *val = FIELD_GET(GENMASK(0, 0), ret);
+ }
+ break;
+ default:
+ ret = -EINVAL;
+ break;
+ }
+
+ mutex_unlock(&data->lock);
+
+ return ret;
+}
+
+static int
+mp2845_identify_iout_scale(struct mp2845_data *data, int page)
+{
+ int gain;
+ int ret;
+
+ ret = i2c_smbus_write_byte_data(data->client, 0x00, page);
+ if (ret < 0)
+ return ret;
+
+ ret = i2c_smbus_read_word_data(data->client, page == 0 ? MFR_MFG_ID_SCALE_VI2 :
+ MFR_MFG_ID_SCALE_VI1);
+ if (ret < 0)
+ return ret;
+
+ gain = page == 0 ? FIELD_GET(MP2845_IOUT_SCALE_MASK1, ret) :
+ FIELD_GET(MP2845_IOUT_SCALE_MASK2, ret);
+ switch (gain) {
+ case 1:
+ data->iout_gain[page] = 1;
+ break;
+ case 2:
+ data->iout_gain[page] = 2;
+ break;
+ case 3:
+ data->iout_gain[page] = 4;
+ break;
+ case 4:
+ data->iout_gain[page] = 8;
+ break;
+ case 5:
+ data->iout_gain[page] = 16;
+ break;
+ case 6:
+ data->iout_gain[page] = 32;
+ break;
+ case 7:
+ data->iout_gain[page] = 64;
+ break;
+ default:
+ data->iout_gain[page] = 1;
+ break;
+ }
+
+ return 0;
+}
+
+static const struct hwmon_channel_info *mp2845_info[] = {
+ HWMON_CHANNEL_INFO(in,
+ HWMON_I_INPUT | HWMON_I_MIN | HWMON_I_CRIT | HWMON_I_CRIT_ALARM |
+ HWMON_I_LCRIT_ALARM | HWMON_I_LABEL,
+ HWMON_I_INPUT | HWMON_I_CRIT_ALARM | HWMON_I_LCRIT_ALARM |
+ HWMON_I_LABEL,
+ HWMON_I_INPUT | HWMON_I_CRIT_ALARM | HWMON_I_LCRIT_ALARM |
+ HWMON_I_LABEL,
+ HWMON_I_INPUT | HWMON_I_CRIT_ALARM | HWMON_I_LCRIT_ALARM |
+ HWMON_I_LABEL,
+ HWMON_I_INPUT | HWMON_I_CRIT_ALARM | HWMON_I_LCRIT_ALARM |
+ HWMON_I_LABEL),
+ HWMON_CHANNEL_INFO(temp,
+ HWMON_T_INPUT | HWMON_T_CRIT_ALARM,
+ HWMON_T_INPUT | HWMON_T_CRIT_ALARM,
+ HWMON_T_INPUT | HWMON_T_CRIT_ALARM,
+ HWMON_T_INPUT | HWMON_T_CRIT_ALARM),
+ HWMON_CHANNEL_INFO(curr,
+ HWMON_C_INPUT | HWMON_C_CRIT | HWMON_C_CRIT_ALARM | HWMON_C_MAX |
+ HWMON_C_LABEL,
+ HWMON_C_INPUT | HWMON_C_CRIT | HWMON_C_CRIT_ALARM | HWMON_C_MAX |
+ HWMON_C_LABEL,
+ HWMON_C_INPUT | HWMON_C_CRIT | HWMON_C_CRIT_ALARM | HWMON_C_MAX |
+ HWMON_C_LABEL,
+ HWMON_C_INPUT | HWMON_C_CRIT | HWMON_C_CRIT_ALARM | HWMON_C_MAX |
+ HWMON_C_LABEL),
+ NULL
+};
+
+static const struct hwmon_ops mp2845_hwmon_ops = {
+ .is_visible = mp2845_is_visible,
+ .read = mp2845_read,
+ .read_string = mp2845_read_string,
+};
+
+static const struct hwmon_chip_info mp2845_chip_info = {
+ .ops = &mp2845_hwmon_ops,
+ .info = mp2845_info,
+};
+
+static int mp2845_probe(struct i2c_client *client)
+{
+ struct device *dev = &client->dev;
+ struct device *hwmon_dev;
+ struct mp2845_data *data;
+ int ret;
+
+ if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA |
+ I2C_FUNC_SMBUS_WORD_DATA)) {
+ dev_err(dev, "check failed, smbus byte and/or word data not supported!\n");
+ return -ENODEV;
+ }
+
+ data = devm_kzalloc(dev, sizeof(struct mp2845_data), GFP_KERNEL);
+ if (!data)
+ return -ENOMEM;
+
+ mutex_init(&data->lock);
+ data->client = client;
+
+ ret = mp2845_identify_iout_scale(data, 0);
+ if (ret < 0) {
+ dev_err(dev, "unable to identify rail1 iout scale, errno = %d\n", ret);
+ return ret;
+ }
+
+ ret = mp2845_identify_iout_scale(data, 1);
+ if (ret < 0) {
+ dev_err(dev, "unable to identify rail2 iout scale, errno = %d\n", ret);
+ return ret;
+ }
+
+ ret = mp2845_identify_iout_scale(data, 2);
+ if (ret < 0) {
+ dev_err(dev, "unable to identify rail3 iout scale, errno = %d\n", ret);
+ return ret;
+ }
+
+ ret = mp2845_identify_iout_scale(data, 3);
+ if (ret < 0) {
+ dev_err(dev, "unable to identify rail4 iout scale, errno = %d\n", ret);
+ return ret;
+ }
+
+ hwmon_dev = devm_hwmon_device_register_with_info(dev, client->name,
+ data, &mp2845_chip_info,
+ NULL);
+ if (IS_ERR(hwmon_dev)) {
+ dev_err(dev, "unable to register mp2845 hwmon device\n");
+ return PTR_ERR(hwmon_dev);
+ }
+
+ dev_info(dev, "%s: sensor '%s'\n", dev_name(hwmon_dev), client->name);
+
+ return 0;
+}
+
+static const struct i2c_device_id mp2845_ids[] = {
+ {"mp2845", 0},
+ {}
+};
+MODULE_DEVICE_TABLE(i2c, mp2845_ids);
+
+static const struct of_device_id __maybe_unused mp2845_of_match[] = {
+ {.compatible = "mps,mp2845"},
+ {}
+};
+MODULE_DEVICE_TABLE(of, mp2845_of_match);
+
+static struct i2c_driver mp2845_driver = {
+ .class = I2C_CLASS_HWMON,
+ .driver = {
+ .name = "mp2845",
+ .of_match_table = mp2845_of_match,
+ },
+ .probe = mp2845_probe,
+ .id_table = mp2845_ids,
+};
+module_i2c_driver(mp2845_driver);
+
+MODULE_AUTHOR("Wensheng Wang <wenswang@yeah.net>");
+MODULE_DESCRIPTION("MP2845 driver");
+MODULE_LICENSE("GPL");
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] dt-bindings: hwmon: Add MPS mp2845
2026-02-25 8:56 ` [PATCH 1/2] dt-bindings: hwmon: Add MPS mp2845 wenswang
2026-02-25 8:56 ` [PATCH 2/2] hwmon: add MP2845 driver wenswang
@ 2026-02-26 7:45 ` Krzysztof Kozlowski
1 sibling, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2026-02-26 7:45 UTC (permalink / raw)
To: wenswang
Cc: robh, krzk+dt, conor+dt, linux, corbet, skhan, devicetree,
linux-kernel, linux-hwmon, linux-doc
On Wed, Feb 25, 2026 at 04:56:30PM +0800, wenswang@yeah.net wrote:
> From: Wensheng Wang <wenswang@yeah.net>
>
> Add support for MPS mp2845 controller.
>
> Signed-off-by: Wensheng Wang <wenswang@yeah.net>
> ---
> Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
> index a482aeadcd44..2c6c84185bb3 100644
> --- a/Documentation/devicetree/bindings/trivial-devices.yaml
> +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
> @@ -303,6 +303,8 @@ properties:
> - miramems,da280
> # MiraMEMS DA311 3-axis 12-bit digital accelerometer
> - miramems,da311
> + # Monolithic Power Systems Inc. multi-phase controller mp2845
Broken alignment.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] hwmon: add MP2845 driver
2026-02-25 8:56 ` [PATCH 2/2] hwmon: add MP2845 driver wenswang
@ 2026-02-26 7:48 ` Krzysztof Kozlowski
2026-02-26 14:22 ` Guenter Roeck
1 sibling, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2026-02-26 7:48 UTC (permalink / raw)
To: wenswang
Cc: robh, krzk+dt, conor+dt, linux, corbet, skhan, devicetree,
linux-kernel, linux-hwmon, linux-doc
On Wed, Feb 25, 2026 at 04:56:31PM +0800, wenswang@yeah.net wrote:
> +#define MFR_VIN_OV_UV_SET 0x71
> +#define MFR_OVUV_OCWARN_THRES 0x75
> +#define MFR_TOTAL_OCP_SET 0x76
> +#define MFR_PROTECT_STATUS1 0x80
> +#define MFR_PROTECT_STATUS2 0x81
> +
> +#define MP2845_VIN_LIMIT_UINT 125
> +#define MP2845_READ_VIN_UINT 3125
> +#define MP2845_READ_VIN_DIV 100
> +#define MP2845_READ_IOUT_UINT 3125
> +#define MP2845_READ_IOUT_DIV 100
> +#define MP2845_READ_VOUT_UINT 5
> +#define MP2845_TEMP_UINT 1000
> +
> +#define MFR_READ_VIN 0xA6
> +#define MFR_READ_VOUT 0xA7
> +#define MFR_READ_IOUT 0xA8
> +#define MFR_READ_TEMP 0xA9
> +#define MFR_MFG_ID_SCALE_VI1 0x77
> +#define MFR_MFG_ID_SCALE_VI2 0x78
> +
> +struct mp2845_data {
> + struct i2c_client *client;
> + int iout_gain[4];
> + /* lock for preventing concurrency issue */
This is completely useless comment. The definition of lock is to prevent
concurrency issues. It's like adding a comment to a function: "it is a
function".
You must here explain which data or code logic is protected by this.
> + struct mutex lock;
> +};
...
> +
> +static int mp2845_probe(struct i2c_client *client)
> +{
> + struct device *dev = &client->dev;
> + struct device *hwmon_dev;
> + struct mp2845_data *data;
> + int ret;
> +
> + if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA |
> + I2C_FUNC_SMBUS_WORD_DATA)) {
> + dev_err(dev, "check failed, smbus byte and/or word data not supported!\n");
> + return -ENODEV;
> + }
> +
> + data = devm_kzalloc(dev, sizeof(struct mp2845_data), GFP_KERNEL);
sizeof(*)
> + if (!data)
> + return -ENOMEM;
> +
> + mutex_init(&data->lock);
> + data->client = client;
> +
> + ret = mp2845_identify_iout_scale(data, 0);
> + if (ret < 0) {
> + dev_err(dev, "unable to identify rail1 iout scale, errno = %d\n", ret);
> + return ret;
> + }
> +
> + ret = mp2845_identify_iout_scale(data, 1);
> + if (ret < 0) {
> + dev_err(dev, "unable to identify rail2 iout scale, errno = %d\n", ret);
> + return ret;
> + }
> +
> + ret = mp2845_identify_iout_scale(data, 2);
> + if (ret < 0) {
> + dev_err(dev, "unable to identify rail3 iout scale, errno = %d\n", ret);
> + return ret;
> + }
> +
> + ret = mp2845_identify_iout_scale(data, 3);
> + if (ret < 0) {
> + dev_err(dev, "unable to identify rail4 iout scale, errno = %d\n", ret);
> + return ret;
> + }
> +
> + hwmon_dev = devm_hwmon_device_register_with_info(dev, client->name,
> + data, &mp2845_chip_info,
> + NULL);
> + if (IS_ERR(hwmon_dev)) {
> + dev_err(dev, "unable to register mp2845 hwmon device\n");
> + return PTR_ERR(hwmon_dev);
> + }
> +
> + dev_info(dev, "%s: sensor '%s'\n", dev_name(hwmon_dev), client->name);
Driver should be silent on success probe. See also coding style and
driver development docs.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] hwmon: add MP2845 driver
2026-02-25 8:56 ` [PATCH 2/2] hwmon: add MP2845 driver wenswang
2026-02-26 7:48 ` Krzysztof Kozlowski
@ 2026-02-26 14:22 ` Guenter Roeck
1 sibling, 0 replies; 6+ messages in thread
From: Guenter Roeck @ 2026-02-26 14:22 UTC (permalink / raw)
To: wenswang
Cc: robh, krzk+dt, conor+dt, corbet, skhan, devicetree, linux-kernel,
linux-hwmon, linux-doc
On Wed, Feb 25, 2026 at 04:56:31PM +0800, wenswang@yeah.net wrote:
> From: Wensheng Wang <wenswang@yeah.net>
>
> Add support for MPS VR controller mp2845. This driver exposes
> telemetry and limit value readings.
>
> Signed-off-by: Wensheng Wang <wenswang@yeah.net>
> ---
> Documentation/hwmon/index.rst | 1 +
> Documentation/hwmon/mp2845.rst | 143 ++++++++++
> MAINTAINERS | 7 +
> drivers/hwmon/Kconfig | 10 +
> drivers/hwmon/Makefile | 1 +
> drivers/hwmon/mp2845.c | 493 +++++++++++++++++++++++++++++++++
> 6 files changed, 655 insertions(+)
> create mode 100644 Documentation/hwmon/mp2845.rst
> create mode 100644 drivers/hwmon/mp2845.c
>
> diff --git a/Documentation/hwmon/index.rst b/Documentation/hwmon/index.rst
> index d91dbb20c7dc..0a2176e5b694 100644
> --- a/Documentation/hwmon/index.rst
> +++ b/Documentation/hwmon/index.rst
> @@ -176,6 +176,7 @@ Hardware Monitoring Kernel Drivers
> mcp3021
> menf21bmc
> mlxreg-fan
> + mp2845
> mp2856
> mp2869
> mp2888
> diff --git a/Documentation/hwmon/mp2845.rst b/Documentation/hwmon/mp2845.rst
> new file mode 100644
> index 000000000000..dc6328855e27
> --- /dev/null
> +++ b/Documentation/hwmon/mp2845.rst
> @@ -0,0 +1,143 @@
> +.. SPDX-License-Identifier: GPL-2.0
> +
> +Kernel driver mp2845
> +====================
> +
> +Supported chips:
> +
> + * MPS mp2845
> +
> + Prefix: 'mp2845'
> +
> + * MPS mp2845
> +
> + Prefix: 'mp2845'
> +
> +Author:
> +
> + Wensheng Wang <wenswang@yeah.net>
> +
> +Description
> +-----------
> +
> +This driver implements support for Monolithic Power Systems, Inc. (MPS)
> +MP2845 Dual Loop Digital Multi-phase Controller.
> +
> +Device compliant with:
> +
> +- Smbus interface.
> +
> +The driver exports the following attributes via the 'sysfs' files
> +for input voltage:
> +
> +**in0_input**
> +
> +**in0_label**
> +
> +**in0_crit**
> +
> +**in0_crit_alarm**
> +
> +**in0_lcrit_alarm**
> +
> +**in0_min**
> +
> +The driver provides the following attributes for output voltage:
> +
> +**in1_input**
> +
> +**in1_label**
> +
> +**in1_crit_alarm**
> +
> +**in1_lcrit_alarm**
> +
> +**in2_input**
> +
> +**in2_label**
> +
> +**in2_crit_alarm**
> +
> +**in2_lcrit_alarm**
> +
> +**in3_input**
> +
> +**in3_label**
> +
> +**in3_crit_alarm**
> +
> +**in3_lcrit_alarm**
> +
> +**in4_input**
> +
> +**in4_label**
> +
> +**in4_crit_alarm**
> +
> +**in4_lcrit_alarm**
> +
> +The driver provides the following attributes for input current:
> +
> +**curr1_input**
> +
> +**curr1_label**
> +
> +The driver provides the following attributes for output current:
> +
> +**curr1_input**
> +
> +**curr1_label**
> +
> +**curr1_crit**
> +
> +**curr1_crit_alarm**
> +
> +**curr1_max**
> +
> +**curr2_input**
> +
> +**curr2_label**
> +
> +**curr2_crit**
> +
> +**curr2_crit_alarm**
> +
> +**curr2_max**
> +
> +**curr3_input**
> +
> +**curr3_label**
> +
> +**curr3_crit**
> +
> +**curr3_crit_alarm**
> +
> +**curr3_max**
> +
> +**curr4_input**
> +
> +**curr4_label**
> +
> +**curr4_crit**
> +
> +**curr4_crit_alarm**
> +
> +**curr4_max**
> +
> +The driver provides the following attributes for temperature:
> +
> +**temp1_input**
> +
> +**temp1_crit_alarm**
> +
> +**temp2_input**
> +
> +**temp2_crit_alarm**
> +
> +**temp3_input**
> +
> +**temp3_crit_alarm**
> +
> +**temp4_input**
> +
> +**temp4_crit_alarm**
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 55af015174a5..f1539307de5f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17905,6 +17905,13 @@ F: drivers/resctrl/mpam_*
> F: drivers/resctrl/test_mpam_*
> F: include/linux/arm_mpam.h
>
> +MPS MP2845 DRIVER
> +M: Wensheng Wang <wenswang@yeah.net>
> +L: linux-hwmon@vger.kernel.org
> +S: Maintained
> +F: Documentation/hwmon/mp2845.rst
> +F: drivers/hwmon/mp2845.c
> +
> MPS MP2869 DRIVER
> M: Wensheng Wang <wenswang@yeah.net>
> L: linux-hwmon@vger.kernel.org
> diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
> index 41c381764c2b..3f77982e50ea 100644
> --- a/drivers/hwmon/Kconfig
> +++ b/drivers/hwmon/Kconfig
> @@ -1432,6 +1432,16 @@ config SENSORS_MENF21BMC_HWMON
> This driver can also be built as a module. If so the module
> will be called menf21bmc_hwmon.
>
> +config SENSORS_MP2845
> + tristate "MPS MP2845"
> + depends on I2C
> + help
> + If you say yes here you get hardware monitoring support for
> + MPS MP2845 Dual Loop Digital Multi-Phase Controller.
> +
> + This driver can also be built as a module. If so, the module
> + will be called mp2845.
> +
> config SENSORS_MR75203
> tristate "Moortec Semiconductor MR75203 PVT Controller"
> select REGMAP_MMIO
> diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile
> index eade8e3b1bde..4372759f2e60 100644
> --- a/drivers/hwmon/Makefile
> +++ b/drivers/hwmon/Makefile
> @@ -174,6 +174,7 @@ obj-$(CONFIG_SENSORS_TC654) += tc654.o
> obj-$(CONFIG_SENSORS_TPS23861) += tps23861.o
> obj-$(CONFIG_SENSORS_MLXREG_FAN) += mlxreg-fan.o
> obj-$(CONFIG_SENSORS_MENF21BMC_HWMON) += menf21bmc_hwmon.o
> +obj-$(CONFIG_SENSORS_MP2845) += mp2845.o
> obj-$(CONFIG_SENSORS_MR75203) += mr75203.o
> obj-$(CONFIG_SENSORS_NCT6683) += nct6683.o
> obj-$(CONFIG_SENSORS_NCT6694) += nct6694-hwmon.o
> diff --git a/drivers/hwmon/mp2845.c b/drivers/hwmon/mp2845.c
> new file mode 100644
> index 000000000000..adecde7c9b71
> --- /dev/null
> +++ b/drivers/hwmon/mp2845.c
I just realized, while reviewing: This is a PMBus chip. YOu are trying
to bypass the PMBus core. This is an absolute no-go. Try again, this
time as PMBus driver.
> @@ -0,0 +1,493 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +/*
> + * Hardware monitoring driver for MPS Digital Controller(MP2845)
> + */
> +#include <linux/bitfield.h>
> +#include <linux/delay.h>
> +#include <linux/err.h>
> +#include <linux/hwmon.h>
> +#include <linux/hwmon-sysfs.h>
Unnecessary include.
> +#include <linux/i2c.h>
> +#include <linux/init.h>
> +#include <linux/kernel.h>
> +#include <linux/module.h>
> +
> +#define MP2845_IOUT_SCALE_MASK1 GENMASK(5, 3)
> +#define MP2845_IOUT_SCALE_MASK2 GENMASK(13, 11)
> +
> +#define MFR_VIN_OV_UV_SET 0x71
> +#define MFR_OVUV_OCWARN_THRES 0x75
> +#define MFR_TOTAL_OCP_SET 0x76
> +#define MFR_PROTECT_STATUS1 0x80
> +#define MFR_PROTECT_STATUS2 0x81
> +
> +#define MP2845_VIN_LIMIT_UINT 125
> +#define MP2845_READ_VIN_UINT 3125
> +#define MP2845_READ_VIN_DIV 100
> +#define MP2845_READ_IOUT_UINT 3125
> +#define MP2845_READ_IOUT_DIV 100
> +#define MP2845_READ_VOUT_UINT 5
> +#define MP2845_TEMP_UINT 1000
> +
> +#define MFR_READ_VIN 0xA6
> +#define MFR_READ_VOUT 0xA7
> +#define MFR_READ_IOUT 0xA8
> +#define MFR_READ_TEMP 0xA9
> +#define MFR_MFG_ID_SCALE_VI1 0x77
> +#define MFR_MFG_ID_SCALE_VI2 0x78
> +
> +struct mp2845_data {
> + struct i2c_client *client;
> + int iout_gain[4];
> + /* lock for preventing concurrency issue */
> + struct mutex lock;
> +};
> +
> +static umode_t mp2845_is_visible(const void *data, enum hwmon_sensor_types type,
> + u32 attr, int channel)
> +{
> + switch (type) {
> + case hwmon_temp:
> + case hwmon_in:
> + case hwmon_curr:
> + return 0444;
> + default:
> + break;
> + }
> +
> + return 0;
> +}
> +
> +static int mp2845_read_string(struct device *dev, enum hwmon_sensor_types type,
> + u32 attr, int channel, const char **str)
> +{
> + struct mp2845_data *data;
> + int ret = 0;
> +
> + data = dev_get_drvdata(dev);
> +
> + mutex_lock(&data->lock);
What concurrency issue is this supposed to protect against, and why is the
concurrency protection in the hwmon core insufficient ?
> +
> + switch (type) {
> + case hwmon_in:
> + if (channel == 0)
> + *str = "vin";
> + else if (channel == 1)
> + *str = "vout1";
> + else if (channel == 2)
> + *str = "vout2";
> + else if (channel == 3)
> + *str = "vout3";
> + else
> + *str = "vout4";
> + break;
> + case hwmon_curr:
> + if (channel == 0)
> + *str = "iout1";
> + else if (channel == 1)
> + *str = "iout2";
> + else if (channel == 2)
> + *str = "iout3";
> + else
> + *str = "iout4";
> + break;
> + default:
> + ret = -EINVAL;
> + break;
> + }
> +
The channel labels are supposed to reflect system values. The above just
reports PMBus values, and those are already provided by the PMBus core.
This is pointless code.
> + mutex_unlock(&data->lock);
> +
> + return ret;
> +}
> +
> +static int mp2845_read(struct device *dev, enum hwmon_sensor_types type,
> + u32 attr, int channel, long *val)
> +{
> + int ret;
> + struct mp2845_data *data;
> +
> + data = dev_get_drvdata(dev);
> + if (!data)
> + return -ENOMEM;
How would this ever happen ?
> +
> + mutex_lock(&data->lock);
Why is the lock provided by the hwmon core insufficient ?
> +
> + switch (type) {
> + case hwmon_in:
> + if (channel == 0) {
> + if (attr == hwmon_in_input) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, 0);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_READ_VIN);
> + if (ret < 0)
> + break;
> +
> + *val = DIV_ROUND_CLOSEST((ret & GENMASK(9, 0)) *
> + MP2845_READ_VIN_UINT,
> + MP2845_READ_VIN_DIV);
> + } else if (attr == hwmon_in_crit) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, 0);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_VIN_OV_UV_SET);
> + if (ret < 0)
> + break;
> +
> + *val = FIELD_GET(GENMASK(15, 8), ret) * MP2845_VIN_LIMIT_UINT;
> + } else if (attr == hwmon_in_min) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, 0);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_VIN_OV_UV_SET);
> + if (ret < 0)
> + break;
> +
> + *val = FIELD_GET(GENMASK(7, 0), ret) * MP2845_VIN_LIMIT_UINT;
> + } else if (attr == hwmon_in_crit_alarm) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, 0);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS2);
> + if (ret < 0)
> + break;
> +
> + *val = FIELD_GET(GENMASK(5, 5), ret);
> + } else {
> + ret = i2c_smbus_write_byte_data(data->client, 0, 0);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS2);
> + if (ret < 0)
> + break;
> +
> + *val = FIELD_GET(GENMASK(4, 4), ret);
> + }
> + } else {
> + if (attr == hwmon_in_input) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, channel - 1);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_READ_VOUT);
> + if (ret < 0)
> + break;
> +
> + *val = (ret & GENMASK(9, 0)) * MP2845_READ_VOUT_UINT;
> + } else if (attr == hwmon_in_crit_alarm) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, 0);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS1);
> + if (ret < 0)
> + break;
> +
> + if (channel == 1)
> + *val = FIELD_GET(GENMASK(10, 10), ret);
> + else if (channel == 2)
> + *val = FIELD_GET(GENMASK(7, 7), ret);
> + else if (channel == 3)
> + *val = FIELD_GET(GENMASK(4, 4), ret);
> + else
> + *val = FIELD_GET(GENMASK(1, 1), ret);
> + } else {
> + ret = i2c_smbus_write_byte_data(data->client, 0, 0);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS1);
> + if (ret < 0)
> + break;
> +
> + if (channel == 1)
> + *val = FIELD_GET(GENMASK(11, 11), ret);
> + else if (channel == 2)
> + *val = FIELD_GET(GENMASK(8, 8), ret);
> + else if (channel == 3)
> + *val = FIELD_GET(GENMASK(5, 5), ret);
> + else
> + *val = FIELD_GET(GENMASK(2, 2), ret);
> + }
> + }
> + break;
> + case hwmon_temp:
> + if (attr == hwmon_temp_input) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, channel);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_READ_TEMP);
> + if (ret < 0)
> + break;
> +
> + *val = ((ret & GENMASK(7, 0)) - 40) * MP2845_TEMP_UINT;
> + } else {
> + ret = i2c_smbus_write_byte_data(data->client, 0, 0);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS1);
> + if (ret < 0)
> + break;
> +
> + if (channel == 0) {
> + *val = FIELD_GET(GENMASK(12, 12), ret);
> + } else if (channel == 1) {
> + *val = FIELD_GET(GENMASK(14, 14), ret);
> + } else if (channel == 2) {
> + ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS2);
> + if (ret < 0)
> + break;
> +
> + *val = FIELD_GET(GENMASK(0, 0), ret);
> + } else {
> + ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS2);
> + if (ret < 0)
> + break;
> +
> + *val = FIELD_GET(GENMASK(2, 2), ret);
> + }
> + }
> + break;
> + case hwmon_curr:
> + if (attr == hwmon_curr_input) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, channel);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_READ_IOUT);
> + if (ret < 0)
> + break;
> +
> + *val = DIV_ROUND_CLOSEST((ret & GENMASK(9, 0)) * data->iout_gain[channel] *
> + MP2845_READ_IOUT_UINT, MP2845_READ_IOUT_DIV);
> + } else if (attr == hwmon_curr_max) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, channel);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_OVUV_OCWARN_THRES);
> + if (ret < 0)
> + break;
> +
> + *val = DIV_ROUND_CLOSEST(FIELD_GET(GENMASK(15, 8), ret) *
> + data->iout_gain[channel] * MP2845_READ_IOUT_UINT *
> + 4 * 2, MP2845_READ_IOUT_DIV);
> + } else if (attr == hwmon_curr_crit) {
> + ret = i2c_smbus_write_byte_data(data->client, 0, channel);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_TOTAL_OCP_SET);
> + if (ret < 0)
> + break;
> +
> + *val = DIV_ROUND_CLOSEST(FIELD_GET(GENMASK(15, 8), ret) *
> + data->iout_gain[channel] * MP2845_READ_IOUT_UINT *
> + 4 * 2, MP2845_READ_IOUT_DIV);
> + } else {
> + ret = i2c_smbus_write_byte_data(data->client, 0, 0);
> + if (ret < 0)
> + break;
> +
> + ret = i2c_smbus_read_word_data(data->client, MFR_PROTECT_STATUS1);
> + if (ret < 0)
> + break;
> +
> + if (channel == 0)
> + *val = FIELD_GET(GENMASK(9, 9), ret);
> + else if (channel == 2)
> + *val = FIELD_GET(GENMASK(6, 6), ret);
> + else if (channel == 3)
> + *val = FIELD_GET(GENMASK(3, 3), ret);
> + else
> + *val = FIELD_GET(GENMASK(0, 0), ret);
> + }
> + break;
> + default:
> + ret = -EINVAL;
> + break;
> + }
> +
> + mutex_unlock(&data->lock);
> +
> + return ret;
> +}
> +
> +static int
> +mp2845_identify_iout_scale(struct mp2845_data *data, int page)
> +{
> + int gain;
> + int ret;
> +
> + ret = i2c_smbus_write_byte_data(data->client, 0x00, page);
> + if (ret < 0)
> + return ret;
> +
> + ret = i2c_smbus_read_word_data(data->client, page == 0 ? MFR_MFG_ID_SCALE_VI2 :
> + MFR_MFG_ID_SCALE_VI1);
> + if (ret < 0)
> + return ret;
> +
> + gain = page == 0 ? FIELD_GET(MP2845_IOUT_SCALE_MASK1, ret) :
> + FIELD_GET(MP2845_IOUT_SCALE_MASK2, ret);
> + switch (gain) {
> + case 1:
> + data->iout_gain[page] = 1;
> + break;
> + case 2:
> + data->iout_gain[page] = 2;
> + break;
> + case 3:
> + data->iout_gain[page] = 4;
> + break;
> + case 4:
> + data->iout_gain[page] = 8;
> + break;
> + case 5:
> + data->iout_gain[page] = 16;
> + break;
> + case 6:
> + data->iout_gain[page] = 32;
> + break;
> + case 7:
> + data->iout_gain[page] = 64;
> + break;
> + default:
> + data->iout_gain[page] = 1;
> + break;
> + }
> +
> + return 0;
> +}
> +
> +static const struct hwmon_channel_info *mp2845_info[] = {
> + HWMON_CHANNEL_INFO(in,
> + HWMON_I_INPUT | HWMON_I_MIN | HWMON_I_CRIT | HWMON_I_CRIT_ALARM |
> + HWMON_I_LCRIT_ALARM | HWMON_I_LABEL,
> + HWMON_I_INPUT | HWMON_I_CRIT_ALARM | HWMON_I_LCRIT_ALARM |
> + HWMON_I_LABEL,
> + HWMON_I_INPUT | HWMON_I_CRIT_ALARM | HWMON_I_LCRIT_ALARM |
> + HWMON_I_LABEL,
> + HWMON_I_INPUT | HWMON_I_CRIT_ALARM | HWMON_I_LCRIT_ALARM |
> + HWMON_I_LABEL,
> + HWMON_I_INPUT | HWMON_I_CRIT_ALARM | HWMON_I_LCRIT_ALARM |
> + HWMON_I_LABEL),
> + HWMON_CHANNEL_INFO(temp,
> + HWMON_T_INPUT | HWMON_T_CRIT_ALARM,
> + HWMON_T_INPUT | HWMON_T_CRIT_ALARM,
> + HWMON_T_INPUT | HWMON_T_CRIT_ALARM,
> + HWMON_T_INPUT | HWMON_T_CRIT_ALARM),
> + HWMON_CHANNEL_INFO(curr,
> + HWMON_C_INPUT | HWMON_C_CRIT | HWMON_C_CRIT_ALARM | HWMON_C_MAX |
> + HWMON_C_LABEL,
> + HWMON_C_INPUT | HWMON_C_CRIT | HWMON_C_CRIT_ALARM | HWMON_C_MAX |
> + HWMON_C_LABEL,
> + HWMON_C_INPUT | HWMON_C_CRIT | HWMON_C_CRIT_ALARM | HWMON_C_MAX |
> + HWMON_C_LABEL,
> + HWMON_C_INPUT | HWMON_C_CRIT | HWMON_C_CRIT_ALARM | HWMON_C_MAX |
> + HWMON_C_LABEL),
> + NULL
> +};
> +
> +static const struct hwmon_ops mp2845_hwmon_ops = {
> + .is_visible = mp2845_is_visible,
> + .read = mp2845_read,
> + .read_string = mp2845_read_string,
> +};
> +
> +static const struct hwmon_chip_info mp2845_chip_info = {
> + .ops = &mp2845_hwmon_ops,
> + .info = mp2845_info,
> +};
> +
> +static int mp2845_probe(struct i2c_client *client)
> +{
> + struct device *dev = &client->dev;
> + struct device *hwmon_dev;
> + struct mp2845_data *data;
> + int ret;
> +
> + if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA |
> + I2C_FUNC_SMBUS_WORD_DATA)) {
> + dev_err(dev, "check failed, smbus byte and/or word data not supported!\n");
> + return -ENODEV;
> + }
> +
> + data = devm_kzalloc(dev, sizeof(struct mp2845_data), GFP_KERNEL);
> + if (!data)
> + return -ENOMEM;
> +
> + mutex_init(&data->lock);
> + data->client = client;
> +
> + ret = mp2845_identify_iout_scale(data, 0);
> + if (ret < 0) {
> + dev_err(dev, "unable to identify rail1 iout scale, errno = %d\n", ret);
> + return ret;
> + }
> +
> + ret = mp2845_identify_iout_scale(data, 1);
> + if (ret < 0) {
> + dev_err(dev, "unable to identify rail2 iout scale, errno = %d\n", ret);
> + return ret;
> + }
> +
> + ret = mp2845_identify_iout_scale(data, 2);
> + if (ret < 0) {
> + dev_err(dev, "unable to identify rail3 iout scale, errno = %d\n", ret);
> + return ret;
> + }
> +
> + ret = mp2845_identify_iout_scale(data, 3);
> + if (ret < 0) {
> + dev_err(dev, "unable to identify rail4 iout scale, errno = %d\n", ret);
> + return ret;
> + }
> +
> + hwmon_dev = devm_hwmon_device_register_with_info(dev, client->name,
> + data, &mp2845_chip_info,
> + NULL);
> + if (IS_ERR(hwmon_dev)) {
> + dev_err(dev, "unable to register mp2845 hwmon device\n");
> + return PTR_ERR(hwmon_dev);
> + }
> +
> + dev_info(dev, "%s: sensor '%s'\n", dev_name(hwmon_dev), client->name);
> +
> + return 0;
> +}
> +
> +static const struct i2c_device_id mp2845_ids[] = {
> + {"mp2845", 0},
> + {}
> +};
> +MODULE_DEVICE_TABLE(i2c, mp2845_ids);
> +
> +static const struct of_device_id __maybe_unused mp2845_of_match[] = {
> + {.compatible = "mps,mp2845"},
> + {}
> +};
> +MODULE_DEVICE_TABLE(of, mp2845_of_match);
> +
> +static struct i2c_driver mp2845_driver = {
> + .class = I2C_CLASS_HWMON,
> + .driver = {
> + .name = "mp2845",
> + .of_match_table = mp2845_of_match,
> + },
> + .probe = mp2845_probe,
> + .id_table = mp2845_ids,
> +};
> +module_i2c_driver(mp2845_driver);
> +
> +MODULE_AUTHOR("Wensheng Wang <wenswang@yeah.net>");
> +MODULE_DESCRIPTION("MP2845 driver");
> +MODULE_LICENSE("GPL");
> --
> 2.25.1
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-02-26 14:22 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-25 8:55 [PATCH 0/2] hwmon: Add support for MPS mp2845 chip wenswang
2026-02-25 8:56 ` [PATCH 1/2] dt-bindings: hwmon: Add MPS mp2845 wenswang
2026-02-25 8:56 ` [PATCH 2/2] hwmon: add MP2845 driver wenswang
2026-02-26 7:48 ` Krzysztof Kozlowski
2026-02-26 14:22 ` Guenter Roeck
2026-02-26 7:45 ` [PATCH 1/2] dt-bindings: hwmon: Add MPS mp2845 Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox