[PATCH] drm/nvd0/disp: mask off high 16 bit of negative cursor x-coordinate

[PATCH] drm/nvd0/disp: mask off high 16 bit of negative cursor x-coordinate

[Christoph Bumiller]

---
 drivers/gpu/drm/nouveau/nvd0_display.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/nvd0_display.c b/drivers/gpu/drm/nouveau/nvd0_display.c
index c486d3c..c50b075 100644
--- a/drivers/gpu/drm/nouveau/nvd0_display.c
+++ b/drivers/gpu/drm/nouveau/nvd0_display.c
@@ -790,7 +790,7 @@ nvd0_crtc_cursor_move(struct drm_crtc *crtc, int x, int y)
 	struct nouveau_crtc *nv_crtc = nouveau_crtc(crtc);
 	int ch = EVO_CURS(nv_crtc->index);
 
-	evo_piow(crtc->dev, ch, 0x0084, (y << 16) | x);
+	evo_piow(crtc->dev, ch, 0x0084, (y << 16) | (x & 0xffff));
 	evo_piow(crtc->dev, ch, 0x0080, 0x00000000);
 	return 0;
 }
-- 
1.7.3.4

Re: [PATCH] drm/nvd0/disp: mask off high 16 bit of negative cursor x-coordinate

[Maarten Lankhorst]

Op 26-07-12 20:53, Christoph Bumiller schreef:
> ---
>  drivers/gpu/drm/nouveau/nvd0_display.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/gpu/drm/nouveau/nvd0_display.c b/drivers/gpu/drm/nouveau/nvd0_display.c
> index c486d3c..c50b075 100644
> --- a/drivers/gpu/drm/nouveau/nvd0_display.c
> +++ b/drivers/gpu/drm/nouveau/nvd0_display.c
> @@ -790,7 +790,7 @@ nvd0_crtc_cursor_move(struct drm_crtc *crtc, int x, int y)
>  	struct nouveau_crtc *nv_crtc = nouveau_crtc(crtc);
>  	int ch = EVO_CURS(nv_crtc->index);
>  
> -	evo_piow(crtc->dev, ch, 0x0084, (y << 16) | x);
> +	evo_piow(crtc->dev, ch, 0x0084, (y << 16) | (x & 0xffff));
>  	evo_piow(crtc->dev, ch, 0x0080, 0x00000000);
>  	return 0;
>  }

Signed-off-by: Maarten Lankhorst <maarten.lankhorst-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org [3.4]+

3.2 didn't have d9 support yet.

Re: [PATCH] drm/nvd0/disp: mask off high 16 bit of negative cursor x-coordinate

[Ben Skeggs]

On Fri, Jul 27, 2012 at 6:04 AM, Maarten Lankhorst
<m.b.lankhorst-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> Op 26-07-12 20:53, Christoph Bumiller schreef:
>> ---
>>  drivers/gpu/drm/nouveau/nvd0_display.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/nouveau/nvd0_display.c b/drivers/gpu/drm/nouveau/nvd0_display.c
>> index c486d3c..c50b075 100644
>> --- a/drivers/gpu/drm/nouveau/nvd0_display.c
>> +++ b/drivers/gpu/drm/nouveau/nvd0_display.c
>> @@ -790,7 +790,7 @@ nvd0_crtc_cursor_move(struct drm_crtc *crtc, int x, int y)
>>       struct nouveau_crtc *nv_crtc = nouveau_crtc(crtc);
>>       int ch = EVO_CURS(nv_crtc->index);
>>
>> -     evo_piow(crtc->dev, ch, 0x0084, (y << 16) | x);
>> +     evo_piow(crtc->dev, ch, 0x0084, (y << 16) | (x & 0xffff));
>>       evo_piow(crtc->dev, ch, 0x0080, 0x00000000);
>>       return 0;
>>  }
>
> Signed-off-by: Maarten Lankhorst <maarten.lankhorst-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
> Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org [3.4]+
I've already picked this up for nouveau git also.

>
> 3.2 didn't have d9 support yet.
>
> _______________________________________________
> Nouveau mailing list
> Nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
> http://lists.freedesktop.org/mailman/listinfo/nouveau