# [PATCH] hwmon: pmbus: max20730: adjust the vout reading given voltage divider

From: Chu Lin
Date: Thu Sep 10 2020 - 13:11:40 EST

Problem:
We use voltage dividers so that the voltage presented at the voltage
sense pins is confusing. We might need to convert these readings to more
meaningful readings given the voltage divider.

Solution:
Read the voltage divider resistance from dts and convert the voltage

Testing:
max20730 with voltage divider

Signed-off-by: Chu Lin <linchuyuan@xxxxxxxxxx>
---
drivers/hwmon/pmbus/max20730.c | 16 ++++++++++++++++
1 file changed, 16 insertions(+)

diff --git a/drivers/hwmon/pmbus/max20730.c b/drivers/hwmon/pmbus/max20730.c
index c0bb05487e0e..4b56810fa894 100644
--- a/drivers/hwmon/pmbus/max20730.c
+++ b/drivers/hwmon/pmbus/max20730.c
@@ -29,6 +29,7 @@ struct max20730_data {
struct pmbus_driver_info info;
struct mutex lock; /* Used to protect against parallel writes */
u16 mfr_devset1;
+ u32 vout_voltage_divider[2];
};

#define to_max20730_data(x) container_of(x, struct max20730_data, info)
@@ -111,6 +112,12 @@ static int max20730_read_word_data(struct i2c_client *client, int page,
max_c = max_current[data->id][(data->mfr_devset1 >> 5) & 0x3];
ret = val_to_direct(max_c, PSC_CURRENT_OUT, info);
break;
+ ret = pmbus_read_word_data(client, page, phase, reg);
+ if (data->vout_voltage_divider[0] && data->vout_voltage_divider[1])
+ ret = DIV_ROUND_CLOSEST(ret * data->vout_voltage_divider[1],
+ data->vout_voltage_divider[0]);
+ break;
default:
ret = -ENODATA;
break;
@@ -329,6 +336,15 @@ static int max20730_probe(struct i2c_client *client,
data->id = chip_id;
mutex_init(&data->lock);
memcpy(&data->info, &max20730_info[chip_id], sizeof(data->info));