Sakari, thanks. I did not find that on patchwork, so resend it again. :) On 10/21/20 5:20 PM, Sakari Ailus wrote: > Hi Bingbu, > > On Wed, Oct 21, 2020 at 10:44:10AM +0800, Bingbu Cao wrote: >> The minimal valid exposure value should be 4 lines instead of 8 for >> ov2740 mannual exposure control. >> >> Signed-off-by: Bingbu Cao <bingbu.cao@xxxxxxxxx> >> --- >> drivers/media/i2c/ov2740.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/media/i2c/ov2740.c b/drivers/media/i2c/ov2740.c >> index bd0d45b0d43f..0dbae4949cb3 100644 >> --- a/drivers/media/i2c/ov2740.c >> +++ b/drivers/media/i2c/ov2740.c >> @@ -37,7 +37,7 @@ >> >> /* Exposure controls from sensor */ >> #define OV2740_REG_EXPOSURE 0x3500 >> -#define OV2740_EXPOSURE_MIN 8 >> +#define OV2740_EXPOSURE_MIN 4 >> #define OV2740_EXPOSURE_MAX_MARGIN 8 >> #define OV2740_EXPOSURE_STEP 1 > > This is already in my tree --- I may forgotten to push master after > applying many patches, it is up-to-date now. > -- Best regards, Bingbu Cao