[PATCH] drm/amdgpu: Don't save new cursor size before updating CUR_SIZE register.

Daniel Vetter daniel at ffwll.ch
Sun Dec 18 13:46:00 UTC 2016


On Sat, Dec 17, 2016 at 04:35:09PM +0000, 'Liviu Dudau' wrote:
> On Fri, Dec 16, 2016 at 07:16:25PM +0000, Deucher, Alexander wrote:
> > > -----Original Message-----
> > > From: Liviu Dudau [mailto:liviu at dudau.co.uk]
> > > Sent: Friday, December 16, 2016 2:11 PM
> > > To: Daenzer, Michel; Deucher, Alexander
> > > Cc: Koenig, Christian; David Airlie; dri-devel at lists.freedesktop.org; amd-
> > > gfx at lists.freedesktop.org; Liviu Dudau
> > > Subject: [PATCH] drm/amdgpu: Don't save new cursor size before updating
> > > CUR_SIZE register.
> > > 
> > > Commit 7c83d7abc999 ("drm/amdgpu: Only update the CUR_SIZE register
> > > when
> > > necessary") did not cleanup correctly the old code for DCE v6 and v8.
> > > As a consequence, cursor updates stopped working for my Radeon R9
> > > 1002:67b0
> > > dual-monitor setup.
> > > 
> > > Fixes: 7c83d7abc999 ("drm/amdgpu: Only update the CUR_SIZE register
> > > when necessary")
> > > Signed-off-by: Liviu Dudau <liviu at dudau.co.uk>
> > 
> > Already fixed:
> > https://lists.freedesktop.org/archives/amd-gfx/2016-December/003985.html
> 
> Sorry, I've only checked the dri-devel list, I've only noticed there is an
> amd-gfx one while preparing the patch for sending.

Hm yeah, MAINTAINERS is not updated to include the new list. Alex, can you
pls fix that?
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the dri-devel mailing list