From: Jonathan Cameron Date: Sun, 15 Dec 2024 18:28:53 +0000 (+0000) Subject: io: adc: ina2xx-adc: Fix sign and use aligned_s64 for timestamp. X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=1b54068b5934a871f1895adc5e5ca4355781eeb7;p=users%2Fjedix%2Flinux-maple.git io: adc: ina2xx-adc: Fix sign and use aligned_s64 for timestamp. 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: Andy Shevchenko Link: https://patch.msgid.link/20241215182912.481706-3-jic23@kernel.org Signed-off-by: Jonathan Cameron --- diff --git a/drivers/iio/adc/ina2xx-adc.c b/drivers/iio/adc/ina2xx-adc.c index 48c95e12e791f..40d14faa71c50 100644 --- a/drivers/iio/adc/ina2xx-adc.c +++ b/drivers/iio/adc/ina2xx-adc.c @@ -150,7 +150,7 @@ struct ina2xx_chip_info { /* data buffer needs space for channel data and timestamp */ struct { u16 chan[4]; - u64 ts __aligned(8); + aligned_s64 ts; } scan; };