lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <6esdq4e4r7vmxwlyfs4a3sgdomazxospr3go2oaqkwhpgw6mkh@hdca2d3f6zwp>
Date: Tue, 12 Aug 2025 09:08:30 -0400
From: Ben Collins <bcollins@...nel.org>
To: linux-iio@...r.kernel.org
Cc: Andrew Hepp <andrew.hepp@...pp.dev>, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, Nuno Sá <nuno.sa@...log.com>
Subject: [PATCH] mcp9600: Add support for mcp9601 and sensor config

The mcp9600 dt binding doc claims to support thermocouple-type but
I don't see where this is implemented.

- Add support to detect mcp9601 device type
- Add support to use thermocouple-type dt prop
- Add thrermocouple iio info to get/set this from sysfs
- Add filter-level dt prop to set the filtering level of the chip
- Update dt binding docs

Signed-off-by: Ben Collins <bcollins@...nel.org>
Cc: Andrew Hepp <andrew.hepp@...pp.dev>
Cc: devicetree@...r.kernel.org
Cc: linux-kernel@...r.kernel.org
Cc: linux-iio@...r.kernel.org
Cc: "Nuno Sá" <nuno.sa@...log.com>
---
 .../iio/temperature/microchip,mcp9600.yaml    |   9 +
 drivers/iio/temperature/mcp9600.c             | 185 ++++++++++++++++--
 2 files changed, 181 insertions(+), 13 deletions(-)

diff --git a/Documentation/devicetree/bindings/iio/temperature/microchip,mcp9600.yaml b/Documentation/devicetree/bindings/iio/temperature/microchip,mcp9600.yaml
index d2cafa38a5442..0ee0259471c6a 100644
--- a/Documentation/devicetree/bindings/iio/temperature/microchip,mcp9600.yaml
+++ b/Documentation/devicetree/bindings/iio/temperature/microchip,mcp9600.yaml
@@ -42,6 +42,14 @@ properties:
       Use defines in dt-bindings/iio/temperature/thermocouple.h.
       Supported types are B, E, J, K, N, R, S, T.
 
+  filter-level:
+    $ref: /schemas/types.yaml#/definitions/uint32
+    description:
+      Level of chip filtering to use. 0 means filtering is disabled.
+      See reference manual 5.2.2 - THERMOCOUPLE SENSOR CONFIGURATION
+      REGISTER, FIGURE 5-6 for Filter Step Response graph. Supported
+      values are in the range of 0-7.
+
   vdd-supply: true
 
 required:
@@ -65,6 +73,7 @@ examples:
             interrupts = <25 IRQ_TYPE_EDGE_RISING>;
             interrupt-names = "open-circuit";
             thermocouple-type = <THERMOCOUPLE_TYPE_K>;
+            filter-level = <1>;
             vdd-supply = <&vdd>;
         };
     };
diff --git a/drivers/iio/temperature/mcp9600.c b/drivers/iio/temperature/mcp9600.c
index 6e9108d5cf75f..c1fe1e530786c 100644
--- a/drivers/iio/temperature/mcp9600.c
+++ b/drivers/iio/temperature/mcp9600.c
@@ -18,15 +18,23 @@
 #include <linux/minmax.h>
 #include <linux/mod_devicetable.h>
 #include <linux/module.h>
+#include <dt-bindings/iio/temperature/thermocouple.h>
 
 #include <linux/iio/events.h>
 #include <linux/iio/iio.h>
 
 /* MCP9600 registers */
-#define MCP9600_HOT_JUNCTION 0x0
-#define MCP9600_COLD_JUNCTION 0x2
+#define MCP9600_HOT_JUNCTION		0x0
+#define MCP9600_COLD_JUNCTION		0x2
 #define MCP9600_STATUS			0x4
 #define MCP9600_STATUS_ALERT(x)		BIT(x)
+#define MCP9600_STATUS_OC_IR		BIT(4)
+#define MCP9601_STATUS_SC		BIT(5)
+#define MCP9600_SENSOR_CFG		0x5
+#define MCP9600_SENSOR_TYPE_MASK	GENMASK(6, 4)
+#define MCP9600_SENSOR_TYPE(x)		((x << 4) & MCP9600_SENSOR_TYPE_MASK)
+#define MCP9600_FILTER_MASK		GENMASK(2, 0)
+#define MCP9600_FILTER(x)		((x << 0) & MCP9600_FILTER_MASK)
 #define MCP9600_ALERT_CFG1		0x8
 #define MCP9600_ALERT_CFG(x)		(MCP9600_ALERT_CFG1 + (x - 1))
 #define MCP9600_ALERT_CFG_ENABLE	BIT(0)
@@ -38,10 +46,11 @@
 #define MCP9600_ALERT_LIMIT1		0x10
 #define MCP9600_ALERT_LIMIT(x)		(MCP9600_ALERT_LIMIT1 + (x - 1))
 #define MCP9600_ALERT_LIMIT_MASK	GENMASK(15, 2)
