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

[Petre Rodan <petre.rodan@subdimension.ro>]

[-- Attachment #1: Type: text/plain, Size: 1480 bytes --]


Hello Jonathan,

thank you for the review.

I actually heard about structs who's elements are 64byte padded so they don't end up in the same cache update in another context but up until today I did not make the connection with what is going on here.
thank you for the detailed explanation.

On Sat, Dec 06, 2025 at 08:54:07PM +0000, Jonathan Cameron wrote:
> On Thu, 04 Dec 2025 17:12:56 +0200
> Petre Rodan <petre.rodan@subdimension.ro> wrote:
> 
> > Adds driver for digital Honeywell ABP2 series of board mount
> > pressure and temperature sensors.

[..]

> Nice clean driver.

well, thank you. I will be sending in a few modifications to my other Honeywell
drivers based on feedback received here.

> > +#include <linux/jiffies.h>
> 
> I'm not immediately spotting use of this but I might well be missing it!

it's an indirect include for the definition of 'HZ'. requested by Andy.

> > +	data->scan.chan[0] = get_unaligned_be24(&data->rx_buf[1]);
> > +	data->scan.chan[1] = get_unaligned_be24(&data->rx_buf[4]);
> Turn on just the second channel and run the test in tools/iio/ for buffered
> reads.
> 
> You'll see data from the first channel not the one userspace asked for.
> Set available_scan_masks = { 0x3, 0 };
> allows the IIO core to know this is what is happening and shuffle the
> data as necessary for you.

nice catch. it was outputing pure garbage (not the first channel) but your change
fixed it.

best regards,
peter

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]