Re: [PATCH v1 2/3] iio: imu: st_lsm6dsx: Use aligned data type for timestamp

From: Jonathan Cameron
Date: Mon Aug 28 2023 - 12:04:07 EST


On Tue, 15 Aug 2023 18:40:26 +0300
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:

> Use __aligned_s64 for the timestamp field.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> ---
> drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
> index c19237717e81..d3d4b3281aa6 100644
> --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
> +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h
> @@ -442,10 +442,9 @@ struct st_lsm6dsx_hw {
> const struct st_lsm6dsx_settings *settings;
>
> struct iio_mount_matrix orientation;
> - /* Ensure natural alignment of buffer elements */
> struct {
> __le16 channels[3];
> - s64 ts __aligned(8);
> + __aligned_s64 ts;

aligned_s64 as it's internal to the kernel?

> } scan[ST_LSM6DSX_ID_MAX];
> };
>