Re: [PATCH resend] ocfb: fix tgdel and tvdel timing parameters

[Tomi Valkeinen <tomi.valkeinen@ti.com>]

[-- Attachment #1: Type: text/plain, Size: 1296 bytes --]



On 26/01/16 16:48, Andrea Merello wrote:
> According to the ocfb documentation:
> Fix tgdel HW param should be left margin, not right.
> Fix tvdel HW param should upper margin, not lower.
> 
> This seems to fix lock issues on certain monitors (tested on a
> slightly customized IP, but the FPGA guy said that it should
> be the same wrt this changes).
> 
> Signed-off-by: Andrea Merello <andrea.merello@gmail.com>
> 
> diff --git a/drivers/video/fbdev/ocfb.c b/drivers/video/fbdev/ocfb.c
> index c9293ae..a970edc2 100644
> --- a/drivers/video/fbdev/ocfb.c
> +++ b/drivers/video/fbdev/ocfb.c
> @@ -123,11 +123,11 @@ static int ocfb_setupfb(struct ocfb_dev *fbdev)
> 
>  	/* Horizontal timings */
>  	ocfb_writereg(fbdev, OCFB_HTIM, (var->hsync_len - 1) << 24 |
> -		      (var->right_margin - 1) << 16 | (var->xres - 1));
> +		      (var->left_margin - 1) << 16 | (var->xres - 1));
> 
>  	/* Vertical timings */
>  	ocfb_writereg(fbdev, OCFB_VTIM, (var->vsync_len - 1) << 24 |
> -		      (var->lower_margin - 1) << 16 | (var->yres - 1));
> +		      (var->upper_margin - 1) << 16 | (var->yres - 1));
> 
>  	/* Total length of frame */
>  	hlen = var->left_margin + var->right_margin + var->hsync_len +
> --
> 1.9.1
> 

Thanks, queued for 4.5 fixes.

 Tomi


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]