drivers: video: ov5640: Fix constrast value sign

trunghieulenxp · ngphibang · commit 45fc595673a9 · 2025-05-16T10:19:12.000+02:00
Fix sign's register for constrast value.

Signed-off-by: Trung Hieu Le &lt;trunghieu.le@nxp.com&gt;
Signed-off-by: Phi Bang Nguyen &lt;phibang.nguyen@nxp.com&gt;
diff --git a/drivers/video/ov5640.c b/drivers/video/ov5640.c
@@ -1049,6 +1049,11 @@ static int ov5640_set_ctrl_contrast(const struct device *dev, int value)
 		return ret;
 	}
 
+	ret = ov5640_modify_reg(&cfg->i2c, SDE_CTRL6_REG, BIT(2), value >= 0 ? 0 : BIT(2));
+	if (ret) {
+		return ret;
+	}
+
 	return ov5640_write_reg(&cfg->i2c, SDE_CTRL6_REG, value & 0xff);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1049,6 +1049,11 @@ static int ov5640_set_ctrl_contrast(const struct device *dev, int value)`
`1049`	`1049`	`return ret;`
`1050`	`1050`	`}`
`1051`	`1051`
	`1052`	`+ ret = ov5640_modify_reg(&cfg->i2c, SDE_CTRL6_REG, BIT(2), value >= 0 ? 0 : BIT(2));`
	`1053`	`+ if (ret) {`
	`1054`	`+ return ret;`
	`1055`	`+ }`
	`1056`	`+`
`1052`	`1057`	`return ov5640_write_reg(&cfg->i2c, SDE_CTRL6_REG, value & 0xff);`
`1053`	`1058`	`}`
`1054`	`1059`