Re: [PATCH 79/92] iio: imu: mpu6050: Fix alignment for DMA safety

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Acked-by: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@xxxxxxx>


From: Jonathan Cameron <jic23@xxxxxxxxxx>
Sent: Tuesday, May 3, 2022 10:59
To: linux-iio@xxxxxxxxxxxxxxx <linux-iio@xxxxxxxxxxxxxxx>
Cc: Akinobu Mita <akinobu.mita@xxxxxxxxx>; Alexandru Lazar <alazar@xxxxxxxxxxxxx>; Alexandru Tachici <alexandru.tachici@xxxxxxxxxx>; Antoniu Miclaus <antoniu.miclaus@xxxxxxxxxx>; Charles-Antoine Couret <charles-antoine.couret@xxxxxxxxxxxxx>; Cosmin Tanislav <cosmin.tanislav@xxxxxxxxxx>; Cristian Pop <cristian.pop@xxxxxxxxxx>; David Lechner <david@xxxxxxxxxxxxxx>; Ivan Mikhaylov <i.mikhaylov@xxxxxxxxx>; Jacopo Mondi <jacopo+renesas@xxxxxxxxxx>; Jean-Baptiste Maneyrol <jmaneyrol@xxxxxxxxxxxxxx>; Lars-Peter Clausen <lars@xxxxxxxxxx>; Marcelo Schmitt <marcelo.schmitt1@xxxxxxxxx>; Mårten Lindahl <martenli@xxxxxxxx>; Matt Ranostay <mranostay@xxxxxxxxx>; Michael Hennerich <michael.hennerich@xxxxxxxxxx>; Michael Welling <mwelling@xxxxxxxx>; Mugilraj Dhavachelvan <dmugil2000@xxxxxxxxx>; Navin Sankar Velliangiri <navin@xxxxxxxxxxx>; Nuno Sá <nuno.sa@xxxxxxxxxx>; Paul Cercueil <paul@xxxxxxxxxxxxxxx>; Phil Reid <preid@xxxxxxxxxxxxxxxxx>; Puranjay Mohan <puranjay12@xxxxxxxxx>; Ricardo Ribalda <ribalda@xxxxxxxxxx>; Robert Jones <rjones@xxxxxxxxxxxxx>; Rui Miguel Silva <rui.silva@xxxxxxxxxx>; Sean Nyekjaer <sean.nyekjaer@xxxxxxxxx>; Tomas Melin <tomas.melin@xxxxxxxxxxx>; Tomislav Denis <tomislav.denis@xxxxxxx>; Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>; Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
Subject: [PATCH 79/92] iio: imu: mpu6050: Fix alignment for DMA safety 
 
[CAUTION] This is EXTERNAL email. Do not click any links or open attachments unless you recognize the sender and know the content is safe.

======================================================================
From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>

____cacheline_aligned is an insufficient guarantee for non-coherent DMA
on platforms with 128 byte cachelines above L1.  Switch to the updated
IIO_ALIGN definition.

Fixes: 6b0cc5dce072 ("iio:imu:inv_mpu6050 Fix dma and ts alignment and data leak issues.")
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
Cc: Jean-Baptiste Maneyrol <jmaneyrol@xxxxxxxxxxxxxx>
---
 drivers/iio/imu/inv_mpu6050/inv_mpu_iio.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_iio.h b/drivers/iio/imu/inv_mpu6050/inv_mpu_iio.h
index c6aa36ee966a..18bd1a50463c 100644
--- a/drivers/iio/imu/inv_mpu6050/inv_mpu_iio.h
+++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_iio.h
@@ -203,7 +203,7 @@ struct inv_mpu6050_state {
         s32 magn_raw_to_gauss[3];
         struct iio_mount_matrix magn_orient;
         unsigned int suspended_sensors;
-       u8 data[INV_MPU6050_OUTPUT_DATA_SIZE] ____cacheline_aligned;
+       u8 data[INV_MPU6050_OUTPUT_DATA_SIZE] __aligned(IIO_ALIGN);
 };
 
 /*register and associated bit definition*/
-- 
2.36.0




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux