Re: [PATCH] drm/exynos: fix source data argument for plane

[Gustavo Padovan <gustavo@padovan.org>]

Hi Joonyoung,

2015-04-16 Joonyoung Shim <jy0922.shim@samsung.com>:

> The exynos_update_plane function needs 16.16 fixed point source data.
> 
> Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>
> ---
>  drivers/gpu/drm/exynos/exynos_drm_crtc.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_crtc.c b/drivers/gpu/drm/exynos/exynos_drm_crtc.c
> index eb49195..f0bdaca 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_crtc.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_crtc.c
> @@ -127,7 +127,8 @@ static int exynos_drm_crtc_mode_set_base(struct drm_crtc *crtc, int x, int y,
>  	crtc_h = fb->height - y;
>  
>  	return exynos_update_plane(crtc->primary, crtc, fb, 0, 0,
> -				   crtc_w, crtc_h, x, y, crtc_w, crtc_h);
> +				   crtc_w, crtc_h, x << 16, y << 16,
> +				   crtc_w << 16, crtc_h << 16);
>  }
>  
>  static void exynos_drm_crtc_disable(struct drm_crtc *crtc)
> @@ -202,8 +203,8 @@ static int exynos_drm_crtc_page_flip(struct drm_crtc *crtc,
>  	crtc_w = fb->width - crtc->x;
>  	crtc_h = fb->height - crtc->y;
>  	ret = exynos_update_plane(crtc->primary, crtc, fb, 0, 0,
> -				  crtc_w, crtc_h, crtc->x, crtc->y,
> -				  crtc_w, crtc_h);
> +				  crtc_w, crtc_h, crtc->x << 16, crtc->y << 16,
> +				  crtc_w << 16, crtc_h << 16);

Assuming we are pushing the atomic patches soon we don't need the change
on this patch.  I've asked about the atomic  modesetting patches in the
other thread.

	Gustavo
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel