* [PATCH 3/4] hwmon: (pmbus) add support for IR35215
@ 2019-04-16 18:45 Ruslan Babayev
0 siblings, 0 replies; 4+ messages in thread
From: Ruslan Babayev @ 2019-04-16 18:45 UTC (permalink / raw)
To: Jean Delvare, Guenter Roeck
Cc: Ruslan Babayev, xe-linux-external, Jonathan Corbet, linux-hwmon,
linux-doc, linux-kernel
From: Ruslan Babayev <ruslan@babayev.com>
Provide support for Infineon Dual Output Digital Multi-Phase Buck
Controllers IR35215 and IR3570B.
Cc: xe-linux-external@cisco.com
Signed-off-by: Ruslan Babayev <ruslan@babayev.com>
---
Documentation/hwmon/ir35215 | 98 +++++++++++++++++++++++++++++++++++
drivers/hwmon/pmbus/Kconfig | 10 ++++
drivers/hwmon/pmbus/Makefile | 1 +
drivers/hwmon/pmbus/ir35215.c | 91 ++++++++++++++++++++++++++++++++
4 files changed, 200 insertions(+)
create mode 100644 Documentation/hwmon/ir35215
create mode 100644 drivers/hwmon/pmbus/ir35215.c
diff --git a/Documentation/hwmon/ir35215 b/Documentation/hwmon/ir35215
new file mode 100644
index 000000000000..b77481340006
--- /dev/null
+++ b/Documentation/hwmon/ir35215
@@ -0,0 +1,98 @@
+Kernel driver ir35215
+=====================
+
+Supported chips:
+ * Infinion IR35215
+ Prefix: 'ir35215'
+ Addresses scanned: -
+ Datasheet: Datasheet is not publicly available.
+ * Infinion IR3570B
+ Prefix: 'ir3570b'
+ Addresses scanned: -
+ Datasheet: Datasheet is not publicly available.
+
+Author: Ruslan Babayev <ruslan@babayev.com>
+
+
+Description
+-----------
+
+IR35215 is a Dual Output Digital Multi-Phase Controller
+
+
+Usage Notes
+-----------
+
+This driver does not probe for PMBus devices. You will have to instantiate
+devices explicitly.
+
+Example: the following commands will load the driver for an IR35215
+at address 0x40 on I2C bus #4:
+
+# modprobe ir35215
+# echo ir35215 0x40 > /sys/bus/i2c/devices/i2c-4/new_device
+
+
+Sysfs attributes
+----------------
+
+curr[1-2]_label "iin[1-2]"
+curr[1-2]_input Measured input current
+curr[1-2]_max Maximum current for
+curr[1-2]_max_alarm Current high alarm
+
+curr[3-4]_label "iout[3-4]"
+curr[3-4]_input Measured output current
+curr[3-4]_crit Critical maximum current
+curr[3-4]_crit_alarm Current critical high alarm
+curr[3-4]_highest Highest output current
+curr[3-4]_lowest Lowest output current
+curr[3-4]_max Maximum current
+curr[3-4]_max_alarm Current high alarm
+
+in[1-2]_label "vin[1-2]"
+in[1-2]_input Measured input voltage
+in[1-2]_crit Critical maximum input voltage
+in[1-2]_crit_alarm Input voltage critical high alarm
+in[1-2]_highest Highest input voltage
+in[1-2]_lowest Lowest input voltage
+in[1-2]_min Minimum input voltage
+in[1-2]_min_alarm Input voltage low alarm
+
+in[3-4]_label "vout[3-4]"
+in[3-4]_input Measured output voltage
+in[3-4]_lcrit Critical minimum output voltage
+in[3-4]_lcrit_alarm Output voltage critical low alarm
+in[3-4]_crit Critical maximum output voltage
+in[3-4]_crit_alarm Output voltage critical high alarm
+in[3-4]_highest Highest output voltage
+in[3-4]_lowest Lowest output voltage
+in[3-4]_max Maximum output voltage
+in[3-4]_max_alarm Output voltage high alarm
+in[3-4]_min Minimum output voltage
+in[3-4]_min_alarm Output voltage low alarm
+
+out[1-2]_command Commanded output voltage
+out[1-2]_margin_high Output voltage when operation is set to "high"
+out[1-2]_margin_low Output voltage when operation is set to "low"
+out[1-2]_max Maximum output voltage
+out[1-2]_max_alarm Output voltage high alarm
+out[1-2]_operation Set the output voltage to "on", "off", "high" or "low"
+
+power1_label "pin"
+power1_input Measured input power
+power1_alarm Input power high alarm
+power1_max Input power limit
+
+power[2-3]_label "pout[1-2]"
+power[2-3]_input Measured output power
+power[2-3]_max Output power limit
+power[2-3]_max_alarm Output power high alarm
+
+temp[1-2]_input Measured temperature
+temp[1-2]_crit Critical high temperature
+temp[1-2]_crit_alarm Chip temperature critical high alarm
+temp[1-2]_highest Highest temperature
+temp[1-2]_lowest Lowest temperature
+temp[1-2]_max Maximum temperature
+temp[1-2]_max_alarm Chip temperature high alarm
diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig
index 629cb45f8557..a7dd5e606061 100644
--- a/drivers/hwmon/pmbus/Kconfig
+++ b/drivers/hwmon/pmbus/Kconfig
@@ -45,6 +45,16 @@ config SENSORS_IBM_CFFPS
This driver can also be built as a module. If so, the module will
be called ibm-cffps.
+config SENSORS_IR35215
+ tristate "Infineon IR35215"
+ default n
+ help
+ If you say yes here you get hardware monitoring support for the
+ Infineon IR35215 controller.
+
+ This driver can also be built as a module. If so, the module will
+ be called ir35215.
+
config SENSORS_IR35221
tristate "Infineon IR35221"
help
diff --git a/drivers/hwmon/pmbus/Makefile b/drivers/hwmon/pmbus/Makefile
index ea0e39518c21..48643e3bab42 100644
--- a/drivers/hwmon/pmbus/Makefile
+++ b/drivers/hwmon/pmbus/Makefile
@@ -7,6 +7,7 @@ obj-$(CONFIG_PMBUS) += pmbus_core.o
obj-$(CONFIG_SENSORS_PMBUS) += pmbus.o
obj-$(CONFIG_SENSORS_ADM1275) += adm1275.o
obj-$(CONFIG_SENSORS_IBM_CFFPS) += ibm-cffps.o
+obj-$(CONFIG_SENSORS_IR35215) += ir35215.o
obj-$(CONFIG_SENSORS_IR35221) += ir35221.o
obj-$(CONFIG_SENSORS_LM25066) += lm25066.o
obj-$(CONFIG_SENSORS_LTC2978) += ltc2978.o
diff --git a/drivers/hwmon/pmbus/ir35215.c b/drivers/hwmon/pmbus/ir35215.c
new file mode 100644
index 000000000000..e50ee465a42d
--- /dev/null
+++ b/drivers/hwmon/pmbus/ir35215.c
@@ -0,0 +1,91 @@
+/*
+ * Hardware monitoring driver for Infineon IR35215
+ *
+ * Copyright (c) 2019 Cisco Systems.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/init.h>
+#include <linux/err.h>
+#include <linux/i2c.h>
+#include "pmbus.h"
+
+#define COMMON_FUNCS (PMBUS_HAVE_VIN | PMBUS_HAVE_VOUT \
+ | PMBUS_HAVE_IIN | PMBUS_HAVE_IOUT \
+ | PMBUS_HAVE_PIN | PMBUS_HAVE_POUT \
+ | PMBUS_HAVE_TEMP \
+ | PMBUS_HAVE_STATUS_VOUT \
+ | PMBUS_HAVE_STATUS_IOUT \
+ | PMBUS_HAVE_STATUS_INPUT \
+ | PMBUS_HAVE_STATUS_TEMP \
+ | PMBUS_HAVE_VOUT_MARGIN_HIGH \
+ | PMBUS_HAVE_VOUT_MARGIN_LOW \
+ )
+
+
+#define IR35215_FUNCS (COMMON_FUNCS | PMBUS_HAVE_VOUT_COMMAND)
+#define IR3570B_FUNCS (COMMON_FUNCS | PMBUS_HAVE_TEMP2)
+
+static struct pmbus_driver_info ir35215_info = {
+ .pages = 2,
+ .format[PSC_VOLTAGE_IN] = linear,
+ .format[PSC_VOLTAGE_OUT] = linear,
+ .format[PSC_CURRENT_IN] = linear,
+ .format[PSC_CURRENT_OUT] = linear,
+ .format[PSC_POWER] = linear,
+ .format[PSC_TEMPERATURE] = linear,
+ .func[0] = IR35215_FUNCS,
+ .func[1] = IR35215_FUNCS,
+};
+
+static struct pmbus_driver_info ir3570b_info = {
+ .pages = 1,
+ .format[PSC_VOLTAGE_IN] = linear,
+ .format[PSC_VOLTAGE_OUT] = linear,
+ .format[PSC_CURRENT_IN] = linear,
+ .format[PSC_CURRENT_OUT] = linear,
+ .format[PSC_POWER] = linear,
+ .format[PSC_TEMPERATURE] = linear,
+ .func[0] = IR3570B_FUNCS,
+ .func[1] = IR3570B_FUNCS,
+};
+
+static int ir35215_probe(struct i2c_client *client,
+ const struct i2c_device_id *id)
+{
+ return pmbus_do_probe(client, id, (void *)id->driver_data);
+}
+
+static const struct i2c_device_id ir35215_id[] = {
+ {"ir35215", (kernel_ulong_t)&ir35215_info},
+ {"ir3570b", (kernel_ulong_t)&ir3570b_info},
+ {}
+};
+
+MODULE_DEVICE_TABLE(i2c, ir35215_id);
+
+static struct i2c_driver ir35215_driver = {
+ .driver = {
+ .name = "ir35215",
+ },
+ .probe = ir35215_probe,
+ .remove = pmbus_do_remove,
+ .id_table = ir35215_id,
+};
+
+module_i2c_driver(ir35215_driver);
+
+MODULE_AUTHOR("Ruslan Babayev <ruslan@babayev.com>");
+MODULE_DESCRIPTION("PMBus driver for IR35215");
+MODULE_LICENSE("GPL");
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 3/4] hwmon: (pmbus) add support for IR35215
2019-04-16 20:16 ` Guenter Roeck
@ 2019-04-17 2:04 ` Ruslan Babayev
0 siblings, 0 replies; 4+ messages in thread
From: Ruslan Babayev @ 2019-04-17 2:04 UTC (permalink / raw)
To: Guenter Roeck
Cc: Ruslan Babayev, Jean Delvare, xe-linux-external, Jonathan Corbet,
linux-hwmon, linux-doc, linux-kernel
Guenter Roeck writes:
>> +
>> +#include <linux/kernel.h>
>> +#include <linux/module.h>
>> +#include <linux/init.h>
>> +#include <linux/err.h>
>> +#include <linux/i2c.h>
>
> Alphabetic include file order please.
Sure
>> +
>> +static struct pmbus_driver_info ir3570b_info = {
>> + .pages = 1,
>
> Either this ...
>
>> + .format[PSC_VOLTAGE_IN] = linear,
>> + .format[PSC_VOLTAGE_OUT] = linear,
>> + .format[PSC_CURRENT_IN] = linear,
>> + .format[PSC_CURRENT_OUT] = linear,
>> + .format[PSC_POWER] = linear,
>> + .format[PSC_TEMPERATURE] = linear,
>> + .func[0] = IR3570B_FUNCS,
>> + .func[1] = IR3570B_FUNCS,
>
> ... or this is wrong.
The latter is wrong. Thanks for catching it.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 3/4] hwmon: (pmbus) add support for IR35215
2019-04-16 18:36 Ruslan Babayev
@ 2019-04-16 20:16 ` Guenter Roeck
2019-04-17 2:04 ` Ruslan Babayev
0 siblings, 1 reply; 4+ messages in thread
From: Guenter Roeck @ 2019-04-16 20:16 UTC (permalink / raw)
To: Ruslan Babayev
Cc: Jean Delvare, xe-linux-external, Jonathan Corbet, linux-hwmon,
linux-doc, linux-kernel
On Tue, Apr 16, 2019 at 11:36:18AM -0700, Ruslan Babayev wrote:
> Provide support for Infineon Dual Output Digital Multi-Phase Buck
> Controllers IR35215 and IR3570B.
>
> Cc: xe-linux-external@cisco.com
> Signed-off-by: Ruslan Babayev <ruslan@babayev.com>
> ---
> Documentation/hwmon/ir35215 | 98 +++++++++++++++++++++++++++++++++++
> drivers/hwmon/pmbus/Kconfig | 10 ++++
> drivers/hwmon/pmbus/Makefile | 1 +
> drivers/hwmon/pmbus/ir35215.c | 91 ++++++++++++++++++++++++++++++++
> 4 files changed, 200 insertions(+)
> create mode 100644 Documentation/hwmon/ir35215
> create mode 100644 drivers/hwmon/pmbus/ir35215.c
>
> diff --git a/Documentation/hwmon/ir35215 b/Documentation/hwmon/ir35215
> new file mode 100644
> index 000000000000..b77481340006
> --- /dev/null
> +++ b/Documentation/hwmon/ir35215
> @@ -0,0 +1,98 @@
> +Kernel driver ir35215
> +=====================
> +
> +Supported chips:
> + * Infinion IR35215
> + Prefix: 'ir35215'
> + Addresses scanned: -
> + Datasheet: Datasheet is not publicly available.
> + * Infinion IR3570B
> + Prefix: 'ir3570b'
> + Addresses scanned: -
> + Datasheet: Datasheet is not publicly available.
> +
> +Author: Ruslan Babayev <ruslan@babayev.com>
> +
> +
> +Description
> +-----------
> +
> +IR35215 is a Dual Output Digital Multi-Phase Controller
> +
> +
> +Usage Notes
> +-----------
> +
> +This driver does not probe for PMBus devices. You will have to instantiate
> +devices explicitly.
> +
> +Example: the following commands will load the driver for an IR35215
> +at address 0x40 on I2C bus #4:
> +
> +# modprobe ir35215
> +# echo ir35215 0x40 > /sys/bus/i2c/devices/i2c-4/new_device
> +
> +
> +Sysfs attributes
> +----------------
> +
> +curr[1-2]_label "iin[1-2]"
> +curr[1-2]_input Measured input current
> +curr[1-2]_max Maximum current for
> +curr[1-2]_max_alarm Current high alarm
> +
> +curr[3-4]_label "iout[3-4]"
> +curr[3-4]_input Measured output current
> +curr[3-4]_crit Critical maximum current
> +curr[3-4]_crit_alarm Current critical high alarm
> +curr[3-4]_highest Highest output current
> +curr[3-4]_lowest Lowest output current
> +curr[3-4]_max Maximum current
> +curr[3-4]_max_alarm Current high alarm
> +
> +in[1-2]_label "vin[1-2]"
> +in[1-2]_input Measured input voltage
> +in[1-2]_crit Critical maximum input voltage
> +in[1-2]_crit_alarm Input voltage critical high alarm
> +in[1-2]_highest Highest input voltage
> +in[1-2]_lowest Lowest input voltage
> +in[1-2]_min Minimum input voltage
> +in[1-2]_min_alarm Input voltage low alarm
> +
> +in[3-4]_label "vout[3-4]"
> +in[3-4]_input Measured output voltage
> +in[3-4]_lcrit Critical minimum output voltage
> +in[3-4]_lcrit_alarm Output voltage critical low alarm
> +in[3-4]_crit Critical maximum output voltage
> +in[3-4]_crit_alarm Output voltage critical high alarm
> +in[3-4]_highest Highest output voltage
> +in[3-4]_lowest Lowest output voltage
> +in[3-4]_max Maximum output voltage
> +in[3-4]_max_alarm Output voltage high alarm
> +in[3-4]_min Minimum output voltage
> +in[3-4]_min_alarm Output voltage low alarm
> +
> +out[1-2]_command Commanded output voltage
> +out[1-2]_margin_high Output voltage when operation is set to "high"
> +out[1-2]_margin_low Output voltage when operation is set to "low"
> +out[1-2]_max Maximum output voltage
> +out[1-2]_max_alarm Output voltage high alarm
> +out[1-2]_operation Set the output voltage to "on", "off", "high" or "low"
> +
> +power1_label "pin"
> +power1_input Measured input power
> +power1_alarm Input power high alarm
> +power1_max Input power limit
> +
> +power[2-3]_label "pout[1-2]"
> +power[2-3]_input Measured output power
> +power[2-3]_max Output power limit
> +power[2-3]_max_alarm Output power high alarm
> +
> +temp[1-2]_input Measured temperature
> +temp[1-2]_crit Critical high temperature
> +temp[1-2]_crit_alarm Chip temperature critical high alarm
> +temp[1-2]_highest Highest temperature
> +temp[1-2]_lowest Lowest temperature
> +temp[1-2]_max Maximum temperature
> +temp[1-2]_max_alarm Chip temperature high alarm
> diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig
> index 629cb45f8557..a7dd5e606061 100644
> --- a/drivers/hwmon/pmbus/Kconfig
> +++ b/drivers/hwmon/pmbus/Kconfig
> @@ -45,6 +45,16 @@ config SENSORS_IBM_CFFPS
> This driver can also be built as a module. If so, the module will
> be called ibm-cffps.
>
> +config SENSORS_IR35215
> + tristate "Infineon IR35215"
> + default n
> + help
> + If you say yes here you get hardware monitoring support for the
> + Infineon IR35215 controller.
> +
> + This driver can also be built as a module. If so, the module will
> + be called ir35215.
> +
> config SENSORS_IR35221
> tristate "Infineon IR35221"
> help
> diff --git a/drivers/hwmon/pmbus/Makefile b/drivers/hwmon/pmbus/Makefile
> index ea0e39518c21..48643e3bab42 100644
> --- a/drivers/hwmon/pmbus/Makefile
> +++ b/drivers/hwmon/pmbus/Makefile
> @@ -7,6 +7,7 @@ obj-$(CONFIG_PMBUS) += pmbus_core.o
> obj-$(CONFIG_SENSORS_PMBUS) += pmbus.o
> obj-$(CONFIG_SENSORS_ADM1275) += adm1275.o
> obj-$(CONFIG_SENSORS_IBM_CFFPS) += ibm-cffps.o
> +obj-$(CONFIG_SENSORS_IR35215) += ir35215.o
> obj-$(CONFIG_SENSORS_IR35221) += ir35221.o
> obj-$(CONFIG_SENSORS_LM25066) += lm25066.o
> obj-$(CONFIG_SENSORS_LTC2978) += ltc2978.o
> diff --git a/drivers/hwmon/pmbus/ir35215.c b/drivers/hwmon/pmbus/ir35215.c
> new file mode 100644
> index 000000000000..e50ee465a42d
> --- /dev/null
> +++ b/drivers/hwmon/pmbus/ir35215.c
> @@ -0,0 +1,91 @@
> +/*
> + * Hardware monitoring driver for Infineon IR35215
> + *
> + * Copyright (c) 2019 Cisco Systems.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/module.h>
> +#include <linux/init.h>
> +#include <linux/err.h>
> +#include <linux/i2c.h>
Alphabetic include file order please.
> +#include "pmbus.h"
> +
> +#define COMMON_FUNCS (PMBUS_HAVE_VIN | PMBUS_HAVE_VOUT \
> + | PMBUS_HAVE_IIN | PMBUS_HAVE_IOUT \
> + | PMBUS_HAVE_PIN | PMBUS_HAVE_POUT \
> + | PMBUS_HAVE_TEMP \
> + | PMBUS_HAVE_STATUS_VOUT \
> + | PMBUS_HAVE_STATUS_IOUT \
> + | PMBUS_HAVE_STATUS_INPUT \
> + | PMBUS_HAVE_STATUS_TEMP \
> + | PMBUS_HAVE_VOUT_MARGIN_HIGH \
> + | PMBUS_HAVE_VOUT_MARGIN_LOW \
> + )
> +
> +
> +#define IR35215_FUNCS (COMMON_FUNCS | PMBUS_HAVE_VOUT_COMMAND)
> +#define IR3570B_FUNCS (COMMON_FUNCS | PMBUS_HAVE_TEMP2)
> +
> +static struct pmbus_driver_info ir35215_info = {
> + .pages = 2,
> + .format[PSC_VOLTAGE_IN] = linear,
> + .format[PSC_VOLTAGE_OUT] = linear,
> + .format[PSC_CURRENT_IN] = linear,
> + .format[PSC_CURRENT_OUT] = linear,
> + .format[PSC_POWER] = linear,
> + .format[PSC_TEMPERATURE] = linear,
> + .func[0] = IR35215_FUNCS,
> + .func[1] = IR35215_FUNCS,
> +};
> +
> +static struct pmbus_driver_info ir3570b_info = {
> + .pages = 1,
Either this ...
> + .format[PSC_VOLTAGE_IN] = linear,
> + .format[PSC_VOLTAGE_OUT] = linear,
> + .format[PSC_CURRENT_IN] = linear,
> + .format[PSC_CURRENT_OUT] = linear,
> + .format[PSC_POWER] = linear,
> + .format[PSC_TEMPERATURE] = linear,
> + .func[0] = IR3570B_FUNCS,
> + .func[1] = IR3570B_FUNCS,
... or this is wrong.
> +};
> +
> +static int ir35215_probe(struct i2c_client *client,
> + const struct i2c_device_id *id)
> +{
> + return pmbus_do_probe(client, id, (void *)id->driver_data);
> +}
> +
> +static const struct i2c_device_id ir35215_id[] = {
> + {"ir35215", (kernel_ulong_t)&ir35215_info},
> + {"ir3570b", (kernel_ulong_t)&ir3570b_info},
> + {}
> +};
> +
> +MODULE_DEVICE_TABLE(i2c, ir35215_id);
> +
> +static struct i2c_driver ir35215_driver = {
> + .driver = {
> + .name = "ir35215",
> + },
> + .probe = ir35215_probe,
> + .remove = pmbus_do_remove,
> + .id_table = ir35215_id,
> +};
> +
> +module_i2c_driver(ir35215_driver);
> +
> +MODULE_AUTHOR("Ruslan Babayev <ruslan@babayev.com>");
> +MODULE_DESCRIPTION("PMBus driver for IR35215");
> +MODULE_LICENSE("GPL");
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 3/4] hwmon: (pmbus) add support for IR35215
@ 2019-04-16 18:36 Ruslan Babayev
2019-04-16 20:16 ` Guenter Roeck
0 siblings, 1 reply; 4+ messages in thread
From: Ruslan Babayev @ 2019-04-16 18:36 UTC (permalink / raw)
To: Jean Delvare, Guenter Roeck
Cc: xe-linux-external, Jonathan Corbet, linux-hwmon, linux-doc, linux-kernel
Provide support for Infineon Dual Output Digital Multi-Phase Buck
Controllers IR35215 and IR3570B.
Cc: xe-linux-external@cisco.com
Signed-off-by: Ruslan Babayev <ruslan@babayev.com>
---
Documentation/hwmon/ir35215 | 98 +++++++++++++++++++++++++++++++++++
drivers/hwmon/pmbus/Kconfig | 10 ++++
drivers/hwmon/pmbus/Makefile | 1 +
drivers/hwmon/pmbus/ir35215.c | 91 ++++++++++++++++++++++++++++++++
4 files changed, 200 insertions(+)
create mode 100644 Documentation/hwmon/ir35215
create mode 100644 drivers/hwmon/pmbus/ir35215.c
diff --git a/Documentation/hwmon/ir35215 b/Documentation/hwmon/ir35215
new file mode 100644
index 000000000000..b77481340006
--- /dev/null
+++ b/Documentation/hwmon/ir35215
@@ -0,0 +1,98 @@
+Kernel driver ir35215
+=====================
+
+Supported chips:
+ * Infinion IR35215
+ Prefix: 'ir35215'
+ Addresses scanned: -
+ Datasheet: Datasheet is not publicly available.
+ * Infinion IR3570B
+ Prefix: 'ir3570b'
+ Addresses scanned: -
+ Datasheet: Datasheet is not publicly available.
+
+Author: Ruslan Babayev <ruslan@babayev.com>
+
+
+Description
+-----------
+
+IR35215 is a Dual Output Digital Multi-Phase Controller
+
+
+Usage Notes
+-----------
+
+This driver does not probe for PMBus devices. You will have to instantiate
+devices explicitly.
+
+Example: the following commands will load the driver for an IR35215
+at address 0x40 on I2C bus #4:
+
+# modprobe ir35215
+# echo ir35215 0x40 > /sys/bus/i2c/devices/i2c-4/new_device
+
+
+Sysfs attributes
+----------------
+
+curr[1-2]_label "iin[1-2]"
+curr[1-2]_input Measured input current
+curr[1-2]_max Maximum current for
+curr[1-2]_max_alarm Current high alarm
+
+curr[3-4]_label "iout[3-4]"
+curr[3-4]_input Measured output current
+curr[3-4]_crit Critical maximum current
+curr[3-4]_crit_alarm Current critical high alarm
+curr[3-4]_highest Highest output current
+curr[3-4]_lowest Lowest output current
+curr[3-4]_max Maximum current
+curr[3-4]_max_alarm Current high alarm
+
+in[1-2]_label "vin[1-2]"
+in[1-2]_input Measured input voltage
+in[1-2]_crit Critical maximum input voltage
+in[1-2]_crit_alarm Input voltage critical high alarm
+in[1-2]_highest Highest input voltage
+in[1-2]_lowest Lowest input voltage
+in[1-2]_min Minimum input voltage
+in[1-2]_min_alarm Input voltage low alarm
+
+in[3-4]_label "vout[3-4]"
+in[3-4]_input Measured output voltage
+in[3-4]_lcrit Critical minimum output voltage
+in[3-4]_lcrit_alarm Output voltage critical low alarm
+in[3-4]_crit Critical maximum output voltage
+in[3-4]_crit_alarm Output voltage critical high alarm
+in[3-4]_highest Highest output voltage
+in[3-4]_lowest Lowest output voltage
+in[3-4]_max Maximum output voltage
+in[3-4]_max_alarm Output voltage high alarm
+in[3-4]_min Minimum output voltage
+in[3-4]_min_alarm Output voltage low alarm
+
+out[1-2]_command Commanded output voltage
+out[1-2]_margin_high Output voltage when operation is set to "high"
+out[1-2]_margin_low Output voltage when operation is set to "low"
+out[1-2]_max Maximum output voltage
+out[1-2]_max_alarm Output voltage high alarm
+out[1-2]_operation Set the output voltage to "on", "off", "high" or "low"
+
+power1_label "pin"
+power1_input Measured input power
+power1_alarm Input power high alarm
+power1_max Input power limit
+
+power[2-3]_label "pout[1-2]"
+power[2-3]_input Measured output power
+power[2-3]_max Output power limit
+power[2-3]_max_alarm Output power high alarm
+
+temp[1-2]_input Measured temperature
+temp[1-2]_crit Critical high temperature
+temp[1-2]_crit_alarm Chip temperature critical high alarm
+temp[1-2]_highest Highest temperature
+temp[1-2]_lowest Lowest temperature
+temp[1-2]_max Maximum temperature
+temp[1-2]_max_alarm Chip temperature high alarm
diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig
index 629cb45f8557..a7dd5e606061 100644
--- a/drivers/hwmon/pmbus/Kconfig
+++ b/drivers/hwmon/pmbus/Kconfig
@@ -45,6 +45,16 @@ config SENSORS_IBM_CFFPS
This driver can also be built as a module. If so, the module will
be called ibm-cffps.
+config SENSORS_IR35215
+ tristate "Infineon IR35215"
+ default n
+ help
+ If you say yes here you get hardware monitoring support for the
+ Infineon IR35215 controller.
+
+ This driver can also be built as a module. If so, the module will
+ be called ir35215.
+
config SENSORS_IR35221
tristate "Infineon IR35221"
help
diff --git a/drivers/hwmon/pmbus/Makefile b/drivers/hwmon/pmbus/Makefile
index ea0e39518c21..48643e3bab42 100644
--- a/drivers/hwmon/pmbus/Makefile
+++ b/drivers/hwmon/pmbus/Makefile
@@ -7,6 +7,7 @@ obj-$(CONFIG_PMBUS) += pmbus_core.o
obj-$(CONFIG_SENSORS_PMBUS) += pmbus.o
obj-$(CONFIG_SENSORS_ADM1275) += adm1275.o
obj-$(CONFIG_SENSORS_IBM_CFFPS) += ibm-cffps.o
+obj-$(CONFIG_SENSORS_IR35215) += ir35215.o
obj-$(CONFIG_SENSORS_IR35221) += ir35221.o
obj-$(CONFIG_SENSORS_LM25066) += lm25066.o
obj-$(CONFIG_SENSORS_LTC2978) += ltc2978.o
diff --git a/drivers/hwmon/pmbus/ir35215.c b/drivers/hwmon/pmbus/ir35215.c
new file mode 100644
index 000000000000..e50ee465a42d
--- /dev/null
+++ b/drivers/hwmon/pmbus/ir35215.c
@@ -0,0 +1,91 @@
+/*
+ * Hardware monitoring driver for Infineon IR35215
+ *
+ * Copyright (c) 2019 Cisco Systems.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/init.h>
+#include <linux/err.h>
+#include <linux/i2c.h>
+#include "pmbus.h"
+
+#define COMMON_FUNCS (PMBUS_HAVE_VIN | PMBUS_HAVE_VOUT \
+ | PMBUS_HAVE_IIN | PMBUS_HAVE_IOUT \
+ | PMBUS_HAVE_PIN | PMBUS_HAVE_POUT \
+ | PMBUS_HAVE_TEMP \
+ | PMBUS_HAVE_STATUS_VOUT \
+ | PMBUS_HAVE_STATUS_IOUT \
+ | PMBUS_HAVE_STATUS_INPUT \
+ | PMBUS_HAVE_STATUS_TEMP \
+ | PMBUS_HAVE_VOUT_MARGIN_HIGH \
+ | PMBUS_HAVE_VOUT_MARGIN_LOW \
+ )
+
+
+#define IR35215_FUNCS (COMMON_FUNCS | PMBUS_HAVE_VOUT_COMMAND)
+#define IR3570B_FUNCS (COMMON_FUNCS | PMBUS_HAVE_TEMP2)
+
+static struct pmbus_driver_info ir35215_info = {
+ .pages = 2,
+ .format[PSC_VOLTAGE_IN] = linear,
+ .format[PSC_VOLTAGE_OUT] = linear,
+ .format[PSC_CURRENT_IN] = linear,
+ .format[PSC_CURRENT_OUT] = linear,
+ .format[PSC_POWER] = linear,
+ .format[PSC_TEMPERATURE] = linear,
+ .func[0] = IR35215_FUNCS,
+ .func[1] = IR35215_FUNCS,
+};
+
+static struct pmbus_driver_info ir3570b_info = {
+ .pages = 1,
+ .format[PSC_VOLTAGE_IN] = linear,
+ .format[PSC_VOLTAGE_OUT] = linear,
+ .format[PSC_CURRENT_IN] = linear,
+ .format[PSC_CURRENT_OUT] = linear,
+ .format[PSC_POWER] = linear,
+ .format[PSC_TEMPERATURE] = linear,
+ .func[0] = IR3570B_FUNCS,
+ .func[1] = IR3570B_FUNCS,
+};
+
+static int ir35215_probe(struct i2c_client *client,
+ const struct i2c_device_id *id)
+{
+ return pmbus_do_probe(client, id, (void *)id->driver_data);
+}
+
+static const struct i2c_device_id ir35215_id[] = {
+ {"ir35215", (kernel_ulong_t)&ir35215_info},
+ {"ir3570b", (kernel_ulong_t)&ir3570b_info},
+ {}
+};
+
+MODULE_DEVICE_TABLE(i2c, ir35215_id);
+
+static struct i2c_driver ir35215_driver = {
+ .driver = {
+ .name = "ir35215",
+ },
+ .probe = ir35215_probe,
+ .remove = pmbus_do_remove,
+ .id_table = ir35215_id,
+};
+
+module_i2c_driver(ir35215_driver);
+
+MODULE_AUTHOR("Ruslan Babayev <ruslan@babayev.com>");
+MODULE_DESCRIPTION("PMBus driver for IR35215");
+MODULE_LICENSE("GPL");
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-04-17 2:05 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-16 18:45 [PATCH 3/4] hwmon: (pmbus) add support for IR35215 Ruslan Babayev
-- strict thread matches above, loose matches on Subject: below --
2019-04-16 18:36 Ruslan Babayev
2019-04-16 20:16 ` Guenter Roeck
2019-04-17 2:04 ` Ruslan Babayev
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).