From: Sakari Ailus <sakari.ailus@linux.intel.com>
Date: Mon, 10 Jan 2022 16:16:51 +0000 (+0100)
Subject: media: ov5648: Don't pack controls struct
X-Git-Tag: v5.18-rc1~153^2~180
X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=edd4fbff5378a8103470304809195dc8f4b1d42a;p=linux-block.git

media: ov5648: Don't pack controls struct

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>
---

diff --git a/drivers/media/i2c/ov5648.c b/drivers/media/i2c/ov5648.c
index 87f9b724cd7f..3478650ee732 100644
--- a/drivers/media/i2c/ov5648.c
+++ b/drivers/media/i2c/ov5648.c
@@ -639,7 +639,7 @@ struct ov5648_ctrls {
 	struct v4l2_ctrl *pixel_rate;
 
 	struct v4l2_ctrl_handler handler;
-} __packed;
+};
 
 struct ov5648_sensor {
 	struct device *dev;