diff options
Diffstat (limited to 'target/linux/generic/backport-5.10/873-v6.0-hwmon-tps23861-fix-byte-order-in-current-and-voltage.patch')
-rw-r--r-- | target/linux/generic/backport-5.10/873-v6.0-hwmon-tps23861-fix-byte-order-in-current-and-voltage.patch | 66 |
1 files changed, 0 insertions, 66 deletions
diff --git a/target/linux/generic/backport-5.10/873-v6.0-hwmon-tps23861-fix-byte-order-in-current-and-voltage.patch b/target/linux/generic/backport-5.10/873-v6.0-hwmon-tps23861-fix-byte-order-in-current-and-voltage.patch deleted file mode 100644 index 45c2d0b4a4..0000000000 --- a/target/linux/generic/backport-5.10/873-v6.0-hwmon-tps23861-fix-byte-order-in-current-and-voltage.patch +++ /dev/null @@ -1,66 +0,0 @@ -From 0eabb1396656f215a5333a9444158b17b0fd3247 Mon Sep 17 00:00:00 2001 -From: Alexandru Gagniuc <mr.nuke.me@gmail.com> -Date: Wed, 20 Jul 2022 22:22:55 -0500 -Subject: hwmon: (tps23861) fix byte order in current and voltage registers - -Trying to use this driver on a big-endian machine results in garbage -values for voltage and current. The tps23861 registers are little- -endian, and regmap_read_bulk() does not do byte order conversion. Thus -on BE machines, the most significant bytes got modified, and were -trimmed by the VOLTAGE_CURRENT_MASK. - -To resolve this use uint16_t values, and convert them to host byte -order using le16_to_cpu(). This results in correct readings on MIPS. - -Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com> -Link: https://lore.kernel.org/r/20220721032255.2850647-1-mr.nuke.me@gmail.com -[groeck: Use __le16 instead of uint16_t] -Signed-off-by: Guenter Roeck <linux@roeck-us.net> ---- - drivers/hwmon/tps23861.c | 14 +++++++++----- - 1 file changed, 9 insertions(+), 5 deletions(-) - ---- a/drivers/hwmon/tps23861.c -+++ b/drivers/hwmon/tps23861.c -@@ -140,7 +140,8 @@ static int tps23861_read_temp(struct tps - static int tps23861_read_voltage(struct tps23861_data *data, int channel, - long *val) - { -- unsigned int regval; -+ __le16 regval; -+ long raw_val; - int err; - - if (channel < TPS23861_NUM_PORTS) { -@@ -155,7 +156,8 @@ static int tps23861_read_voltage(struct - if (err < 0) - return err; - -- *val = (FIELD_GET(VOLTAGE_CURRENT_MASK, regval) * VOLTAGE_LSB) / 1000; -+ raw_val = le16_to_cpu(regval); -+ *val = (FIELD_GET(VOLTAGE_CURRENT_MASK, raw_val) * VOLTAGE_LSB) / 1000; - - return 0; - } -@@ -163,8 +165,9 @@ static int tps23861_read_voltage(struct - static int tps23861_read_current(struct tps23861_data *data, int channel, - long *val) - { -- unsigned int current_lsb; -- unsigned int regval; -+ long raw_val, current_lsb; -+ __le16 regval; -+ - int err; - - if (data->shunt_resistor == SHUNT_RESISTOR_DEFAULT) -@@ -178,7 +181,8 @@ static int tps23861_read_current(struct - if (err < 0) - return err; - -- *val = (FIELD_GET(VOLTAGE_CURRENT_MASK, regval) * current_lsb) / 1000000; -+ raw_val = le16_to_cpu(regval); -+ *val = (FIELD_GET(VOLTAGE_CURRENT_MASK, raw_val) * current_lsb) / 1000000; - - return 0; - } |