media: ov5648: Don't pack controls struct
authorSakari Ailus <sakari.ailus@linux.intel.com>
Mon, 10 Jan 2022 16:16:51 +0000 (17:16 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 28 Jan 2022 18:32:49 +0000 (19:32 +0100)
Don't pack the driver specific struct containing control pointers. This
lead to potential alignment issues when working with the pointers.

Reported-by: kernel test robot <lkp@intel.com>
Fixes: e43ccb0a045f ("media: i2c: Add support for the OV5648 image sensor")
Reviewed-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/i2c/ov5648.c

index 87f9b72..3478650 100644 (file)
@@ -639,7 +639,7 @@ struct ov5648_ctrls {
        struct v4l2_ctrl *pixel_rate;
 
        struct v4l2_ctrl_handler handler;
-} __packed;
+};
 
 struct ov5648_sensor {
        struct device *dev;