[PATCH] drm/radeon: Fix negative cursor position

2016-10-27 Thread Michel Dänzer
On 26/09/16 09:09 PM, Takashi Iwai wrote: > On Mon, 26 Sep 2016 11:42:16 +0200, Takashi Iwai wrote: >> On Mon, 26 Sep 2016 10:57:50 +0200, Michel D4nzer wrote: >>> On 23/09/16 10:06 PM, Takashi Iwai wrote: radeon_cursor_move_unlock() contains a workaround for AVIVO chips that are older th

[PATCH] drm/radeon: Fix negative cursor position

2016-10-27 Thread Takashi Iwai
On Thu, 27 Oct 2016 11:02:30 +0200, Michel D4nzer wrote: > > On 26/09/16 09:09 PM, Takashi Iwai wrote: > > On Mon, 26 Sep 2016 11:42:16 +0200, Takashi Iwai wrote: > >> On Mon, 26 Sep 2016 10:57:50 +0200, Michel D4nzer wrote: > >>> On 23/09/16 10:06 PM, Takashi Iwai wrote: > radeon_cursor_move

[PATCH] drm/radeon: Fix negative cursor position

2016-09-26 Thread Michel Dänzer
On 23/09/16 10:06 PM, Takashi Iwai wrote: > radeon_cursor_move_unlock() contains a workaround for AVIVO chips that > are older than DCE6 when the cursor ends on 128 pixel boundary. It > decreases the position when the calculated end position is on 128 > pixel boundary. However, it hits also the c

[PATCH] drm/radeon: Fix negative cursor position

2016-09-26 Thread Takashi Iwai
On Mon, 26 Sep 2016 11:42:16 +0200, Takashi Iwai wrote: > > On Mon, 26 Sep 2016 10:57:50 +0200, > Michel D4nzer wrote: > > > > On 23/09/16 10:06 PM, Takashi Iwai wrote: > > > radeon_cursor_move_unlock() contains a workaround for AVIVO chips that > > > are older than DCE6 when the cursor ends on 1

[PATCH] drm/radeon: Fix negative cursor position

2016-09-26 Thread Takashi Iwai
On Mon, 26 Sep 2016 10:57:50 +0200, Michel D4nzer wrote: > > On 23/09/16 10:06 PM, Takashi Iwai wrote: > > radeon_cursor_move_unlock() contains a workaround for AVIVO chips that > > are older than DCE6 when the cursor ends on 128 pixel boundary. It > > decreases the position when the calculated e

[PATCH] drm/radeon: Fix negative cursor position

2016-09-23 Thread Takashi Iwai
radeon_cursor_move_unlock() contains a workaround for AVIVO chips that are older than DCE6 when the cursor ends on 128 pixel boundary. It decreases the position when the calculated end position is on 128 pixel boundary. However, it hits also the condition where x=-1 and width=1 are passed, since