media: mt9m114: Fix default hblank and vblank values

The current default hblank and vblank values are based on reaching 30 fps
with the pixel-array outputting 1280x960, but the default format for
the pixel-array source pad and the isp sink pad is 1296x976, correct
the default hblank and vblank values to take this into account.

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Hans de Goede <johannes.goede@oss.qualcomm.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>

diff --git a/drivers/media/i2c/mt9m114.c b/drivers/media/i2c/mt9m114.c
index 0c536d71b01ffd4307a699d95c7b7d161be22e08..0076723c5aa5bb82ab775f161bc9916c5126562d 100644 (file)
--- a/drivers/media/i2c/mt9m114.c
+++ b/drivers/media/i2c/mt9m114.c
@@ -332,12 +332,13 @@
  * minimum values that have been seen in register lists are 303 and 21, use
  * them.
  *
- * Set the default to achieve 1280x960 at 30fps with a 48 MHz pixclock.
+ * Set the default to achieve full resolution (1296x976 analog crop
+ * rectangle, 1280x960 output size) at 30fps with a 48 MHz pixclock.
  */
 #define MT9M114_MIN_HBLANK                             303
 #define MT9M114_MIN_VBLANK                             21
-#define MT9M114_DEF_HBLANK                             323
-#define MT9M114_DEF_VBLANK                             39
+#define MT9M114_DEF_HBLANK                             307
+#define MT9M114_DEF_VBLANK                             23
 
 #define MT9M114_DEF_FRAME_RATE                         30
 #define MT9M114_MAX_FRAME_RATE                         120