Skip to contentSkip to conversation

iio: magnetometer: add support for Melexis MLX90393

PATCH In review Nikhil Gautam 6 participants 8 messages Jun 27 – Jul 1

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

v4 cover iio: magnetometer: add support for Melexis MLX90393

Hi,

This series adds initial Industrial I/O subsystem support for the Melexis MLX90393 3-axis magnetometer and temperature sensor.

The MLX90393 supports both I2C and SPI interfaces. This series implements support for the I2C interface while keeping the driver structure transport-independent to simplify future SPI support.

The device uses a command-based communication protocol rather than a conventional register-addressed interface. A small transport abstraction layer is therefore used instead of regmap to share the common sensor logic between the current I2C implementation and future SPI support.

Currently supported features:

* Raw magnetic field measurements for X/Y/Z axes
* Raw temperature measurements
* Configurable gain/scale selection
* Configurable oversampling ratio
* Direct mode operation through the IIO subsystem
* I2C interface support

The driver has been tested on Raspberry Pi 5 hardware using an MLX90393 sensor connected over I2C. Magnetic field and temperature measurements were verified through the IIO sysfs interface.

Datasheet:
https://www.melexis.com/en/documents/documentation/datasheets/datasheet-mlx90393

Previous submission:
v3: https://lore.kernel.org/linux-iio/[email protected]/T/#u
v2: https://lore.kernel.org/linux-iio/[email protected]/T/#t
v1: https://lore.kernel.org/linux-iio/[email protected]/

Changes in v4:
[DT]
* Updated the example to include the required vdd-supply and vddio-supply properties.
* Added fixed regulator nodes to the example so it passes dt_binding_check.

[IIO]
* No Change

Changes in v3:
[DT]

* Made vdd-supply and vddio-supply required properties.
* Moved the MAINTAINERS entry to the correct alphabetical position.

[IIO]

* Added the publicly available MLX90393 datasheet reference.
* Clarified the motivation for introducing a dedicated MLX90393 driver.
* Added the I2C device ID table and switched to designated initializers.
* Added a forward declaration for struct device to reduce unnecessary header dependencies.
* Added symbol namespaces.
* Improved include usage.
* Improved status validation and consistency of conditional checks.
* Added comments documenting protocol encoding, initialization delays, and temperature conversion constants derived from the datasheet.
* Improved naming, formatting, indentation and general coding style.
* Addressed all review comments from Andy Shevchenko,
Uwe Kleine-König and Krzysztof Kozlowski.

Changes in v2:
[DT]

* Extended the DT binding to document power supply regulators and optional interrupt and trigger GPIOs.

[IIO]

* Removed the RFC tag based on reviewer feedback.
* Added a MAINTAINERS entry as part of the initial submission and expanded it in the driver patch.
* Reworked the scale availability implementation to simplify the data layout and eliminate the need for constructing a temporary table, avoiding potential race conditions.
* Replaced usleep_range() with fsleep() where appropriate and documented initialization delays.
* Simplified helper functions and improved error handling by returning directly where appropriate.
* Reduced unnecessary local variables and line wrapping to improve readability and align with kernel coding style.
* Added comments for lock protection and command definitions to improve code clarity.
* Switched to devm_mutex_init() and cleaned up include usage in accordance with the "include what you use" principle.
* Improved consistency across the driver, including conditional handling, switch statements, formatting and general code style.
* Addressed all review comments from Jonathan Cameron.

Many thanks to Jonathan Cameron, Andy Shevchenko,
Uwe Kleine-König and Krzysztof Kozlowski for their detailed reviews and valuable feedback, which significantly improved this series.

Further review and comments are greatly appreciated.

Thanks,
Nikhil Gautam

Nikhil Gautam (2):
dt-bindings: iio: magnetometer: add Melexis MLX90393
iio: magnetometer: add support for Melexis MLX90393

 .../iio/magnetometer/melexis,mlx90393.yaml    |  63 ++
 MAINTAINERS                                   |   7 +
 drivers/iio/magnetometer/Kconfig              |  10 +
 drivers/iio/magnetometer/Makefile             |   2 +
 drivers/iio/magnetometer/mlx90393.h           |  73 ++
 drivers/iio/magnetometer/mlx90393_core.c      | 701 ++++++++++++++++++
 drivers/iio/magnetometer/mlx90393_i2c.c       |  76 ++
 7 files changed, 932 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/magnetometer/melexis,mlx90393.yaml
 create mode 100644 drivers/iio/magnetometer/mlx90393.h
 create mode 100644 drivers/iio/magnetometer/mlx90393_core.c
 create mode 100644 drivers/iio/magnetometer/mlx90393_i2c.c
Signature · 1 line
2.39.5
#1 lore raw

PATCH v4 1/2 dt-bindings: iio: magnetometer: add Melexis MLX90393

Add devicetree bindings for the Melexis MLX90393
3-axis magnetometer and temperature sensor.

The device supports magnetic field and temperature measurements over I2C and SPI interfaces.

This initial binding documents the I2C interface.

---
 .../iio/magnetometer/melexis,mlx90393.yaml    | 63 +++++++++++++++++++
 MAINTAINERS                                   |  6 ++
 2 files changed, 69 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/magnetometer/melexis,mlx90393.yaml
Documentation/devicetree/bindings/iio/magnetometer/melexis,mlx90393.yaml new file +63 -0
new file mode 100644
index 000000000000..91e984827786
@@ -0,0 +1,63 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/magnetometer/melexis,mlx90393.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Melexis MLX90393 magnetometer sensor
+
+maintainers:
+  - Nikhil Gautam <[email protected]>
+
+description:
+  Melexis MLX90393 3-axis magnetometer and temperature sensor.
+
+properties:
+  compatible:
+    const: melexis,mlx90393
+
+  reg:
+    maxItems: 1
+
+  vdd-supply:
+    description: Regulator that provides power to the sensor
+
+  vddio-supply:
+    description: Regulator that provides power to the bus
+
+  interrupts:
+    maxItems: 1
+
+  trigger-gpios:
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+  - vdd-supply
+  - vddio-supply
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/gpio/gpio.h>
+    #include <dt-bindings/interrupt-controller/irq.h>
+
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        magnetometer@c {
+            compatible = "melexis,mlx90393";
+            reg = <0x0c>;
+
+            interrupt-parent = <&gpio>;
+            interrupts = <17 IRQ_TYPE_EDGE_RISING>;
+
+            trigger-gpios = <&gpio 18 GPIO_ACTIVE_HIGH>;
+
+            vdd-supply = <&vdd>;
+            vddio-supply = <&vddio>;
+        };
+    };
MAINTAINERS +6 -0
index a92290fffa16..469ae3f9aa0f 100644
@@ -15501,6 +15501,12 @@ F:	Documentation/scsi/megaraid.rst
 F:	drivers/scsi/megaraid.*
 F:	drivers/scsi/megaraid/
 
+MELEXIS MLX90393 MAGNETOMETER DRIVER
+M:	Nikhil Gautam <[email protected]>
+L:	[email protected]
+S:	Maintained
+F:	Documentation/devicetree/bindings/iio/magnetometer/melexis,mlx90393.yaml
+
 MELEXIS MLX90614 DRIVER
 M:	Crt Mori <[email protected]>
 L:	[email protected]
Signature · 1 line
2.39.5
Reviewed-by on v4 1/2
#2 lore raw

Re: [PATCH v4 1/2] dt-bindings: iio: magnetometer: add Melexis MLX90393

Quoted 9 lines · Nikhil Gautam

Add devicetree bindings for the Melexis MLX90393
3-axis magnetometer and temperature sensor.

The device supports magnetic field and temperature measurements over I2C and SPI interfaces.

This initial binding documents the I2C interface.

If you are going to send any new version, then fix wrapping in commit msg:
Please wrap commit message according to Linux coding style / submission process (neither too early nor over the limit):
https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst#L597

Best regards,
Krzysztof

#3 lore raw

PATCH v4 2/2 iio: magnetometer: add support for Melexis MLX90393

In reply to Nikhil Gautam

Add Industrial I/O subsystem support for the Melexis MLX90393 3-axis magnetometer and temperature sensor.

The driver currently supports:

raw magnetic field measurements
raw temperature measurements
configurable gain/scale selection
configurable oversampling ratio
direct mode operation

