[PATCH v4 0/2] iio: pressure: add Honeywell ABP2 driver

[PATCH v4 0/2] iio: pressure: add Honeywell ABP2 driver

[Petre Rodan]

Adds driver for digital Honeywell ABP2 series of board mount
pressure and temperature sensors.

This driver differs quite a bit from the Honeywell ABP series that
is already present in the kernel:

- many more pressure-triplet variations of pressure ranges and
units
- extra end of conversion interrupt feature
- implements both i2c and SPI interfaces
- uses a bidirectional data retrieving protocol (4 wire SPI instead of 3 wire)
- 24+24bit resolution vs 14+11bit on the ABP

Specific low level i2c and spi data transfer API is used instead
of regmap because the protocol is based on simple commands instead
of on a memory map.

Given the fact that the sensor can perform gage and differential
pressure measurements with a full-scale range of down to 500 pascals
the offset and scale are calculated in such a way to provide a result
in pascals, not kilopascals. Just like the Honeywell MPR driver.
For a kilopascal output the scale representation would lose too much
precision.

Tested on two sensors (ABP2MRRT001PDSA3 and ABP2DANT001BA2A3)

Datasheet: https://prod-edam.honeywell.com/content/dam/honeywell-edam/sps/siot/en-us/products/sensors/pressure-sensors/board-mount-pressure-sensors/basic-abp2-series/documents/sps-siot-abp2-series-datasheet-32350268-en.pdf
Signed-off-by: Petre Rodan <petre.rodan@subdimension.ro>
---
v1 -> v2:
 - bindings are unchanged, added tag from Krzysztof
 - driver code changes based on Andy's feedback - detailed list in the respective patch
v2 -> v3:
 - driver code changes based on Andy's feedback - detailed list in the respective patch
v3 -> v4
 - driver code changes based on Jonathan's feedback - detailed list in the respective patch
 - Link to v3: https://lore.kernel.org/r/20251204-honeywell_abp2_driver-v3-0-ad3275bad35e@subdimension.ro

---
Petre Rodan (2):
      dt-bindings: iio: pressure: add honeywell,abp2030pa
      iio: pressure: add Honeywell ABP2 driver

 .../bindings/iio/pressure/honeywell,abp2030pa.yaml | 132 +++++
 MAINTAINERS                                        |   7 +
 drivers/iio/pressure/Kconfig                       |  29 ++
 drivers/iio/pressure/Makefile                      |   3 +
 drivers/iio/pressure/abp2030pa.c                   | 544 +++++++++++++++++++++
 drivers/iio/pressure/abp2030pa.h                   |  73 +++
 drivers/iio/pressure/abp2030pa_i2c.c               |  90 ++++
 drivers/iio/pressure/abp2030pa_spi.c               |  67 +++
 8 files changed, 945 insertions(+)
---
base-commit: f9e05791642810a0cf6237d39fafd6fec5e0b4bb
change-id: 20251122-honeywell_abp2_driver-9c796ef0c4b2

Best regards,
-- 
Petre Rodan <petre.rodan@subdimension.ro>

[PATCH v4 1/2] dt-bindings: iio: pressure: add honeywell,abp2030pa

[Petre Rodan]

Adds binding for digital Honeywell ABP2 series pressure and temperature
sensors.
The i2c address is hardcoded and depends on the part number.
There is an optional interrupt that signals the end of conversion.

Signed-off-by: Petre Rodan <petre.rodan@subdimension.ro>
---
    v1->v3 unchanged
    v3->v4 'vdd-supply: true' change requested by Jonathan
---
 .../bindings/iio/pressure/honeywell,abp2030pa.yaml | 132 +++++++++++++++++++++
 MAINTAINERS                                        |   6 +
 2 files changed, 138 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/pressure/honeywell,abp2030pa.yaml b/Documentation/devicetree/bindings/iio/pressure/honeywell,abp2030pa.yaml
