On Tue Oct 14 19:40:26 2025 +0200, Hans de Goede wrote:
> The exposure minimum and step are constant use the defines for this
> instead of retrieving these from the exposure-control.
>
> Signed-off-by: Hans de Goede <[email protected]>
> Tested-by: Mehdi Djait <[email protected]> # Dell XPS 9315
> Reviewed-by: Mehdi Djait <[email protected]>
> Signed-off-by: Sakari Ailus <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>
Patch committed.
Thanks,
Hans Verkuil
drivers/media/i2c/ov01a10.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
---
diff --git a/drivers/media/i2c/ov01a10.c b/drivers/media/i2c/ov01a10.c
index 1af0af00df24..76c75da4d7c8 100644
--- a/drivers/media/i2c/ov01a10.c
+++ b/drivers/media/i2c/ov01a10.c
@@ -366,9 +366,8 @@ static int ov01a10_set_ctrl(struct v4l2_ctrl *ctrl)
exposure_max = fmt->height + ctrl->val -
OV01A10_EXPOSURE_MAX_MARGIN;
__v4l2_ctrl_modify_range(ov01a10->exposure,
- ov01a10->exposure->minimum,
- exposure_max, ov01a10->exposure->step,
- exposure_max);
+ OV01A10_EXPOSURE_MIN, exposure_max,
+ OV01A10_EXPOSURE_STEP, exposure_max);
}
if (!pm_runtime_get_if_in_use(ov01a10->dev))
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]
