Re: [PATCH v2] fbdev: modedb: fix CEA modes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 18 Nov 2010, Arnd Hannemann wrote:

> Some of the modes were missing the correct sync polarities.
> This was causing a corrupt or left shifted picture on my TV.
> Additionally format #35 had a wrong refresh rate and pixel clock.
> 
> This patch fixes those issues.
> 
> Signed-off-by: Arnd Hannemann <arnd@xxxxxxxxxx>

Acked-by: Guennadi Liakhovetski <g.liakhovetski@xxxxxx>

Thanks
Guennadi

> 
> ---
> v2: Format #5 had also wrong sync settings, so include the fix in this patch
> ---
>  drivers/video/modedb.c |   14 +++++++++-----
>  1 files changed, 9 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/video/modedb.c b/drivers/video/modedb.c
> index 9a0ae6c..c3a1825 100644
> --- a/drivers/video/modedb.c
> +++ b/drivers/video/modedb.c
> @@ -289,7 +289,8 @@ const struct fb_videomode cea_modes[64] = {
>  	},
>  	/* #5: 1920x1080i@xxxxx/60Hz */
>  	[5] = {
> -		NULL, 60, 1920, 1080, 13763, 148, 88, 15, 2, 44, 5, 0, FB_VMODE_INTERLACED, 0,
> +		NULL, 60, 1920, 1080, 13763, 148, 88, 15, 2, 44, 5,
> +		FB_SYNC_HOR_HIGH_ACT | FB_SYNC_VERT_HIGH_ACT, FB_VMODE_INTERLACED, 0,
>  	},
>  	/* #7: 720(1440)x480iH@xxxxx/60Hz */
>  	[7] = {
> @@ -305,19 +306,22 @@ const struct fb_videomode cea_modes[64] = {
>  	},
>  	/* #19: 1280x720p@50Hz */
>  	[19] = {
> -		NULL, 50, 1280, 720, 13468, 220, 440, 20, 5, 40, 5, 0, FB_VMODE_NONINTERLACED, 0,
> +		NULL, 50, 1280, 720, 13468, 220, 440, 20, 5, 40, 5,
> +		FB_SYNC_HOR_HIGH_ACT | FB_SYNC_VERT_HIGH_ACT, FB_VMODE_NONINTERLACED, 0,
>  	},
>  	/* #20: 1920x1080i@50Hz */
>  	[20] = {
> -		NULL, 50, 1920, 1080, 13480, 148, 528, 15, 5, 528, 5, 0, FB_VMODE_INTERLACED, 0,
> +		NULL, 50, 1920, 1080, 13480, 148, 528, 15, 5, 528, 5,
> +		FB_SYNC_HOR_HIGH_ACT | FB_SYNC_VERT_HIGH_ACT, FB_VMODE_INTERLACED, 0,
>  	},
>  	/* #32: 1920x1080p@xxxxx/24Hz */
>  	[32] = {
> -		NULL, 24, 1920, 1080, 13468, 148, 638, 36, 4, 44, 5, 0, FB_VMODE_NONINTERLACED, 0,
> +		NULL, 24, 1920, 1080, 13468, 148, 638, 36, 4, 44, 5,
> +		FB_SYNC_HOR_HIGH_ACT | FB_SYNC_VERT_HIGH_ACT, FB_VMODE_NONINTERLACED, 0,
>  	},
>  	/* #35: (2880)x480p4x@xxxxx/60Hz */
>  	[35] = {
> -		NULL, 50, 2880, 480, 11100, 240, 64, 30, 9, 248, 6, 0, FB_VMODE_NONINTERLACED, 0,
> +		NULL, 60, 2880, 480, 9250, 240, 64, 30, 9, 248, 6, 0, FB_VMODE_NONINTERLACED, 0,
>  	},
>  };
>  
> -- 
> 1.7.0.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Tourism]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux