From nobody Sat Jul 25 04:53:44 2026 Received: from mail-lj1-f173.google.com (mail-lj1-f173.google.com [209.85.208.173]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DFC76378828 for ; Fri, 17 Jul 2026 20:11:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.173 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784319105; cv=none; b=TPKMwfSH2X1YH7txXno4Qcy/CakTCF5+3CtJF/SEPEr7fLR/gTDBnNwksu+6gvITfR+bAuHxS0b8oMCWcZH7cXJ1ecljG3LR78nuzCxj54f+phS62s8TLMNFYl46XS/wHTgLqkaaWVZ7rEakwPwY1JZZZ0vTeJzK9VJJowMxjuM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784319105; c=relaxed/simple; bh=BCVGMZJMqKv9mEWFv1asCj8XmBIDC6Y8WkqH06d1nCQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=SFE1vPMk0F2XZOiPRnvk3/sxsQHlAWjpwlkXulQIVf+ic3+i4rpZCu3HZZ1glF6F9SqABm3ynAs1SJE522kpl/GMzGXkAbKtASidrlgfgKnlbEigTnpNjttpridTDSO1zBvj6dt1foivgY2+a3rws81BSLX320itJepS6fFHT1k= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=MuJ3T1B3; arc=none smtp.client-ip=209.85.208.173 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="MuJ3T1B3" Received: by mail-lj1-f173.google.com with SMTP id 38308e7fff4ca-39b3931e259so74857741fa.2 for ; Fri, 17 Jul 2026 13:11:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1784319102; x=1784923902; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=QE9NqWnp+0llPNlHSCBGG32bbwPzbNgAMLIqj11/XOg=; b=MuJ3T1B3HaHfJ45wbixMpH3i+nhcSI7a5ebyshfJEbrbIRU3eDQLmAdDqEWewo1tAE dFr9ftLLJ1Dy05kIqICcDILzFcsLTux77bN8IpuS7CWQEbyQVvjxlBGzxeDUpMdYO5QT 6RQiklm2/8mznmypPP3idA6vXsevZvhFPg68vJGyoidG3ZCbQbbCl7t//NDQY0+b5AbC 9hlauVzHjyAU6SFGOnCbhB5isilgHrglHxj+HTzlS671r6Fr6ZXPui9nMZsDSrrS3tvF BYxRktPGfmUsBvAeCNkj1l26OtUIn6tSaescKvkJnQ6k/OCkscmdBT2yDn7PR0UoiOzP drEg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784319102; x=1784923902; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=QE9NqWnp+0llPNlHSCBGG32bbwPzbNgAMLIqj11/XOg=; b=UMrQFSwMsAof0PJcO9zJ3I08tlZn2EL82yHU3TCxcjtAwA8KIrqAl1uVXJhw/naEBG 9AHL1R8tJzQfr8jQtpkTdkh5YlR4xRjPjEIz+s+3DfhC4zDUW6pTf1Kj1F9fnGPMxXXH c831ns9Z2RyiyVyflkUV0cJiJgGTIhR063MoA7JyAOU7dink0oXPDjxm0Y/uTPOttkIh sGpnxQnBp5D3QqMme3qdJBCcM2cJxfrhekctBUZgaCTwSLC1DL8gCUvFgz05CvvkzdzE XpWgXf/y8yFY9XLH1Gt0DetKmQTslq/9nimg2g3SUi3NdZ00MYAKcRioLLUMSQcc8Wom Xq7w== X-Forwarded-Encrypted: i=1; AHgh+RquhEoxDbFDUA00Iq4tbdtrVKNX4WGVrdRX21WTpvGyqirNECPdNvTymgeuGSiVtRivMkfTHqIk7Nzb6sk=@vger.kernel.org X-Gm-Message-State: AOJu0YzfG2tlbYNJb3iAvUIs2nmrNnej1Qnanop06MfaW/FOEyUyFi9s L8booZlgL7qc2p97hCKhf2lbxjn+j/EzohLzvto7VxJdRdmSIW6ExdU7 X-Gm-Gg: AfdE7clbItd9ljbJ6yLQUK7nIMqKffJiqPdeZAloVhe2vBOP8mM7otg4pMJeagqsfrM Jmx4ssE1JdGv2vWgBcey9snWrFOhj9L52+anGg/0i/r4W/5uiyGH8D3w5c+WmGfOlAPbA9a9FBM xd03motOzVq4xyGJ6T98HoFp3Jgf2OVlqfy6qGDKCZHFzVwXbbdJRx7FvQ3aC7kJfPjtlmugYR1 pRof5UHqQjs8uu42kZhq63FpTfiZPHSkEu3Uio608vJqTTdPdLDS6V8bqQRW5qqMlL3E8HxiKSm DcUzLRyUyn1yqQqHb8JWSuLlzzadyoFPMneCIztfSww2tWjWgiYiYGBwnI2u90cqMlOrcpiqYzg E5d2W54pI6dqxfXsV1VBH28+bkbWRO2zAtMYDvuCvaug3wnVFXJ20FPLMv6xiP7SqA6c4vOcudC 5cT7+OZCxWmeQ4vB7SPG6Dbc70neUONmo= X-Received: by 2002:ac2:484a:0:b0:5b0:1380:ca35 with SMTP id 2adb3069b0e04-5b28f9ac996mr678330e87.41.1784319101662; Fri, 17 Jul 2026 13:11:41 -0700 (PDT) Received: from Shofiq.home (87-92-108-63.bb.dnainternet.fi. [87.92.108.63]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5b291da7bafsm571257e87.33.2026.07.17.13.11.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 17 Jul 2026 13:11:40 -0700 (PDT) From: Md Shofiqul Islam To: linux-iio@vger.kernel.org Cc: Md Shofiqul Islam , jic23@kernel.org, devicetree@vger.kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, andriy.shevchenko@intel.com, u.kleine-koenig@baylibre.com, joshua.crofts1@gmail.com, nuno.sa@analog.com, Michael.Hennerich@analog.com, dlechner@baylibre.com, linux@analog.com, linux-kernel@vger.kernel.org Subject: [PATCH v10 1/2] dt-bindings: iio: health: add adi,max86150 Date: Fri, 17 Jul 2026 23:11:36 +0300 Message-ID: <20260717201138.1078019-2-shofiqtest@gmail.com> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20260717201138.1078019-1-shofiqtest@gmail.com> References: <20260717201138.1078019-1-shofiqtest@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add the Device Tree binding schema for the Analog Devices MAX86150 integrated biosensor, which combines two photoplethysmography (PPG) channels (Red LED and IR LED) and one electrocardiogram (ECG) channel in a single I2C device. Samples are buffered in a 32-entry hardware FIFO with a configurable almost-full interrupt. The device requires four independent supply rails per the datasheet power tree: digital core (vdd), analog core (avdd), ECG reference (vref), and LED anode (leds). Assisted-by: Claude:claude-sonnet-5 Signed-off-by: Md Shofiqul Islam --- .../bindings/iio/health/adi,max86150.yaml | 78 +++++++++++++++++++ MAINTAINERS | 6 ++ 2 files changed, 84 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/health/adi,max861= 50.yaml diff --git a/Documentation/devicetree/bindings/iio/health/adi,max86150.yaml= b/Documentation/devicetree/bindings/iio/health/adi,max86150.yaml new file mode 100644 index 0000000000000..da2ff9b0ada2b --- /dev/null +++ b/Documentation/devicetree/bindings/iio/health/adi,max86150.yaml @@ -0,0 +1,78 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/iio/health/adi,max86150.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Analog Devices MAX86150 ECG and PPG Biosensor + +maintainers: + - Md Shofiqul Islam + +description: | + The MAX86150 is an integrated biosensor SoC that combines: + - Two PPG (photoplethysmography) channels: Red LED and IR LED, + for heart rate and blood-oxygen saturation (SpO2) measurement. + - One ECG (electrocardiogram) channel for biopotential recording. + + The device communicates over I2C at up to 400 kHz and raises an + active-low interrupt when the 32-entry hardware FIFO reaches its + configurable almost-full threshold. + + Datasheet: + https://www.analog.com/media/en/technical-documentation/data-sheets/MA= X86150.pdf + +properties: + compatible: + const: adi,max86150 + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + description: | + Active-low interrupt line. Asserted when the FIFO almost-full + threshold is reached or when a new PPG sample is ready. + + vdd-supply: + description: Digital core power supply (1.8 V). + + avdd-supply: + description: Analog core power supply (1.8 V). + + vref-supply: + description: ECG reference voltage supply. + + leds-supply: + description: LED anode supply, typically 3.3 V. + +required: + - compatible + - reg + - vdd-supply + - avdd-supply + - vref-supply + - leds-supply + +additionalProperties: false + +examples: + - | + #include + + i2c { + #address-cells =3D <1>; + #size-cells =3D <0>; + + heart-rate@5e { + compatible =3D "adi,max86150"; + reg =3D <0x5e>; + interrupt-parent =3D <&gpio1>; + interrupts =3D <5 IRQ_TYPE_LEVEL_LOW>; + vdd-supply =3D <&vdd_1v8>; + avdd-supply =3D <&vdd_1v8>; + vref-supply =3D <&vdd_1v8>; + leds-supply =3D <&vdd_3v3>; + }; + }; diff --git a/MAINTAINERS b/MAINTAINERS index 3115538ce8291..189385966de82 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -15705,6 +15705,12 @@ S: Orphan F: Documentation/hwmon/max6650.rst F: drivers/hwmon/max6650.c =20 +MAX86150 ECG AND PPG BIOSENSOR DRIVER +M: Md Shofiqul Islam +L: linux-iio@vger.kernel.org +S: Maintained +F: Documentation/devicetree/bindings/iio/health/adi,max86150.yaml + MAX9286 QUAD GMSL DESERIALIZER DRIVER M: Jacopo Mondi M: Kieran Bingham --=20 2.51.1 From nobody Sat Jul 25 04:53:44 2026 Received: from mail-lf1-f52.google.com (mail-lf1-f52.google.com [209.85.167.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4FFFF207A20 for ; Fri, 17 Jul 2026 20:11:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.52 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784319108; cv=none; b=FQB/HD+d0n2UE3ztd05o6+sYtS8h93Xobeb+JTMZUU5z3J3nlyxuWh/OMcveDvV/QDMR6+EQLrthmjzgrWM13HX6dRsShKl2VkrJ5UaASV8wkGhELhYcynF8bnyTNWleQ0ui12/rdjYMh868m5wd21gV4H/JeWNMECCnqSQIfh4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784319108; c=relaxed/simple; bh=bzY8FMbFb6Wlf2n1YK3u7w6allCEAwegwWtc+K6OgH4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=eI4Mp0J5i8iZNZOqobMOAFUtH1QKvdlYb3L8c18WENqgbgk9Ez2Vhm/uoBU2jHZvyP9eYiNnvefBBOxI2IRdqdUwvHAFdy3SgyYfwgqCLXqr+1Vl5JwoaV3JRyGMT5MMIdpdJhHPAr3+YvXT1R+T5EQGJWxoZ7VyL3vQd0/5SSc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Y6QcSx3R; arc=none smtp.client-ip=209.85.167.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Y6QcSx3R" Received: by mail-lf1-f52.google.com with SMTP id 2adb3069b0e04-5aeb5e85378so8857537e87.1 for ; Fri, 17 Jul 2026 13:11:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1784319103; x=1784923903; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=YaAy4S5NDbwFfa0CjmghucFyX0/TubQ3Uq9q8Lp9tdM=; b=Y6QcSx3RdEx3q7MaquqWNFVKMc3dI2vEd3ByawIFH6TNqsmtBd71iyNCv2KHE3rJr+ 8pHgNGA906KS1LFEp+B3Z19QRgFV2KgBLsG2ZwLKWrY3Iq8R+B0F1jPfijBhCBqoqIVf bK7CK1+YTVHVJhxUSUrfDpSB2TtjQlhaxfiq6gRA2t0spT47Y4Xa7YrT1ba9Ht2ZxCYv O/LDwYrne3YS/a8P+TkZcsmZfVaVq1OSY4xTa8yQAu9A1VZWyW8HdfVJ6JAnW1Wm9zmd zJ4A4XKq1AS92ZFCPVNvLZcDjFhVqMO67E35eQR0yGCgNGXP0sni6jVmx8s52HjlQykS hm/A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784319103; x=1784923903; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=YaAy4S5NDbwFfa0CjmghucFyX0/TubQ3Uq9q8Lp9tdM=; b=WB5yYLQ0xHeFgK3jRRecV8IiROtRYkEdVLUpYJkrvB2JzNcfnKDTnfOaTtoxwJEcFj DYe2dFtPIamxIxP/eY5/BSwarwn6aG/iyQ5UBPzC4aCdWt95dTJM4iewrrq0LgbEOafa vpDo+ZP2BkMOPENZZotRa8L1xTyo7xxoMJup3sraRApYATt8oDrYyxdIDZRV7C5wciaO K9s8B+55F2/THDMdri2YLn5b7a6ctrmE7cH8IzaQx4uwKxSixthXy6/ZVUjyVTybpPqD cscToAb4QqHZROtlrnRa+RcN6vmEYOJFWN2oIDxzbzHkNbFcQu6T/iLUlNPEaFdohYxM nlkA== X-Forwarded-Encrypted: i=1; AHgh+RolKgSp0FaIS29whhCAxQYK9MqlmUph39ORxzr4qnwCjS2wjmK/nW5jKV5G5FfbRiry8m6wTthzV+17Lbg=@vger.kernel.org X-Gm-Message-State: AOJu0YzG3rLZR3BAJcxciDMPAy3qSX9tLaXOMRzqKpkENKfNF1K16Z1d THlhy/UAKYuJi2VU9fCfzc1P5EmPnB1BayruOMQNGjtvgq2BfgF+FEqU X-Gm-Gg: AfdE7ckKmO+JCSro8soq8pfOC0jkZMkdELQ1kVhJV+3mRl2/q56hhFGwmiYzkqGW+E5 WOn6P4OnvFF3m1hv6m2TF0VJzix+3GKSjIwcqseULRvmaReJzJxWSVZs9YVOTCYkGJh/vFqDULE JALgPzWKvmrZzs7YWdxZW0zTvO1K4OnXcMerUfDPGp8tRzs3A6b4D7ZTofDbCERMIAqStSDNvbU zN3qFFYVpa7kSyCSPb5GDL1D8ZsLvP2twfe5O2Sbz9sBCFWIRkjGXc3ZpqOzYL31feLzvcKEOgt K0CT2jnN1ubq2mdvJbPyzrY5eHy1wLO5ugQXkx6pHdn3oyhOsJBBhwd0g22t7vF2w+lFuDNs5JQ ETeLnmdsKxtaVBk/gPG5vTmHuQelKbpY/uCzUPA0MhxMZnYDke4wnFJ2D9A7yJ+lNRJ0uJnD1i1 PKhDk/EBSpG3UfMxaCwGpdJqfb0I0Qt6DFrXzNuti8gw== X-Received: by 2002:a05:6512:a86:b0:5ae:b13d:e715 with SMTP id 2adb3069b0e04-5b28f998ceemr987217e87.31.1784319103076; Fri, 17 Jul 2026 13:11:43 -0700 (PDT) Received: from Shofiq.home (87-92-108-63.bb.dnainternet.fi. [87.92.108.63]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5b291da7bafsm571257e87.33.2026.07.17.13.11.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 17 Jul 2026 13:11:42 -0700 (PDT) From: Md Shofiqul Islam To: linux-iio@vger.kernel.org Cc: Md Shofiqul Islam , jic23@kernel.org, devicetree@vger.kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, andriy.shevchenko@intel.com, u.kleine-koenig@baylibre.com, joshua.crofts1@gmail.com, nuno.sa@analog.com, Michael.Hennerich@analog.com, dlechner@baylibre.com, linux@analog.com, linux-kernel@vger.kernel.org Subject: [PATCH v10 2/2] iio: health: add MAX86150 ECG and PPG biosensor driver Date: Fri, 17 Jul 2026 23:11:37 +0300 Message-ID: <20260717201138.1078019-3-shofiqtest@gmail.com> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20260717201138.1078019-1-shofiqtest@gmail.com> References: <20260717201138.1078019-1-shofiqtest@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add a new IIO driver for the Analog Devices MAX86150 integrated biosensor, which combines two PPG optical channels (Red/IR LED) and one ECG biopotential channel in a single I2C device. The device has a 32-entry hardware FIFO with a configurable almost-full interrupt. The driver uses the standard IIO hardware-trigger and triggered-buffer framework: a hard-irq handler reads and clears INT_STATUS1 to de-assert the line before calling iio_trigger_poll(), and the threaded trigger handler drains the FIFO and pushes samples, with timestamps back-calculated from the interrupt arrival time by one sample_period_ns per step. Relying on the trigger core's own attach/detach synchronization avoids the need for an explicit iio_buffer_enabled() guard or synchronize_irq() in the interrupt path. Key implementation details: - Part ID register (0xFF) verified against 0x1E on probe; mismatched devices are rejected with -ENODEV so the driver cannot bind to the wrong hardware - Stale A_FULL and PPG_RDY status bits are cleared before arming the interrupt or polling for a sample, so a previously-latched flag cannot fire the handler against garbage state - 24-bit FIFO words decoded directly from the raw byte buffer - regmap_set_bits() / regmap_clear_bits() for single-direction writes - Device remains in shutdown between captures to suppress LED current - vdd, avdd, vref and leds regulators required per the datasheet power tree - iio_get_time_ns() used for timestamps so they respect the IIO device's configured clock source - A_FULL status bit used to detect FIFO exactly full (wr_ptr =3D=3D rd_ptr with OVF_COUNTER =3D=3D 0) so valid samples are not silently dropped - No IRQ trigger-type override: the device has no register to reconfigure interrupt polarity or type, so IRQF_ONESHOT with whatever type firmware provides is sufficient Suggested-by: Jonathan Cameron Suggested-by: Joshua Crofts Assisted-by: Claude:claude-sonnet-5 Signed-off-by: Md Shofiqul Islam --- MAINTAINERS | 1 + drivers/iio/health/Kconfig | 13 + drivers/iio/health/Makefile | 1 + drivers/iio/health/max86150.c | 691 ++++++++++++++++++++++++++++++++++ 4 files changed, 706 insertions(+) create mode 100644 drivers/iio/health/max86150.c diff --git a/MAINTAINERS b/MAINTAINERS index 189385966de82..5e266c03bda6f 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -15710,6 +15710,7 @@ M: Md Shofiqul Islam L: linux-iio@vger.kernel.org S: Maintained F: Documentation/devicetree/bindings/iio/health/adi,max86150.yaml +F: drivers/iio/health/max86150.c =20 MAX9286 QUAD GMSL DESERIALIZER DRIVER M: Jacopo Mondi diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig index a89f3abf11f4a..7438cdfe4238f 100644 --- a/drivers/iio/health/Kconfig +++ b/drivers/iio/health/Kconfig @@ -62,4 +62,17 @@ config MAX30102 =20 endmenu =20 +config MAX86150 + tristate "MAX86150 ECG and PPG biosensor" + depends on I2C + select IIO_BUFFER + select IIO_TRIGGERED_BUFFER + select REGMAP_I2C + help + Say Y here to enable support for the Maxim MAX86150 combined + ECG and photoplethysmography (PPG) biosensor. + + This driver can also be built as a module. If so, the module + will be called max86150. + endmenu diff --git a/drivers/iio/health/Makefile b/drivers/iio/health/Makefile index 9108171122588..04fc73c584449 100644 --- a/drivers/iio/health/Makefile +++ b/drivers/iio/health/Makefile @@ -9,3 +9,4 @@ obj-$(CONFIG_AFE4403) +=3D afe4403.o obj-$(CONFIG_AFE4404) +=3D afe4404.o obj-$(CONFIG_MAX30100) +=3D max30100.o obj-$(CONFIG_MAX30102) +=3D max30102.o +obj-$(CONFIG_MAX86150) +=3D max86150.o diff --git a/drivers/iio/health/max86150.c b/drivers/iio/health/max86150.c new file mode 100644 index 0000000000000..47d96570a95fc --- /dev/null +++ b/drivers/iio/health/max86150.c @@ -0,0 +1,691 @@ +// SPDX-License-Identifier: GPL-2.0-only +/* + * MAX86150 combined ECG and PPG biosensor driver + * + * Copyright (C) 2026 Md Shofiqul Islam + * + * The MAX86150 integrates two PPG optical channels (Red/IR LED) and one + * ECG biopotential channel in a single I2C device. Data is captured + * through a 32-entry hardware FIFO with a configurable almost-full + * interrupt, making it well-suited for continuous monitoring with a + * low-power host. + * + * Datasheet: + * https://www.analog.com/media/en/technical-documentation/data-sheets/M= AX86150.pdf + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include +#include +#include +#include +#include + +#define MAX86150_REG_INT_STATUS1 0x00 +#define MAX86150_REG_INT_STATUS2 0x01 +#define MAX86150_REG_INT_ENABLE1 0x02 +#define MAX86150_REG_INT_ENABLE2 0x03 +#define MAX86150_REG_FIFO_WR_PTR 0x04 +#define MAX86150_REG_OVF_COUNTER 0x05 +#define MAX86150_REG_FIFO_RD_PTR 0x06 +#define MAX86150_REG_FIFO_DATA 0x07 +#define MAX86150_REG_FIFO_CONFIG 0x08 +#define MAX86150_REG_FIFO_DCTRL1 0x09 +#define MAX86150_REG_FIFO_DCTRL2 0x0A +#define MAX86150_REG_SYS_CTRL 0x0D +#define MAX86150_REG_PPG_CONFIG1 0x10 +#define MAX86150_REG_PPG_CONFIG2 0x11 +#define MAX86150_REG_LED1_PA 0x14 +#define MAX86150_REG_LED2_PA 0x15 +#define MAX86150_REG_ECG_CONFIG1 0x3C +#define MAX86150_REG_ECG_CONFIG3 0x3E +#define MAX86150_REG_PART_ID 0xFF + +#define MAX86150_PART_ID_VAL 0x1E + +#define MAX86150_INT_A_FULL BIT(7) +#define MAX86150_INT_PPG_RDY BIT(6) + +#define MAX86150_SYS_SHDN BIT(1) +#define MAX86150_SYS_RESET BIT(0) + +#define MAX86150_FIFO_SMP_AVE GENMASK(7, 5) +#define MAX86150_FIFO_ROLLOVER_EN BIT(4) +#define MAX86150_FIFO_A_FULL GENMASK(3, 0) + +#define MAX86150_FD_NONE 0x0 +#define MAX86150_FD_LED1 0x1 +#define MAX86150_FD_LED2 0x2 +#define MAX86150_FD_ECG 0x9 +#define MAX86150_FIFO_FD1 GENMASK(3, 0) +#define MAX86150_FIFO_FD2 GENMASK(7, 4) +#define MAX86150_FIFO_FD3 GENMASK(3, 0) +#define MAX86150_FIFO_FD4 GENMASK(7, 4) + +#define MAX86150_PPG_ADC_RGE GENMASK(7, 6) +#define MAX86150_PPG_SR GENMASK(5, 1) + +/* PPG ADC full-scale range (ADC_RGE field of PPG_CONFIG1) */ +#define MAX86150_PPG_ADC_RGE_4096 0 /* 4096 nA */ +#define MAX86150_PPG_ADC_RGE_8192 1 /* 8192 nA */ +#define MAX86150_PPG_ADC_RGE_16384 2 /* 16384 nA */ +#define MAX86150_PPG_ADC_RGE_32768 3 /* 32768 nA */ + +/* PPG sample rate (SR field of PPG_CONFIG1) - single-pulse variants */ +#define MAX86150_PPG_SR_SP_10HZ 0 +#define MAX86150_PPG_SR_SP_20HZ 1 +#define MAX86150_PPG_SR_SP_50HZ 2 +#define MAX86150_PPG_SR_SP_84HZ 3 +#define MAX86150_PPG_SR_SP_100HZ 4 +#define MAX86150_PPG_SR_SP_200HZ 5 +#define MAX86150_PPG_SR_SP_400HZ 6 +#define MAX86150_PPG_SR_SP_800HZ 7 +#define MAX86150_PPG_SR_SP_1000HZ 8 +#define MAX86150_PPG_SR_SP_1600HZ 9 +#define MAX86150_PPG_SR_SP_3200HZ 10 +/* Double-pulse variants (two LED pulses averaged per sample) */ +#define MAX86150_PPG_SR_DP_10HZ 11 +#define MAX86150_PPG_SR_DP_20HZ 12 +#define MAX86150_PPG_SR_DP_50HZ 13 +#define MAX86150_PPG_SR_DP_84HZ 14 +#define MAX86150_PPG_SR_DP_100HZ 15 +#define MAX86150_PPG_SR_DP_200HZ 16 +#define MAX86150_PPG_SR_DP_400HZ 17 +#define MAX86150_PPG_SR_DP_800HZ 18 +#define MAX86150_PPG_SR_DP_1000HZ 19 +#define MAX86150_PPG_SR_DP_1600HZ 20 +#define MAX86150_PPG_SR_DP_3200HZ 21 + +/* LED pulse amplitude: 0x00 =3D 0 mA, step ~0.8 mA, 0x3F ~=3D 50 mA, 0xFF= ~=3D 200 mA */ + +#define MAX86150_FIFO_DEPTH 32 +#define MAX86150_BYTES_PER_SLOT 3 +#define MAX86150_NUM_SLOTS 3 +#define MAX86150_SAMPLE_BYTES (MAX86150_NUM_SLOTS * MAX86150_BYTES_PER_SL= OT) + +/* Samples available in the FIFO when the A_FULL interrupt fires */ +#define MAX86150_FIFO_A_FULL_SAMPLES 17 + +#define MAX86150_LED_PA_DEFAULT 0x3F + +enum max86150_scan_idx { + MAX86150_IDX_PPG_RED, + MAX86150_IDX_PPG_IR, + MAX86150_IDX_ECG, + MAX86150_IDX_TS, +}; + +/** + * struct max86150_data - driver private state + * @regmap: register map for this device + * @trig: IIO hardware trigger backed by the device interrupt = line + * @sample_period_ns: sample period in nanoseconds (set from configured ra= te) + * @fifo_raw: scratch buffer for regmap_noinc_read() FIFO bursts; = kept + * in struct (heap) rather than on the stack, since sta= ck + * memory isn't guaranteed DMA-safe (e.g. CONFIG_VMAP_S= TACK) + * and some I2C host controllers DMA the read buffer + * @scan: IIO push buffer; channels[] packed per active_scan_m= ask, + * with a trailing aligned_s64 slot for the timestamp + */ +struct max86150_data { + struct regmap *regmap; + struct iio_trigger *trig; + u32 sample_period_ns; + u8 fifo_raw[MAX86150_SAMPLE_BYTES]; + IIO_DECLARE_DMA_BUFFER_WITH_TS(s32, scan, MAX86150_NUM_SLOTS); +}; + +static const struct iio_chan_spec max86150_channels[] =3D { + { + .type =3D IIO_INTENSITY, + .modified =3D 1, + .channel2 =3D IIO_MOD_LIGHT_RED, + .info_mask_separate =3D BIT(IIO_CHAN_INFO_RAW), + .scan_index =3D MAX86150_IDX_PPG_RED, + .scan_type =3D { + .sign =3D 'u', + .realbits =3D 19, + .storagebits =3D 32, + .endianness =3D IIO_CPU, + }, + }, + { + .type =3D IIO_INTENSITY, + .modified =3D 1, + .channel2 =3D IIO_MOD_LIGHT_IR, + .info_mask_separate =3D BIT(IIO_CHAN_INFO_RAW), + .scan_index =3D MAX86150_IDX_PPG_IR, + .scan_type =3D { + .sign =3D 'u', + .realbits =3D 19, + .storagebits =3D 32, + .endianness =3D IIO_CPU, + }, + }, + { + .type =3D IIO_VOLTAGE, + .channel =3D 0, + .info_mask_separate =3D BIT(IIO_CHAN_INFO_RAW), + .scan_index =3D MAX86150_IDX_ECG, + .scan_type =3D { + .sign =3D 's', + .realbits =3D 18, + .storagebits =3D 32, + .endianness =3D IIO_CPU, + }, + }, + IIO_CHAN_SOFT_TIMESTAMP(MAX86150_IDX_TS), +}; + +static const struct regmap_config max86150_regmap_config =3D { + .reg_bits =3D 8, + .val_bits =3D 8, + .max_register =3D MAX86150_REG_PART_ID, +}; + +static int max86150_read_one_sample(struct max86150_data *data, + u32 *ppg_red, u32 *ppg_ir, s32 *ecg) +{ + int ret; + + ret =3D regmap_noinc_read(data->regmap, MAX86150_REG_FIFO_DATA, + data->fifo_raw, MAX86150_SAMPLE_BYTES); + if (ret) + return ret; + + *ppg_red =3D (data->fifo_raw[0] & 0x07) << 16 | + data->fifo_raw[1] << 8 | data->fifo_raw[2]; + *ppg_ir =3D (data->fifo_raw[3] & 0x07) << 16 | + data->fifo_raw[4] << 8 | data->fifo_raw[5]; + *ecg =3D sign_extend32((data->fifo_raw[6] & 0x03) << 16 | + data->fifo_raw[7] << 8 | data->fifo_raw[8], 17); + return 0; +} + +/* + * Take the device out of shutdown, reset the FIFO pointers, wait for the + * first PPG sample, and read it back. Always returns the device to + * shutdown before returning, whether or not the read succeeded. + */ +static int max86150_do_read_raw(struct max86150_data *data, + u32 *ppg_red, u32 *ppg_ir, s32 *ecg) +{ + unsigned int ppg_rdy_status; + int ret; + + ret =3D regmap_clear_bits(data->regmap, MAX86150_REG_SYS_CTRL, + MAX86150_SYS_SHDN); + if (ret) + return ret; + + ret =3D regmap_write(data->regmap, MAX86150_REG_FIFO_WR_PTR, 0); + if (ret) + goto out_shdn; + + ret =3D regmap_write(data->regmap, MAX86150_REG_OVF_COUNTER, 0); + if (ret) + goto out_shdn; + + ret =3D regmap_write(data->regmap, MAX86150_REG_FIFO_RD_PTR, 0); + if (ret) + goto out_shdn; + + /* + * Clear stale PPG_RDY from a previous session; reading + * INT_STATUS1 de-asserts any pending flags so the poll + * below waits for a genuinely new sample. + */ + ret =3D regmap_read(data->regmap, MAX86150_REG_INT_STATUS1, + &ppg_rdy_status); + if (ret) + goto out_shdn; + + /* + * Poll PPG_RDY rather than sleeping a fixed interval \ufffd\ufffd\ufffd = the + * internal oscillator may start slower than nominal. 25 ms + * covers more than two 100 Hz sample periods. + */ + ret =3D regmap_read_poll_timeout(data->regmap, + MAX86150_REG_INT_STATUS1, + ppg_rdy_status, + ppg_rdy_status & MAX86150_INT_PPG_RDY, + 1000, 25000); + if (ret) + goto out_shdn; + + ret =3D max86150_read_one_sample(data, ppg_red, ppg_ir, ecg); + +out_shdn: + regmap_set_bits(data->regmap, MAX86150_REG_SYS_CTRL, MAX86150_SYS_SHDN); + return ret; +} + +static int max86150_read_raw(struct iio_dev *indio_dev, + struct iio_chan_spec const *chan, + int *val, int *val2, long mask) +{ + struct max86150_data *data =3D iio_priv(indio_dev); + u32 ppg_red, ppg_ir; + s32 ecg; + int ret; + + switch (mask) { + case IIO_CHAN_INFO_RAW: { + IIO_DEV_ACQUIRE_DIRECT_MODE(indio_dev, claim); + + if (IIO_DEV_ACQUIRE_FAILED(claim)) + return -EBUSY; + + ret =3D max86150_do_read_raw(data, &ppg_red, &ppg_ir, &ecg); + if (ret) + return ret; + + switch (chan->scan_index) { + case MAX86150_IDX_PPG_RED: + *val =3D ppg_red; + break; + case MAX86150_IDX_PPG_IR: + *val =3D ppg_ir; + break; + case MAX86150_IDX_ECG: + *val =3D ecg; + break; + default: + return -EINVAL; + } + return IIO_VAL_INT; + } + default: + return -EINVAL; + } +} + +static const struct iio_info max86150_iio_info =3D { + .read_raw =3D max86150_read_raw, + .validate_trigger =3D iio_validate_own_trigger, +}; + +static int max86150_trigger_disable(struct max86150_data *data) +{ + int ret; + + ret =3D regmap_write(data->regmap, MAX86150_REG_INT_ENABLE1, 0); + if (ret) + return ret; + return regmap_set_bits(data->regmap, MAX86150_REG_SYS_CTRL, + MAX86150_SYS_SHDN); +} + +static int max86150_trigger_enable(struct max86150_data *data) +{ + unsigned int dummy; + int ret; + + ret =3D regmap_clear_bits(data->regmap, MAX86150_REG_SYS_CTRL, + MAX86150_SYS_SHDN); + if (ret) + return ret; + + ret =3D regmap_write(data->regmap, MAX86150_REG_FIFO_WR_PTR, 0); + if (ret) + goto err_shdn; + + ret =3D regmap_write(data->regmap, MAX86150_REG_OVF_COUNTER, 0); + if (ret) + goto err_shdn; + + ret =3D regmap_write(data->regmap, MAX86150_REG_FIFO_RD_PTR, 0); + if (ret) + goto err_shdn; + + /* + * Clear a stale A_FULL latched from before this trigger was enabled; + * otherwise arming INT_ENABLE1 below fires the handler immediately + * against a FIFO state that was never actually seen as full. + */ + ret =3D regmap_read(data->regmap, MAX86150_REG_INT_STATUS1, &dummy); + if (ret) + goto err_shdn; + + ret =3D regmap_write(data->regmap, MAX86150_REG_INT_ENABLE1, + MAX86150_INT_A_FULL); + if (ret) + goto err_shdn; + return 0; + +err_shdn: + regmap_set_bits(data->regmap, MAX86150_REG_SYS_CTRL, MAX86150_SYS_SHDN); + return ret; +} + +static int max86150_set_trigger_state(struct iio_trigger *trig, bool state) +{ + struct iio_dev *indio_dev =3D iio_trigger_get_drvdata(trig); + struct max86150_data *data =3D iio_priv(indio_dev); + + if (state) + return max86150_trigger_enable(data); + return max86150_trigger_disable(data); +} + +static const struct iio_trigger_ops max86150_trigger_ops =3D { + .set_trigger_state =3D max86150_set_trigger_state, + .validate_device =3D iio_trigger_validate_own_device, +}; + +/* + * Threaded IRQ handler: reads and clears INT_STATUS1 to de-assert the + * hardware interrupt line BEFORE iio_trigger_poll() re-enables it. This + * prevents an IRQ storm on level-triggered lines where the line would + * remain asserted until max86150_trigger_handler() ran later in its own + * kthread \ufffd\ufffd\ufffd after IRQF_ONESHOT had already unmasked the = IRQ. + */ +static irqreturn_t max86150_irq_handler(int irq, void *private) +{ + struct iio_trigger *trig =3D private; + struct iio_dev *indio_dev =3D iio_trigger_get_drvdata(trig); + struct max86150_data *data =3D iio_priv(indio_dev); + unsigned int status; + int ret; + + ret =3D regmap_read(data->regmap, MAX86150_REG_INT_STATUS1, &status); + if (ret || !(status & MAX86150_INT_A_FULL)) + return IRQ_NONE; + + iio_trigger_poll(trig); + return IRQ_HANDLED; +} + +static irqreturn_t max86150_trigger_handler(int irq, void *p) +{ + struct iio_poll_func *pf =3D p; + struct iio_dev *idev =3D pf->indio_dev; + struct max86150_data *data =3D iio_priv(idev); + unsigned int wr_ptr, rd_ptr, ovf, n_avail; + u32 ppg_red, ppg_ir; + s32 ecg; + s64 t_drain =3D 0; + int ret; + + /* + * INT_STATUS1 was already read (and the interrupt de-asserted) by + * max86150_irq_handler(). Read only the FIFO pointers here. + */ + ret =3D regmap_read(data->regmap, MAX86150_REG_FIFO_WR_PTR, &wr_ptr); + if (ret) + goto done; + ret =3D regmap_read(data->regmap, MAX86150_REG_FIFO_RD_PTR, &rd_ptr); + if (ret) + goto done; + ret =3D regmap_read(data->regmap, MAX86150_REG_OVF_COUNTER, &ovf); + if (ret) + goto done; + + if (ovf > 0) { + n_avail =3D MAX86150_FIFO_DEPTH; + t_drain =3D iio_get_time_ns(idev); + } else { + n_avail =3D (wr_ptr - rd_ptr) & (MAX86150_FIFO_DEPTH - 1); + /* + * wr_ptr =3D=3D rd_ptr with no overflow means either empty or + * exactly 32 slots filled (pointer wrapped). Since this + * handler is only called when A_FULL fired, the FIFO must + * be full \ufffd\ufffd\ufffd treat as 32 available. + */ + if (n_avail =3D=3D 0) + n_avail =3D MAX86150_FIFO_DEPTH; + } + + for (unsigned int i =3D 0; i < n_avail; i++) { + unsigned int j =3D 0; + s64 ts; + + if (ovf > 0) + ts =3D t_drain - + (s64)(n_avail - 1 - i) * data->sample_period_ns; + else + ts =3D pf->timestamp + + ((s64)i - (MAX86150_FIFO_A_FULL_SAMPLES - 1)) * + data->sample_period_ns; + + ret =3D max86150_read_one_sample(data, &ppg_red, &ppg_ir, &ecg); + if (ret) + break; + + memset(data->scan, 0, sizeof(data->scan)); + + if (test_bit(MAX86150_IDX_PPG_RED, idev->active_scan_mask)) + data->scan[j++] =3D ppg_red; + if (test_bit(MAX86150_IDX_PPG_IR, idev->active_scan_mask)) + data->scan[j++] =3D ppg_ir; + if (test_bit(MAX86150_IDX_ECG, idev->active_scan_mask)) + data->scan[j++] =3D ecg; + + iio_push_to_buffers_with_timestamp(idev, data->scan, ts); + } + +done: + iio_trigger_notify_done(idev->trig); + return IRQ_HANDLED; +} + +static void max86150_powerdown(void *arg) +{ + struct max86150_data *data =3D arg; + struct device *dev =3D regmap_get_device(data->regmap); + int ret; + + ret =3D regmap_write(data->regmap, MAX86150_REG_INT_ENABLE1, 0); + if (ret) + dev_warn(dev, "Failed to disable interrupts: %d\n", ret); + + ret =3D regmap_set_bits(data->regmap, MAX86150_REG_SYS_CTRL, + MAX86150_SYS_SHDN); + if (ret) + dev_warn(dev, "Failed to shut down device: %d\n", ret); +} + +static int max86150_chip_init(struct max86150_data *data) +{ + int ret; + + ret =3D regmap_write(data->regmap, MAX86150_REG_SYS_CTRL, + MAX86150_SYS_RESET); + if (ret) + return ret; + + /* SYS_RESET self-clears within 1 ms (datasheet SYS_CTRL register) */ + fsleep(10 * USEC_PER_MSEC); + + /* + * FIFO_A_FULL holds (FIFO depth - samples available), i.e. how many + * free slots remain when the interrupt should fire. + */ + ret =3D regmap_write(data->regmap, MAX86150_REG_FIFO_CONFIG, + MAX86150_FIFO_ROLLOVER_EN | + FIELD_PREP(MAX86150_FIFO_A_FULL, + MAX86150_FIFO_DEPTH - + MAX86150_FIFO_A_FULL_SAMPLES)); + if (ret) + return ret; + + ret =3D regmap_write(data->regmap, MAX86150_REG_FIFO_DCTRL1, + FIELD_PREP(MAX86150_FIFO_FD1, MAX86150_FD_LED1) | + FIELD_PREP(MAX86150_FIFO_FD2, MAX86150_FD_LED2)); + if (ret) + return ret; + + ret =3D regmap_write(data->regmap, MAX86150_REG_FIFO_DCTRL2, + FIELD_PREP(MAX86150_FIFO_FD3, MAX86150_FD_ECG) | + FIELD_PREP(MAX86150_FIFO_FD4, MAX86150_FD_NONE)); + if (ret) + return ret; + + ret =3D regmap_write(data->regmap, MAX86150_REG_PPG_CONFIG1, + FIELD_PREP(MAX86150_PPG_ADC_RGE, + MAX86150_PPG_ADC_RGE_16384) | + FIELD_PREP(MAX86150_PPG_SR, + MAX86150_PPG_SR_SP_100HZ)); + if (ret) + return ret; + + data->sample_period_ns =3D 10000000; /* matches MAX86150_PPG_SR_SP_100HZ = above */ + + ret =3D regmap_write(data->regmap, MAX86150_REG_LED1_PA, + MAX86150_LED_PA_DEFAULT); + if (ret) + return ret; + + ret =3D regmap_write(data->regmap, MAX86150_REG_LED2_PA, + MAX86150_LED_PA_DEFAULT); + if (ret) + return ret; + + return regmap_write(data->regmap, MAX86150_REG_SYS_CTRL, + MAX86150_SYS_SHDN); +} + +static int max86150_probe(struct i2c_client *client) +{ + struct device *dev =3D &client->dev; + struct iio_dev *indio_dev; + struct max86150_data *data; + unsigned int part_id; + int ret; + + indio_dev =3D devm_iio_device_alloc(dev, sizeof(*data)); + if (!indio_dev) + return -ENOMEM; + + data =3D iio_priv(indio_dev); + + ret =3D devm_regulator_get_enable(dev, "vdd"); + if (ret) + return dev_err_probe(dev, ret, "Failed to enable vdd supply\n"); + + ret =3D devm_regulator_get_enable(dev, "avdd"); + if (ret) + return dev_err_probe(dev, ret, "Failed to enable avdd supply\n"); + + ret =3D devm_regulator_get_enable(dev, "vref"); + if (ret) + return dev_err_probe(dev, ret, "Failed to enable vref supply\n"); + + ret =3D devm_regulator_get_enable(dev, "leds"); + if (ret) + return dev_err_probe(dev, ret, "Failed to enable leds supply\n"); + + data->regmap =3D devm_regmap_init_i2c(client, &max86150_regmap_config); + if (IS_ERR(data->regmap)) + return dev_err_probe(dev, PTR_ERR(data->regmap), + "Failed to init regmap\n"); + + ret =3D regmap_read(data->regmap, MAX86150_REG_PART_ID, &part_id); + if (ret) + return dev_err_probe(dev, ret, "Cannot read part ID\n"); + + if (part_id !=3D MAX86150_PART_ID_VAL) + return dev_err_probe(dev, -ENODEV, + "Unexpected part ID 0x%02x (expected 0x%02x)\n", + part_id, MAX86150_PART_ID_VAL); + + ret =3D max86150_chip_init(data); + if (ret) + return dev_err_probe(dev, ret, "Chip initialisation failed\n"); + + ret =3D devm_add_action_or_reset(dev, max86150_powerdown, data); + if (ret) + return ret; + + indio_dev->name =3D "max86150"; + indio_dev->channels =3D max86150_channels; + indio_dev->num_channels =3D ARRAY_SIZE(max86150_channels); + indio_dev->info =3D &max86150_iio_info; + indio_dev->modes =3D INDIO_DIRECT_MODE; + + if (client->irq > 0) { + data->trig =3D devm_iio_trigger_alloc(dev, "%s-dev%d", + indio_dev->name, + iio_device_id(indio_dev)); + if (!data->trig) + return -ENOMEM; + + data->trig->ops =3D &max86150_trigger_ops; + iio_trigger_set_drvdata(data->trig, indio_dev); + + /* + * The device only ever drives an active-low interrupt line; + * there is no register to reconfigure its polarity or type, + * so the trigger type from firmware needs no help here. + */ + ret =3D devm_request_threaded_irq(dev, client->irq, + NULL, + max86150_irq_handler, + IRQF_ONESHOT, + "max86150", data->trig); + if (ret) + return ret; + + ret =3D devm_iio_trigger_register(dev, data->trig); + if (ret) + return dev_err_probe(dev, ret, + "Failed to register trigger\n"); + } + + ret =3D devm_iio_triggered_buffer_setup(dev, indio_dev, + iio_pollfunc_store_time, + max86150_trigger_handler, + NULL); + if (ret) + return ret; + + /* + * Set the default trigger AFTER buffer setup succeeds. Setting it + * before would leak the iio_trigger_get() reference if buffer setup + * failed: INDIO_BUFFER_TRIGGERED is not set on that path so + * iio_device_release() skips iio_trigger_put(). + */ + if (data->trig) + indio_dev->trig =3D iio_trigger_get(data->trig); + + return devm_iio_device_register(dev, indio_dev); +} + +static const struct i2c_device_id max86150_id[] =3D { + { "max86150" }, + { } +}; +MODULE_DEVICE_TABLE(i2c, max86150_id); + +static const struct of_device_id max86150_of_match[] =3D { + { .compatible =3D "adi,max86150" }, + { } +}; +MODULE_DEVICE_TABLE(of, max86150_of_match); + +static struct i2c_driver max86150_driver =3D { + .driver =3D { + .name =3D "max86150", + .of_match_table =3D max86150_of_match, + }, + .probe =3D max86150_probe, + .id_table =3D max86150_id, +}; +module_i2c_driver(max86150_driver); + +MODULE_AUTHOR("Md Shofiqul Islam "); +MODULE_DESCRIPTION("MAX86150 ECG and PPG biosensor driver"); +MODULE_LICENSE("GPL"); --=20 2.51.1