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

[Joonyoung Shim <jy0922.shim@samsung.com>]

Hi Gustavo,

On 04/25/2015 03:15 AM, Gustavo Padovan wrote:
> 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.
> 

Yeah, but this fixed patch needs until merged your atomic patches, if
not, i cannot output updated screen from primary plane on current
exynos drm driver.

Thanks.

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