* [PATCH 1/3] dt-bindings: hwmon: Add TI TPS23861 bindings
@ 2020-12-18 13:03 Robert Marko
2020-12-18 13:03 ` [PATCH 2/3] hwmon: add Texas Instruments TPS23861 driver Robert Marko
2020-12-18 13:03 ` [PATCH 3/3] MAINTAINERS: Add entry for Texas Instruments TPS23861 PoE PSE Robert Marko
0 siblings, 2 replies; 5+ messages in thread
From: Robert Marko @ 2020-12-18 13:03 UTC (permalink / raw)
To: jdelvare, linux, robh+dt, linux-hwmon, devicetree, linux-kernel
Cc: Robert Marko, Luka Perkov
Document bindings for the Texas Instruments TPS23861 driver.
Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Cc: Luka Perkov <luka.perkov@sartura.hr>
---
.../bindings/hwmon/ti,tps23861.yaml | 53 +++++++++++++++++++
1 file changed, 53 insertions(+)
create mode 100644 Documentation/devicetree/bindings/hwmon/ti,tps23861.yaml
diff --git a/Documentation/devicetree/bindings/hwmon/ti,tps23861.yaml b/Documentation/devicetree/bindings/hwmon/ti,tps23861.yaml
new file mode 100644
index 000000000000..a7a801f54398
--- /dev/null
+++ b/Documentation/devicetree/bindings/hwmon/ti,tps23861.yaml
@@ -0,0 +1,53 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+
+$id: http://devicetree.org/schemas/hwmon/ti,tps23861.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: TI TPS23861 PoE PSE
+
+maintainers:
+ - Robert Marko <robert.marko@sartura.hr>
+
+description: |
+ The TPS23861 is a IEEE 802.3at Quad Port Power-over-Ethernet PSE Controller.
+
+ Datasheets:
+ https://www.ti.com/lit/gpn/tps23861
+
+
+properties:
+ compatible:
+ enum:
+ - ti,tps23861
+
+ reg:
+ maxItems: 1
+
+ shunt-resistor-micro-ohms:
+ description: |
+ The value of curent sense resistor in microohms.
+ items:
+ default: 255000
+ minimum: 250000
+ maximum: 255000
+
+required:
+ - compatible
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ tps23861@30 {
+ compatible = "ti,tps23861";
+ reg = <0x30>;
+ shunt-resistor-micro-ohms = <255000>;
+ };
+ };
--
2.29.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/3] hwmon: add Texas Instruments TPS23861 driver
2020-12-18 13:03 [PATCH 1/3] dt-bindings: hwmon: Add TI TPS23861 bindings Robert Marko
@ 2020-12-18 13:03 ` Robert Marko
2020-12-18 14:50 ` Guenter Roeck
2020-12-18 13:03 ` [PATCH 3/3] MAINTAINERS: Add entry for Texas Instruments TPS23861 PoE PSE Robert Marko
1 sibling, 1 reply; 5+ messages in thread
From: Robert Marko @ 2020-12-18 13:03 UTC (permalink / raw)
To: jdelvare, linux, robh+dt, linux-hwmon, devicetree, linux-kernel
Cc: Robert Marko, Luka Perkov
Add basic monitoring support as well as port on/off control for Texas
Instruments TPS23861 PoE PSE IC.
Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Cc: Luka Perkov <luka.perkov@sartura.hr>
---
drivers/hwmon/Kconfig | 11 ++
drivers/hwmon/Makefile | 1 +
drivers/hwmon/tps23861.c | 398 +++++++++++++++++++++++++++++++++++++++
3 files changed, 410 insertions(+)
create mode 100644 drivers/hwmon/tps23861.c
diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
index a850e4f0e0bd..3368ecfa7a9c 100644
--- a/drivers/hwmon/Kconfig
+++ b/drivers/hwmon/Kconfig
@@ -1102,6 +1102,17 @@ config SENSORS_TC654
This driver can also be built as a module. If so, the module
will be called tc654.
+config SENSORS_TPS23861
+ tristate "Texas Instruments TPS23861 PoE PSE"
+ depends on I2C
+ select REGMAP_I2C
+ help
+ If you say yes here you get support for Texas Instruments
+ TPS23861 802.3at PoE PSE chips.
+
+ This driver can also be built as a module. If so, the module
+ will be called tps23861.
+
config SENSORS_MENF21BMC_HWMON
tristate "MEN 14F021P00 BMC Hardware Monitoring"
depends on MFD_MENF21BMC
diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile
index 9db2903b61e5..7493e9d5dc43 100644
--- a/drivers/hwmon/Makefile
+++ b/drivers/hwmon/Makefile
@@ -141,6 +141,7 @@ obj-$(CONFIG_SENSORS_MAX31790) += max31790.o
obj-$(CONFIG_SENSORS_MC13783_ADC)+= mc13783-adc.o
obj-$(CONFIG_SENSORS_MCP3021) += mcp3021.o
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_MR75203) += mr75203.o
diff --git a/drivers/hwmon/tps23861.c b/drivers/hwmon/tps23861.c
new file mode 100644
index 000000000000..e8b614267084
--- /dev/null
+++ b/drivers/hwmon/tps23861.c
@@ -0,0 +1,398 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Driver for the TI TPS23861 PoE PSE.
+ *
+ * Author: Robert Marko <robert.marko@sartura.hr>
+ */
+
+#include <linux/module.h>
+#include <linux/of_device.h>
+#include <linux/hwmon.h>
+#include <linux/hwmon-sysfs.h>
+#include <linux/i2c.h>
+#include <linux/regmap.h>
+#include <linux/bitfield.h>
+#include <linux/delay.h>
+
+#define FW_VERSION 0x41
+#define TEMPERATURE 0x2c
+#define INPUT_VOLTAGE_LSB 0x2e
+#define INPUT_VOLTAGE_MSB 0x2f
+#define PORT_1_CURRENT_LSB 0x30
+#define PORT_1_CURRENT_MSB 0x31
+#define PORT_1_VOLTAGE_LSB 0x32
+#define PORT_1_VOLTAGE_MSB 0x33
+#define PORT_2_CURRENT_LSB 0x34
+#define PORT_2_CURRENT_MSB 0x35
+#define PORT_2_VOLTAGE_LSB 0x36
+#define PORT_2_VOLTAGE_MSB 0x37
+#define PORT_3_CURRENT_LSB 0x38
+#define PORT_3_CURRENT_MSB 0x39
+#define PORT_3_VOLTAGE_LSB 0x3a
+#define PORT_3_VOLTAGE_MSB 0x3b
+#define PORT_4_CURRENT_LSB 0x3c
+#define PORT_4_CURRENT_MSB 0x3d
+#define PORT_4_VOLTAGE_LSB 0x3e
+#define PORT_4_VOLTAGE_MSB 0x3f
+#define VOLTAGE_CURRENT_MASK GENMASK(13, 0)
+#define PORT_1_STATUS 0x0c
+#define PORT_2_STATUS 0x0d
+#define PORT_3_STATUS 0x0e
+#define PORT_4_STATUS 0x0f
+#define PORT_STATUS_CLASS_MASK GENMASK(7, 4)
+#define PORT_STATUS_DETECT_MASK GENMASK(3, 0)
+#define PORT_CLASS_UNKNOWN 0
+#define PORT_CLASS_1 1
+#define PORT_CLASS_2 2
+#define PORT_CLASS_3 3
+#define PORT_CLASS_4 4
+#define PORT_CLASS_RESERVED 5
+#define PORT_CLASS_0 6
+#define PORT_CLASS_OVERCURRENT 7
+#define PORT_CLASS_MISMATCH 8
+#define PORT_DETECT_UNKNOWN 0
+#define PORT_DETECT_SHORT 1
+#define PORT_DETECT_RESERVED 2
+#define PORT_DETECT_RESISTANCE_LOW 3
+#define PORT_DETECT_RESISTANCE_OK 4
+#define PORT_DETECT_RESISTANCE_HIGH 5
+#define PORT_DETECT_OPEN_CIRCUIT 6
+#define PORT_DETECT_RESERVED_2 7
+#define PORT_DETECT_MOSFET_FAULT 8
+#define PORT_DETECT_LEGACY 9
+/* Measurment beyond clamp voltage */
+#define PORT_DETECT_CAPACITANCE_INVALID_BEYOND 10
+/* Insufficient voltage delta */
+#define PORT_DETECT_CAPACITANCE_INVALID_DELTA 11
+#define PORT_DETECT_CAPACITANCE_OUT_OF_RANGE 12
+#define POE_PLUS 0x40
+
+#define DETECT_CLASS_RESTART 0x18
+#define POWER_ENABLE 0x19
+
+#define TEMPERATURE_LSB 652 /* 0.652 degrees Celsius */
+#define VOLTAGE_LSB 3662 /* 3.662 mV */
+#define SHUNT_RESISTOR_DEFAULT 255000 /* 255 mOhm */
+#define CURRENT_LSB_255 62260 /* 62.260 uA */
+#define CURRENT_LSB_250 61039 /* 61.039 uA */
+
+struct tps23861_data {
+ struct regmap *regmap;
+ u32 shunt_resistor;
+};
+
+static struct regmap_config tps23861_regmap_config = {
+ .reg_bits = 8,
+ .val_bits = 8,
+};
+
+static ssize_t tps23861_simple_value_show(struct device *dev,
+ struct device_attribute *da, char *buf)
+{
+ struct sensor_device_attribute *attr = to_sensor_dev_attr(da);
+ struct tps23861_data *data = dev_get_drvdata(dev);
+ unsigned int regval;
+
+ regmap_read(data->regmap, attr->index, ®val);
+
+ return snprintf(buf, PAGE_SIZE, "%d\n", regval);
+}
+
+static ssize_t tps23861_temperature_show(struct device *dev,
+ struct device_attribute *da, char *buf)
+{
+ struct sensor_device_attribute *attr = to_sensor_dev_attr(da);
+ struct tps23861_data *data = dev_get_drvdata(dev);
+ unsigned int regval;
+ int temperature;
+
+ regmap_read(data->regmap, attr->index, ®val);
+
+ temperature = (regval * TEMPERATURE_LSB) - 20000;
+
+ return snprintf(buf, PAGE_SIZE, "%d\n", temperature);
+}
+
+static ssize_t tps23861_voltage_show(struct device *dev,
+ struct device_attribute *da, char *buf)
+{
+ struct sensor_device_attribute *attr = to_sensor_dev_attr(da);
+ struct tps23861_data *data = dev_get_drvdata(dev);
+ u16 regval;
+ unsigned int voltage;
+
+ regmap_bulk_read(data->regmap, attr->index, ®val, 2);
+
+ voltage = (FIELD_GET(VOLTAGE_CURRENT_MASK, regval) * VOLTAGE_LSB) / 1000;
+
+ return snprintf(buf, PAGE_SIZE, "%u\n", voltage);
+}
+
+static ssize_t tps23861_current_show(struct device *dev,
+ struct device_attribute *da, char *buf)
+{
+ struct sensor_device_attribute *attr = to_sensor_dev_attr(da);
+ struct tps23861_data *data = dev_get_drvdata(dev);
+ u16 regval;
+ unsigned int current_mA;
+ unsigned int current_lsb;
+
+ if (data->shunt_resistor == SHUNT_RESISTOR_DEFAULT)
+ current_lsb = CURRENT_LSB_255;
+ else
+ current_lsb = CURRENT_LSB_250;
+
+ regmap_bulk_read(data->regmap, attr->index, ®val, 2);
+
+ current_mA = (FIELD_GET(VOLTAGE_CURRENT_MASK, regval) * current_lsb) / 1000000;
+
+ return snprintf(buf, PAGE_SIZE, "%u\n", current_mA);
+}
+
+static ssize_t tps23861_port_status_show(struct device *dev,
+ struct device_attribute *da, char *buf)
+{
+ struct sensor_device_attribute *attr = to_sensor_dev_attr(da);
+ struct tps23861_data *data = dev_get_drvdata(dev);
+ char *detect_status, *class_status;
+ unsigned int regval;
+ u8 class, detect;
+
+ regmap_read(data->regmap, attr->index, ®val);
+ detect = FIELD_GET(PORT_STATUS_DETECT_MASK, regval);
+ class = FIELD_GET(PORT_STATUS_CLASS_MASK, regval);
+
+ switch (detect) {
+ case PORT_DETECT_UNKNOWN:
+ detect_status = "Unknown device";
+ break;
+ case PORT_DETECT_SHORT:
+ detect_status = "Short circuit detected";
+ break;
+ case PORT_DETECT_RESISTANCE_LOW:
+ detect_status = "Too low resistance detected";
+ break;
+ case PORT_DETECT_RESISTANCE_OK:
+ detect_status = "Valid resistance detected";
+ break;
+ case PORT_DETECT_RESISTANCE_HIGH:
+ detect_status = "Too high resistance detected";
+ break;
+ case PORT_DETECT_OPEN_CIRCUIT:
+ detect_status = "Open circuit detected";
+ break;
+ case PORT_DETECT_MOSFET_FAULT:
+ detect_status = "MOSFET fault detected";
+ break;
+ case PORT_DETECT_LEGACY:
+ detect_status = "Legacy device detected";
+ break;
+ case PORT_DETECT_CAPACITANCE_INVALID_BEYOND:
+ detect_status = "Invalid capacitance, beyond clamp voltage";
+ break;
+ case PORT_DETECT_CAPACITANCE_INVALID_DELTA:
+ detect_status = "Invalid capacitance, insufficient voltage delta";
+ break;
+ case PORT_DETECT_CAPACITANCE_OUT_OF_RANGE:
+ detect_status = "Valid capacitance, outside of legacy range";
+ break;
+ case PORT_DETECT_RESERVED:
+ case PORT_DETECT_RESERVED_2:
+ default:
+ return -EINVAL;
+ }
+
+ switch (class) {
+ case PORT_CLASS_UNKNOWN:
+ class_status = "Unknown class";
+ break;
+ case PORT_CLASS_RESERVED:
+ case PORT_CLASS_0:
+ class_status = "Class 0";
+ break;
+ case PORT_CLASS_1:
+ class_status = "Class 1";
+ break;
+ case PORT_CLASS_2:
+ class_status = "Class 2";
+ break;
+ case PORT_CLASS_3:
+ class_status = "Class 3";
+ break;
+ case PORT_CLASS_4:
+ class_status = "Class 4";
+ break;
+ case PORT_CLASS_OVERCURRENT:
+ class_status = "Overcurrent";
+ break;
+ case PORT_CLASS_MISMATCH:
+ class_status = "Class mismatch";
+ break;
+ default:
+ return -EINVAL;
+ }
+
+ return snprintf(buf, PAGE_SIZE, "%s, %s\n", detect_status, class_status);
+}
+
+static ssize_t tps23861_poe_plus_show(struct device *dev,
+ struct device_attribute *da, char *buf)
+{
+ struct sensor_device_attribute *attr = to_sensor_dev_attr(da);
+ struct tps23861_data *data = dev_get_drvdata(dev);
+ char *ports[4];
+ unsigned int regval;
+ int i;
+
+ regmap_read(data->regmap, attr->index, ®val);
+
+ for (i = 0; i < 4; i++) {
+ if (BIT(i + 4) & regval)
+ ports[i] = "Yes";
+ else
+ ports[i] = "No";
+ }
+
+ return snprintf(buf, PAGE_SIZE, "Port 1: %s\nPort 2: %s\nPort 3: %s\nPort 4: %s\n",
+ ports[0], ports[1], ports[2], ports[3]);
+}
+
+static ssize_t tps23861_port_disable_store(struct device *dev,
+ struct device_attribute *da,
+ const char *buf, size_t count)
+{
+ struct sensor_device_attribute *attr = to_sensor_dev_attr(da);
+ struct tps23861_data *data = dev_get_drvdata(dev);
+ unsigned int regval, port;
+ int ret;
+
+ ret = kstrtouint(buf, 0, &port);
+ if (ret < 0)
+ return ret;
+
+ regmap_read(data->regmap, attr->index, ®val);
+ regval |= BIT(port + 3);
+ regmap_write(data->regmap, attr->index, regval);
+
+ return count;
+}
+
+static ssize_t tps23861_port_enable_store(struct device *dev,
+ struct device_attribute *da,
+ const char *buf, size_t count)
+{
+ struct sensor_device_attribute *attr = to_sensor_dev_attr(da);
+ struct tps23861_data *data = dev_get_drvdata(dev);
+ unsigned int regval, port;
+ int ret;
+
+ ret = kstrtouint(buf, 0, &port);
+ if (ret < 0)
+ return ret;
+
+ regmap_read(data->regmap, attr->index, ®val);
+ regval |= BIT(port - 1);
+ regval |= BIT(port + 3);
+ regmap_write(data->regmap, attr->index, regval);
+
+ return count;
+}
+
+static SENSOR_DEVICE_ATTR_RO(fw_ver, tps23861_simple_value, FW_VERSION);
+static SENSOR_DEVICE_ATTR_RO(temperature, tps23861_temperature, TEMPERATURE);
+static SENSOR_DEVICE_ATTR_RO(in_voltage, tps23861_voltage, INPUT_VOLTAGE_LSB);
+static SENSOR_DEVICE_ATTR_RO(port1_current, tps23861_current, PORT_1_CURRENT_LSB);
+static SENSOR_DEVICE_ATTR_RO(port1_voltage, tps23861_voltage, PORT_1_VOLTAGE_LSB);
+static SENSOR_DEVICE_ATTR_RO(port1_status, tps23861_port_status, PORT_1_STATUS);
+static SENSOR_DEVICE_ATTR_RO(port2_current, tps23861_current, PORT_2_CURRENT_LSB);
+static SENSOR_DEVICE_ATTR_RO(port2_voltage, tps23861_voltage, PORT_2_VOLTAGE_LSB);
+static SENSOR_DEVICE_ATTR_RO(port2_status, tps23861_port_status, PORT_2_STATUS);
+static SENSOR_DEVICE_ATTR_RO(port3_current, tps23861_current, PORT_3_CURRENT_LSB);
+static SENSOR_DEVICE_ATTR_RO(port3_voltage, tps23861_voltage, PORT_3_VOLTAGE_LSB);
+static SENSOR_DEVICE_ATTR_RO(port3_status, tps23861_port_status, PORT_3_STATUS);
+static SENSOR_DEVICE_ATTR_RO(port4_current, tps23861_current, PORT_4_CURRENT_LSB);
+static SENSOR_DEVICE_ATTR_RO(port4_voltage, tps23861_voltage, PORT_4_VOLTAGE_LSB);
+static SENSOR_DEVICE_ATTR_RO(port4_status, tps23861_port_status, PORT_4_STATUS);
+static SENSOR_DEVICE_ATTR_RO(poe_plus_status, tps23861_poe_plus, POE_PLUS);
+static SENSOR_DEVICE_ATTR_WO(port_disable, tps23861_port_disable, POWER_ENABLE);
+static SENSOR_DEVICE_ATTR_WO(port_enable, tps23861_port_enable, DETECT_CLASS_RESTART);
+
+static struct attribute *tps23861_attrs[] = {
+ &sensor_dev_attr_fw_ver.dev_attr.attr,
+ &sensor_dev_attr_temperature.dev_attr.attr,
+ &sensor_dev_attr_in_voltage.dev_attr.attr,
+ &sensor_dev_attr_port1_current.dev_attr.attr,
+ &sensor_dev_attr_port1_voltage.dev_attr.attr,
+ &sensor_dev_attr_port1_status.dev_attr.attr,
+ &sensor_dev_attr_port2_current.dev_attr.attr,
+ &sensor_dev_attr_port2_voltage.dev_attr.attr,
+ &sensor_dev_attr_port2_status.dev_attr.attr,
+ &sensor_dev_attr_port3_current.dev_attr.attr,
+ &sensor_dev_attr_port3_voltage.dev_attr.attr,
+ &sensor_dev_attr_port3_status.dev_attr.attr,
+ &sensor_dev_attr_port4_current.dev_attr.attr,
+ &sensor_dev_attr_port4_voltage.dev_attr.attr,
+ &sensor_dev_attr_port4_status.dev_attr.attr,
+ &sensor_dev_attr_poe_plus_status.dev_attr.attr,
+ &sensor_dev_attr_port_disable.dev_attr.attr,
+ &sensor_dev_attr_port_enable.dev_attr.attr,
+ NULL,
+};
+
+static const struct attribute_group tps23861_group = {
+ .attrs = tps23861_attrs,
+};
+
+static const struct attribute_group *tps23861_groups[] = {
+ &tps23861_group,
+ NULL,
+};
+
+static int tps23861_probe(struct i2c_client *client)
+{
+ struct device *dev = &client->dev;
+ struct tps23861_data *data;
+ struct device *hwmon_dev;
+ u32 shunt_resistor;
+
+ data = devm_kzalloc(dev, sizeof(*data), GFP_KERNEL);
+ if (!data)
+ return -ENOMEM;
+
+ data->regmap = devm_regmap_init_i2c(client, &tps23861_regmap_config);
+ if (IS_ERR(data->regmap)) {
+ dev_err(dev, "failed to allocate register map\n");
+ return PTR_ERR(data->regmap);
+ }
+
+ if (of_property_read_u32(dev->of_node, "shunt-resistor-micro-ohms", &shunt_resistor) < 0)
+ data->shunt_resistor = shunt_resistor;
+ else
+ data->shunt_resistor = SHUNT_RESISTOR_DEFAULT;
+
+ hwmon_dev = devm_hwmon_device_register_with_groups(dev, client->name,
+ data, tps23861_groups);
+ if (IS_ERR(hwmon_dev))
+ return PTR_ERR(hwmon_dev);
+
+ return 0;
+}
+
+static const struct of_device_id __maybe_unused tps23861_of_match[] = {
+ { .compatible = "ti,tps23861", },
+ { },
+};
+MODULE_DEVICE_TABLE(of, tps23861_of_match);
+
+static struct i2c_driver tps23861_driver = {
+ .probe_new = tps23861_probe,
+ .driver = {
+ .name = "tps23861",
+ .of_match_table = of_match_ptr(tps23861_of_match),
+ },
+};
+module_i2c_driver(tps23861_driver);
+
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Robert Marko <robert.marko@sartura.hr>");
+MODULE_DESCRIPTION("TI TPS23861 PoE PSE");
--
2.29.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 3/3] MAINTAINERS: Add entry for Texas Instruments TPS23861 PoE PSE
2020-12-18 13:03 [PATCH 1/3] dt-bindings: hwmon: Add TI TPS23861 bindings Robert Marko
2020-12-18 13:03 ` [PATCH 2/3] hwmon: add Texas Instruments TPS23861 driver Robert Marko
@ 2020-12-18 13:03 ` Robert Marko
1 sibling, 0 replies; 5+ messages in thread
From: Robert Marko @ 2020-12-18 13:03 UTC (permalink / raw)
To: jdelvare, linux, robh+dt, linux-hwmon, devicetree, linux-kernel
Cc: Robert Marko, Luka Perkov
Add maintainers entry for the Texas Instruments TPS23861 PoE PSE driver.
Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Cc: Luka Perkov <luka.perkov@sartura.hr>
---
MAINTAINERS | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 830244166a7c..5441be7a5c26 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17336,6 +17336,14 @@ F: include/dt-bindings/soc/ti,sci_pm_domain.h
F: include/linux/soc/ti/ti_sci_inta_msi.h
F: include/linux/soc/ti/ti_sci_protocol.h
+TEXAS INSTRUMENTS TPS23861 PoE PSE DRIVER
+M: Robert Marko <robert.marko@sartura.hr>
+M: Luka Perkov <luka.perkov@sartura.hr>
+L: linux-hwmon@vger.kernel.org
+S: Maintained
+F: Documentation/devicetree/bindings/hwmon/ti,tps23861.yaml
+F: drivers/hwmon/tps23861.c
+
THANKO'S RAREMONO AM/FM/SW RADIO RECEIVER USB DRIVER
M: Hans Verkuil <hverkuil@xs4all.nl>
L: linux-media@vger.kernel.org
--
2.29.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/3] hwmon: add Texas Instruments TPS23861 driver
2020-12-18 13:03 ` [PATCH 2/3] hwmon: add Texas Instruments TPS23861 driver Robert Marko
@ 2020-12-18 14:50 ` Guenter Roeck
2020-12-19 19:14 ` Robert Marko
0 siblings, 1 reply; 5+ messages in thread
From: Guenter Roeck @ 2020-12-18 14:50 UTC (permalink / raw)
To: Robert Marko, jdelvare, robh+dt, linux-hwmon, devicetree, linux-kernel
Cc: Luka Perkov
On 12/18/20 5:03 AM, Robert Marko wrote:
> Add basic monitoring support as well as port on/off control for Texas
> Instruments TPS23861 PoE PSE IC.
>
> Signed-off-by: Robert Marko <robert.marko@sartura.hr>
> Cc: Luka Perkov <luka.perkov@sartura.hr>
> ---
> drivers/hwmon/Kconfig | 11 ++
> drivers/hwmon/Makefile | 1 +
> drivers/hwmon/tps23861.c | 398 +++++++++++++++++++++++++++++++++++++++
> 3 files changed, 410 insertions(+)
> create mode 100644 drivers/hwmon/tps23861.c
>
> diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
> index a850e4f0e0bd..3368ecfa7a9c 100644
> --- a/drivers/hwmon/Kconfig
> +++ b/drivers/hwmon/Kconfig
> @@ -1102,6 +1102,17 @@ config SENSORS_TC654
> This driver can also be built as a module. If so, the module
> will be called tc654.
>
> +config SENSORS_TPS23861
> + tristate "Texas Instruments TPS23861 PoE PSE"
> + depends on I2C
> + select REGMAP_I2C
> + help
> + If you say yes here you get support for Texas Instruments
> + TPS23861 802.3at PoE PSE chips.
> +
> + This driver can also be built as a module. If so, the module
> + will be called tps23861.
> +
> config SENSORS_MENF21BMC_HWMON
> tristate "MEN 14F021P00 BMC Hardware Monitoring"
> depends on MFD_MENF21BMC
> diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile
> index 9db2903b61e5..7493e9d5dc43 100644
> --- a/drivers/hwmon/Makefile
> +++ b/drivers/hwmon/Makefile
> @@ -141,6 +141,7 @@ obj-$(CONFIG_SENSORS_MAX31790) += max31790.o
> obj-$(CONFIG_SENSORS_MC13783_ADC)+= mc13783-adc.o
> obj-$(CONFIG_SENSORS_MCP3021) += mcp3021.o
> 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_MR75203) += mr75203.o
> diff --git a/drivers/hwmon/tps23861.c b/drivers/hwmon/tps23861.c
> new file mode 100644
> index 000000000000..e8b614267084
> --- /dev/null
> +++ b/drivers/hwmon/tps23861.c
> @@ -0,0 +1,398 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +/*
> + * Driver for the TI TPS23861 PoE PSE.
> + *
> + * Author: Robert Marko <robert.marko@sartura.hr>
> + */
> +
> +#include <linux/module.h>
> +#include <linux/of_device.h>
> +#include <linux/hwmon.h>
> +#include <linux/hwmon-sysfs.h>
> +#include <linux/i2c.h>
> +#include <linux/regmap.h>
> +#include <linux/bitfield.h>
> +#include <linux/delay.h>
Alphabetic include file order, please.
[ ... ]
> + hwmon_dev = devm_hwmon_device_register_with_groups(dev, client->name,
> + data, tps23861_groups);
Please rework to use devm_hwmon_device_register_with_info().
Thanks,
Guenter
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/3] hwmon: add Texas Instruments TPS23861 driver
2020-12-18 14:50 ` Guenter Roeck
@ 2020-12-19 19:14 ` Robert Marko
0 siblings, 0 replies; 5+ messages in thread
From: Robert Marko @ 2020-12-19 19:14 UTC (permalink / raw)
To: Guenter Roeck
Cc: jdelvare, robh+dt, linux-hwmon, devicetree, linux-kernel, Luka Perkov
On Fri, Dec 18, 2020 at 3:50 PM Guenter Roeck <linux@roeck-us.net> wrote:
>
> On 12/18/20 5:03 AM, Robert Marko wrote:
> > Add basic monitoring support as well as port on/off control for Texas
> > Instruments TPS23861 PoE PSE IC.
> >
> > Signed-off-by: Robert Marko <robert.marko@sartura.hr>
> > Cc: Luka Perkov <luka.perkov@sartura.hr>
> > ---
> > drivers/hwmon/Kconfig | 11 ++
> > drivers/hwmon/Makefile | 1 +
> > drivers/hwmon/tps23861.c | 398 +++++++++++++++++++++++++++++++++++++++
> > 3 files changed, 410 insertions(+)
> > create mode 100644 drivers/hwmon/tps23861.c
> >
> > diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
> > index a850e4f0e0bd..3368ecfa7a9c 100644
> > --- a/drivers/hwmon/Kconfig
> > +++ b/drivers/hwmon/Kconfig
> > @@ -1102,6 +1102,17 @@ config SENSORS_TC654
> > This driver can also be built as a module. If so, the module
> > will be called tc654.
> >
> > +config SENSORS_TPS23861
> > + tristate "Texas Instruments TPS23861 PoE PSE"
> > + depends on I2C
> > + select REGMAP_I2C
> > + help
> > + If you say yes here you get support for Texas Instruments
> > + TPS23861 802.3at PoE PSE chips.
> > +
> > + This driver can also be built as a module. If so, the module
> > + will be called tps23861.
> > +
> > config SENSORS_MENF21BMC_HWMON
> > tristate "MEN 14F021P00 BMC Hardware Monitoring"
> > depends on MFD_MENF21BMC
> > diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile
> > index 9db2903b61e5..7493e9d5dc43 100644
> > --- a/drivers/hwmon/Makefile
> > +++ b/drivers/hwmon/Makefile
> > @@ -141,6 +141,7 @@ obj-$(CONFIG_SENSORS_MAX31790) += max31790.o
> > obj-$(CONFIG_SENSORS_MC13783_ADC)+= mc13783-adc.o
> > obj-$(CONFIG_SENSORS_MCP3021) += mcp3021.o
> > 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_MR75203) += mr75203.o
> > diff --git a/drivers/hwmon/tps23861.c b/drivers/hwmon/tps23861.c
> > new file mode 100644
> > index 000000000000..e8b614267084
> > --- /dev/null
> > +++ b/drivers/hwmon/tps23861.c
> > @@ -0,0 +1,398 @@
> > +// SPDX-License-Identifier: GPL-2.0-or-later
> > +/*
> > + * Driver for the TI TPS23861 PoE PSE.
> > + *
> > + * Author: Robert Marko <robert.marko@sartura.hr>
> > + */
> > +
> > +#include <linux/module.h>
> > +#include <linux/of_device.h>
> > +#include <linux/hwmon.h>
> > +#include <linux/hwmon-sysfs.h>
> > +#include <linux/i2c.h>
> > +#include <linux/regmap.h>
> > +#include <linux/bitfield.h>
> > +#include <linux/delay.h>
>
> Alphabetic include file order, please.
Done in v2.
>
> [ ... ]
>
> > + hwmon_dev = devm_hwmon_device_register_with_groups(dev, client->name,
> > + data, tps23861_groups);
>
> Please rework to use devm_hwmon_device_register_with_info().
Done in v2.
Thanks for the remarks.
Regards,
Robert
>
> Thanks,
> Guenter
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-12-19 19:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-18 13:03 [PATCH 1/3] dt-bindings: hwmon: Add TI TPS23861 bindings Robert Marko
2020-12-18 13:03 ` [PATCH 2/3] hwmon: add Texas Instruments TPS23861 driver Robert Marko
2020-12-18 14:50 ` Guenter Roeck
2020-12-19 19:14 ` Robert Marko
2020-12-18 13:03 ` [PATCH 3/3] MAINTAINERS: Add entry for Texas Instruments TPS23861 PoE PSE Robert Marko
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.