From e4ab55fc35664d18638d558cfba13b7149bcf3bb Mon Sep 17 00:00:00 2001 From: Matt Sealey Date: Thu, 27 Jan 2011 10:55:26 -0600 Subject: [PATCH] siihdmi: add comment about vsync_len being too short (HDMI spec) --- drivers/video/mxc/siihdmi.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/video/mxc/siihdmi.c b/drivers/video/mxc/siihdmi.c index dc142d4ddfa..21a7001cbf0 100644 --- a/drivers/video/mxc/siihdmi.c +++ b/drivers/video/mxc/siihdmi.c @@ -636,7 +636,12 @@ static void siihdmi_sanitize_modelist(struct siihdmi_tx * const tx, mode->xres, mode->yres, mode->refresh); remove = true; } else if (mode->vsync_len < 2) { - DEBUG("Removing mode %ux%u@%u (vertical sync period too short)\n", + /* + *HDMI specification requires at least 2 lines of + * vertical sync, as detailed in SII9022 datasheet + * (page 22) and HDMI 1.3 spec (5.1.2) + */ + DEBUG("Removing mode %ux%u@%u (vertical sync period too short)\n", mode->xres, mode->yres, mode->refresh); remove = true; } else {