diff mbox series

[v3,03/20] iio: adc: dln2: Use aligned_s64 for timestamp

Message ID	20250413103443.2420727-4-jic23@kernel.org (mailing list archive)
State	Accepted
Headers	show Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DFBD22AD3E for <linux-iio@vger.kernel.org>; Sun, 13 Apr 2025 10:35:02 +0000 (UTC) From: Jonathan Cameron <jic23@kernel.org> To: linux-iio@vger.kernel.org, David Lechner <dlechner@baylibre.com>, =?utf-8?q?Nuno_S=C3=A1?= <nuno.sa@analog.com>, Andy Shevchenko <andy@kernel.org> Cc: Jonathan Cameron <Jonathan.Cameron@huawei.com> Subject: [PATCH v3 03/20] iio: adc: dln2: Use aligned_s64 for timestamp Date: Sun, 13 Apr 2025 11:34:26 +0100 Message-ID: <20250413103443.2420727-4-jic23@kernel.org> In-Reply-To: <20250413103443.2420727-1-jic23@kernel.org> References: <20250413103443.2420727-1-jic23@kernel.org> Precedence: bulk MIME-Version: 1.0 Content-Transfer-Encoding: 8bit
Series	IIO: Introduce iio_push_to_buffers_with_ts() taking an input buffer length argument. \| expand [v3,00/20] IIO: Introduce iio_push_to_buffers_with_ts() taking an input buffer length argument. [v3,01/20] iio: adc: ad7266: Fix potential timestamp alignment issue. [v3,02/20] iio: adc: ad7768-1: Fix insufficient alignment of timestamp. [v3,03/20] iio: adc: dln2: Use aligned_s64 for timestamp [v3,04/20] iio: accel: adxl355: Make timestamp 64-bit aligned using aligned_s64 [v3,05/20] iio: introduced iio_push_to_buffers_with_ts() that takes a data_total_len argument. [v3,06/20] iio: dummy: Use a fixed structure to build up scan to push to buffers. [v3,07/20] iio: dummy: Switch to iio_push_to_buffers_with_ts() and provide size of storage [v3,08/20] iio: adc: ti-ads131e08: Use new iio_push_to_buffers_with_ts() to provide length sanity c… [v3,09/20] iio: adc: Use iio_push_to_buffers_with_ts() to provide length for runtime checks. [v3,10/20] iio: accel: Use iio_push_to_buffers_with_ts() to provide length for runtime checks. [v3,11/20] iio: accel: hid: Use iio_push_to_buffers_with_ts() to provide length for runtime checks. [v3,12/20] iio: chemical: Use iio_push_to_buffers_with_ts() to provide length for runtime checks. [v3,13/20] iio: temp: maxim-thermocouple: Fix potential lack of DMA safe buffer. [v3,14/20] iio: temperature: Use iio_push_to_buffers_with_ts() to provide length for runtime checks. [v3,15/20] iio: resolver: Use iio_push_to_buffers_with_ts() to provide length for runtime checks. [v3,16/20] iio: proximity: irsd200: Use a struct for scan and iio_push_to_buffers_with_ts() [v3,17/20] iio: proximity: Use iio_push_to_buffers_with_ts() to provide length for runtime checks. [v3,18/20] iio: pressure: zpa2326: Use aligned_s64 for the timestamp [v3,19/20] iio: pressure: Use iio_push_to_buffers_with_ts() to provide length for runtime checks. [v3,20/20] iio: magnetometer: Use iio_push_to_buffers_with_ts() to provide length for runtime check…

Commit Message

Jonathan Cameron April 13, 2025, 10:34 a.m. UTC

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Here the lack of marking allows the overall structure to not be
sufficiently aligned resulting in misplacement of the timestamp
in iio_push_to_buffers_with_timestamp(). Use aligned_s64 to
force the alignment on all architectures.

Reported-by: David Lechner <dlechner@baylibre.com>
Reviewed-by: Andy Shevchenko <andy@kernel.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
---
 drivers/iio/adc/dln2-adc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff mbox series

Patch

diff --git a/drivers/iio/adc/dln2-adc.c b/drivers/iio/adc/dln2-adc.c
index a1e48a756a7b..359e26e3f5bc 100644
--- a/drivers/iio/adc/dln2-adc.c
+++ b/drivers/iio/adc/dln2-adc.c
@@ -466,7 +466,7 @@  static irqreturn_t dln2_adc_trigger_h(int irq, void *p)
 	struct iio_dev *indio_dev = pf->indio_dev;
 	struct {
 		__le16 values[DLN2_ADC_MAX_CHANNELS];
-		int64_t timestamp_space;
+		aligned_s64 timestamp_space;
 	} data;
 	struct dln2_adc_get_all_vals dev_data;
 	struct dln2_adc *dln2 = iio_priv(indio_dev);