new file mode 100644
index 000000000000..e82897ffac3b
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/pressure/honeywell,abp2030pa.yaml
@@ -0,0 +1,132 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/pressure/honeywell,abp2030pa.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Honeywell abp2030pa pressure sensor
+
+maintainers:
+  - Petre Rodan <petre.rodan@subdimension.ro>
+
+description: |
+  Honeywell pressure sensor of model abp2030pa.
+
+  This sensor has an I2C and SPI interface.
+
+  There are many models with different pressure ranges available. The vendor
+  calls them "ABP2 series". All of them have an identical programming model and
+  differ in the pressure range and measurement unit.
+
+  To support different models one needs to specify its pressure triplet.
+
+  For custom silicon chips not covered by the Honeywell ABP2 series datasheet,
+  the pressure values can be specified manually via honeywell,pmin-pascal and
+  honeywell,pmax-pascal.
+
+  Specifications about the devices can be found at:
+  https://prod-edam.honeywell.com/content/dam/honeywell-edam/sps/siot/en-us/products/sensors/pressure-sensors/board-mount-pressure-sensors/basic-abp2-series/documents/sps-siot-abp2-series-datasheet-32350268-en.pdf
+
+properties:
+  compatible:
+    const: honeywell,abp2030pa
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    description:
+      Optional interrupt for indicating end of conversion.
+      SPI variants of ABP2 chips do not provide this feature.
+    maxItems: 1
+
+  honeywell,pressure-triplet:
+    description: |
+      Case-sensitive five character string that defines pressure range, unit
+      and type as part of the device nomenclature. In the unlikely case of a
+      custom chip, unset and provide pmin-pascal and pmax-pascal instead.
+    enum: [001BA, 1.6BA, 2.5BA, 004BA, 006BA, 008BA, 010BA, 012BA, 001BD,
+           1.6BD, 2.5BD, 004BD, 001BG, 1.6BG, 2.5BG, 004BG, 006BG, 008BG,
+           010BG, 012BG, 001GG, 1.2GG, 100KA, 160KA, 250KA, 001KD, 1.6KD,
+           2.5KD, 004KD, 006KD, 010KD, 016KD, 025KD, 040KD, 060KD, 100KD,
+           160KD, 250KD, 400KD, 001KG, 1.6KG, 2.5KG, 004KG, 006KG, 010KG,
+           016KG, 025KG, 040KG, 060KG, 100KG, 160KG, 250KG, 400KG, 600KG,
+           800KG, 250LD, 600LD, 600LG, 2.5MD, 006MD, 010MD, 016MD, 025MD,
+           040MD, 060MD, 100MD, 160MD, 250MD, 400MD, 600MD, 006MG, 010MG,
+           016MG, 025MG, 040MG, 060MG, 100MG, 160MG, 250MG, 400MG, 600MG,
+           001ND, 002ND, 004ND, 005ND, 010ND, 020ND, 030ND, 002NG, 004NG,
+           005NG, 010NG, 020NG, 030NG, 015PA, 030PA, 060PA, 100PA, 150PA,
+           175PA, 001PD, 005PD, 015PD, 030PD, 060PD, 001PG, 005PG, 015PG,
+           030PG, 060PG, 100PG, 150PG, 175PG]
+    $ref: /schemas/types.yaml#/definitions/string
+
+  honeywell,pmin-pascal:
+    description:
+      Minimum pressure value the sensor can measure in pascal.
+
+  honeywell,pmax-pascal:
+    description:
+      Maximum pressure value the sensor can measure in pascal.
+
+  spi-max-frequency:
+    maximum: 800000
+
+  vdd-supply: true
+
+required:
+  - compatible
+  - reg
+  - vdd-supply
+
+oneOf:
+  - required:
+      - honeywell,pressure-triplet
+  - required:
+      - honeywell,pmin-pascal
+      - honeywell,pmax-pascal
+
+allOf:
+  - $ref: /schemas/spi/spi-peripheral-props.yaml
+  - if:
+      required:
+        - honeywell,pressure-triplet
+    then:
+      properties:
+        honeywell,pmin-pascal: false
+        honeywell,pmax-pascal: false
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/gpio/gpio.h>
+    #include <dt-bindings/interrupt-controller/irq.h>
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pressure@18 {
+            compatible = "honeywell,abp2030pa";
+            reg = <0x18>;
+            interrupt-parent = <&gpio3>;
+            interrupts = <21 IRQ_TYPE_EDGE_RISING>;
+
+            honeywell,pressure-triplet = "001BA";
+            vdd-supply = <&vcc_3v3>;
+        };
+    };
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pressure@0 {
+            compatible = "honeywell,abp2030pa";
+            reg = <0>;
+            spi-max-frequency = <800000>;
+
+            honeywell,pressure-triplet = "001PD";
+            vdd-supply = <&vcc_3v3>;
+        };
+    };
+...
diff --git a/MAINTAINERS b/MAINTAINERS
index 31d98efb1ad1..15b92300acbc 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11404,6 +11404,12 @@ F:	lib/test_hmm*
 F:	mm/hmm*
 F:	tools/testing/selftests/mm/*hmm*
 
+HONEYWELL ABP2030PA PRESSURE SENSOR SERIES IIO DRIVER
+M:	Petre Rodan <petre.rodan@subdimension.ro>
+L:	linux-iio@vger.kernel.org
+S:	Maintained
+F:	Documentation/devicetree/bindings/iio/pressure/honeywell,abp2030pa.yaml
+
 HONEYWELL HSC030PA PRESSURE SENSOR SERIES IIO DRIVER
 M:	Petre Rodan <petre.rodan@subdimension.ro>
 L:	linux-iio@vger.kernel.org

-- 
2.51.2

[PATCH v4 2/2] iio: pressure: add Honeywell ABP2 driver

[Petre Rodan]

Adds driver for digital Honeywell ABP2 series of board mount
pressure and temperature sensors.

This driver covers 113 different pressure ranges and units on
both i2c and SPI buses.

The communication protocol involves sending two simple commands
to the sensor and there is no register access or a memory map.
For this reason the regmap API was not used.

The i2c address is hardcoded and depends on the part number.

Optional end of conversion interrupt control is present on the
i2c variants of the chips.
The EOC can also be defined for the SPI variants if a non-ABP2
but compatible chip is to be driven.

Tested on two sensors (ABP2MRRT001PDSA3 and ABP2DANT001BA2A3).

Datasheet: https://prod-edam.honeywell.com/content/dam/honeywell-edam/sps/siot/en-us/products/sensors/pressure-sensors/board-mount-pressure-sensors/basic-abp2-series/d
ocuments/sps-siot-abp2-series-datasheet-32350268-en.pdf
Signed-off-by: Petre Rodan <petre.rodan@subdimension.ro>
---
v1 -> v2 changes based on Andy's review
commit msg:
   - Datasheet tag
   - mention the lack of regmap use
Kconfig:
   - user can nicely select bus drivers separately
abp2030pa.c:
   - add a few includes
   - blank line trimming
   - enum without assignments
   - use the recommended trailing in the arrays
   - less const
   - comment for the 5 * USEC_PER_MSEC fsleep()
   - curly braces for else branch
   - improved error codes
   - remove INDIO_DIRECT_MODE line
   - remove call to _ops->init
abp2030pa.h:
   - add completion header
   - remove a few structs
   - rename buffer -> rx_buf in abp2_data struct
   - add tx_buf to abp2_data struct
abp2030pa_i2c.c, abp2030pa_spi.c:
   - remove _init() ops
   - remove redundant else
   - remove device.h include from _spi.c
v2 -> v3 changes based on Andy's review
   - move a few private constants from .h to .c
   - replace ABP2_VARIANTS_MAX with a static_assert
   - Kconfig select dependencies in _core symbol
   - move memset call
   - rename jump label
   - change indentation
   - s/IRQF_TRIGGER_RISING/IRQF_ONESHOT/
   - add comment about latch-up sensitivity right above the invalid status
  error
v3 -> v4 changes based on Jonathan's review
   - remove sysfs include
   - add back INDIO_DIRECT_MODE
   - add available_scan_masks
   - tweak cacheline alignment as per Jonathan's input
   - change abp2_i2c_write() to be more generic, just as it's spi counterpart
   - zero out xfer struct in abp2_spi_xfer()
---
 MAINTAINERS                          |   1 +
 drivers/iio/pressure/Kconfig         |  29 ++
 drivers/iio/pressure/Makefile        |   3 +
 drivers/iio/pressure/abp2030pa.c     | 544 +++++++++++++++++++++++++++++++++++
 drivers/iio/pressure/abp2030pa.h     |  73 +++++
 drivers/iio/pressure/abp2030pa_i2c.c |  90 ++++++
 drivers/iio/pressure/abp2030pa_spi.c |  67 +++++
 7 files changed, 807 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 15b92300acbc..0aa9cba68137 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11409,6 +11409,7 @@ M:	Petre Rodan <petre.rodan@subdimension.ro>
 L:	linux-iio@vger.kernel.org
 S:	Maintained
 F:	Documentation/devicetree/bindings/iio/pressure/honeywell,abp2030pa.yaml
+F:	drivers/iio/pressure/abp2030pa*
 
 HONEYWELL HSC030PA PRESSURE SENSOR SERIES IIO DRIVER
 M:	Petre Rodan <petre.rodan@subdimension.ro>
diff --git a/drivers/iio/pressure/Kconfig b/drivers/iio/pressure/Kconfig
index 2fe9dc90cceb..b2720be51506 100644
--- a/drivers/iio/pressure/Kconfig
+++ b/drivers/iio/pressure/Kconfig
@@ -16,6 +16,35 @@ config ABP060MG
 	  To compile this driver as a module, choose M here: the module
 	  will be called abp060mg.
 
+config ABP2030PA
+	tristate
+	select IIO_BUFFER
+	select IIO_TRIGGERED_BUFFER
+
+config ABP2030PA_I2C
+	tristate "Honeywell ABP2 pressure sensor series I2C driver"
+	depends on I2C
+	select ABP2030PA
+	help
+	  Say Y here to build I2C bus support for the Honeywell ABP2
+	  series pressure and temperature digital sensor.
+
+	  To compile this driver as a module, choose M here: the module
+	  will be called abp2030pa_i2c and you will also get abp2030pa
+	  for the core module.
+
+config ABP2030PA_SPI
+	tristate "Honeywell ABP2 pressure sensor series SPI driver"
+	depends on SPI_MASTER
+	select ABP2030PA
+	help
+	  Say Y here to build I2C bus support for the Honeywell ABP2
+	  series pressure and temperature digital sensor.
+
+	  To compile this driver as a module, choose M here: the module
+	  will be called abp2030pa_spi and you will also get abp2030pa
+	  for the core module.
+
 config ROHM_BM1390
 	tristate "ROHM BM1390GLV-Z pressure sensor driver"
 	depends on I2C
diff --git a/drivers/iio/pressure/Makefile b/drivers/iio/pressure/Makefile
index a21443e992b9..bc0d11a20acc 100644
--- a/drivers/iio/pressure/Makefile
+++ b/drivers/iio/pressure/Makefile
@@ -5,6 +5,9 @@
 
 # When adding new entries keep the list in alphabetical order
 obj-$(CONFIG_ABP060MG) += abp060mg.o
+obj-$(CONFIG_ABP2030PA) += abp2030pa.o
+obj-$(CONFIG_ABP2030PA_I2C) += abp2030pa_i2c.o
+obj-$(CONFIG_ABP2030PA_SPI) += abp2030pa_spi.o
 obj-$(CONFIG_ADP810) += adp810.o
 obj-$(CONFIG_ROHM_BM1390) += rohm-bm1390.o
 obj-$(CONFIG_BMP280) += bmp280.o
diff --git a/drivers/iio/pressure/abp2030pa.c b/drivers/iio/pressure/abp2030pa.c
new file mode 100644
index 000000000000..be11e28208ec
--- /dev/null
+++ b/drivers/iio/pressure/abp2030pa.c
@@ -0,0 +1,544 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Honeywell ABP2 series pressure sensor driver
+ *
+ * Copyright (c) 2025 Petre Rodan <petre.rodan@subdimension.ro>
+ *
+ * Datasheet: https://prod-edam.honeywell.com/content/dam/honeywell-edam/sps/siot/en-us/products/sensors/pressure-sensors/board-mount-pressure-sensors/basic-abp2-series/documents/sps-siot-abp2-series-datasheet-32350268-en.pdf
+ */
+
+#include <linux/array_size.h>
+#include <linux/bits.h>
+#include <linux/completion.h>
+#include <linux/delay.h>
+#include <linux/dev_printk.h>
+#include <linux/device.h>
+#include <linux/errno.h>
+#include <linux/export.h>
+#include <linux/gpio/consumer.h>
+#include <linux/interrupt.h>
+#include <linux/jiffies.h>
+#include <linux/math64.h>
+#include <linux/module.h>
+#include <linux/property.h>
+#include <linux/regulator/consumer.h>
+#include <linux/string.h>
+#include <linux/time.h>
+#include <linux/types.h>
+#include <linux/unaligned.h>
+#include <linux/units.h>
+
+#include <linux/iio/buffer.h>
+#include <linux/iio/iio.h>
+#include <linux/iio/trigger_consumer.h>
+#include <linux/iio/triggered_buffer.h>
+
+#include "abp2030pa.h"
+
+/* Status byte flags */
+#define ABP2_ST_POWER     BIT(6) /* 1 if device is powered */
+#define ABP2_ST_BUSY      BIT(5) /* 1 if device is busy */
+
+#define ABP2_CMD_NOP      0xf0
+#define ABP2_CMD_SYNC     0xaa
+#define ABP2_PKT_SYNC_LEN 3
+#define ABP2_PKT_NOP_LEN  ABP2_MEASUREMENT_RD_SIZE
+
+struct abp2_func_spec {
+	u32 output_min;
+	u32 output_max;
+};
+
+/* transfer function A: 10%   to 90%   of 2^24 */
+static const struct abp2_func_spec abp2_func_spec[] = {
+	[ABP2_FUNCTION_A] = { .output_min = 1677722, .output_max = 15099494 },
+};
+
+enum abp2_variants {
+	ABP2001BA, ABP21_6BA, ABP22_5BA, ABP2004BA, ABP2006BA, ABP2008BA,
+	ABP2010BA, ABP2012BA, ABP2001BD, ABP21_6BD, ABP22_5BD, ABP2004BD,
+	ABP2001BG, ABP21_6BG, ABP22_5BG, ABP2004BG, ABP2006BG, ABP2008BG,
+	ABP2010BG, ABP2012BG, ABP2001GG, ABP21_2GG, ABP2100KA, ABP2160KA,
+	ABP2250KA, ABP2001KD, ABP21_6KD, ABP22_5KD, ABP2004KD, ABP2006KD,
+	ABP2010KD, ABP2016KD, ABP2025KD, ABP2040KD, ABP2060KD, ABP2100KD,
+	ABP2160KD, ABP2250KD, ABP2400KD, ABP2001KG, ABP21_6KG, ABP22_5KG,
+	ABP2004KG, ABP2006KG, ABP2010KG, ABP2016KG, ABP2025KG, ABP2040KG,
+	ABP2060KG, ABP2100KG, ABP2160KG, ABP2250KG, ABP2400KG, ABP2600KG,
+	ABP2800KG, ABP2250LD, ABP2600LD, ABP2600LG, ABP22_5MD, ABP2006MD,
+	ABP2010MD, ABP2016MD, ABP2025MD, ABP2040MD, ABP2060MD, ABP2100MD,
+	ABP2160MD, ABP2250MD, ABP2400MD, ABP2600MD, ABP2006MG, ABP2010MG,
+	ABP2016MG, ABP2025MG, ABP2040MG, ABP2060MG, ABP2100MG, ABP2160MG,
+	ABP2250MG, ABP2400MG, ABP2600MG, ABP2001ND, ABP2002ND, ABP2004ND,
+	ABP2005ND, ABP2010ND, ABP2020ND, ABP2030ND, ABP2002NG, ABP2004NG,
+	ABP2005NG, ABP2010NG, ABP2020NG, ABP2030NG, ABP2015PA, ABP2030PA,
+	ABP2060PA, ABP2100PA, ABP2150PA, ABP2175PA, ABP2001PD, ABP2005PD,
+	ABP2015PD, ABP2030PD, ABP2060PD, ABP2001PG, ABP2005PG, ABP2015PG,
+	ABP2030PG, ABP2060PG, ABP2100PG, ABP2150PG, ABP2175PG,
+};
+
+static const char * const abp2_triplet_variants[] = {
+	[ABP2001BA] = "001BA", [ABP21_6BA] = "1.6BA", [ABP22_5BA] = "2.5BA",
+	[ABP2004BA] = "004BA", [ABP2006BA] = "006BA", [ABP2008BA] = "008BA",
+	[ABP2010BA] = "010BA", [ABP2012BA] = "012BA", [ABP2001BD] = "001BD",
+	[ABP21_6BD] = "1.6BD", [ABP22_5BD] = "2.5BD", [ABP2004BD] = "004BD",
+	[ABP2001BG] = "001BG", [ABP21_6BG] = "1.6BG", [ABP22_5BG] = "2.5BG",
+	[ABP2004BG] = "004BG", [ABP2006BG] = "006BG", [ABP2008BG] = "008BG",
+	[ABP2010BG] = "010BG", [ABP2012BG] = "012BG", [ABP2001GG] = "001GG",
+	[ABP21_2GG] = "1.2GG", [ABP2100KA] = "100KA", [ABP2160KA] = "160KA",
+	[ABP2250KA] = "250KA", [ABP2001KD] = "001KD", [ABP21_6KD] = "1.6KD",
+	[ABP22_5KD] = "2.5KD", [ABP2004KD] = "004KD", [ABP2006KD] = "006KD",
+	[ABP2010KD] = "010KD", [ABP2016KD] = "016KD", [ABP2025KD] = "025KD",
+	[ABP2040KD] = "040KD", [ABP2060KD] = "060KD", [ABP2100KD] = "100KD",
+	[ABP2160KD] = "160KD", [ABP2250KD] = "250KD", [ABP2400KD] = "400KD",
+	[ABP2001KG] = "001KG", [ABP21_6KG] = "1.6KG", [ABP22_5KG] = "2.5KG",
+	[ABP2004KG] = "004KG", [ABP2006KG] = "006KG", [ABP2010KG] = "010KG",
+	[ABP2016KG] = "016KG", [ABP2025KG] = "025KG", [ABP2040KG] = "040KG",
+	[ABP2060KG] = "060KG", [ABP2100KG] = "100KG", [ABP2160KG] = "160KG",
+	[ABP2250KG] = "250KG", [ABP2400KG] = "400KG", [ABP2600KG] = "600KG",
+	[ABP2800KG] = "800KG", [ABP2250LD] = "250LD", [ABP2600LD] = "600LD",
+	[ABP2600LG] = "600LG", [ABP22_5MD] = "2.5MD", [ABP2006MD] = "006MD",
+	[ABP2010MD] = "010MD", [ABP2016MD] = "016MD", [ABP2025MD] = "025MD",
+	[ABP2040MD] = "040MD", [ABP2060MD] = "060MD", [ABP2100MD] = "100MD",
+	[ABP2160MD] = "160MD", [ABP2250MD] = "250MD", [ABP2400MD] = "400MD",
+	[ABP2600MD] = "600MD", [ABP2006MG] = "006MG", [ABP2010MG] = "010MG",
+	[ABP2016MG] = "016MG", [ABP2025MG] = "025MG", [ABP2040MG] = "040MG",
+	[ABP2060MG] = "060MG", [ABP2100MG] = "100MG", [ABP2160MG] = "160MG",
+	[ABP2250MG] = "250MG", [ABP2400MG] = "400MG", [ABP2600MG] = "600MG",
+	[ABP2001ND] = "001ND", [ABP2002ND] = "002ND", [ABP2004ND] = "004ND",
+	[ABP2005ND] = "005ND", [ABP2010ND] = "010ND", [ABP2020ND] = "020ND",
+	[ABP2030ND] = "030ND", [ABP2002NG] = "002NG", [ABP2004NG] = "004NG",
+	[ABP2005NG] = "005NG", [ABP2010NG] = "010NG", [ABP2020NG] = "020NG",
+	[ABP2030NG] = "030NG", [ABP2015PA] = "015PA", [ABP2030PA] = "030PA",
+	[ABP2060PA] = "060PA", [ABP2100PA] = "100PA", [ABP2150PA] = "150PA",
+	[ABP2175PA] = "175PA", [ABP2001PD] = "001PD", [ABP2005PD] = "005PD",
+	[ABP2015PD] = "015PD", [ABP2030PD] = "030PD", [ABP2060PD] = "060PD",
+	[ABP2001PG] = "001PG", [ABP2005PG] = "005PG", [ABP2015PG] = "015PG",
+	[ABP2030PG] = "030PG", [ABP2060PG] = "060PG", [ABP2100PG] = "100PG",
+	[ABP2150PG] = "150PG", [ABP2175PG] = "175PG",
+};
+
+/**
+ * struct abp2_range_config - list of pressure ranges based on nomenclature
+ * @pmin: lowest pressure that can be measured
+ * @pmax: highest pressure that can be measured
+ */
+struct abp2_range_config {
+	s32 pmin;
+	s32 pmax;
+};
+
+/* All min max limits have been converted to pascals */
+static const struct abp2_range_config abp2_range_config[] = {
+	[ABP2001BA] = { .pmin =       0, .pmax =  100000 },
+	[ABP21_6BA] = { .pmin =       0, .pmax =  160000 },
+	[ABP22_5BA] = { .pmin =       0, .pmax =  250000 },
+	[ABP2004BA] = { .pmin =       0, .pmax =  400000 },
+	[ABP2006BA] = { .pmin =       0, .pmax =  600000 },
+	[ABP2008BA] = { .pmin =       0, .pmax =  800000 },
+	[ABP2010BA] = { .pmin =       0, .pmax = 1000000 },
+	[ABP2012BA] = { .pmin =       0, .pmax = 1200000 },
+	[ABP2001BD] = { .pmin = -100000, .pmax =  100000 },
+	[ABP21_6BD] = { .pmin = -160000, .pmax =  160000 },
+	[ABP22_5BD] = { .pmin = -250000, .pmax =  250000 },
+	[ABP2004BD] = { .pmin = -400000, .pmax =  400000 },
+	[ABP2001BG] = { .pmin =       0, .pmax =  100000 },
+	[ABP21_6BG] = { .pmin =       0, .pmax =  160000 },
+	[ABP22_5BG] = { .pmin =       0, .pmax =  250000 },
+	[ABP2004BG] = { .pmin =       0, .pmax =  400000 },
+	[ABP2006BG] = { .pmin =       0, .pmax =  600000 },
+	[ABP2008BG] = { .pmin =       0, .pmax =  800000 },
+	[ABP2010BG] = { .pmin =       0, .pmax = 1000000 },
+	[ABP2012BG] = { .pmin =       0, .pmax = 1200000 },
+	[ABP2001GG] = { .pmin =       0, .pmax = 1000000 },
+	[ABP21_2GG] = { .pmin =       0, .pmax = 1200000 },
+	[ABP2100KA] = { .pmin =       0, .pmax =  100000 },
+	[ABP2160KA] = { .pmin =       0, .pmax =  160000 },
+	[ABP2250KA] = { .pmin =       0, .pmax =  250000 },
+	[ABP2001KD] = { .pmin =   -1000, .pmax =    1000 },
+	[ABP21_6KD] = { .pmin =   -1600, .pmax =    1600 },
+	[ABP22_5KD] = { .pmin =   -2500, .pmax =    2500 },
+	[ABP2004KD] = { .pmin =   -4000, .pmax =    4000 },
+	[ABP2006KD] = { .pmin =   -6000, .pmax =    6000 },
+	[ABP2010KD] = { .pmin =  -10000, .pmax =   10000 },
+	[ABP2016KD] = { .pmin =  -16000, .pmax =   16000 },
+	[ABP2025KD] = { .pmin =  -25000, .pmax =   25000 },
+	[ABP2040KD] = { .pmin =  -40000, .pmax =   40000 },
+	[ABP2060KD] = { .pmin =  -60000, .pmax =   60000 },
+	[ABP2100KD] = { .pmin = -100000, .pmax =  100000 },
+	[ABP2160KD] = { .pmin = -160000, .pmax =  160000 },
+	[ABP2250KD] = { .pmin = -250000, .pmax =  250000 },
+	[ABP2400KD] = { .pmin = -400000, .pmax =  400000 },
+	[ABP2001KG] = { .pmin =       0, .pmax =    1000 },
+	[ABP21_6KG] = { .pmin =       0, .pmax =    1600 },
+	[ABP22_5KG] = { .pmin =       0, .pmax =    2500 },
+	[ABP2004KG] = { .pmin =       0, .pmax =    4000 },
+	[ABP2006KG] = { .pmin =       0, .pmax =    6000 },
+	[ABP2010KG] = { .pmin =       0, .pmax =   10000 },
+	[ABP2016KG] = { .pmin =       0, .pmax =   16000 },
+	[ABP2025KG] = { .pmin =       0, .pmax =   25000 },
+	[ABP2040KG] = { .pmin =       0, .pmax =   40000 },
+	[ABP2060KG] = { .pmin =       0, .pmax =   60000 },
+	[ABP2100KG] = { .pmin =       0, .pmax =  100000 },
+	[ABP2160KG] = { .pmin =       0, .pmax =  160000 },
+	[ABP2250KG] = { .pmin =       0, .pmax =  250000 },
+	[ABP2400KG] = { .pmin =       0, .pmax =  400000 },
+	[ABP2600KG] = { .pmin =       0, .pmax =  600000 },
+	[ABP2800KG] = { .pmin =       0, .pmax =  800000 },
+	[ABP2250LD] = { .pmin =    -250, .pmax =     250 },
+	[ABP2600LD] = { .pmin =    -600, .pmax =     600 },
+	[ABP2600LG] = { .pmin =       0, .pmax =     600 },
+	[ABP22_5MD] = { .pmin =    -250, .pmax =     250 },
+	[ABP2006MD] = { .pmin =    -600, .pmax =     600 },
+	[ABP2010MD] = { .pmin =   -1000, .pmax =    1000 },
+	[ABP2016MD] = { .pmin =   -1600, .pmax =    1600 },
+	[ABP2025MD] = { .pmin =   -2500, .pmax =    2500 },
+	[ABP2040MD] = { .pmin =   -4000, .pmax =    4000 },
+	[ABP2060MD] = { .pmin =   -6000, .pmax =    6000 },
+	[ABP2100MD] = { .pmin =  -10000, .pmax =   10000 },
+	[ABP2160MD] = { .pmin =  -16000, .pmax =   16000 },
+	[ABP2250MD] = { .pmin =  -25000, .pmax =   25000 },
+	[ABP2400MD] = { .pmin =  -40000, .pmax =   40000 },
+	[ABP2600MD] = { .pmin =  -60000, .pmax =   60000 },
+	[ABP2006MG] = { .pmin =       0, .pmax =     600 },
+	[ABP2010MG] = { .pmin =       0, .pmax =    1000 },
+	[ABP2016MG] = { .pmin =       0, .pmax =    1600 },
+	[ABP2025MG] = { .pmin =       0, .pmax =    2500 },
+	[ABP2040MG] = { .pmin =       0, .pmax =    4000 },
+	[ABP2060MG] = { .pmin =       0, .pmax =    6000 },
+	[ABP2100MG] = { .pmin =       0, .pmax =   10000 },
+	[ABP2160MG] = { .pmin =       0, .pmax =   16000 },
+	[ABP2250MG] = { .pmin =       0, .pmax =   25000 },
+	[ABP2400MG] = { .pmin =       0, .pmax =   40000 },
+	[ABP2600MG] = { .pmin =       0, .pmax =   60000 },
+	[ABP2001ND] = { .pmin =    -249, .pmax =     249 },
+	[ABP2002ND] = { .pmin =    -498, .pmax =     498 },
+	[ABP2004ND] = { .pmin =    -996, .pmax =     996 },
+	[ABP2005ND] = { .pmin =   -1245, .pmax =    1245 },
+	[ABP2010ND] = { .pmin =   -2491, .pmax =    2491 },
+	[ABP2020ND] = { .pmin =   -4982, .pmax =    4982 },
+	[ABP2030ND] = { .pmin =   -7473, .pmax =    7473 },
+	[ABP2002NG] = { .pmin =       0, .pmax =     498 },
+	[ABP2004NG] = { .pmin =       0, .pmax =     996 },
+	[ABP2005NG] = { .pmin =       0, .pmax =    1245 },
+	[ABP2010NG] = { .pmin =       0, .pmax =    2491 },
+	[ABP2020NG] = { .pmin =       0, .pmax =    4982 },
+	[ABP2030NG] = { .pmin =       0, .pmax =    7473 },
+	[ABP2015PA] = { .pmin =       0, .pmax =  103421 },
+	[ABP2030PA] = { .pmin =       0, .pmax =  206843 },
+	[ABP2060PA] = { .pmin =       0, .pmax =  413685 },
+	[ABP2100PA] = { .pmin =       0, .pmax =  689476 },
+	[ABP2150PA] = { .pmin =       0, .pmax = 1034214 },
+	[ABP2175PA] = { .pmin =       0, .pmax = 1206583 },
+	[ABP2001PD] = { .pmin =   -6895, .pmax =    6895 },
+	[ABP2005PD] = { .pmin =  -34474, .pmax =   34474 },
+	[ABP2015PD] = { .pmin = -103421, .pmax =  103421 },
+	[ABP2030PD] = { .pmin = -206843, .pmax =  206843 },
+	[ABP2060PD] = { .pmin = -413685, .pmax =  413685 },
+	[ABP2001PG] = { .pmin =       0, .pmax =    6895 },
+	[ABP2005PG] = { .pmin =       0, .pmax =   34474 },
+	[ABP2015PG] = { .pmin =       0, .pmax =  103421 },
+	[ABP2030PG] = { .pmin =       0, .pmax =  206843 },
+	[ABP2060PG] = { .pmin =       0, .pmax =  413685 },
+	[ABP2100PG] = { .pmin =       0, .pmax =  689476 },
+	[ABP2150PG] = { .pmin =       0, .pmax = 1034214 },
+	[ABP2175PG] = { .pmin =       0, .pmax = 1206583 },
+};
+
+static_assert(ARRAY_SIZE(abp2_triplet_variants) == ARRAY_SIZE(abp2_range_config));
+
+static int abp2_get_measurement(struct abp2_data *data)
+{
+	struct device *dev = data->dev;
+	int ret;
+
+	reinit_completion(&data->completion);
+
+	ret = data->ops->write(data, ABP2_CMD_SYNC, ABP2_PKT_SYNC_LEN);
+	if (ret < 0)
+		return ret;
+
+	if (data->irq > 0) {
+		ret = wait_for_completion_timeout(&data->completion, HZ);
+		if (!ret) {
+			dev_err(dev, "timeout waiting for EOC interrupt\n");
+			return -ETIMEDOUT;
+		}
+	} else {
+		fsleep(5 * USEC_PER_MSEC);
+	}
+
+	memset(data->rx_buf, 0, sizeof(data->rx_buf));
+	ret = data->ops->read(data, ABP2_CMD_NOP, ABP2_PKT_NOP_LEN);
+	if (ret < 0)
+		return ret;
+
+	/*
+	 * Status byte flags
+	 *  bit7 SANITY_CHK     - must always be 0
+	 *  bit6 ABP2_ST_POWER  - 1 if device is powered
+	 *  bit5 ABP2_ST_BUSY   - 1 if device has no new conversion ready
+	 *  bit4 SANITY_CHK     - must always be 0
+	 *  bit3 SANITY_CHK     - must always be 0
+	 *  bit2 MEMORY_ERR     - 1 if integrity test has failed
+	 *  bit1 SANITY_CHK     - must always be 0
+	 *  bit0 MATH_ERR       - 1 during internal math saturation error
+	 */
+
+	if (data->rx_buf[0] == (ABP2_ST_POWER | ABP2_ST_BUSY))
+		return -EBUSY;
+
+	/*
+	 * The ABP2 sensor series seem to have a noticeable latch-up sensitivity.
+	 * A partial latch-up condition manifests as either
+	 *   - output of invalid status bytes
+	 *   - zeroed out conversions (despite a normal status byte)
+	 *   - the MOSI line being pulled low randomly in sync with the SCLK
+	 * signal (visible during the ABP2_CMD_NOP command).
+	 * https://e2e.ti.com/support/processors-group/processors/f/processors-forum/1588325/am3358-spi-tx-data-corruption
+	 */
+
+	if (data->rx_buf[0] != ABP2_ST_POWER) {
+		dev_err(data->dev,
+			"unexpected status byte 0x%02x\n", data->rx_buf[0]);
+		return -EIO;
+	}
+
+	return 0;
+}
+
+static irqreturn_t abp2_eoc_handler(int irq, void *private)
+{
+	struct abp2_data *data = private;
+
+	complete(&data->completion);
+
+	return IRQ_HANDLED;
+}
+
+static irqreturn_t abp2_trigger_handler(int irq, void *private)
+{
+	int ret;
+	struct iio_poll_func *pf = private;
+	struct iio_dev *indio_dev = pf->indio_dev;
+	struct abp2_data *data = iio_priv(indio_dev);
+
+	ret = abp2_get_measurement(data);
+	if (ret < 0)
+		goto out_notify_done;
+
+	data->scan.chan[0] = get_unaligned_be24(&data->rx_buf[1]);
+	data->scan.chan[1] = get_unaligned_be24(&data->rx_buf[4]);
+
+	iio_push_to_buffers_with_ts(indio_dev, &data->scan, sizeof(data->scan),
+				    iio_get_time_ns(indio_dev));
+
+out_notify_done:
+	iio_trigger_notify_done(indio_dev->trig);
+
+	return IRQ_HANDLED;
+}
+
+/*
+ * IIO ABI expects
+ * value = (conv + offset) * scale
+ *
+ * temp[C] = conv * a + b
+ *   where a = 200/16777215; b = -50
+ *
+ *  temp[C] = (conv + (b/a)) * a * (1000)
+ *  =>
+ *  scale = a * 1000 = .0000119209296 * 1000 = .01192092966562
+ *  offset = b/a = -50 * 16777215 / 200 = -4194303.75
+ *
+ *  pressure = (conv - Omin) * Q + Pmin =
+ *          ((conv - Omin) + Pmin/Q) * Q
+ *  =>
+ *  scale = Q = (Pmax - Pmin) / (Omax - Omin)
+ *  offset = Pmin/Q - Omin = Pmin * (Omax - Omin) / (Pmax - Pmin) - Omin
+ */
+static int abp2_read_raw(struct iio_dev *indio_dev,
+			 struct iio_chan_spec const *channel, int *val,
+			 int *val2, long mask)
+{
+	struct abp2_data *data = iio_priv(indio_dev);
+	int ret;
+
+	switch (mask) {
+	case IIO_CHAN_INFO_RAW:
+		ret = abp2_get_measurement(data);
+		if (ret < 0)
+			return ret;
+
+		switch (channel->type) {
+		case IIO_PRESSURE:
+			*val = get_unaligned_be24(&data->rx_buf[1]);
+			return IIO_VAL_INT;
+		case IIO_TEMP:
+			*val = get_unaligned_be24(&data->rx_buf[4]);
+			return IIO_VAL_INT;
+		default:
+			return -EINVAL;
+		}
+		return IIO_VAL_INT;
+
+	case IIO_CHAN_INFO_SCALE:
+		switch (channel->type) {
+		case IIO_TEMP:
+			*val = 0;
+			*val2 = 11920929;
+			return IIO_VAL_INT_PLUS_NANO;
+		case IIO_PRESSURE:
+			*val = data->p_scale;
+			*val2 = data->p_scale_dec;
+			return IIO_VAL_INT_PLUS_NANO;
+		default:
+			return -EINVAL;
+		}
+
+	case IIO_CHAN_INFO_OFFSET:
+		switch (channel->type) {
+		case IIO_TEMP:
+			*val = -4194304;
+			return IIO_VAL_INT;
+		case IIO_PRESSURE:
+			*val = data->p_offset;
+			return IIO_VAL_INT;
+		default:
+			return -EINVAL;
+		}
+
+	default:
+		return -EINVAL;
+	}
+}
+
+static const struct iio_info abp2_info = {
+	.read_raw = &abp2_read_raw,
+};
+
+static const unsigned long abp2_scan_masks[] = {0x3, 0};
+
+static const struct iio_chan_spec abp2_channels[] = {
+	{
+		.type = IIO_PRESSURE,
+		.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |
+					BIT(IIO_CHAN_INFO_SCALE) |
+					BIT(IIO_CHAN_INFO_OFFSET),
+		.scan_index = 0,
+		.scan_type = {
+			.sign = 'u',
+			.realbits = 24,
+			.storagebits = 32,
+			.endianness = IIO_CPU,
+		},
+	},
+	{
+		.type = IIO_TEMP,
+		.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |
+				      BIT(IIO_CHAN_INFO_SCALE) |
+				      BIT(IIO_CHAN_INFO_OFFSET),
+		.scan_index = 1,
+		.scan_type = {
+			.sign = 'u',
+			.realbits = 24,
+			.storagebits = 32,
+			.endianness = IIO_CPU,
+		},
+	},
+	IIO_CHAN_SOFT_TIMESTAMP(2),
+};
+
+int abp2_common_probe(struct device *dev, const struct abp2_ops *ops, int irq)
+{
+	int ret;
+	struct abp2_data *data;
+	struct iio_dev *indio_dev;
+	const char *triplet;
+	s32 tmp;
+	s64 odelta, pdelta;
+
+	indio_dev = devm_iio_device_alloc(dev, sizeof(*data));
+	if (!indio_dev)
+		return -ENOMEM;
+
+	data = iio_priv(indio_dev);
+	data->dev = dev;
+	data->ops = ops;
+	data->irq = irq;
+
+	init_completion(&data->completion);
+
+	indio_dev->name = "abp2030pa";
+	indio_dev->info = &abp2_info;
+	indio_dev->channels = abp2_channels;
+	indio_dev->num_channels = ARRAY_SIZE(abp2_channels);
+	indio_dev->modes = INDIO_DIRECT_MODE;
+	indio_dev->available_scan_masks = abp2_scan_masks;
+
+	ret = devm_regulator_get_enable(dev, "vdd");
+	if (ret)
+		return dev_err_probe(dev, ret, "can't get and enable vdd supply\n");
+
+	ret = device_property_read_string(dev, "honeywell,pressure-triplet",
+					  &triplet);
+	if (ret) {
+		ret = device_property_read_u32(dev, "honeywell,pmin-pascal",
+					       &data->pmin);
+		if (ret)
+			return dev_err_probe(dev, ret,
+					     "honeywell,pmin-pascal could not be read\n");
+
+		ret = device_property_read_u32(dev, "honeywell,pmax-pascal",
+					       &data->pmax);
+		if (ret)
+			return dev_err_probe(dev, ret,
+					     "honeywell,pmax-pascal could not be read\n");
+	} else {
+		ret = device_property_match_property_string(dev,
+							    "honeywell,pressure-triplet",
+							    abp2_triplet_variants,
+							    ARRAY_SIZE(abp2_triplet_variants));
+		if (ret < 0)
+			return dev_err_probe(dev, -EINVAL, "honeywell,pressure-triplet is invalid\n");
+
+		data->pmin = abp2_range_config[ret].pmin;
+		data->pmax = abp2_range_config[ret].pmax;
+	}
+
+	if (data->pmin >= data->pmax)
+		return dev_err_probe(dev, -EINVAL, "pressure limits are invalid\n");
+
+	data->outmin = abp2_func_spec[data->function].output_min;
+	data->outmax = abp2_func_spec[data->function].output_max;
+
+	odelta = data->outmax - data->outmin;
+	pdelta = data->pmax - data->pmin;
+
+	data->p_scale = div_s64_rem(div_s64(pdelta * NANO, odelta), NANO, &tmp);
+	data->p_scale_dec = tmp;
+
+	data->p_offset = div_s64(odelta * data->pmin, pdelta) - data->outmin;
+
+	if (data->irq > 0) {
+		ret = devm_request_irq(dev, irq, abp2_eoc_handler, IRQF_ONESHOT,
+				       dev_name(dev), data);
+		if (ret)
+			return dev_err_probe(dev, ret, "request irq %d failed\n", data->irq);
+	}
+
+	ret = devm_iio_triggered_buffer_setup(dev, indio_dev, NULL,
+					      abp2_trigger_handler, NULL);
+	if (ret)
+		return dev_err_probe(dev, ret, "iio triggered buffer setup failed\n");
+
+	ret = devm_iio_device_register(dev, indio_dev);
+	if (ret)
+		return dev_err_probe(dev, ret, "unable to register iio device\n");
+
+	return 0;
+}
+EXPORT_SYMBOL_NS_GPL(abp2_common_probe, "IIO_HONEYWELL_ABP2030PA");
+
+MODULE_AUTHOR("Petre Rodan <petre.rodan@subdimension.ro>");
+MODULE_DESCRIPTION("Honeywell ABP2 pressure sensor core driver");
+MODULE_LICENSE("GPL");
diff --git a/drivers/iio/pressure/abp2030pa.h b/drivers/iio/pressure/abp2030pa.h
new file mode 100644
index 000000000000..57e5ed784686
--- /dev/null
+++ b/drivers/iio/pressure/abp2030pa.h
@@ -0,0 +1,73 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Honeywell ABP2 series pressure sensor driver
+ *
+ * Copyright (c) 2025 Petre Rodan <petre.rodan@subdimension.ro>
+ */
+
+#ifndef _ABP2030PA_H
+#define _ABP2030PA_H
+
+#include <linux/completion.h>
+#include <linux/types.h>
+
+#include <linux/iio/iio.h>
+
+#define ABP2_MEASUREMENT_RD_SIZE 7
+
+struct device;
+
+struct abp2_data;
+struct abp2_ops;
+
+enum abp2_func_id {
+	ABP2_FUNCTION_A,
+};
+
+/**
+ * struct abp2_data
+ * @dev: current device structure
+ * @ops: pointers for bus specific read and write functions
+ * @pmin: minimal pressure in pascal
+ * @pmax: maximal pressure in pascal
+ * @outmin: minimum raw pressure in counts (based on transfer function)
+ * @outmax: maximum raw pressure in counts (based on transfer function)
+ * @function: transfer function
+ * @p_scale: pressure scale
+ * @p_scale_dec: pressure scale, decimal number
+ * @p_offset: pressure offset
+ * @irq: end of conversion - applies only to the i2c sensor
+ * @completion: handshake from irq to read
+ * @scan: channel values for buffered mode
+ * @tx_buf: transmit buffer used during the SPI communication
+ * @rx_buf: raw data provided by sensor
+ */
+struct abp2_data {
+	struct device *dev;
+	const struct abp2_ops *ops;
+	s32 pmin;
+	s32 pmax;
+	u32 outmin;
+	u32 outmax;
+	enum abp2_func_id function;
+	int p_scale;
+	int p_scale_dec;
+	int p_offset;
+	int irq;
+	struct completion completion;
+	struct {
+		u32 chan[2];
+		aligned_s64 timestamp;
+	} scan;
+	u8 rx_buf[ABP2_MEASUREMENT_RD_SIZE] __aligned(IIO_DMA_MINALIGN);
+	u8 tx_buf[ABP2_MEASUREMENT_RD_SIZE];
+};
+
+struct abp2_ops {
+	int (*read)(struct abp2_data *data, u8 cmd, u8 nbytes);
+	int (*write)(struct abp2_data *data, u8 cmd, u8 nbytes);
+};
+
+int abp2_common_probe(struct device *dev, const struct abp2_ops *ops, int irq);
+
+#endif
diff --git a/drivers/iio/pressure/abp2030pa_i2c.c b/drivers/iio/pressure/abp2030pa_i2c.c
new file mode 100644
index 000000000000..9f1c1c8a9afb
--- /dev/null
+++ b/drivers/iio/pressure/abp2030pa_i2c.c
@@ -0,0 +1,90 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Honeywell ABP2 series pressure sensor driver
+ *
+ * Copyright (c) 2025 Petre Rodan <petre.rodan@subdimension.ro>
+ */
+
+#include <linux/device.h>
+#include <linux/errno.h>
+#include <linux/i2c.h>
+#include <linux/mod_devicetable.h>
+#include <linux/module.h>
+#include <linux/types.h>
+
+#include "abp2030pa.h"
+
+static int abp2_i2c_read(struct abp2_data *data, u8 unused, u8 nbytes)
+{
+	struct i2c_client *client = to_i2c_client(data->dev);
+	int ret;
+
+	if (nbytes > ABP2_MEASUREMENT_RD_SIZE)
+		return -EOVERFLOW;
+
+	ret = i2c_master_recv(client, data->rx_buf, nbytes);
+	if (ret < 0)
+		return ret;
+	if (ret != nbytes)
+		return -EIO;
+
+	return 0;
+}
+
+static int abp2_i2c_write(struct abp2_data *data, u8 cmd, u8 nbytes)
+{
+	struct i2c_client *client = to_i2c_client(data->dev);
+	int ret;
+
+	if (nbytes > ABP2_MEASUREMENT_RD_SIZE)
+		return -EOVERFLOW;
+
+	data->tx_buf[0] = cmd;
+	ret = i2c_master_send(client, data->tx_buf, nbytes);
+	if (ret < 0)
+		return ret;
+	if (ret != nbytes)
+		return -EIO;
+
+	return 0;
+}
+
+static const struct abp2_ops abp2_i2c_ops = {
+	.read = abp2_i2c_read,
+	.write = abp2_i2c_write,
+};
+
+static int abp2_i2c_probe(struct i2c_client *client)
+{
+	if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C))
+		return -EOPNOTSUPP;
+
+	return abp2_common_probe(&client->dev, &abp2_i2c_ops, client->irq);
+}
+
+static const struct of_device_id abp2_i2c_match[] = {
+	{ .compatible = "honeywell,abp2030pa" },
+	{ }
+};
+MODULE_DEVICE_TABLE(of, abp2_i2c_match);
+
+static const struct i2c_device_id abp2_i2c_id[] = {
+	{ "abp2030pa" },
+	{ }
+};
+MODULE_DEVICE_TABLE(i2c, abp2_i2c_id);
+
+static struct i2c_driver abp2_i2c_driver = {
+	.driver = {
+		.name = "abp2030pa",
+		.of_match_table = abp2_i2c_match,
+	},
+	.probe = abp2_i2c_probe,
+	.id_table = abp2_i2c_id,
+};
+module_i2c_driver(abp2_i2c_driver);
+
+MODULE_AUTHOR("Petre Rodan <petre.rodan@subdimension.ro>");
+MODULE_DESCRIPTION("Honeywell ABP2 pressure sensor i2c driver");
+MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS("IIO_HONEYWELL_ABP2030PA");
diff --git a/drivers/iio/pressure/abp2030pa_spi.c b/drivers/iio/pressure/abp2030pa_spi.c
new file mode 100644
index 000000000000..eaea9a3ebf11
--- /dev/null
+++ b/drivers/iio/pressure/abp2030pa_spi.c
@@ -0,0 +1,67 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * Honeywell ABP2 series pressure sensor driver
+ *
+ * Copyright (c) 2025 Petre Rodan <petre.rodan@subdimension.ro>
+ */
+
+#include <linux/errno.h>
+#include <linux/mod_devicetable.h>
+#include <linux/module.h>
+#include <linux/spi/spi.h>
+#include <linux/types.h>
+
+#include "abp2030pa.h"
+
+static int abp2_spi_xfer(struct abp2_data *data, u8 cmd, u8 nbytes)
+{
+	struct spi_device *spi = to_spi_device(data->dev);
+	struct spi_transfer xfer = { };
+
+	if (nbytes > ABP2_MEASUREMENT_RD_SIZE)
+		return -EOVERFLOW;
+
+	data->tx_buf[0] = cmd;
+	xfer.tx_buf = data->tx_buf;
+	xfer.rx_buf = data->rx_buf;
+	xfer.len = nbytes;
+
+	return spi_sync_transfer(spi, &xfer, 1);
+}
+
+static const struct abp2_ops abp2_spi_ops = {
+	.read = abp2_spi_xfer,
+	.write = abp2_spi_xfer,
+};
+
+static int abp2_spi_probe(struct spi_device *spi)
+{
+	return abp2_common_probe(&spi->dev, &abp2_spi_ops, spi->irq);
+}
+
+static const struct of_device_id abp2_spi_match[] = {
+	{ .compatible = "honeywell,abp2030pa" },
+	{ }
+};
+MODULE_DEVICE_TABLE(of, abp2_spi_match);
+
+static const struct spi_device_id abp2_spi_id[] = {
+	{ "abp2030pa" },
+	{ }
+};
+MODULE_DEVICE_TABLE(spi, abp2_spi_id);
+
+static struct spi_driver abp2_spi_driver = {
+	.driver = {
+		.name = "abp2030pa",
+		.of_match_table = abp2_spi_match,
+	},
+	.probe = abp2_spi_probe,
+	.id_table = abp2_spi_id,
+};
+module_spi_driver(abp2_spi_driver);
+
+MODULE_AUTHOR("Petre Rodan <petre.rodan@subdimension.ro>");
+MODULE_DESCRIPTION("Honeywell ABP2 pressure sensor spi driver");
+MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS("IIO_HONEYWELL_ABP2030PA");

