Am 14.11.2016 um 10:02 schrieb Daniel Vetter: > This old code pattern was reintroduced in > > Author: Ken Wang <Qingqing.Wang at amd.com> > Date: Tue Jan 19 14:03:24 2016 +0800 > > drm/amdgpu: add display controller implementation for si v10 > > Realign it with later display code. Tsk, tsk for massive copypasting > ;-) > > Cc: Christian König <christian.koenig at amd.com> > Cc: Ken Wang <Qingqing.Wang at amd.com> > Cc: Alex Deucher <alexander.deucher at amd.com> > Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
Acked-by: Christian König <christian.koenig at amd.com> for the whole series. Not that I'm deeply into this part of the driver, but it is clearly good to have a clean interface to use. Regards, Christian. > --- > drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c > b/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c > index c1bd1beab655..bc9f2f423270 100644 > --- a/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c > @@ -2040,13 +2040,13 @@ static void dce_v6_0_crtc_dpms(struct drm_crtc *crtc, > int mode) > type = amdgpu_crtc_idx_to_irq_type(adev, amdgpu_crtc->crtc_id); > amdgpu_irq_update(adev, &adev->crtc_irq, type); > amdgpu_irq_update(adev, &adev->pageflip_irq, type); > - drm_vblank_post_modeset(dev, amdgpu_crtc->crtc_id); > + drm_crtc_vblank_on(crtc); > dce_v6_0_crtc_load_lut(crtc); > break; > case DRM_MODE_DPMS_STANDBY: > case DRM_MODE_DPMS_SUSPEND: > case DRM_MODE_DPMS_OFF: > - drm_vblank_pre_modeset(dev, amdgpu_crtc->crtc_id); > + drm_crtc_vblank_off(crtc); > if (amdgpu_crtc->enabled) > amdgpu_atombios_crtc_blank(crtc, ATOM_ENABLE); > amdgpu_atombios_crtc_enable(crtc, ATOM_DISABLE);
