lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20210105093803.5247-1-paul.kocialkowski@bootlin.com>
Date:   Tue,  5 Jan 2021 10:38:03 +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] 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 | 2 +-
 drivers/media/i2c/ov8865.c | 7 +++----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/drivers/media/i2c/ov5648.c b/drivers/media/i2c/ov5648.c
index 609aa67b54ce..752a2e035c49 100644
--- a/drivers/media/i2c/ov5648.c
+++ b/drivers/media/i2c/ov5648.c
@@ -496,7 +496,7 @@
 	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..704d75b601e3 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,7 @@
 	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 +1597,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.29.2

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