]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
iio: adc: ad7944: Fix sign and use aligned_s64 for timestamp.
authorJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 15 Dec 2024 18:28:52 +0000 (18:28 +0000)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sat, 28 Dec 2024 14:28:14 +0000 (14:28 +0000)
Whilst it doesn't actually make any difference because the code
that fills this field doesn't care, timestamps are all signed.
Use the new aligned_s64 instead of open coding alignment to avoid
confusing static analyzers and give slightly cleaner code.

Reviewed-by: David Lechner <dlechner@baylibre.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://patch.msgid.link/20241215182912.481706-2-jic23@kernel.org
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/adc/ad7944.c

index a5aea4e9f1a7bdd8ca10f9f4580ad3216ddcdfcb..0ec9cda10f5f8f61727581b152fb921f2e0b4bff 100644 (file)
@@ -75,7 +75,7 @@ struct ad7944_adc {
                        u16 u16;
                        u32 u32;
                } raw;
-               u64 timestamp __aligned(8);
+               aligned_s64 timestamp;
         } sample __aligned(IIO_DMA_MINALIGN);
 };