* [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710
@ 2024-04-29 3:38 Peter Yin
2024-04-29 3:38 ` [PATCH v3 1/2] " Peter Yin
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Peter Yin @ 2024-04-29 3:38 UTC (permalink / raw)
To: patrick, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Jean Delvare, Guenter Roeck, Jonathan Corbet, Delphine CC Chiu,
Patrick Rudolph, Lukas Wunner, Javier Carrasco, Bjorn Helgaas,
devicetree, linux-kernel, linux-hwmon, linux-doc, linux-i2c
Add support for Infineon XDP710. This is a Hot-Swap Controller.
Change log:
v2 -> v3
- Fixed Ack quotes
- rename "microOhmRsense" to "micro_ohm_rsense"
v1 -> v2
- Merged patch 3 into patch 1.
- Modified comments.
- Fixed the microOhmRsense array.
- Improved operational accuracy.
v1
- Add support for xdp710 hot-swap device from Infineon vendor
---
Peter Yin (2):
hwmon: (pmbus) Add support for Infineon XDP710
dt-bindings: hwmon: Add infineon xdp710 driver bindings
.../devicetree/bindings/trivial-devices.yaml | 2 +
Documentation/hwmon/index.rst | 1 +
Documentation/hwmon/xdp710.rst | 83 +++++++++++
drivers/hwmon/pmbus/Kconfig | 9 ++
drivers/hwmon/pmbus/Makefile | 1 +
drivers/hwmon/pmbus/xdp710.c | 132 ++++++++++++++++++
6 files changed, 228 insertions(+)
create mode 100644 Documentation/hwmon/xdp710.rst
create mode 100644 drivers/hwmon/pmbus/xdp710.c
--
2.25.1
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH v3 1/2] hwmon: (pmbus) Add support for Infineon XDP710 2024-04-29 3:38 [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 Peter Yin @ 2024-04-29 3:38 ` Peter Yin 2024-04-29 3:38 ` [PATCH v3 2/2] dt-bindings: hwmon: Add infineon xdp710 driver bindings Peter Yin ` (2 subsequent siblings) 3 siblings, 0 replies; 6+ messages in thread From: Peter Yin @ 2024-04-29 3:38 UTC (permalink / raw) To: patrick, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Jean Delvare, Guenter Roeck, Jonathan Corbet, Delphine CC Chiu, Patrick Rudolph, Javier Carrasco, Andre Werner, Alexander Stein, Bjorn Helgaas, Lukas Wunner, devicetree, linux-kernel, linux-hwmon, linux-doc, linux-i2c Add support for Infineon XDP710.This is a Hot-Swap Controller. Signed-off-by: Peter Yin <peteryin.openbmc@gmail.com> --- Documentation/hwmon/index.rst | 1 + Documentation/hwmon/xdp710.rst | 83 +++++++++++++++++++++ drivers/hwmon/pmbus/Kconfig | 9 +++ drivers/hwmon/pmbus/Makefile | 1 + drivers/hwmon/pmbus/xdp710.c | 132 +++++++++++++++++++++++++++++++++ 5 files changed, 226 insertions(+) create mode 100644 Documentation/hwmon/xdp710.rst create mode 100644 drivers/hwmon/pmbus/xdp710.c diff --git a/Documentation/hwmon/index.rst b/Documentation/hwmon/index.rst index 1ca7a4fe1f8f..b2546925fb15 100644 --- a/Documentation/hwmon/index.rst +++ b/Documentation/hwmon/index.rst @@ -250,6 +250,7 @@ Hardware Monitoring Kernel Drivers wm831x wm8350 xgene-hwmon + xdp710 xdpe12284 xdpe152c4 zl6100 diff --git a/Documentation/hwmon/xdp710.rst b/Documentation/hwmon/xdp710.rst new file mode 100644 index 000000000000..083891f27818 --- /dev/null +++ b/Documentation/hwmon/xdp710.rst @@ -0,0 +1,83 @@ +.. SPDX-License-Identifier: GPL-2.0 + +Kernel driver xdp710 +==================== + +Supported chips: + + * Infineon XDP710 + + Prefix: 'xdp710' + + * Datasheet + + Publicly available at the Infineon website : https://www.infineon.com/dgdl/Infineon-XDP710-001-DataSheet-v01_00-EN.pdf?fileId=8ac78c8c8412f8d301848a5316290b97 + +Author: + + Peter Yin <peteryin.openbmc@gmail.com> + +Description +----------- + +This driver implements support for Infineon XDP710 Hot-Swap Controller. + +Device compliant with: + +- PMBus rev 1.3 interface. + +Device supports direct and linear format for reading input voltage, +output voltage, output current, input power and temperature. + +The driver exports the following attributes via the 'sysfs' files +for input voltage: + +**in1_input** + +**in1_label** + +**in1_max** + +**in1_max_alarm** + +**in1_min** + +**in1_min_alarm** + +The driver provides the following attributes for output voltage: + +**in2_input** + +**in2_label** + +**in2_alarm** + +The driver provides the following attributes for output current: + +**curr1_input** + +**curr1_label** + +**curr1_alarm** + +**curr1_max** + +The driver provides the following attributes for input power: + +**power1_input** + +**power1_label** + +**power1_alarm** + +The driver provides the following attributes for temperature: + +**temp1_input** + +**temp1_max** + +**temp1_max_alarm** + +**temp1_crit** + +**temp1_crit_alarm** diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig index 557ae0c414b0..c775128c2ea7 100644 --- a/drivers/hwmon/pmbus/Kconfig +++ b/drivers/hwmon/pmbus/Kconfig @@ -511,6 +511,15 @@ config SENSORS_UCD9200 This driver can also be built as a module. If so, the module will be called ucd9200. +config SENSORS_XDP710 + tristate "Infineon XDP710 family" + help + If you say yes here you get hardware monitoring support for Infineon + XDP710. + + This driver can also be built as a module. If so, the module will + be called xdp710. + config SENSORS_XDPE152 tristate "Infineon XDPE152 family" help diff --git a/drivers/hwmon/pmbus/Makefile b/drivers/hwmon/pmbus/Makefile index f14ecf03ad77..4fe630793721 100644 --- a/drivers/hwmon/pmbus/Makefile +++ b/drivers/hwmon/pmbus/Makefile @@ -51,6 +51,7 @@ obj-$(CONFIG_SENSORS_TPS53679) += tps53679.o obj-$(CONFIG_SENSORS_TPS546D24) += tps546d24.o obj-$(CONFIG_SENSORS_UCD9000) += ucd9000.o obj-$(CONFIG_SENSORS_UCD9200) += ucd9200.o +obj-$(CONFIG_SENSORS_XDP710) += xdp710.o obj-$(CONFIG_SENSORS_XDPE122) += xdpe12284.o obj-$(CONFIG_SENSORS_XDPE152) += xdpe152c4.o obj-$(CONFIG_SENSORS_ZL6100) += zl6100.o diff --git a/drivers/hwmon/pmbus/xdp710.c b/drivers/hwmon/pmbus/xdp710.c new file mode 100644 index 000000000000..4f858eafc5e8 --- /dev/null +++ b/drivers/hwmon/pmbus/xdp710.c @@ -0,0 +1,132 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * Driver for Infineon XDP710 Hot-Swap Controller + */ + +#include <linux/bitops.h> +#include <linux/i2c.h> +#include <linux/module.h> +#include <linux/of_device.h> +#include "pmbus.h" + +#define XDP710_REG_CFG 0xD3 +#define XDP710_V_SNS_CFG 0xD4 +#define XDP710_CS_RNG 0xD5 + +/* + * The table to map configuration register values + * to sense resistor values + */ +const int micro_ohm_rsense[] = { + 200, 250, 300, 330, 400, 470, 500, 600, + 670, 700, 750, 800, 900, 1000, 1100, 1200, + 1250, 1300, 1400, 1500, 1600, 1700, 1800, 1900, + 2000, 2100, 2200, 2300, 2400, 2500, 2600, 2700, + 2800, 3000, 3100, 3200, 3300, 3400, 3500, 3600, + 3700, 3800, 3900, 4000, 4100, 4200, 4300, 4400, + 4500, 4600, 4700, 4800, 4900, 5000, 5500, 6000, + 6500, 7000, 7500, 8000, 8500, 9000, 9500, 10000 +}; + +static struct pmbus_driver_info xdp710_info = { + .pages = 1, + .format[PSC_VOLTAGE_IN] = direct, + .format[PSC_VOLTAGE_OUT] = direct, + .format[PSC_CURRENT_OUT] = direct, + .format[PSC_POWER] = direct, + .format[PSC_TEMPERATURE] = direct, + .m[PSC_VOLTAGE_IN] = 4653, + .b[PSC_VOLTAGE_IN] = 0, + .R[PSC_VOLTAGE_IN] = -2, + .m[PSC_VOLTAGE_OUT] = 4653, + .b[PSC_VOLTAGE_OUT] = 0, + .R[PSC_VOLTAGE_OUT] = -2, + .m[PSC_CURRENT_OUT] = 23165, + .b[PSC_CURRENT_OUT] = 0, + .R[PSC_CURRENT_OUT] = -2, + .m[PSC_POWER] = 4211, + .b[PSC_POWER] = 0, + .R[PSC_POWER] = -2, + .m[PSC_TEMPERATURE] = 52, + .b[PSC_TEMPERATURE] = 14321, + .R[PSC_TEMPERATURE] = -1, + .func[0] = + PMBUS_HAVE_VIN | PMBUS_HAVE_VOUT | PMBUS_HAVE_PIN | + PMBUS_HAVE_TEMP | PMBUS_HAVE_IOUT | + PMBUS_HAVE_STATUS_INPUT | PMBUS_HAVE_STATUS_TEMP, +}; + +static int xdp710_probe(struct i2c_client *client) +{ + struct pmbus_driver_info *info; + u8 cs_rng; + u8 vtlm_rng; + int rsense; + int ret; + int m = 0; + + info = devm_kmemdup(&client->dev, &xdp710_info, sizeof(*info), + GFP_KERNEL); + if (!info) + return -ENOMEM; + + ret = i2c_smbus_read_word_data(client, XDP710_CS_RNG); + if (ret < 0) { + dev_err(&client->dev, "Can't get CS_RNG"); + return ret; + } + cs_rng = (ret >> 6) & GENMASK(1, 0); + + ret = i2c_smbus_read_word_data(client, XDP710_V_SNS_CFG); + if (ret < 0) { + dev_err(&client->dev, "Can't get V_SNS_CFG"); + return ret; + } + vtlm_rng = ret & GENMASK(1, 0); + + ret = i2c_smbus_read_word_data(client, XDP710_REG_CFG); + if (ret < 0) { + dev_err(&client->dev, "Can't get REG_CFG"); + return ret; + } + ret &= GENMASK(5, 0); + rsense = micro_ohm_rsense[ret]; + + info->m[PSC_VOLTAGE_IN] <<= vtlm_rng; + info->m[PSC_VOLTAGE_OUT] <<= vtlm_rng; + + m = info->m[PSC_CURRENT_OUT]; + info->m[PSC_CURRENT_OUT] = DIV_ROUND_CLOSEST(m * rsense >> cs_rng, + 1000); + + m = info->m[PSC_POWER]; + info->m[PSC_POWER] = DIV_ROUND_CLOSEST(m * rsense >> cs_rng, 1000); + + return pmbus_do_probe(client, info); +} + +static const struct of_device_id xdp710_of_match[] = { + { .compatible = "infineon,xdp710" }, + {} +}; + +static const struct i2c_device_id xdp710_id[] = { + {"xdp710", 0}, + { } +}; +MODULE_DEVICE_TABLE(i2c, xdp710_id); + +static struct i2c_driver xdp710_driver = { + .driver = { + .name = "xdp710", + .of_match_table = xdp710_of_match, + }, + .probe = xdp710_probe, + .id_table = xdp710_id, +}; +module_i2c_driver(xdp710_driver); + +MODULE_AUTHOR("Peter Yin <peter.yin@quantatw.com>"); +MODULE_DESCRIPTION("PMBus driver for XDP710 HSC"); +MODULE_LICENSE("GPL"); +MODULE_IMPORT_NS(PMBUS); -- 2.25.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v3 2/2] dt-bindings: hwmon: Add infineon xdp710 driver bindings 2024-04-29 3:38 [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 Peter Yin 2024-04-29 3:38 ` [PATCH v3 1/2] " Peter Yin @ 2024-04-29 3:38 ` Peter Yin 2024-04-29 4:59 ` [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 Krzysztof Kozlowski 2024-04-29 5:00 ` Guenter Roeck 3 siblings, 0 replies; 6+ messages in thread From: Peter Yin @ 2024-04-29 3:38 UTC (permalink / raw) To: patrick, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Jean Delvare, Guenter Roeck, Jonathan Corbet, Delphine CC Chiu, Patrick Rudolph, Cosmo Chou, Marek Vasut, Bjorn Helgaas, Lukas Wunner, devicetree, linux-kernel, linux-hwmon, linux-doc, linux-i2c Add a device tree bindings for xdp710 device Acked-by: Rob Herring (Arm) <robh@kernel.org> Signed-off-by: Peter Yin <peteryin.openbmc@gmail.com> --- 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 e07be7bf8395..f982de168c4c 100644 --- a/Documentation/devicetree/bindings/trivial-devices.yaml +++ b/Documentation/devicetree/bindings/trivial-devices.yaml @@ -134,6 +134,8 @@ properties: - infineon,irps5401 # Infineon TLV493D-A1B6 I2C 3D Magnetic Sensor - infineon,tlv493d-a1b6 + # Infineon Hot-swap controller xdp710 + - infineon,xdp710 # Infineon Multi-phase Digital VR Controller xdpe11280 - infineon,xdpe11280 # Infineon Multi-phase Digital VR Controller xdpe12254 -- 2.25.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 2024-04-29 3:38 [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 Peter Yin 2024-04-29 3:38 ` [PATCH v3 1/2] " Peter Yin 2024-04-29 3:38 ` [PATCH v3 2/2] dt-bindings: hwmon: Add infineon xdp710 driver bindings Peter Yin @ 2024-04-29 4:59 ` Krzysztof Kozlowski 2024-04-29 5:00 ` Guenter Roeck 3 siblings, 0 replies; 6+ messages in thread From: Krzysztof Kozlowski @ 2024-04-29 4:59 UTC (permalink / raw) To: Peter Yin, patrick, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Jean Delvare, Guenter Roeck, Jonathan Corbet, Delphine CC Chiu, Patrick Rudolph, Lukas Wunner, Javier Carrasco, Bjorn Helgaas, devicetree, linux-kernel, linux-hwmon, linux-doc, linux-i2c On 29/04/2024 05:38, Peter Yin wrote: > Add support for Infineon XDP710. This is a Hot-Swap Controller. > > Change log: > > v2 -> v3 > - Fixed Ack quotes > - rename "microOhmRsense" to "micro_ohm_rsense" You got the message it was applied, didn't you? Best regards, Krzysztof ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 2024-04-29 3:38 [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 Peter Yin ` (2 preceding siblings ...) 2024-04-29 4:59 ` [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 Krzysztof Kozlowski @ 2024-04-29 5:00 ` Guenter Roeck 2024-04-29 5:18 ` Peter Yin 3 siblings, 1 reply; 6+ messages in thread From: Guenter Roeck @ 2024-04-29 5:00 UTC (permalink / raw) To: Peter Yin, patrick, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Jean Delvare, Jonathan Corbet, Delphine CC Chiu, Patrick Rudolph, Lukas Wunner, Javier Carrasco, Bjorn Helgaas, devicetree, linux-kernel, linux-hwmon, linux-doc, linux-i2c On 4/28/24 20:38, Peter Yin wrote: > Add support for Infineon XDP710. This is a Hot-Swap Controller. > > Change log: > > v2 -> v3 > - Fixed Ack quotes > - rename "microOhmRsense" to "micro_ohm_rsense" > > v1 -> v2 > - Merged patch 3 into patch 1. > - Modified comments. > - Fixed the microOhmRsense array. > - Improved operational accuracy. > > v1 > - Add support for xdp710 hot-swap device from Infineon vendor > --- > > Peter Yin (2): > hwmon: (pmbus) Add support for Infineon XDP710 > dt-bindings: hwmon: Add infineon xdp710 driver bindings > > .../devicetree/bindings/trivial-devices.yaml | 2 + > Documentation/hwmon/index.rst | 1 + > Documentation/hwmon/xdp710.rst | 83 +++++++++++ > drivers/hwmon/pmbus/Kconfig | 9 ++ > drivers/hwmon/pmbus/Makefile | 1 + > drivers/hwmon/pmbus/xdp710.c | 132 ++++++++++++++++++ > 6 files changed, 228 insertions(+) > create mode 100644 Documentation/hwmon/xdp710.rst > create mode 100644 drivers/hwmon/pmbus/xdp710.c > What is the point of resending this ? I said I applied the series. Guenter ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 2024-04-29 5:00 ` Guenter Roeck @ 2024-04-29 5:18 ` Peter Yin 0 siblings, 0 replies; 6+ messages in thread From: Peter Yin @ 2024-04-29 5:18 UTC (permalink / raw) To: Guenter Roeck Cc: patrick, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Jean Delvare, Jonathan Corbet, Delphine CC Chiu, Patrick Rudolph, Lukas Wunner, Javier Carrasco, Bjorn Helgaas, devicetree, linux-kernel, linux-hwmon, linux-doc, linux-i2c Sorry, I thought it was going to be applied after I fixed it. On Mon, Apr 29, 2024 at 1:00 PM Guenter Roeck <linux@roeck-us.net> wrote: > > On 4/28/24 20:38, Peter Yin wrote: > > Add support for Infineon XDP710. This is a Hot-Swap Controller. > > > > Change log: > > > > v2 -> v3 > > - Fixed Ack quotes > > - rename "microOhmRsense" to "micro_ohm_rsense" > > > > v1 -> v2 > > - Merged patch 3 into patch 1. > > - Modified comments. > > - Fixed the microOhmRsense array. > > - Improved operational accuracy. > > > > v1 > > - Add support for xdp710 hot-swap device from Infineon vendor > > --- > > > > Peter Yin (2): > > hwmon: (pmbus) Add support for Infineon XDP710 > > dt-bindings: hwmon: Add infineon xdp710 driver bindings > > > > .../devicetree/bindings/trivial-devices.yaml | 2 + > > Documentation/hwmon/index.rst | 1 + > > Documentation/hwmon/xdp710.rst | 83 +++++++++++ > > drivers/hwmon/pmbus/Kconfig | 9 ++ > > drivers/hwmon/pmbus/Makefile | 1 + > > drivers/hwmon/pmbus/xdp710.c | 132 ++++++++++++++++++ > > 6 files changed, 228 insertions(+) > > create mode 100644 Documentation/hwmon/xdp710.rst > > create mode 100644 drivers/hwmon/pmbus/xdp710.c > > > > What is the point of resending this ? I said I applied the series. > > Guenter > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-04-29 5:18 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-04-29 3:38 [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 Peter Yin 2024-04-29 3:38 ` [PATCH v3 1/2] " Peter Yin 2024-04-29 3:38 ` [PATCH v3 2/2] dt-bindings: hwmon: Add infineon xdp710 driver bindings Peter Yin 2024-04-29 4:59 ` [PATCH v3 0/2] hwmon: (pmbus) Add support for Infineon XDP710 Krzysztof Kozlowski 2024-04-29 5:00 ` Guenter Roeck 2024-04-29 5:18 ` Peter Yin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).