Hi Andy, Thank you for your patch. On 1/3/23 15:52, Andy Shevchenko wrote: > There is no point to specify asm-generic for the unaligned.h. > Drop the 'generic' suffix and move the inclusion to be after > the non-media linux/* ones. > > Cc: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > --- > drivers/media/i2c/st-vgxy61.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/i2c/st-vgxy61.c b/drivers/media/i2c/st-vgxy61.c > index 826baf4e064d..5dcabee6677d 100644 > --- a/drivers/media/i2c/st-vgxy61.c > +++ b/drivers/media/i2c/st-vgxy61.c > @@ -5,7 +5,6 @@ > * Copyright (C) 2022 STMicroelectronics SA > */ > > -#include <asm-generic/unaligned.h> > #include <linux/clk.h> > #include <linux/delay.h> > #include <linux/gpio/consumer.h> > @@ -15,6 +14,9 @@ > #include <linux/pm_runtime.h> > #include <linux/regulator/consumer.h> > #include <linux/units.h> > + > +#include <asm/unaligned.h> > + Any particular reason not to keep the alphabetical sort ? > #include <media/mipi-csi2.h> > #include <media/v4l2-async.h> > #include <media/v4l2-ctrls.h> -- Regards, Benjamin