-#define MCP9600_DEVICE_ID 0x20
+#define MCP9600_DEVICE_ID		0x20
 
 /* MCP9600 device id value */
-#define MCP9600_DEVICE_ID_MCP9600 0x40
+#define MCP9600_DEVICE_ID_MCP9600	0x40
+#define MCP9600_DEVICE_ID_MCP9601	0x41
 
 #define MCP9600_ALERT_COUNT		4
 
@@ -58,6 +67,21 @@ enum mcp9600_alert {
 	MCP9600_ALERT4
 };
 
+static const unsigned int mcp9600_type_map[] = {
+	[THERMOCOUPLE_TYPE_K] = 0,
+	[THERMOCOUPLE_TYPE_J] = 1,
+	[THERMOCOUPLE_TYPE_T] = 2,
+	[THERMOCOUPLE_TYPE_N] = 3,
+	[THERMOCOUPLE_TYPE_S] = 4,
+	[THERMOCOUPLE_TYPE_E] = 5,
+	[THERMOCOUPLE_TYPE_B] = 6,
+	[THERMOCOUPLE_TYPE_R] = 7,
+};
+
+static const int mcp9600_tc_types[] = {
+	'B', 'E', 'J', 'K', 'N', 'R', 'S', 'T'
+};
+
 static const char * const mcp9600_alert_name[MCP9600_ALERT_COUNT] = {
 	[MCP9600_ALERT1] = "alert1",
 	[MCP9600_ALERT2] = "alert2",
@@ -87,8 +111,12 @@ static const struct iio_event_spec mcp9600_events[] = {
 		{							       \
 			.type = IIO_TEMP,				       \
 			.address = MCP9600_HOT_JUNCTION,		       \
-			.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |	       \
-					      BIT(IIO_CHAN_INFO_SCALE),	       \
+			.info_mask_separate =                                  \
+					BIT(IIO_CHAN_INFO_RAW) |	       \
+					BIT(IIO_CHAN_INFO_THERMOCOUPLE_TYPE) | \
+					BIT(IIO_CHAN_INFO_SCALE),              \
+			.info_mask_separate_available =                        \
+					BIT(IIO_CHAN_INFO_THERMOCOUPLE_TYPE),  \
 			.event_spec = &mcp9600_events[hj_ev_spec_off],	       \
 			.num_event_specs = hj_num_ev,			       \
 		},							       \
@@ -125,6 +153,9 @@ static const struct iio_chan_spec mcp9600_channels[][2] = {
 
 struct mcp9600_data {
 	struct i2c_client *client;
+	unsigned char dev_id;
+	u32 thermocouple_type;
+	u32 filter;
 };
 
 static int mcp9600_read(struct mcp9600_data *data,
@@ -155,13 +186,82 @@ static int mcp9600_read_raw(struct iio_dev *indio_dev,
 		if (ret)
 			return ret;
 		return IIO_VAL_INT;
+
 	case IIO_CHAN_INFO_SCALE:
 		*val = 62;
 		*val2 = 500000;
 		return IIO_VAL_INT_PLUS_MICRO;
+
+	case IIO_CHAN_INFO_THERMOCOUPLE_TYPE:
+		*val = mcp9600_tc_types[data->thermocouple_type];
+		return IIO_VAL_CHAR;
+
+	default:
+		return -EINVAL;
+	}
+}
+
+static int mcp9600_write_raw_get_fmt(struct iio_dev *indio_dev,
+				     struct iio_chan_spec const *chan,
+				     long mask)
+{
+	switch (mask) {
+	case IIO_CHAN_INFO_THERMOCOUPLE_TYPE:
+		return IIO_VAL_CHAR;
+	default:
+		return -EINVAL;
+	}
+}
+
+static int mcp9600_config(struct mcp9600_data *data)
+{
+	struct i2c_client *client = data->client;
+	int ret, cfg;
+
+	cfg  = MCP9600_SENSOR_TYPE(mcp9600_type_map[data->thermocouple_type]) |
+		MCP9600_FILTER(data->filter);
+
+	ret = i2c_smbus_write_byte_data(client, MCP9600_SENSOR_CFG, cfg);
+
+	if (ret < 0) {
+		dev_err(&client->dev, "Failed to set sensor configuration\n");
+		return ret;
+	}
+
+	return 0;
+}
+
+static int mcp9600_write_raw(struct iio_dev *indio_dev,
+			     struct iio_chan_spec const *chan,
+			     int val, int val2, long mask)
+{
+	struct mcp9600_data *data = iio_priv(indio_dev);
+	int tc_type = -1;
+	int i, ret;
+
+	switch (mask) {
+	case IIO_CHAN_INFO_THERMOCOUPLE_TYPE:
+		for (i = 0; i < ARRAY_SIZE(mcp9600_tc_types); i++) {
+			if (mcp9600_tc_types[i] == toupper(val)) {
+				tc_type = i;
+				break;
+			}
+		}
+		if (tc_type < 0)
+			return -EINVAL;
+
+		data->thermocouple_type = tc_type;
+		ret = mcp9600_config(data);
+		if (ret < 0)
+			return ret;
+
+		break;
+
 	default:
 		return -EINVAL;
 	}
+
+	return 0;
 }
 
 static int mcp9600_get_alert_index(int channel2, enum iio_event_direction dir)
@@ -299,12 +399,32 @@ static int mcp9600_write_thresh(struct iio_dev *indio_dev,
 	}
 }
 
+static int mcp9600_read_avail(struct iio_dev *indio_dev,
+			      struct iio_chan_spec const *chan,
+			      const int **vals, int *type, int *length,
+			      long mask)
+{
+	switch (mask) {
+	case IIO_CHAN_INFO_THERMOCOUPLE_TYPE:
+		*vals = mcp9600_tc_types;
+		*type = IIO_VAL_CHAR;
+		*length = ARRAY_SIZE(mcp9600_tc_types);
+		return IIO_AVAIL_LIST;
+
+	default:
+		return -EINVAL;
+	}
+}
+
 static const struct iio_info mcp9600_info = {
 	.read_raw = mcp9600_read_raw,
+	.write_raw = mcp9600_write_raw,
+	.write_raw_get_fmt = mcp9600_write_raw_get_fmt,
 	.read_event_config = mcp9600_read_event_config,
 	.write_event_config = mcp9600_write_event_config,
 	.read_event_value = mcp9600_read_thresh,
 	.write_event_value = mcp9600_write_thresh,
+	.read_avail = mcp9600_read_avail,
 };
 
 static irqreturn_t mcp9600_alert_handler(void *private,
@@ -418,26 +538,65 @@ static int mcp9600_probe(struct i2c_client *client)
 {
 	struct iio_dev *indio_dev;
 	struct mcp9600_data *data;
-	int ret, ch_sel;
+	int ch_sel, dev_id, ret;
 
-	ret = i2c_smbus_read_byte_data(client, MCP9600_DEVICE_ID);
-	if (ret < 0)
-		return dev_err_probe(&client->dev, ret, "Failed to read device ID\n");
-	if (ret != MCP9600_DEVICE_ID_MCP9600)
-		dev_warn(&client->dev, "Expected ID %x, got %x\n",
-				MCP9600_DEVICE_ID_MCP9600, ret);
+	dev_id = i2c_smbus_read_byte_data(client, MCP9600_DEVICE_ID);
+	if (dev_id < 0)
+		return dev_err_probe(&client->dev, dev_id, "Failed to read device ID\n");
+
+	switch (dev_id) {
+	case MCP9600_DEVICE_ID_MCP9600:
+		dev_info(&client->dev, "Identified as mcp9600");
+		break;
+	case MCP9600_DEVICE_ID_MCP9601:
+		dev_info(&client->dev, "Identified as mcp9601");
+		break;
+
+	default:
+		return dev_err_probe(&client->dev, -EINVAL, "Unknown device ID: %x\n",
+				     dev_id);
+	}
 
 	indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
 	if (!indio_dev)
 		return -ENOMEM;
 
 	data = iio_priv(indio_dev);
+
+	ret = device_property_read_u32(&client->dev, "thermocouple-type",
+				       &data->thermocouple_type);
+	if (ret) {
+		dev_warn(&client->dev,
+			 "Missing thermocouple-type property, using Type-K\n");
+		data->thermocouple_type = THERMOCOUPLE_TYPE_K;
+	} else if (data->thermocouple_type < 0 || data->thermocouple_type >=
+		   ARRAY_SIZE(mcp9600_type_map)) {
+		dev_warn(&client->dev,
+			 "Invalid thermocouple-type property, using Type-K\n");
+		data->thermocouple_type = THERMOCOUPLE_TYPE_K;
+	}
+
+	ret = device_property_read_u32(&client->dev, "filter-level",
+				       &data->filter);
+	if (ret) {
+		dev_warn(&client->dev,
+			 "Missing filter-level property, using 0\n");
+		data->filter = 0;
+	} else if (data->filter < 0 || data->filter > 7) {
+		dev_warn(&client->dev,
+			 "Invalid filter-level property, using 0\n");
+		data->filter = 0;
+	}
+
+	data->dev_id = dev_id;
 	data->client = client;
 
 	ch_sel = mcp9600_probe_alerts(indio_dev);
 	if (ch_sel < 0)
 		return ch_sel;
 
+	mcp9600_config(data);
+
 	indio_dev->info = &mcp9600_info;
 	indio_dev->name = "mcp9600";
 	indio_dev->modes = INDIO_DIRECT_MODE;
-- 
2.50.1

-- 
 Ben Collins
 https://libjwt.io
 https://github.com/benmcollins
 --
 3EC9 7598 1672 961A 1139  173A 5D5A 57C7 242B 22CF

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