-- 
2.51.2

Re: [PATCH v4 1/2] dt-bindings: iio: pressure: add honeywell,abp2030pa

[Krzysztof Kozlowski]

On 07/12/2025 07:00, Petre Rodan wrote:
> Adds binding for digital Honeywell ABP2 series pressure and temperature
> sensors.
> The i2c address is hardcoded and depends on the part number.
> There is an optional interrupt that signals the end of conversion.
> 
> Signed-off-by: Petre Rodan <petre.rodan@subdimension.ro>

<form letter>
This is a friendly reminder during the review process.

It looks like you received a tag and forgot to add it.

If you do not know the process, here is a short explanation:
Please add Acked-by/Reviewed-by/Tested-by tags when posting new versions
of patchset, under or above your Signed-off-by tag, unless patch changed
significantly (e.g. new properties added to the DT bindings). Tag is
"received", when provided in a message replied to you on the mailing
list. Tools like b4 can help here. However, there's no need to repost
patches *only* to add the tags. The upstream maintainer will do that for
tags received on the version they apply.

Please read:
https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577

If a tag was not added on purpose, please state why and what changed.
</form letter>

I will not be doing work twice and, without explanation, forcing me to
do that is disrespectful to my time.

Best regards,
Krzysztof

Re: [PATCH v4 1/2] dt-bindings: iio: pressure: add honeywell,abp2030pa

