Skip to content

Commit

Permalink
media: i2c: ov5648/ov8865: Minor cosmetic fixes
Browse files Browse the repository at this point in the history
This solves a few minor cosmetic issues picked up by checkpatch for
the OV5648 and OV8865 drivers.

Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
  • Loading branch information
Paul Kocialkowski authored and Mauro Carvalho Chehab committed Jan 26, 2021
1 parent 066a94e commit 36e4f2b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
3 changes: 2 additions & 1 deletion drivers/media/i2c/ov5648.c
Original file line number Diff line number Diff line change
Expand Up @@ -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 */

Expand Down
8 changes: 4 additions & 4 deletions drivers/media/i2c/ov8865.c
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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 */

Expand Down Expand Up @@ -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;

Expand Down

0 comments on commit 36e4f2b

Please sign in to comment.