Re: [PATCH v2 0/3] iio: Introduce and use aligned_s64 type

Jonathan Cameron
2024-09-07T15:38:00Z

On Tue, 3 Sep 2024 20:59:03 +0300
Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:

> Instead of having open coded idea of aligned member, use
> a newly defined type like it's done in, e.g., u64 case.
> Update a few IIO drivers to show how to use it.
>
> v2 (took only one year from v1, not bad!):
:)

Applied with that tweak for patch 2 that you called out.

Will probably be next cycle though before these go upstream
(so think of this as queuing them up very early for 6.13 :)

> - avoided touching unrelated comments, code lines, etc. (Jonathan)
> - used kernel internal type for the in-kernel code (Jonathan)
>
> Andy Shevchenko (3):
> types: Complement the aligned types with signed 64-bit one
> iio: imu: st_lsm6dsx: Use aligned data type for timestamp
> iio: hid-sensor: Use aligned data type for timestamp
>
> drivers/iio/accel/hid-sensor-accel-3d.c | 2 +-
> drivers/iio/gyro/hid-sensor-gyro-3d.c | 2 +-
> drivers/iio/humidity/hid-sensor-humidity.c | 2 +-
> drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 3 +--
> drivers/iio/light/hid-sensor-als.c | 2 +-
> drivers/iio/orientation/hid-sensor-incl-3d.c | 2 +-
> drivers/iio/orientation/hid-sensor-rotation.c | 2 +-
> drivers/iio/position/hid-sensor-custom-intel-hinge.c | 2 +-
> drivers/iio/pressure/hid-sensor-press.c | 2 +-
> drivers/iio/temperature/hid-sensor-temperature.c | 2 +-
> include/linux/types.h | 3 ++-
> include/uapi/linux/types.h | 1 +
> 12 files changed, 13 insertions(+), 12 deletions(-)
>