The MLX90393 supports both I2C and SPI interfaces. This initial implementation adds support for the I2C interface.

The device uses a command-based communication protocol rather than a conventional register-addressed interface. A small transport abstraction layer is therefore used instead of regmap to share the common sensor logic between the current I2C implementation and future SPI support without duplicating code.

---
 MAINTAINERS                              |   1 +
 drivers/iio/magnetometer/Kconfig         |  10 +
 drivers/iio/magnetometer/Makefile        |   2 +
 drivers/iio/magnetometer/mlx90393.h      |  73 +++
 drivers/iio/magnetometer/mlx90393_core.c | 701 +++++++++++++++++++++++
 drivers/iio/magnetometer/mlx90393_i2c.c  |  76 +++
 6 files changed, 863 insertions(+)
 create mode 100644 drivers/iio/magnetometer/mlx90393.h
 create mode 100644 drivers/iio/magnetometer/mlx90393_core.c
 create mode 100644 drivers/iio/magnetometer/mlx90393_i2c.c
MAINTAINERS +1 -0
index 469ae3f9aa0f..177dda261801 100644
@@ -15506,6 +15506,7 @@ M:	Nikhil Gautam <[email protected]>
 L:	[email protected]
 S:	Maintained
 F:	Documentation/devicetree/bindings/iio/magnetometer/melexis,mlx90393.yaml
+F:	drivers/iio/magnetometer/mlx90393*
 
 MELEXIS MLX90614 DRIVER
 M:	Crt Mori <[email protected]>
drivers/iio/magnetometer/Kconfig +10 -0
index 3debf1320ad1..e6b74e7e3317 100644
@@ -128,6 +128,16 @@ config HID_SENSOR_MAGNETOMETER_3D
 	  Say yes here to build support for the HID SENSOR
 	  Magnetometer 3D.
 
+config MLX90393
+	tristate "MELEXIS MLX90393 3-axis magnetometer sensor"
+	depends on I2C
+	help
+	  Say yes here to build support for the MELEXIS MLX90393 3-axis
+	  magnetometer.
+
+	  To compile this driver as a module, choose M here: the module
+	  will be called mlx90393.
+
 config MMC35240
 	tristate "MEMSIC MMC35240 3-axis magnetic sensor"
 	select REGMAP_I2C
drivers/iio/magnetometer/Makefile +2 -0
index 9297723a97d8..542c89d38a59 100644
@@ -14,6 +14,8 @@ obj-$(CONFIG_BMC150_MAGN_SPI) += bmc150_magn_spi.o
 
 obj-$(CONFIG_MAG3110)	+= mag3110.o
 obj-$(CONFIG_HID_SENSOR_MAGNETOMETER_3D) += hid-sensor-magn-3d.o
+obj-$(CONFIG_MLX90393)		+= mlx90393_core.o
+obj-$(CONFIG_MLX90393)		+= mlx90393_i2c.o
 obj-$(CONFIG_MMC35240)	+= mmc35240.o
 
 obj-$(CONFIG_IIO_ST_MAGN_3AXIS) += st_magn.o
drivers/iio/magnetometer/mlx90393.h new file +73 -0

73 lines not on this page · raw message

drivers/iio/magnetometer/mlx90393_core.c new file +701 -0

701 lines not on this page · raw message

drivers/iio/magnetometer/mlx90393_i2c.c new file +76 -0

76 lines not on this page · raw message

Signature · 1 line
2.39.5
#4 lore raw
Thank you for your contribution! Sashiko AI review found 6 potential issue(s) to consider: - [High] Data race and state corruption due to function-static `scale_avail` array. - [High]…

Thank you for your contribution! Sashiko AI review found 6 potential issue(s) to consider:
- [High] Data race and state corruption due to function-static scale_avail array.
- [High] Stack-allocated buffers are passed to i2c_transfer, violating DMA safety rules.
- [Medium] Writing to temperature scale incorrectly modifies magnetometer hardware gain.
- [Medium] Temperature oversampling ratio exposes raw register values instead of actual multipliers.
- [Medium] IIO scale factors are incorrectly calculated, violating the expected IIO ABI units.
- [Low] Use of floating-point literal in delay calculation causes build failures.
--

commit 46a132b5223abb9cadead7640985442ba357e7bc
Author: Nikhil Gautam <[email protected]>

iio: magnetometer: add support for Melexis MLX90393

This patch adds Industrial I/O subsystem support for the Melexis MLX90393 3-axis magnetometer and temperature sensor. The driver supports raw magnetic field and temperature measurements, scale and oversampling selection, and currently uses the I2C interface.

Quoted 6 lines
drivers/iio/magnetometer/mlx90393_core.c new file +0 -0
new file mode 100644
index 0000000000000..78fcab8a2b566
@@ -0,0 +1,701 @@

[ ... ]

drivers/iio/magnetometer/mlx90393_core.c line 84

Quoted 3 lines
drivers/iio/magnetometer/mlx90393_core.c new file +3 -0
+static const int mlx90393_osr2_avail[MLX90393_OSR2_MAX] = {
+	0, 1, 2, 3,
+};

[Severity: Medium]
Does this expose raw register values for the temperature oversampling ratio instead of the actual multipliers?

The IIO ABI requires exposing the actual ratio (1, 2, 4, 8) rather than the raw hardware indices. Users reading or writing the in_temp_oversampling_ratio sysfs attribute might be confused by these hardware-specific indices.

[ ... ]

drivers/iio/magnetometer/mlx90393_core.c line 229

