Re: [PATCH 6/8] drm/mgag200: Rewrite cursor handling

2019-06-12 Thread Daniel Vetter
On Wed, Jun 12, 2019 at 09:27:21AM +0200, Thomas Zimmermann wrote: > Hi > > Am 11.06.19 um 17:33 schrieb Daniel Vetter: > > On Tue, Jun 11, 2019 at 2:32 PM Thomas Zimmermann > > wrote: > >> > >> Hi > >> > >> Am 05.06.19 um 11:58 schrieb Gerd Hoffmann: > >>> On Tue, Jun 04, 2019 at 05:41:59PM +02

Re: [PATCH 6/8] drm/mgag200: Rewrite cursor handling

2019-06-12 Thread Thomas Zimmermann
Hi Am 11.06.19 um 17:33 schrieb Daniel Vetter: > On Tue, Jun 11, 2019 at 2:32 PM Thomas Zimmermann wrote: >> >> Hi >> >> Am 05.06.19 um 11:58 schrieb Gerd Hoffmann: >>> On Tue, Jun 04, 2019 at 05:41:59PM +0200, Thomas Zimmermann wrote: The cursor handling in mgag200 is complicated to underst

Re: [PATCH 6/8] drm/mgag200: Rewrite cursor handling

2019-06-11 Thread Daniel Vetter
On Tue, Jun 11, 2019 at 2:32 PM Thomas Zimmermann wrote: > > Hi > > Am 05.06.19 um 11:58 schrieb Gerd Hoffmann: > > On Tue, Jun 04, 2019 at 05:41:59PM +0200, Thomas Zimmermann wrote: > >> The cursor handling in mgag200 is complicated to understand. It touches a > >> number of different BOs, but do

Re: [PATCH 6/8] drm/mgag200: Rewrite cursor handling

2019-06-11 Thread Thomas Zimmermann
Hi Am 05.06.19 um 11:58 schrieb Gerd Hoffmann: > On Tue, Jun 04, 2019 at 05:41:59PM +0200, Thomas Zimmermann wrote: >> The cursor handling in mgag200 is complicated to understand. It touches a >> number of different BOs, but doesn't really use all of them. >> >> Rewriting the cursor update reduces

Re: [PATCH 6/8] drm/mgag200: Rewrite cursor handling

2019-06-05 Thread Gerd Hoffmann
On Tue, Jun 04, 2019 at 05:41:59PM +0200, Thomas Zimmermann wrote: > The cursor handling in mgag200 is complicated to understand. It touches a > number of different BOs, but doesn't really use all of them. > > Rewriting the cursor update reduces the amount of cursor state. There are > two BOs for

[PATCH 6/8] drm/mgag200: Rewrite cursor handling

2019-06-04 Thread Thomas Zimmermann
The cursor handling in mgag200 is complicated to understand. It touches a number of different BOs, but doesn't really use all of them. Rewriting the cursor update reduces the amount of cursor state. There are two BOs for double-buffered HW updates. The source BO updates the one that is currently n