Reviewed-By: Emily Deng <Emily.Deng at amd.com> > -----Original Message----- > From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf > Of Alex Deucher > Sent: Saturday, October 01, 2016 1:09 AM > To: amd-gfx at lists.freedesktop.org > Cc: Deucher, Alexander <Alexander.Deucher at amd.com> > Subject: [PATCH 1/7] drm/amdgpu/virtual_dce: drop pageflip_irq funcs > > Never used. > > Signed-off-by: Alex Deucher <alexander.deucher at amd.com> > --- > drivers/gpu/drm/amd/amdgpu/dce_virtual.c | 35 ++++++----------------------- > --- > 1 file changed, 6 insertions(+), 29 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/dce_virtual.c > b/drivers/gpu/drm/amd/amdgpu/dce_virtual.c > index 359fe56..cf43caa 100644 > --- a/drivers/gpu/drm/amd/amdgpu/dce_virtual.c > +++ b/drivers/gpu/drm/amd/amdgpu/dce_virtual.c > @@ -209,10 +209,9 @@ static void dce_virtual_crtc_dpms(struct drm_crtc > *crtc, int mode) > switch (mode) { > case DRM_MODE_DPMS_ON: > amdgpu_crtc->enabled = true; > - /* Make sure VBLANK and PFLIP interrupts are still enabled */ > + /* Make sure VBLANK interrupts are still enabled */ > 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_on(dev, amdgpu_crtc->crtc_id); > break; > case DRM_MODE_DPMS_STANDBY: > @@ -672,8 +671,8 @@ static enum hrtimer_restart > dce_virtual_vblank_timer_handle(struct hrtimer *vbla } > > static void dce_virtual_set_crtc_vblank_interrupt_state(struct > amdgpu_device *adev, > - int crtc, > - enum > amdgpu_interrupt_state state) > + int crtc, > + enum > amdgpu_interrupt_state state) > { > if (crtc >= adev->mode_info.num_crtc) { > DRM_DEBUG("invalid crtc %d\n", crtc); @@ -697,9 +696,9 > @@ static void dce_virtual_set_crtc_vblank_interrupt_state(struct > amdgpu_device *ad > > > static int dce_virtual_set_crtc_irq_state(struct amdgpu_device *adev, > - struct amdgpu_irq_src *source, > - unsigned type, > - enum amdgpu_interrupt_state state) > + struct amdgpu_irq_src *source, > + unsigned type, > + enum amdgpu_interrupt_state state) > { > switch (type) { > case AMDGPU_CRTC_IRQ_VBLANK1: > @@ -726,20 +725,6 @@ static int dce_virtual_crtc_irq(struct > amdgpu_device *adev, > return 0; > } > > -static int dce_virtual_set_pageflip_irq_state(struct amdgpu_device *adev, > - struct amdgpu_irq_src *src, > - unsigned type, > - enum amdgpu_interrupt_state > state) > -{ > - if (type >= adev->mode_info.num_crtc) { > - DRM_ERROR("invalid pageflip crtc %d\n", type); > - return -EINVAL; > - } > - DRM_DEBUG("[FM]set pageflip irq type %d state %d\n", type, state); > - > - return 0; > -} > - > static int dce_virtual_pageflip_irq(struct amdgpu_device *adev, > struct amdgpu_irq_src *source, > struct amdgpu_iv_entry *entry) > @@ -793,17 +778,9 @@ static const struct amdgpu_irq_src_funcs > dce_virtual_crtc_irq_funcs = { > .process = dce_virtual_crtc_irq, > }; > > -static const struct amdgpu_irq_src_funcs dce_virtual_pageflip_irq_funcs = { > - .set = dce_virtual_set_pageflip_irq_state, > - .process = dce_virtual_pageflip_irq, > -}; > - > static void dce_virtual_set_irq_funcs(struct amdgpu_device *adev) { > adev->crtc_irq.num_types = AMDGPU_CRTC_IRQ_LAST; > adev->crtc_irq.funcs = &dce_virtual_crtc_irq_funcs; > - > - adev->pageflip_irq.num_types = AMDGPU_PAGEFLIP_IRQ_LAST; > - adev->pageflip_irq.funcs = &dce_virtual_pageflip_irq_funcs; > } > > -- > 2.5.5 > > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx