[<prev] [next>] [day] [month] [year] [list]
Message-Id: <20210105144720.9578-1-paul.kocialkowski@bootlin.com>
Date: Tue, 5 Jan 2021 15:47:19 +0100
From: Paul Kocialkowski <paul.kocialkowski@...tlin.com>
To: linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Cc: Mauro Carvalho Chehab <mchehab@...nel.org>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Paul Kocialkowski <paul.kocialkowski@...tlin.com>
Subject: [PATCH v2] media: i2c: ov5648/ov8865: Minor cosmetic fixes
This solves a few minor cosmetic issues picked up by checkpatch for
the OV5648 and OV8865 drivers.
Signed-off-by: Paul Kocialkowski <paul.kocialkowski@...tlin.com>
---
drivers/media/i2c/ov5648.c | 3 ++-
drivers/media/i2c/ov8865.c | 8 ++++----
2 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/media/i2c/ov5648.c b/drivers/media/i2c/ov5648.c
index 609aa67b54ce..110190b322e2 100644
--- a/drivers/media/i2c/ov5648.c
+++ b/drivers/media/i2c/ov5648.c
@@ -496,7 +496,8 @@
container_of(s, struct ov5648_sensor, subdev)
#define ov5648_ctrl_subdev(c) \
- (&container_of(c->handler, struct ov5648_sensor, ctrls.handler)->subdev)
+ (&container_of((c)->handler, struct ov5648_sensor, \
+ ctrls.handler)->subdev)
/* Data structures */
diff --git a/drivers/media/i2c/ov8865.c b/drivers/media/i2c/ov8865.c
index fda5a55979aa..ab4804d5b285 100644
--- a/drivers/media/i2c/ov8865.c
+++ b/drivers/media/i2c/ov8865.c
@@ -223,7 +223,6 @@
#define OV8865_FORMAT2_FLIP_HORZ_SENSOR_EN BIT(1)
#define OV8865_FORMAT2_SYNC_HBIN_EN BIT(0)
-
#define OV8865_INC_Y_ODD_REG 0x382a
#define OV8865_INC_Y_ODD(v) ((v) & GENMASK(4, 0))
#define OV8865_INC_Y_EVEN_REG 0x382b
@@ -460,7 +459,8 @@
container_of(s, struct ov8865_sensor, subdev)
#define ov8865_ctrl_subdev(c) \
- (&container_of(c->handler, struct ov8865_sensor, ctrls.handler)->subdev)
+ (&container_of((c)->handler, struct ov8865_sensor, \
+ ctrls.handler)->subdev)
/* Data structures */
@@ -1598,8 +1598,8 @@ static int ov8865_mode_pll1_configure(struct ov8865_sensor *sensor,
return ret;
ret = ov8865_update_bits(sensor, OV8865_PCLK_SEL_REG,
- OV8865_PCLK_SEL_PCLK_DIV_MASK,
- OV8865_PCLK_SEL_PCLK_DIV(config->pclk_div));
+ OV8865_PCLK_SEL_PCLK_DIV_MASK,
+ OV8865_PCLK_SEL_PCLK_DIV(config->pclk_div));
if (ret)
return ret;
--
2.30.0
Powered by blists - more mailing lists