[Petre Rodan]

hello Krzysztof, 

On Mon, Dec 08, 2025 at 06:01:46AM +0100, Krzysztof Kozlowski wrote:
> On 07/12/2025 07:00, Petre Rodan wrote:
> > Adds binding for digital Honeywell ABP2 series pressure and temperature
> > sensors.
> > The i2c address is hardcoded and depends on the part number.
> > There is an optional interrupt that signals the end of conversion.
> > 
> > Signed-off-by: Petre Rodan <petre.rodan@subdimension.ro>
> 
> <form letter>
> This is a friendly reminder during the review process.
> 
> It looks like you received a tag and forgot to add it.

[..]

> If a tag was not added on purpose, please state why and what changed.
> </form letter>

I removed the tag on purpose since the yaml file was modified in v4 as per the
changelog that is below the commit message.

> I will not be doing work twice and, without explanation, forcing me to
> do that is disrespectful to my time.

sorry about that. the single thing that changed was that

---- 8< --------------------------------------------
@@ -76,8 +71,7 @@ properties:
   spi-max-frequency:
     maximum: 800000
 
-  vdd-supply:
-    description: provide VDD power to the sensor.
+  vdd-supply: true
 
 required:
   - compatible
---- >8 --------------------------------------------

I checked the resulting file with make dt_binding_check, yet I was not sure if your previous tag would still validate the work.

best regards,
peter

Re: [PATCH v4 1/2] dt-bindings: iio: pressure: add honeywell,abp2030pa

[Krzysztof Kozlowski]

On 08/12/2025 08:15, Petre Rodan wrote:
> 
> hello Krzysztof, 
> 
> On Mon, Dec 08, 2025 at 06:01:46AM +0100, Krzysztof Kozlowski wrote:
>> On 07/12/2025 07:00, Petre Rodan wrote:
>>> Adds binding for digital Honeywell ABP2 series pressure and temperature
>>> sensors.
>>> The i2c address is hardcoded and depends on the part number.
>>> There is an optional interrupt that signals the end of conversion.
>>>
>>> Signed-off-by: Petre Rodan <petre.rodan@subdimension.ro>
>>
>> <form letter>
>> This is a friendly reminder during the review process.
>>
>> It looks like you received a tag and forgot to add it.
> 
> [..]
> 
>> If a tag was not added on purpose, please state why and what changed.
>> </form letter>
> 
> I removed the tag on purpose since the yaml file was modified in v4 as per the
> changelog that is below the commit message.

Please read my form letter and please read linked submitting patches
doc. I pointed to a very specific paragraph, so you don't need to read
everything...

> 
>> I will not be doing work twice and, without explanation, forcing me to
>> do that is disrespectful to my time.
> 
> sorry about that. the single thing that changed was that
> 
> ---- 8< --------------------------------------------
> @@ -76,8 +71,7 @@ properties:
>    spi-max-frequency:
>      maximum: 800000
>  
> -  vdd-supply:
> -    description: provide VDD power to the sensor.
> +  vdd-supply: true
>  


