On 5/3/2022 11:58 AM, Jonathan Cameron wrote:
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: 8a80a71d9020 ("iio: adc: Add MAX1241 driver")
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
Cc: Alexandru Lazar <alazar@xxxxxxxxxxxxx>
---
drivers/iio/adc/max1241.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/iio/adc/max1241.c b/drivers/iio/adc/max1241.c
index a5afd84af58b..e3dbcc2e4e77 100644
--- a/drivers/iio/adc/max1241.c
+++ b/drivers/iio/adc/max1241.c
@@ -26,7 +26,7 @@ struct max1241 {
struct regulator *vref;
struct gpio_desc *shutdown;
- __be16 data ____cacheline_aligned;
+ __be16 data __aligned(IIO_ALIGN);
};
static const struct iio_chan_spec max1241_channels[] = {
Looks good, I think we can call this
Acked-by: Alexandru Lazar <alazar@xxxxxxxxxxxxx>
Thanks!
In addition to that, I also think the changes in max1118 and max1027 are
OK but I can't test either of them so take this with a grain of salt.
All the best,
Alex
(P.S. Fingers crossed that Thunderbird doesn't screw this up, I'm
nowhere near my development machine for the next two weeks :-( .
Apologies if it does!)
