[PATCH] fixup! media: i2c: ov08d10: add support for reset and power management

Matthias Fend posted 1 patch 1 week ago
drivers/media/i2c/ov08d10.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
[PATCH] fixup! media: i2c: ov08d10: add support for reset and power management
Posted by Matthias Fend 1 week ago
Unconditionally wait for the "XSHUTDN pull up to SCCB start" delay at
startup and fix a violation of the preferred line limit of 80 characters.

Signed-off-by: Matthias Fend <matthias.fend@emfend.at>
---
 drivers/media/i2c/ov08d10.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/drivers/media/i2c/ov08d10.c b/drivers/media/i2c/ov08d10.c
index 9adef5446a61f3204fb809ca3f077c1afb5f7a47..da517eb2181b83ed2ad786bd5e197fe755a47ada 100644
--- a/drivers/media/i2c/ov08d10.c
+++ b/drivers/media/i2c/ov08d10.c
@@ -1358,11 +1358,11 @@ static int ov08d10_power_on(struct device *dev)
 		fsleep(5 * USEC_PER_MSEC);
 
 		reset_control_deassert(ov08d10->reset);
-
-		/* Delay from XSHUTDN pull up to SCCB start: 8ms */
-		fsleep(8 * USEC_PER_MSEC);
 	}
 
+	/* Delay from XSHUTDN pull up to SCCB start: 8ms */
+	fsleep(8 * USEC_PER_MSEC);
+
 	return 0;
 }
 
@@ -1515,7 +1515,8 @@ static int ov08d10_probe(struct i2c_client *client)
 		return ret;
 	}
 
-	ov08d10->reset = devm_reset_control_get_optional_exclusive(ov08d10->dev, NULL);
+	ov08d10->reset = devm_reset_control_get_optional_exclusive(ov08d10->dev,
+								   NULL);
 	if (IS_ERR(ov08d10->reset))
 		return dev_err_probe(ov08d10->dev, PTR_ERR(ov08d10->reset),
 				     "failed to get reset\n");

---
base-commit: 80a7ee2c54995130472b02c66acd34c2e171b90d
change-id: 20260326-ov08d10-fixups-d1317bfa59d6

Best regards,
-- 
Matthias Fend <matthias.fend@emfend.at>