Re: [PATCH v10 0/2] iio: health: add MAX86150 ECG and PPG biosensor driver
From: Siratul Islam
Date: Sat Jul 18 2026 - 04:37:14 EST
On Fri, 2026-07-17 at 23:11 +0300, Md Shofiqul Islam wrote:
> Sorry for the long gap since v9. Between then and now the driver's
> buffer implementation was rewritten from the kfifo-based design onto
> the standard IIO hardware-trigger and triggered-buffer framework, and
> I read through every open comment from every reviewer before sending
> this rather than sending again too quickly.
>
...
>
> Changes since v9:
> - Rewrote the buffer implementation around devm_iio_trigger_alloc() +
> devm_iio_triggered_buffer_setup() instead of the kfifo buffer and
> manual postenable/predisable pair. The hard-irq handler now only
> reads and clears INT_STATUS1 before calling iio_trigger_poll();
> the threaded trigger handler does the FIFO drain. This relies on
> the trigger core's own attach/detach synchronization instead of an
> explicit iio_buffer_enabled() guard or synchronize_irq(), which the
> old design needed to avoid a NULL active_scan_mask race on teardown
> (Sashiko, both HIGH severity findings)
>
...
>
> --
> 2.51.1
Not sure if it was said already, but please include change log from older versions too.
You could keep the description short, and include Links to each version as well. It is
difficult to hunt them down one by one from lore.
Also, don't send "To" a list. send your patches to Maintainers and
Cc Reviewers and lists.
--
Best regards,
Sirat