media: ov5640: correct comments for default VGA to avoid confusion
authorGuoniu.zhou <guoniu.zhou@nxp.com>
Tue, 9 May 2023 06:56:45 +0000 (14:56 +0800)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Thu, 25 May 2023 14:21:22 +0000 (16:21 +0200)
When OV5640 work at DVP mode, the default initialization settings
make it output 30 frames per second. But when it work at CSI-2 mode
the default link frequency will make it output 60 frames per second,
so correct the comments to make it more clear.

Signed-off-by: Guoniu.zhou <guoniu.zhou@nxp.com>
Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/i2c/ov5640.c

index 3a9dc7d11decdbbdea3d875ea3212489831b1273..5b43bbfe4774407670b2b241a133312d37e66c49 100644 (file)
@@ -3851,7 +3851,7 @@ static int ov5640_probe(struct i2c_client *client)
 
        /*
         * default init sequence initialize sensor to
-        * YUV422 UYVY VGA@30fps
+        * YUV422 UYVY VGA(30FPS in parallel mode, 60 in MIPI CSI-2 mode)
         */
        sensor->frame_interval.numerator = 1;
        sensor->frame_interval.denominator = ov5640_framerates[OV5640_30_FPS];