From: "Guoniu.zhou" <guoniu.zhou@xxxxxxx> If run OV5640 with default setting after power up, the real frame rate for it is 60, not 30. The reason is default frame interval parameter initialized in probe is 30 but default link frequency is to generate 60 frame rate, so correct it. Signed-off-by: Guoniu.zhou <guoniu.zhou@xxxxxxx> Reviewed-by: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> --- v1->v2: 1) fix typo issue(s/runn/run) 2) keep original OV5640 default link frequency 3) correct comments and frame_interval parameters to match actual frame rate --- drivers/media/i2c/ov5640.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/media/i2c/ov5640.c b/drivers/media/i2c/ov5640.c index 1536649b9e90..5c01bb9414c9 100644 --- a/drivers/media/i2c/ov5640.c +++ b/drivers/media/i2c/ov5640.c @@ -3851,11 +3851,11 @@ static int ov5640_probe(struct i2c_client *client) /* * default init sequence initialize sensor to - * YUV422 UYVY VGA@30fps + * YUV422 UYVY VGA@60fps */ sensor->frame_interval.numerator = 1; - sensor->frame_interval.denominator = ov5640_framerates[OV5640_30_FPS]; - sensor->current_fr = OV5640_30_FPS; + sensor->frame_interval.denominator = ov5640_framerates[OV5640_60_FPS]; + sensor->current_fr = OV5640_60_FPS; sensor->current_mode = &ov5640_mode_data[OV5640_MODE_VGA_640_480]; sensor->last_mode = sensor->current_mode; -- 2.37.1