This is trivial and absolutely NOT a reason to drop the tag. Did you
read submitting patches document? Please consider TIME of reviewers,
even time spent on replying on this.

Best regards,
Krzysztof

Re: [PATCH v4 1/2] dt-bindings: iio: pressure: add honeywell,abp2030pa

[Jonathan Cameron]

On Mon, 8 Dec 2025 08:21:49 +0100
Krzysztof Kozlowski <krzk@kernel.org> wrote:

> On 08/12/2025 08:15, Petre Rodan wrote:
> > 
> > hello Krzysztof, 
> > 
> > On Mon, Dec 08, 2025 at 06:01:46AM +0100, Krzysztof Kozlowski wrote:  
> >> On 07/12/2025 07:00, Petre Rodan wrote:  
> >>> Adds binding for digital Honeywell ABP2 series pressure and temperature
> >>> sensors.
> >>> The i2c address is hardcoded and depends on the part number.
> >>> There is an optional interrupt that signals the end of conversion.
> >>>
> >>> Signed-off-by: Petre Rodan <petre.rodan@subdimension.ro>  
> >>
> >> <form letter>
> >> This is a friendly reminder during the review process.
> >>
> >> It looks like you received a tag and forgot to add it.  
> > 
> > [..]
> >   
> >> If a tag was not added on purpose, please state why and what changed.
> >> </form letter>  
> > 
> > I removed the tag on purpose since the yaml file was modified in v4 as per the
> > changelog that is below the commit message.  
> 
> Please read my form letter and please read linked submitting patches
> doc. I pointed to a very specific paragraph, so you don't need to read
> everything...
> 
> >   
> >> I will not be doing work twice and, without explanation, forcing me to
> >> do that is disrespectful to my time.  
> > 
> > sorry about that. the single thing that changed was that
> > 
> > ---- 8< --------------------------------------------
> > @@ -76,8 +71,7 @@ properties:
> >    spi-max-frequency:
> >      maximum: 800000
> >  
> > -  vdd-supply:
> > -    description: provide VDD power to the sensor.
> > +  vdd-supply: true
> >    
> 
> 
> This is trivial and absolutely NOT a reason to drop the tag. Did you
> read submitting patches document? Please consider TIME of reviewers,
> even time spent on replying on this.

I put the tag back and applied to the testing branch of iio.git.
I'll rebase that on rc1 once it is available.

Thanks,

Jonathan