Quoted 11 lines
drivers/iio/magnetometer/mlx90393_core.c new file +11 -0
+static int mlx90393_read_reg(struct mlx90393_data *data, u8 reg, u16 *val)
+{
+	u8 tx[2];
+	u8 rx[3];
+	int ret;
+
+	tx[0] = MLX90393_CMD_RR;
+	/* Register address is encoded in bits [7:2] */
+	tx[1] = reg << 2;
+
+	ret = mlx90393_xfer(data, tx, sizeof(tx), rx, sizeof(rx));

[Severity: High]
Can passing these stack-allocated buffers to the I2C transport cause memory corruption or DMA mapping errors?

Since CONFIG_VMAP_STACK is enabled by default in modern kernels, stack memory is not physically contiguous and cannot be safely mapped for DMA. When sysfs operations trigger communication with the device, the I2C controller might use DMA to transfer data, which expects DMA-safe memory.

This pattern is also present in mlx90393_write_reg() and mlx90393_read_measurement().

[ ... ]

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+static int mlx90393_get_scale(struct mlx90393_data *data,
+			      const struct iio_chan_spec *chan,
+			      int *val, int *val2)
+{
+	enum mlx90393_axis_type axis;
+	u8 res;
+
+	if (chan->channel2 == IIO_MOD_Z) {
+		axis = MLX90393_AXIS_TYPE_Z;
+		res = data->res_z;

drivers/iio/magnetometer/mlx90393_core.c line 341

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+	} else {
+		axis = MLX90393_AXIS_TYPE_XY;
+		res = data->res_xy;
+	}
+
+	*val = 0;
+	*val2 = mlx90393_scale_table[axis][data->gain_sel][res];
+
+	return IIO_VAL_INT_PLUS_NANO;
+}

[Severity: Medium]
Does this calculate the scale factor correctly for the IIO ABI?

The magnetometer scale values in mlx90393_scale_table are returned using IIO_VAL_INT_PLUS_NANO, yielding a value like 0.000000751. However, since the value is 0.751 uT/LSB, this corresponds to 0.00751 Gauss/LSB. Returning it as nanounits without adjusting the multiplier makes the reported scale off by a factor of 10,000.

[ ... ]

drivers/iio/magnetometer/mlx90393_core.c line 486

Quoted 12 lines
drivers/iio/magnetometer/mlx90393_core.c new file +12 -0
+static int mlx90393_write_raw(struct iio_dev *indio_dev,
+			      const struct iio_chan_spec *chan,
+			      int val, int val2,
+			      long mask)
+{
+	struct mlx90393_data *data = iio_priv(indio_dev);
+
+	switch (mask) {
+	case IIO_CHAN_INFO_SCALE: {
+		guard(mutex)(&data->lock);
+		return mlx90393_set_scale(data, chan, val, val2);
+	}

[Severity: Medium]
Does this allow writing to the temperature scale?

Since IIO_CHAN_INFO_SCALE is checked unconditionally without validating chan->type, writes to the in_temp_scale sysfs attribute fall through to mlx90393_set_scale(). Because chan->channel2 is 0 for the temperature channel, this unintentionally modifies the magnetometer's GAIN_SEL configuration instead of returning -EINVAL.

[ ... ]

Quoted 4 lines
drivers/iio/magnetometer/mlx90393_core.c new file +4 -0
+static int mlx90393_read_raw(struct iio_dev *indio_dev,
+			     const struct iio_chan_spec *chan,
+			     int *val, int *val2, long mask)
+{

[ ... ]

drivers/iio/magnetometer/mlx90393_core.c line 533

Quoted 12 lines
drivers/iio/magnetometer/mlx90393_core.c new file +12 -0
+	case IIO_CHAN_INFO_SCALE:
+		switch (chan->type) {
+		case IIO_MAGN:
+			return mlx90393_get_scale(data, chan, val, val2);
+
+		case IIO_TEMP:
+			/*
+			 * Datasheet Table 7: Thermal Specification
+			 */
+			*val = 0;
+			*val2 = 22124;
+			return IIO_VAL_INT_PLUS_MICRO;

[Severity: Medium]
Does this return the temperature scale in degrees Celsius instead of milli-Celsius?

The IIO ABI specifies that temperature should be exposed in milli-Celsius, but returning 0.022124 yields degrees Celsius.

[ ... ]

drivers/iio/magnetometer/mlx90393_core.c line 571

Quoted 9 lines
drivers/iio/magnetometer/mlx90393_core.c new file +9 -0
+static int mlx90393_read_avail(struct iio_dev *indio_dev,
+			       const struct iio_chan_spec *chan,
+			       const int **vals,
+			       int *type,
+			       int *length,
+			       long mask)
+{
+	struct mlx90393_data *data = iio_priv(indio_dev);
+	static int scale_avail[MLX90393_GAIN_MAX][MLX90393_AXIS_MAX];

[Severity: High]
Can concurrent reads of this function corrupt the scale_available sysfs output?

Because scale_avail is declared as a function-static array, it is shared across all channels and devices. When the IIO core processes the returned pointer outside of the driver's lock, concurrent reads (e.g., from multiple axes or probed devices) will overwrite the array and return interleaved or incorrect values to userspace.

[ ... ]

Quoted 16 lines
drivers/iio/magnetometer/mlx90393_core.c new file +16 -0
+static int mlx90393_init(struct mlx90393_data *data)
+{
+	int ret;
+	u16 reg;
+
+	/* Exit mode */
+	ret = mlx90393_write_cmd(data, MLX90393_CMD_EX);
+	if (ret)
+		return ret;
+
+	/*
+	 * Datasheet section 15.4.1.2 (RT command), Figure 16:
+	 * Wait 1 ms after EX command before issuing RT.
+	 */
+	fsleep(1 * USEC_PER_MSEC);
+

drivers/iio/magnetometer/mlx90393_core.c line 642

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+	/* Reset device */
+	ret = mlx90393_write_cmd(data, MLX90393_CMD_RT);
+	if (ret)
+		return ret;
+
+	/*
+	 * Datasheet section 15.4.1.2 (RT command), Figure 16:
+	 * Wait 1.5 ms for the start-up sequence to complete.
+	 */
+	fsleep(1.5 * USEC_PER_MSEC);

[Severity: Low]
Will this floating-point literal cause build failures?

The expression uses 1.5, which is a double-precision floating-point literal. The Linux kernel does not support floating-point arithmetic, which leads to compilation errors on most architectures that do not support soft-float or have strict compiler flags against floating-point usage.

Signature · 1 line
Sashiko AI review · https://sashiko.dev/#/patchset/[email protected]?part=2
reviewer
#5 lore raw
In reply to Nikhil Gautam
Quoted 72 lines · Nikhil Gautam

Add Industrial I/O subsystem support for the Melexis MLX90393 3-axis magnetometer and temperature sensor.

The driver currently supports:

raw magnetic field measurements
raw temperature measurements
configurable gain/scale selection
configurable oversampling ratio
direct mode operation

The MLX90393 supports both I2C and SPI interfaces. This initial implementation adds support for the I2C interface.

The device uses a command-based communication protocol rather than a conventional register-addressed interface. A small transport abstraction layer is therefore used instead of regmap to share the common sensor logic between the current I2C implementation and future SPI support without duplicating code.

---
 MAINTAINERS                              |   1 +
 drivers/iio/magnetometer/Kconfig         |  10 +
 drivers/iio/magnetometer/Makefile        |   2 +
 drivers/iio/magnetometer/mlx90393.h      |  73 +++
 drivers/iio/magnetometer/mlx90393_core.c | 701 +++++++++++++++++++++++
 drivers/iio/magnetometer/mlx90393_i2c.c  |  76 +++
 6 files changed, 863 insertions(+)
 create mode 100644 drivers/iio/magnetometer/mlx90393.h
 create mode 100644 drivers/iio/magnetometer/mlx90393_core.c
 create mode 100644 drivers/iio/magnetometer/mlx90393_i2c.c
MAINTAINERS +1 -0
index 469ae3f9aa0f..177dda261801 100644
@@ -15506,6 +15506,7 @@ M:	Nikhil Gautam <[email protected]>
 L:	[email protected]
 S:	Maintained
 F:	Documentation/devicetree/bindings/iio/magnetometer/melexis,mlx90393.yaml
+F:	drivers/iio/magnetometer/mlx90393*
 
 MELEXIS MLX90614 DRIVER
 M:	Crt Mori <[email protected]>
drivers/iio/magnetometer/Kconfig +10 -0
index 3debf1320ad1..e6b74e7e3317 100644
@@ -128,6 +128,16 @@ config HID_SENSOR_MAGNETOMETER_3D
 	  Say yes here to build support for the HID SENSOR
 	  Magnetometer 3D.
 
+config MLX90393
+	tristate "MELEXIS MLX90393 3-axis magnetometer sensor"
+	depends on I2C
+	help
+	  Say yes here to build support for the MELEXIS MLX90393 3-axis
+	  magnetometer.
+
+	  To compile this driver as a module, choose M here: the module
+	  will be called mlx90393.
+
 config MMC35240
 	tristate "MEMSIC MMC35240 3-axis magnetic sensor"
 	select REGMAP_I2C
drivers/iio/magnetometer/Makefile +0 -0
index 9297723a97d8..542c89d38a59 100644

drivers/iio/magnetometer/Makefile obj-$(CONFIG_BMC150_MAGN_SPI) += bmc150_magn_spi.o · line 18

Quoted 5 lines
drivers/iio/magnetometer/Makefile +2 -0
@@ -14,6 +14,8 @@ obj-$(CONFIG_BMC150_MAGN_SPI) += bmc150_magn_spi.o
 
 obj-$(CONFIG_MAG3110)	+= mag3110.o
 obj-$(CONFIG_HID_SENSOR_MAGNETOMETER_3D) += hid-sensor-magn-3d.o
+obj-$(CONFIG_MLX90393)		+= mlx90393_core.o
+obj-$(CONFIG_MLX90393)		+= mlx90393_i2c.o

Seems like the tabs are just making it more messy. I would just go with one space.

Quoted 133 lines
drivers/iio/magnetometer/Makefile +0 -0
 obj-$(CONFIG_MMC35240)	+= mmc35240.o
 
 obj-$(CONFIG_IIO_ST_MAGN_3AXIS) += st_magn.o
drivers/iio/magnetometer/mlx90393.h new file +73 -0
new file mode 100644
index 000000000000..6a646ac05198
@@ -0,0 +1,73 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+/*
+ * MLX90393 magnetometer & temperature sensor driver
+ *
+ * Copyright (c) 2026 Nikhil Gautam <[email protected]>
+ */
+
+#ifndef MLX90393_H
+#define MLX90393_H
+
+#include <linux/bits.h>
+#include <linux/types.h>
+
+#define MLX90393_AXIS_MAX		2
+#define MLX90393_GAIN_MAX		8
+#define MLX90393_RES_MAX		4
+#define MLX90393_OSR2_MAX		4
+#define MLX90393_OSR_MAX		4
+
+#define MLX90393_CMD_MASK        GENMASK(7, 4)
+
+/* Commands (datasheet, Table 11 - Command List) */
+#define MLX90393_CMD_SB        0x10	/* Start Burst Mode */
+#define MLX90393_CMD_SW        0x20	/* Start Wake-up on Change Mode */
+#define MLX90393_CMD_SM        0x30	/* Start Single Measurement Mode */
+#define MLX90393_CMD_RM        0x40	/* Read Measurement */
+#define MLX90393_CMD_RR        0x50	/* Read Register */
+#define MLX90393_CMD_WR        0x60	/* Write Register */
+#define MLX90393_CMD_EX        0x80	/* Exit Mode */
+#define MLX90393_CMD_HR        0xD0	/* Memory Recall */
+#define MLX90393_CMD_HS        0xE0	/* Memory Store */
+#define MLX90393_CMD_RT        0xF0	/* Reset Device */
+
+#define MLX90393_MEASURE_Z    BIT(0)
+#define MLX90393_MEASURE_Y    BIT(1)
+#define MLX90393_MEASURE_X    BIT(2)
+#define MLX90393_MEASURE_TEMP BIT(3)
+
+#define MLX90393_MEASURE_ALL	(MLX90393_MEASURE_TEMP | \
+		MLX90393_MEASURE_X | MLX90393_MEASURE_Y | MLX90393_MEASURE_Z)
+
+#define MLX90393_NUM_CHANNELS	4
+
+#define MLX90393_STATUS_RESP    GENMASK(1, 0)
+#define MLX90393_STATUS_RT	BIT(2)
+#define MLX90393_STATUS_ERROR   BIT(4)
+
+#define MLX90393_REG_CONF1          0x00
+#define MLX90393_REG_CONF2          0x01
+#define MLX90393_REG_CONF3          0x02
+#define MLX90393_REG_CONF4          0x03
+
+#define MLX90393_CONF1_GAIN_SEL     GENMASK(6, 4)
+#define MLX90393_CONF1_HALLCONF     GENMASK(3, 0)
+
+#define MLX90393_CONF3_OSR        GENMASK(1, 0)
+#define MLX90393_CONF3_DIG_FILT   GENMASK(4, 2)
+#define MLX90393_CONF3_RES_X      GENMASK(6, 5)
+#define MLX90393_CONF3_RES_Y      GENMASK(8, 7)
+#define MLX90393_CONF3_RES_Z      GENMASK(10, 9)
+#define MLX90393_CONF3_OSR2       GENMASK(12, 11)
+
+struct device;
+
+struct mlx90393_transfer_ops {
+	int (*xfer)(void *context, const u8 *tx, int tx_len, u8 *rx, int rx_len);
+};
+
+int mlx90393_core_probe(struct device *dev,
+			const struct mlx90393_transfer_ops *ops,
+			void *context);
+
+#endif
drivers/iio/magnetometer/mlx90393_core.c new file +45 -0
new file mode 100644
index 000000000000..78fcab8a2b56
@@ -0,0 +1,701 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * MLX90393 magnetometer & temperature sensor driver
+ *
+ * Copyright (c) 2026 Nikhil Gautam <[email protected]>
+ *
+ * Datasheet: https://media.melexis.com/-/media/files/documents/datasheets/mlx90393-datasheet-melexis.pdf
+ */
+
+#include <linux/array_size.h>
+#include <linux/bitfield.h>
+#include <linux/bitops.h>
+#include <linux/errno.h>
+#include <linux/delay.h>
+#include <linux/module.h>
+#include <linux/mutex.h>
+#include <linux/types.h>
+#include <linux/time64.h>
+#include <linux/unaligned.h>
+#include <linux/units.h>
+
+#include <linux/iio/iio.h>
+
+#include "mlx90393.h"
+
+struct mlx90393_data {
+	/* Protects sensor configuration and measurement operations */
+	struct mutex lock;
+	struct device *dev;
+	void *bus_context;
+	const struct mlx90393_transfer_ops *ops;
+	u8 gain_sel;
+	u8 hallconf;
+
+	u8 res_xy;
+	u8 res_z;
+
+	u8 dig_filt;
+	u8 osr;
+	u8 osr2;
+};
+
+enum mlx90393_channels {
+	MLX90393_CHAN_X,
+	MLX90393_CHAN_Y,

drivers/iio/magnetometer/mlx90393_core.c line 55

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+	MLX90393_CHAN_Z,
+	MLX90393_CHAN_TEMP,
+};
+
+enum mlx90393_axis_type {
+	MLX90393_AXIS_TYPE_XY,
+	MLX90393_AXIS_TYPE_Z,
+};
+
+/* Datasheet: Table no.17 */

It would be more helpful to also say that this is nanotesela per LSB at 25 deg C. And probably also explain why we don't care about the 35 deg C table.

Quoted 18 lines
drivers/iio/magnetometer/mlx90393_core.c new file +18 -0
+static const int mlx90393_scale_table[MLX90393_AXIS_MAX][MLX90393_GAIN_MAX]
+				      [MLX90393_RES_MAX] = {
+	/* XY axis */
+	{
+		{ 751, 1502, 3004, 6009},
+		{ 601, 1202, 2403, 4840},
+		{ 451, 901, 1803, 3605},
+		{ 376, 751, 1502, 3004},
+		{ 300, 601, 1202, 2403},
+		{ 250, 501, 1001, 2003},
+		{ 200, 401, 801, 1602},
+		{ 150, 300, 601, 1202},
+	},
+	/* Z axis */
+	{
+		{ 1210, 2420, 4840, 9680},
+		{ 968, 1936, 3872, 7744},
+		{ 726, 1452, 2904, 5808},

drivers/iio/magnetometer/mlx90393_core.c line 83

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+		{ 605, 1210, 2420, 4840},
+		{ 484, 968, 1936, 3872},
+		{ 403, 807, 1613, 3227},
+		{ 323, 645, 1291, 2581},
+		{ 242, 484, 968, 1936},
+	}
+};
+
+static const int mlx90393_osr2_avail[MLX90393_OSR2_MAX] = {
+	0, 1, 2, 3,

OSR of 0 does not make sense. It would mean 0 samples, which would be no data. Should this be the same as OSR below?

Quoted 24 lines
drivers/iio/magnetometer/mlx90393_core.c new file +24 -0
+};
+
+static const int mlx90393_osr_avail[MLX90393_OSR_MAX] = {
+	1, 2, 4, 8,
+};
+
+#define MLX90393_CHAN(idx, axis, addr) { \
+	.type = IIO_MAGN, \
+	.modified = 1, \
+	.channel = idx, \
+	.address = addr, \
+	.channel2 = IIO_MOD_##axis, \
+	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |	\
+		BIT(IIO_CHAN_INFO_SCALE), \
+	.info_mask_shared_by_type = BIT(IIO_CHAN_INFO_OVERSAMPLING_RATIO),\
+	.info_mask_separate_available = \
+		BIT(IIO_CHAN_INFO_SCALE),  \
+	.info_mask_shared_by_type_available = \
+		BIT(IIO_CHAN_INFO_OVERSAMPLING_RATIO), \
+}
+
+static const struct iio_chan_spec mlx90393_channels[] = {
+	MLX90393_CHAN(0, X, MLX90393_CHAN_X),
+	MLX90393_CHAN(1, Y, MLX90393_CHAN_Y),

drivers/iio/magnetometer/mlx90393_core.c line 117

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+	MLX90393_CHAN(2, Z, MLX90393_CHAN_Z),
+	{
+		.type = IIO_TEMP,
+		.address = MLX90393_CHAN_TEMP,
+		.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |
+			BIT(IIO_CHAN_INFO_OFFSET) |
+			BIT(IIO_CHAN_INFO_OVERSAMPLING_RATIO),
+		.info_mask_separate_available =
+			BIT(IIO_CHAN_INFO_OVERSAMPLING_RATIO),
+		.info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE),

It would make the attribute names a bit more symetric if IIO_CHAN_INFO_OVERSAMPLING_RATIO was shared_by_type and IIO_CHAN_INFO_SCALE was separate just like the mag channels.

Quoted 16 lines
drivers/iio/magnetometer/mlx90393_core.c new file +16 -0
+	},
+};
+
+/*
+ * Calculate total conversion time in microseconds.
+ *
+ * Formula derived from datasheet timing equations.
+ * Datasheet: Table 8, Page no. 12
+ *
+ * Abbreviations:
+ * TCONVM	-> Single Magnetic axis conversion time
+ * TCONVT	-> Temperature conversion time
+ * TSTBY	-> Time from IDLE to STANDBY
+ * TACTIVE	-> Time from STBY to ACTIVE
+ * TCONV_END	-> Time to end analog active mode
+ */

drivers/iio/magnetometer/mlx90393_core.c line 143

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+static int mlx90393_get_tconv_us(struct mlx90393_data *data)
+{
+	const int osr = data->osr;
+	const int osr2 = data->osr2;
+	const int df = data->dig_filt;
+
+	int tconvm;
+	int tconvt;
+
+	int m = 3; /* X,Y,Z */

const

drivers/iio/magnetometer/mlx90393_core.c line 147

Quoted 4 lines
drivers/iio/magnetometer/mlx90393_core.c new file +4 -0
+
+	/*
+	 * TCONVM = 67 + 64 * 2^OSR * (2 + 2^DIG_FILT)
+	 */

Can be single line comment.

drivers/iio/magnetometer/mlx90393_core.c line 152

Quoted 5 lines
drivers/iio/magnetometer/mlx90393_core.c new file +5 -0
+	tconvm = 67 + (64 * BIT(osr) * (2 + BIT(df)));
+
+	/*
+	 * TCONVT = 67 + 192 * 2^OSR2
+	 */

ditto

drivers/iio/magnetometer/mlx90393_core.c line 158

Quoted 6 lines
drivers/iio/magnetometer/mlx90393_core.c new file +6 -0
+	tconvt = 67 + (192 * BIT(osr2));
+	/*
+	 * Total conversion time:
+	 * TSTBY + TACTIVE + m * TCONVM + TCONVT + TCONV_END
+	 */
+	return 220 + 360 + (m * tconvm) + tconvt + 120;

Unnecessary ().

Quoted 15 lines
drivers/iio/magnetometer/mlx90393_core.c new file +15 -0
+}
+
+static int mlx90393_xfer(struct mlx90393_data *data,
+			 const u8 *tx, int tx_len,
+			 u8 *rx, int rx_len)
+{
+	return data->ops->xfer(data->bus_context, tx, tx_len, rx, rx_len);
+}
+
+static int mlx90393_check_status(u8 cmd, u8 status)
+{
+	/* Datasheet: Table 12: Status byte definition */
+
+	/* Always validate error bit */
+	if (status & MLX90393_STATUS_ERROR)

drivers/iio/magnetometer/mlx90393_core.c line 183

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+		return -EIO;
+
+	switch (cmd & MLX90393_CMD_MASK) {
+	case MLX90393_CMD_RM:
+		/*
+		 * D1:D0 indicates response availability
+		 * 00 means invalid/no measurement
+		 */
+		if ((status & MLX90393_STATUS_RESP) == 0)
+			return -EIO;

Should this have a different error code? In other words, does this just mean "try again later and there might be data"?

drivers/iio/magnetometer/mlx90393_core.c line 188

Quoted 5 lines
drivers/iio/magnetometer/mlx90393_core.c new file +5 -0
+		return 0;
+	case MLX90393_CMD_RT:
+		/* Reset acknowledge */
+		if ((status & MLX90393_STATUS_RT) == 0)
+			return -EIO;

Datasheet calls this RS bit, not RT (which is the cooresponding command).

Quoted 73 lines
drivers/iio/magnetometer/mlx90393_core.c new file +73 -0
+		return 0;
+	default:
+		return 0;
+	}
+}
+
+static int mlx90393_write_cmd(struct mlx90393_data *data, u8 cmd)
+{
+	u8 status;
+	int ret;
+
+	ret = mlx90393_xfer(data, &cmd, 1, &status, 1);
+	if (ret)
+		return ret;
+
+	return mlx90393_check_status(cmd, status);
+}
+
+static int mlx90393_read_cmd(struct mlx90393_data *data, u8 cmd, u8 *rx,
+			     int rx_len)
+{
+	int ret;
+
+	ret = mlx90393_xfer(data, &cmd, 1, rx, rx_len);
+	if (ret)
+		return ret;
+
+	return mlx90393_check_status(cmd, rx[0]);
+}
+
+static int mlx90393_read_reg(struct mlx90393_data *data, u8 reg, u16 *val)
+{
+	u8 tx[2];
+	u8 rx[3];
+	int ret;
+
+	tx[0] = MLX90393_CMD_RR;
+	/* Register address is encoded in bits [7:2] */
+	tx[1] = reg << 2;
+
+	ret = mlx90393_xfer(data, tx, sizeof(tx), rx, sizeof(rx));
+	if (ret)
+		return ret;
+
+	ret = mlx90393_check_status(tx[0], rx[0]);
+	if (ret)
+		return ret;
+
+	*val = get_unaligned_be16(&rx[1]);
+
+	return 0;
+}
+
+static int mlx90393_write_reg(struct mlx90393_data *data, u8 reg, u16 val)
+{
+	u8 tx[4];
+	u8 status;
+	int ret;
+
+	tx[0] = MLX90393_CMD_WR;
+	put_unaligned_be16(val, &tx[1]);
+	/* Register address is encoded in bits [7:2] */
+	tx[3] = reg << 2;
+
+	ret = mlx90393_xfer(data, tx, sizeof(tx), &status, 1);
+	if (ret)
+		return ret;
+
+	return mlx90393_check_status(tx[0], status);
+}
+
+static int mlx90393_update_bits(struct mlx90393_data *data, u8 reg,
+				u16 mask, u16 val)

drivers/iio/magnetometer/mlx90393_core.c line 271

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+{
+	u16 reg_val;
+	int ret;
+
+	ret = mlx90393_read_reg(data, reg, &reg_val);
+	if (ret)
+		return ret;
+
+	reg_val &= ~mask;
+	reg_val |= (val << __ffs(mask)) & mask;

Usually, we would call FIELD_PREP() before passing val rather than doing this here. As it is, this doesn't allow updating multiple fields at the same time.

drivers/iio/magnetometer/mlx90393_core.c line 274

Quoted 3 lines
drivers/iio/magnetometer/mlx90393_core.c new file +3 -0
+
+	return mlx90393_write_reg(data, reg, reg_val);
+}

Why not create a regmap with a custom bus type instead of reimplementing functions like this?

Quoted 13 lines
drivers/iio/magnetometer/mlx90393_core.c new file +13 -0
+
+static int mlx90393_read_measurement(struct mlx90393_data *data,
+				     enum mlx90393_channels chan, int *val)
+{
+	u8 rx[9];
+	int ret;
+
+	/* Start measurement */
+	ret = mlx90393_write_cmd(data, MLX90393_CMD_SM | MLX90393_MEASURE_ALL);
+	if (ret)
+		return ret;
+
+	/* Wait conversion */

drivers/iio/magnetometer/mlx90393_core.c line 297

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+	fsleep(mlx90393_get_tconv_us(data));
+
+	/* Read measurement */
+	ret = mlx90393_read_cmd(data, MLX90393_CMD_RM | MLX90393_MEASURE_ALL,
+				rx, sizeof(rx));
+	if (ret)
+		return ret;
+	/*
+	 * Measurement response layout:
+	 * [status][temp][x][y][z]

Would be more useful if this also said how many bytes each field is. Although not sure how useful comment is since we can see this below. Other than mentioning that status is ignored.

Quoted 15 lines
drivers/iio/magnetometer/mlx90393_core.c new file +15 -0
+	 */
+
+	switch (chan) {
+	case MLX90393_CHAN_TEMP:
+		*val = get_unaligned_be16(&rx[1]);
+		return 0;
+
+	case MLX90393_CHAN_X:
+		*val = sign_extend32(get_unaligned_be16(&rx[3]), 15);
+		return 0;
+
+	case MLX90393_CHAN_Y:
+		*val = sign_extend32(get_unaligned_be16(&rx[5]), 15);
+		return 0;
+

drivers/iio/magnetometer/mlx90393_core.c line 322

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+	case MLX90393_CHAN_Z:
+		*val = sign_extend32(get_unaligned_be16(&rx[7]), 15);
+		return 0;
+
+	default:
+		return -EINVAL;
+	}
+}
+
+static int mlx90393_get_scale(struct mlx90393_data *data,

Would be nice if the function name included _mag_ since this doesn't include the temperature scale.

Quoted 11 lines
drivers/iio/magnetometer/mlx90393_core.c new file +11 -0
+			      const struct iio_chan_spec *chan,
+			      int *val, int *val2)
+{
+	enum mlx90393_axis_type axis;
+	u8 res;
+
+	if (chan->channel2 == IIO_MOD_Z) {
+		axis = MLX90393_AXIS_TYPE_Z;
+		res = data->res_z;
+	} else {
+		axis = MLX90393_AXIS_TYPE_XY;

drivers/iio/magnetometer/mlx90393_core.c line 343

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+		res = data->res_xy;
+	}
+
+	*val = 0;
+	*val2 = mlx90393_scale_table[axis][data->gain_sel][res];
+
+	return IIO_VAL_INT_PLUS_NANO;
+}
+
+static int mlx90393_find_scale(struct mlx90393_data *data, bool z_axis,

same here.

drivers/iio/magnetometer/mlx90393_core.c line 345

Quoted 2 lines
drivers/iio/magnetometer/mlx90393_core.c new file +2 -0
+			       int val, int val2,
+			       int *gain)

don't need to wrap this last line.

drivers/iio/magnetometer/mlx90393_core.c line 348

Quoted 3 lines
drivers/iio/magnetometer/mlx90393_core.c new file +3 -0
+{
+	u8 res;
+	enum mlx90393_axis_type axis;

Putting enum first (like the previous funtion) looks nicer.

Quoted 52 lines
drivers/iio/magnetometer/mlx90393_core.c new file +52 -0
+
+	if (z_axis) {
+		axis = MLX90393_AXIS_TYPE_Z;
+		res = data->res_z;
+	} else {
+		axis = MLX90393_AXIS_TYPE_XY;
+		res = data->res_xy;
+	}
+
+	if (val != 0)
+		return -EINVAL;
+
+	for (unsigned int i = 0; i < ARRAY_SIZE(mlx90393_scale_table[0]); i++) {
+		if (mlx90393_scale_table[axis][i][res] == val2) {
+			*gain = i;
+			return 0;
+		}
+	}
+
+	return -EINVAL;
+}
+
+static int mlx90393_set_scale(struct mlx90393_data *data,
+			      const struct iio_chan_spec *chan,
+			      int val, int val2)
+{
+	bool z_axis;
+	int gain;
+	int ret;
+
+	z_axis = chan->channel2 == IIO_MOD_Z;
+
+	ret = mlx90393_find_scale(data, z_axis, val, val2, &gain);
+	if (ret)
+		return ret;
+
+	ret = mlx90393_update_bits(data, MLX90393_REG_CONF1, MLX90393_CONF1_GAIN_SEL,
+				   gain);
+	if (ret)
+		return ret;
+
+	data->gain_sel = gain;
+	return 0;
+}
+
+static int mlx90393_get_osr(struct mlx90393_data *data, int *val)
+{
+	*val = mlx90393_osr_avail[data->osr];
+	return IIO_VAL_INT;
+}
+
+static int mlx90393_find_osr(int val, int *osr)

drivers/iio/magnetometer/mlx90393_core.c line 410

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+{
+	for (unsigned int i = 0; i < MLX90393_OSR_MAX;  i++) {
+		if (mlx90393_osr_avail[i] == val) {
+			*osr = i;
+			return 0;
+		}
+	}
+
+	return -EINVAL;
+}

The two functions above are quite simple and only used once each. They could just be done inline instead.

drivers/iio/magnetometer/mlx90393_core.c line 418

Quoted 8 lines
drivers/iio/magnetometer/mlx90393_core.c new file +8 -0
+
+static int mlx90393_get_temp_osr2(struct mlx90393_data *data, int *val)
+{
+	*val = mlx90393_osr2_avail[data->osr2];
+	return IIO_VAL_INT;
+}
+
+static int mlx90393_set_osr(struct mlx90393_data *data, int val)

Would be more logical to keep this with other osr functions instead of being in the middle of osr2 functions.

Quoted 16 lines
drivers/iio/magnetometer/mlx90393_core.c new file +16 -0
+{
+	int osr;
+	int ret;
+
+	ret = mlx90393_find_osr(val, &osr);
+	if (ret)
+		return ret;
+
+	if (osr == data->osr)
+		return 0;
+
+	ret = mlx90393_update_bits(data, MLX90393_REG_CONF3, MLX90393_CONF3_OSR,
+				   osr);
+	if (ret)
+		return ret;
+

drivers/iio/magnetometer/mlx90393_core.c line 444

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+	data->osr = osr;
+	return 0;
+}
+
+static int mlx90393_set_temp_osr2(struct mlx90393_data *data, int val)
+{
+	int ret;
+
+	if (val < 0 || val >= MLX90393_OSR2_MAX)
+		return -EINVAL;

As mentioned already, allowing 0 doesn't make sense for oversampling_ratio.

Quoted 67 lines
drivers/iio/magnetometer/mlx90393_core.c new file +67 -0
+
+	if (val == data->osr2)
+		return 0;
+
+	ret = mlx90393_update_bits(data, MLX90393_REG_CONF3, MLX90393_CONF3_OSR2,
+				   val);
+	if (ret)
+		return ret;
+
+	data->osr2 = val;
+
+	return 0;
+}
+
+static int mlx90393_write_raw_get_fmt(struct iio_dev *indio_dev,
+				      struct iio_chan_spec const *chan,
+				      long mask)
+{
+	switch (mask) {
+	case IIO_CHAN_INFO_SCALE:
+		return IIO_VAL_INT_PLUS_NANO;
+
+	case IIO_CHAN_INFO_OVERSAMPLING_RATIO:
+		return IIO_VAL_INT;
+
+	default:
+		return -EINVAL;
+	}
+}
+
+static int mlx90393_write_raw(struct iio_dev *indio_dev,
+			      const struct iio_chan_spec *chan,
+			      int val, int val2,
+			      long mask)
+{
+	struct mlx90393_data *data = iio_priv(indio_dev);
+
+	switch (mask) {
+	case IIO_CHAN_INFO_SCALE: {
+		guard(mutex)(&data->lock);
+		return mlx90393_set_scale(data, chan, val, val2);
+	}
+
+	case IIO_CHAN_INFO_OVERSAMPLING_RATIO: {
+		guard(mutex)(&data->lock);
+		switch (chan->type) {
+		case IIO_TEMP:
+			return mlx90393_set_temp_osr2(data, val);
+
+		case IIO_MAGN:
+			return mlx90393_set_osr(data, val);
+
+		default:
+			return -EINVAL;
+		}
+	}
+	default:
+		return -EINVAL;
+	}
+}
+
+static int mlx90393_read_raw(struct iio_dev *indio_dev,
+			     const struct iio_chan_spec *chan,
+			     int *val, int *val2, long mask)
+{
+	struct mlx90393_data *data = iio_priv(indio_dev);
+	int ret;

drivers/iio/magnetometer/mlx90393_core.c line 521

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+
+	switch (mask) {
+	case IIO_CHAN_INFO_RAW: {
+		guard(mutex)(&data->lock);
+		ret = mlx90393_read_measurement(data, chan->address, val);
+		if (ret)
+			return ret;
+
+		return IIO_VAL_INT;
+		}

One too many tabs before }.

drivers/iio/magnetometer/mlx90393_core.c line 530

Quoted 9 lines
drivers/iio/magnetometer/mlx90393_core.c new file +9 -0
+	case IIO_CHAN_INFO_SCALE:
+		switch (chan->type) {
+		case IIO_MAGN:
+			return mlx90393_get_scale(data, chan, val, val2);
+
+		case IIO_TEMP:
+			/*
+			 * Datasheet Table 7: Thermal Specification
+			 */

Comment fits on one line.

drivers/iio/magnetometer/mlx90393_core.c line 532

Quoted 2 lines
drivers/iio/magnetometer/mlx90393_core.c new file +2 -0
+			*val = 0;
+			*val2 = 22124;

It this converting to millideg C?

From 45.2 LSB/deg C, I got:

			*val = 22;
			*val2 = 123893;
Quoted 47 lines
drivers/iio/magnetometer/mlx90393_core.c new file +47 -0
+			return IIO_VAL_INT_PLUS_MICRO;
+
+		default:
+			return -EINVAL;
+		}
+
+	case IIO_CHAN_INFO_OFFSET:
+		if (chan->type != IIO_TEMP)
+			return -EINVAL;
+		/*
+		 * Datasheet Table 7: Thermal Specification
+		 */
+
+		*val = -45114;
+		return IIO_VAL_INT;
+
+	case IIO_CHAN_INFO_OVERSAMPLING_RATIO:
+		switch (chan->type) {
+		case IIO_TEMP:
+			return mlx90393_get_temp_osr2(data, val);
+		case IIO_MAGN:
+			return mlx90393_get_osr(data, val);
+		default:
+			return -EINVAL;
+		}
+	default:
+		return -EINVAL;
+	}
+}
+
+static int mlx90393_read_avail(struct iio_dev *indio_dev,
+			       const struct iio_chan_spec *chan,
+			       const int **vals,
+			       int *type,
+			       int *length,
+			       long mask)
+{
+	struct mlx90393_data *data = iio_priv(indio_dev);
+	static int scale_avail[MLX90393_GAIN_MAX][MLX90393_AXIS_MAX];
+	enum mlx90393_axis_type axis;
+	u8 res;
+
+	switch (mask) {
+	case IIO_CHAN_INFO_SCALE: {
+		guard(mutex)(&data->lock);
+		axis = chan->channel2 == IIO_MOD_Z;
+		res = axis ? data->res_z : data->res_xy;

drivers/iio/magnetometer/mlx90393_core.c line 589

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+
+		for (unsigned int i = 0; i < MLX90393_GAIN_MAX; i++) {
+			scale_avail[i][0] = 0;
+			scale_avail[i][1] =
+				mlx90393_scale_table[axis][i][res];
+		}
+
+		*vals = &scale_avail[0][0];
+		*type = IIO_VAL_INT_PLUS_NANO;
+		*length = MLX90393_GAIN_MAX * MLX90393_AXIS_MAX;

Although MLX90393_AXIS_MAX has the correct value of 2, it doesn't makes sense semantically. The 2 just comes from the fact we have 2 ints per value in the array. Just use 2 here directly.

Quoted 50 lines
drivers/iio/magnetometer/mlx90393_core.c new file +50 -0
+		return IIO_AVAIL_LIST;
+	}
+
+	case IIO_CHAN_INFO_OVERSAMPLING_RATIO:
+		if (chan->type == IIO_TEMP) {
+			*vals = mlx90393_osr2_avail;
+			*type = IIO_VAL_INT;
+			*length = MLX90393_OSR2_MAX;
+		} else {
+			*vals = mlx90393_osr_avail;
+			*type = IIO_VAL_INT;
+			*length = MLX90393_OSR_MAX;
+		}
+		return IIO_AVAIL_LIST;
+
+	default:
+		return -EINVAL;
+	}
+}
+
+static const struct iio_info mlx90393_info = {
+	.read_raw = mlx90393_read_raw,
+	.write_raw = mlx90393_write_raw,
+	.read_avail = mlx90393_read_avail,
+	.write_raw_get_fmt = mlx90393_write_raw_get_fmt,
+};
+
+static int mlx90393_init(struct mlx90393_data *data)
+{
+	int ret;
+	u16 reg;
+
+	/* Exit mode */
+	ret = mlx90393_write_cmd(data, MLX90393_CMD_EX);
+	if (ret)
+		return ret;
+
+	/*
+	 * Datasheet section 15.4.1.2 (RT command), Figure 16:
+	 * Wait 1 ms after EX command before issuing RT.
+	 */
+	fsleep(1 * USEC_PER_MSEC);
+
+	/* Reset device */
+	ret = mlx90393_write_cmd(data, MLX90393_CMD_RT);
+	if (ret)
+		return ret;
+
+	/*
+	 * Datasheet section 15.4.1.2 (RT command), Figure 16:

drivers/iio/magnetometer/mlx90393_core.c line 649

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+	 * Wait 1.5 ms for the start-up sequence to complete.
+	 */
+	fsleep(1.5 * USEC_PER_MSEC);
+
+	ret = mlx90393_read_reg(data, MLX90393_REG_CONF1, &reg);
+	if (ret)
+		return ret;
+
+	data->gain_sel = FIELD_GET(MLX90393_CONF1_GAIN_SEL, reg);
+	data->hallconf = FIELD_GET(MLX90393_CONF1_HALLCONF, reg);

We should know the default values after reset already, so could hard-code them instead, no?

drivers/iio/magnetometer/mlx90393_core.c line 659

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+
+	ret = mlx90393_read_reg(data, MLX90393_REG_CONF3, &reg);
+	if (ret)
+		return ret;
+
+	data->res_xy = FIELD_GET(MLX90393_CONF3_RES_X, reg);
+	data->res_z = FIELD_GET(MLX90393_CONF3_RES_Z, reg);
+	data->dig_filt = FIELD_GET(MLX90393_CONF3_DIG_FILT, reg);
+	data->osr = FIELD_GET(MLX90393_CONF3_OSR, reg);
+	data->osr2 = FIELD_GET(MLX90393_CONF3_OSR2, reg);

Same here.

Quoted 124 lines
drivers/iio/magnetometer/mlx90393_core.c new file +42 -0
+
+	return 0;
+}
+
+int mlx90393_core_probe(struct device *dev,
+			const struct mlx90393_transfer_ops *ops,
+			void *context)
+{
+	struct iio_dev *indio_dev;
+	struct mlx90393_data *data;
+	int ret;
+
+	indio_dev = devm_iio_device_alloc(dev, sizeof(*data));
+	if (!indio_dev)
+		return -ENOMEM;
+
+	data = iio_priv(indio_dev);
+	ret = devm_mutex_init(dev, &data->lock);
+	if (ret)
+		return ret;
+
+	data->dev = dev;
+	data->ops = ops;
+	data->bus_context = context;
+
+	indio_dev->name = "mlx90393";
+	indio_dev->info = &mlx90393_info;
+	indio_dev->modes = INDIO_DIRECT_MODE;
+	indio_dev->channels = mlx90393_channels;
+	indio_dev->num_channels = ARRAY_SIZE(mlx90393_channels);
+
+	ret = mlx90393_init(data);
+	if (ret)
+		return dev_err_probe(dev, ret, "failed to initialize device\n");
+
+	return devm_iio_device_register(dev, indio_dev);
+}
+EXPORT_SYMBOL_NS_GPL(mlx90393_core_probe, "IIO_MLX90393");
+
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Nikhil Gautam <[email protected]>");
+MODULE_DESCRIPTION("MLX90393 magnetometer sensor driver");
drivers/iio/magnetometer/mlx90393_i2c.c new file +76 -0
new file mode 100644
index 000000000000..a9f0a40d15e8
@@ -0,0 +1,76 @@
+// SPDX-License-Identifier: GPL-2.0-only
+#include <linux/array_size.h>
+#include <linux/errno.h>
+#include <linux/i2c.h>
+#include <linux/module.h>
+#include <linux/mod_devicetable.h>
+
+#include "mlx90393.h"
+
+/*
+ * MLX90393 commands use repeated-start transfers where
+ * every command is followed by a status/data response.
+ */
+static int mlx90393_i2c_xfer(void *context,
+			     const u8 *tx, int tx_len,
+			     u8 *rx, int rx_len)
+{
+	struct i2c_client *client = context;
+	int ret;
+	struct i2c_msg msgs[2] = {
+		[0] = {
+			.addr = client->addr,
+			.len = tx_len,
+			.buf = (u8 *)tx,
+		},
+		[1] = {
+			.addr = client->addr,
+			.flags = I2C_M_RD,
+			.len = rx_len,
+			.buf = rx,
+		},
+	};
+
+	ret = i2c_transfer(client->adapter, msgs, ARRAY_SIZE(msgs));
+	if (ret != ARRAY_SIZE(msgs))
+		return ret < 0 ? ret : -EIO;
+
+	return 0;
+}
+
+static const struct mlx90393_transfer_ops mlx90393_i2c_ops = {
+	.xfer = mlx90393_i2c_xfer,
+};
+
+static int mlx90393_i2c_probe(struct i2c_client *client)
+{
+	return mlx90393_core_probe(&client->dev, &mlx90393_i2c_ops, client);
+}
+
+static const struct i2c_device_id mlx90393_id[] = {
+	{ .name = "mlx90393" },
+	{ }
+};
+MODULE_DEVICE_TABLE(i2c, mlx90393_id);
+
+static const struct of_device_id mlx90393_of_match[] = {
+	{ .compatible = "melexis,mlx90393" },
+	{ }
+};
+MODULE_DEVICE_TABLE(of, mlx90393_of_match);
+
+static struct i2c_driver mlx90393_i2c_driver = {
+	.driver = {
+		.name = "mlx90393",
+		.of_match_table = mlx90393_of_match,
+	},
+	.probe = mlx90393_i2c_probe,
+	.id_table = mlx90393_id,
+};
+
+module_i2c_driver(mlx90393_i2c_driver);
+
+MODULE_IMPORT_NS("IIO_MLX90393");
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Nikhil Gautam <[email protected]>");
+MODULE_DESCRIPTION("MLX90393 magnetometer sensor driver");
maintainer
#6 lore raw
In reply to Nikhil Gautam
Quoted 14 lines · On Sat, 27 Jun 2026 15:25:19 +0530 Nikhil Gautam <[email protected]> wrote:

Add Industrial I/O subsystem support for the Melexis MLX90393 3-axis magnetometer and temperature sensor.

The driver currently supports:

raw magnetic field measurements
raw temperature measurements
configurable gain/scale selection
configurable oversampling ratio
direct mode operation

The MLX90393 supports both I2C and SPI interfaces. This initial implementation adds support for the I2C interface.

Quoted 8 lines

The device uses a command-based communication protocol rather than a conventional register-addressed interface. A small transport abstraction layer is therefore used instead of regmap to share the common sensor logic between the current I2C implementation and future SPI support without duplicating code.

Hi Nikhil,

You have a lot of review already so I'm only going to take a very quick look at this version.

Also take a look at:
https://sashiko.dev/#/patchset/20260627095519.8377-1-nikhilgtr%40gmail.com

The osr2_avail comment at least looks superficially valid as an oversampling ratio of 0 means you don't read the value at all.

I2C dma safety one is wrong (that keeps coming up) as you have to opt in to use DMA directly on buffers when using the Linux I2C stack.

Some of the later ones look plausible but over to you to analyse them. It can be useful to address any you don't agree with by replying to this thread to quote them and say why they are wrong

Thanks,

Jonathan

Quoted 12 lines
drivers/iio/magnetometer/mlx90393_core.c new file +6 -0
new file mode 100644
index 000000000000..78fcab8a2b56
 
+static const int mlx90393_scale_table[MLX90393_AXIS_MAX][MLX90393_GAIN_MAX]
+				      [MLX90393_RES_MAX] = {
+	/* XY axis */
+	{
+		{ 751, 1502, 3004, 6009},
+		{ 601, 1202, 2403, 4840},

drivers/iio/magnetometer/mlx90393_core.c line 71

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+		{ 451, 901, 1803, 3605},
+		{ 376, 751, 1502, 3004},
+		{ 300, 601, 1202, 2403},
+		{ 250, 501, 1001, 2003},
+		{ 200, 401, 801, 1602},
+		{ 150, 300, 601, 1202},
+	},
+	/* Z axis */
+	{
+		{ 1210, 2420, 4840, 9680},

Space before } on all these

Quoted 29 lines
drivers/iio/magnetometer/mlx90393_core.c new file +28 -0
+		{ 968, 1936, 3872, 7744},
+		{ 726, 1452, 2904, 5808},
+		{ 605, 1210, 2420, 4840},
+		{ 484, 968, 1936, 3872},
+		{ 403, 807, 1613, 3227},
+		{ 323, 645, 1291, 2581},
+		{ 242, 484, 968, 1936},
+	}
+};
+
+static const int mlx90393_osr2_avail[MLX90393_OSR2_MAX] = {
+	0, 1, 2, 3,
+};
 
+static int mlx90393_read_raw(struct iio_dev *indio_dev,
+			     const struct iio_chan_spec *chan,
+			     int *val, int *val2, long mask)
+{
+	struct mlx90393_data *data = iio_priv(indio_dev);
+	int ret;
+
+	switch (mask) {
+	case IIO_CHAN_INFO_RAW: {
+		guard(mutex)(&data->lock);
+		ret = mlx90393_read_measurement(data, chan->address, val);
+		if (ret)
+			return ret;
+
+		return IIO_VAL_INT;

drivers/iio/magnetometer/mlx90393_core.c line 181

Quoted 10 lines
drivers/iio/magnetometer/mlx90393_core.c new file +10 -0
+		}
+	case IIO_CHAN_INFO_SCALE:
+		switch (chan->type) {
+		case IIO_MAGN:
+			return mlx90393_get_scale(data, chan, val, val2);
+
+		case IIO_TEMP:
+			/*
+			 * Datasheet Table 7: Thermal Specification
+			 */

Single line comment.

drivers/iio/magnetometer/mlx90393_core.c line 543

Quoted 13 lines
drivers/iio/magnetometer/mlx90393_core.c new file +13 -0
+			*val = 0;
+			*val2 = 22124;
+			return IIO_VAL_INT_PLUS_MICRO;
+
+		default:
+			return -EINVAL;
+		}
+
+	case IIO_CHAN_INFO_OFFSET:
+		if (chan->type != IIO_TEMP)
+			return -EINVAL;
+		/*
+		 * Datasheet Table 7: Thermal Specification

Single line comment.

Quoted 25 lines
drivers/iio/magnetometer/mlx90393_core.c new file +18 -0
+		 */
+
+		*val = -45114;
+		return IIO_VAL_INT;
+
+	case IIO_CHAN_INFO_OVERSAMPLING_RATIO:
+		switch (chan->type) {
+		case IIO_TEMP:
+			return mlx90393_get_temp_osr2(data, val);
+		case IIO_MAGN:
+			return mlx90393_get_osr(data, val);
+		default:
+			return -EINVAL;
+		}
+	default:
+		return -EINVAL;
+	}
+}
drivers/iio/magnetometer/mlx90393_i2c.c new file +0 -0
new file mode 100644
index 000000000000..a9f0a40d15e8

...

drivers/iio/magnetometer/mlx90393_i2c.c line 48

Quoted 9 lines
drivers/iio/magnetometer/mlx90393_i2c.c new file +9 -0
+
+static const struct mlx90393_transfer_ops mlx90393_i2c_ops = {
+	.xfer = mlx90393_i2c_xfer,
+};
+
+static int mlx90393_i2c_probe(struct i2c_client *client)
+{
+	return mlx90393_core_probe(&client->dev, &mlx90393_i2c_ops, client);
+}

Drop this blank line. It is a common convention to not have one here as it visually associates the macro with the data it is using (which isn't used anywhere else!)

Quoted 6 lines
drivers/iio/magnetometer/mlx90393_i2c.c new file +6 -0
+module_i2c_driver(mlx90393_i2c_driver);
+
+MODULE_IMPORT_NS("IIO_MLX90393");
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Nikhil Gautam <[email protected]>");
+MODULE_DESCRIPTION("MLX90393 magnetometer sensor driver");
#7 lore raw
In reply to Nikhil Gautam
Quoted 26 lines · Nikhil Gautam

Hi,

This series adds initial Industrial I/O subsystem support for the Melexis MLX90393 3-axis magnetometer and temperature sensor.

The MLX90393 supports both I2C and SPI interfaces. This series implements support for the I2C interface while keeping the driver structure transport-independent to simplify future SPI support.

The device uses a command-based communication protocol rather than a conventional register-addressed interface. A small transport abstraction layer is therefore used instead of regmap to share the common sensor logic between the current I2C implementation and future SPI support.

Currently supported features:

* Raw magnetic field measurements for X/Y/Z axes
* Raw temperature measurements
* Configurable gain/scale selection
* Configurable oversampling ratio
* Direct mode operation through the IIO subsystem
* I2C interface support

The driver has been tested on Raspberry Pi 5 hardware using an MLX90393 sensor connected over I2C. Magnetic field and temperature measurements were verified through the IIO sysfs interface.

...

Quoted 3 lines

v3: https://lore.kernel.org/linux-iio/[email protected]/T/#u
v2: https://lore.kernel.org/linux-iio/[email protected]/T/#t
v1: https://lore.kernel.org/linux-iio/[email protected]/

No, do not issue versions so often, especially for a new driver. *Minimum* of one week must be between two versions.

NAK. Please, go to v3, study, address, and settle down the comments given there.

Signature · 2 lines
With Best Regards,
Andy Shevchenko