From mboxrd@z Thu Jan 1 00:00:00 1970 From: Inki Dae Subject: RE: [PATCH 05/11] drm/exynos: use drm_send_vblank_event() helper Date: Wed, 22 May 2013 11:28:06 +0900 Message-ID: <006001ce5693$fe7eff00$fb7cfd00$%dae@samsung.com> References: <1349725849-22433-1-git-send-email-rob.clark@linaro.org> <1349725849-22433-6-git-send-email-rob.clark@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mailout3.samsung.com (mailout3.samsung.com [203.254.224.33]) by gabe.freedesktop.org (Postfix) with ESMTP id 28FE2E6026 for ; Tue, 21 May 2013 19:28:09 -0700 (PDT) Received: from epcpsbgr5.samsung.com (u145.gpu120.samsung.co.kr [203.254.230.145]) by mailout3.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0MN6005IWHI9VFB0@mailout3.samsung.com> for dri-devel@lists.freedesktop.org; Wed, 22 May 2013 11:28:07 +0900 (KST) In-reply-to: Content-language: ko List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: 'Dave Airlie' Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org Hi Dave, Got it. we have to re-work this patch. Will send it soon. Thanks, Inki Dae > -----Original Message----- > From: Dave Airlie [mailto:airlied@gmail.com] > Sent: Wednesday, May 22, 2013 8:20 AM > To: Inki Dae > Cc: dri-devel@lists.freedesktop.org > Subject: Re: [PATCH 05/11] drm/exynos: use drm_send_vblank_event() helper > > Hi Inki, > > this doesn't apply cleanly anymore, and I think we want exynos to use > drm_send_vblank_event where possible, > > please apply to fixes tree and send to me. > > Dave. > > On Tue, Oct 9, 2012 at 5:50 AM, Rob Clark wrote: > > From: Rob Clark > > > > Signed-off-by: Rob Clark > > --- > > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 10 ++-------- > > drivers/gpu/drm/exynos/exynos_drm_vidi.c | 10 ++-------- > > drivers/gpu/drm/exynos/exynos_mixer.c | 9 ++------- > > 3 files changed, 6 insertions(+), 23 deletions(-) > > > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c > b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > > index b19cd93..fe8fb78 100644 > > --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c > > +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > > @@ -587,7 +587,6 @@ static void fimd_finish_pageflip(struct drm_device > *drm_dev, int crtc) > > { > > struct exynos_drm_private *dev_priv = drm_dev->dev_private; > > struct drm_pending_vblank_event *e, *t; > > - struct timeval now; > > unsigned long flags; > > bool is_checked = false; > > > > @@ -601,13 +600,8 @@ static void fimd_finish_pageflip(struct drm_device > *drm_dev, int crtc) > > > > is_checked = true; > > > > - do_gettimeofday(&now); > > - e->event.sequence = 0; > > - e->event.tv_sec = now.tv_sec; > > - e->event.tv_usec = now.tv_usec; > > - > > - list_move_tail(&e->base.link, &e->base.file_priv- > >event_list); > > - wake_up_interruptible(&e->base.file_priv->event_wait); > > + list_del(&e->base.link); > > + drm_send_vblank_event(drm_dev, -1, e); > > } > > > > if (is_checked) { > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_vidi.c > b/drivers/gpu/drm/exynos/exynos_drm_vidi.c > > index e364165..4549efb 100644 > > --- a/drivers/gpu/drm/exynos/exynos_drm_vidi.c > > +++ b/drivers/gpu/drm/exynos/exynos_drm_vidi.c > > @@ -370,7 +370,6 @@ static void vidi_finish_pageflip(struct drm_device > *drm_dev, int crtc) > > { > > struct exynos_drm_private *dev_priv = drm_dev->dev_private; > > struct drm_pending_vblank_event *e, *t; > > - struct timeval now; > > unsigned long flags; > > bool is_checked = false; > > > > @@ -384,13 +383,8 @@ static void vidi_finish_pageflip(struct drm_device > *drm_dev, int crtc) > > > > is_checked = true; > > > > - do_gettimeofday(&now); > > - e->event.sequence = 0; > > - e->event.tv_sec = now.tv_sec; > > - e->event.tv_usec = now.tv_usec; > > - > > - list_move_tail(&e->base.link, &e->base.file_priv- > >event_list); > > - wake_up_interruptible(&e->base.file_priv->event_wait); > > + list_del(&e->base.link); > > + drm_send_vblank_event(drm_dev, -1, e); > > } > > > > if (is_checked) { > > diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c > b/drivers/gpu/drm/exynos/exynos_mixer.c > > index 25b97d5..325aefd 100644 > > --- a/drivers/gpu/drm/exynos/exynos_mixer.c > > +++ b/drivers/gpu/drm/exynos/exynos_mixer.c > > @@ -828,7 +828,6 @@ static void mixer_finish_pageflip(struct drm_device > *drm_dev, int crtc) > > { > > struct exynos_drm_private *dev_priv = drm_dev->dev_private; > > struct drm_pending_vblank_event *e, *t; > > - struct timeval now; > > unsigned long flags; > > bool is_checked = false; > > > > @@ -841,13 +840,9 @@ static void mixer_finish_pageflip(struct drm_device > *drm_dev, int crtc) > > continue; > > > > is_checked = true; > > - do_gettimeofday(&now); > > - e->event.sequence = 0; > > - e->event.tv_sec = now.tv_sec; > > - e->event.tv_usec = now.tv_usec; > > > > - list_move_tail(&e->base.link, &e->base.file_priv- > >event_list); > > - wake_up_interruptible(&e->base.file_priv->event_wait); > > + list_del(&e->base.link); > > + drm_send_vblank_event(drm_dev, -1, e); > > } > > > > if (is_checked) > > -- > > 1.7.9.5 > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/dri-devel