linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hans de Goede <hansg@kernel.org>
To: Sakari Ailus <sakari.ailus@linux.intel.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Hans de Goede <hdegoede@redhat.com>,
	Mathis Foerst <mathis.foerst@mt.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	linux-media@vger.kernel.org, Hans de Goede <hansg@kernel.org>
Subject: [PATCH v2 04/12] media: mt9m114: Lower minimum vblank value
Date: Sat, 31 May 2025 18:31:39 +0200	[thread overview]
Message-ID: <20250531163148.83497-5-hansg@kernel.org> (raw)
In-Reply-To: <20250531163148.83497-1-hansg@kernel.org>

As the comment above the defines says, the minimum values are undocumented
so the lowest values seen in register lists are used.

The version of the mt9m114 driver shipped together with the atomisp code
uses 21 for vblank in its register lists, lower MT9M114_MIN_VBLANK
accordingly.

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Hans de Goede <hansg@kernel.org>
---
 drivers/media/i2c/mt9m114.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/i2c/mt9m114.c b/drivers/media/i2c/mt9m114.c
index e12c69dc9df0..5f59bfd83402 100644
--- a/drivers/media/i2c/mt9m114.c
+++ b/drivers/media/i2c/mt9m114.c
@@ -325,13 +325,13 @@
 
 /*
  * The minimum amount of horizontal and vertical blanking is undocumented. The
- * minimum values that have been seen in register lists are 303 and 38, use
+ * 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.
  */
 #define MT9M114_MIN_HBLANK				303
-#define MT9M114_MIN_VBLANK				38
+#define MT9M114_MIN_VBLANK				21
 #define MT9M114_DEF_HBLANK				323
 #define MT9M114_DEF_VBLANK				39
 
-- 
2.49.0


  parent reply	other threads:[~2025-05-31 16:32 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-31 16:31 [PATCH v2 00/12] media: mt9m114: Changes to make it work with atomisp devices Hans de Goede
2025-05-31 16:31 ` [PATCH v2 01/12] media: aptina-pll: Debug log p1 min and max values Hans de Goede
2025-05-31 16:31 ` [PATCH v2 02/12] media: mt9m114: Add support for clock-frequency property Hans de Goede
2025-05-31 16:31 ` [PATCH v2 03/12] media: mt9m114: Use aptina-PLL helper to get PLL values Hans de Goede
2025-06-03 10:57   ` Laurent Pinchart
2025-06-03 13:29     ` Hans de Goede
2025-06-03 14:05       ` Laurent Pinchart
2025-06-27 14:33         ` Hans de Goede
2025-06-27 18:06           ` Laurent Pinchart
2025-06-28  9:27             ` Hans de Goede
2025-06-29 20:46               ` Laurent Pinchart
2025-05-31 16:31 ` Hans de Goede [this message]
2025-05-31 16:31 ` [PATCH v2 05/12] media: mt9m114: Fix default hblank and vblank values Hans de Goede
2025-05-31 16:31 ` [PATCH v2 06/12] media: mt9m114: Tweak default hblank and vblank for more accurate fps Hans de Goede
2025-05-31 16:31 ` [PATCH v2 07/12] media: mt9m114: Avoid a reset low spike during probe() Hans de Goede
2025-05-31 16:31 ` [PATCH v2 08/12] media: mt9m114: Put sensor in reset on power-down Hans de Goede
2025-06-03 10:59   ` Laurent Pinchart
2025-05-31 16:31 ` [PATCH v2 09/12] media: mt9m114: Fix scaler bypass mode Hans de Goede
2025-06-03 12:48   ` Laurent Pinchart
2025-06-29 15:28     ` Hans de Goede
2025-05-31 16:31 ` [PATCH v2 10/12] media: mt9m114: Drop start-, stop-streaming sequence from initialize Hans de Goede
2025-06-03 11:33   ` Laurent Pinchart
2025-06-29 15:38     ` Hans de Goede
2025-06-29 17:11     ` Hans de Goede
2025-06-29 21:05       ` Laurent Pinchart
2025-05-31 16:31 ` [PATCH v2 11/12] media: mt9m114: Return -EPROBE_DEFER if no endpoint is found Hans de Goede
2025-06-03 11:03   ` Laurent Pinchart
2025-06-03 13:27     ` Hans de Goede
2025-06-05  9:55       ` Sakari Ailus
2025-05-31 16:31 ` [PATCH v2 12/12] media: mt9m114: Add ACPI enumeration support Hans de Goede

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250531163148.83497-5-hansg@kernel.org \
    --to=hansg@kernel.org \
    --cc=hdegoede@redhat.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=mathis.foerst@mt.com \
    --cc=mchehab@kernel.org \
    --cc=sakari.ailus@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).