[PATCH] drm/exynos: use drm_send_vblank_event() helper
Inki Dae
inki.dae at samsung.com
Tue May 21 23:56:30 PDT 2013
2013/5/22 Joonyoung Shim <jy0922.shim at samsung.com>
> Hi,
>
>
> On 05/22/2013 01:04 PM, Inki Dae wrote:
>
>> From: Rob Clark <rob at ti.com>
>>
>> Rebased.
>>
>> Signed-off-by: Rob Clark <rob at ti.com>
>> Signed-off-by: Inki Dae <inki.dae at samsung.com>
>> ---
>> drivers/gpu/drm/exynos/exynos_**drm_crtc.c | 10 ++--------
>> 1 files changed, 2 insertions(+), 8 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/exynos/**exynos_drm_crtc.c
>> b/drivers/gpu/drm/exynos/**exynos_drm_crtc.c
>> index e8894bc..1e7825a 100644
>> --- a/drivers/gpu/drm/exynos/**exynos_drm_crtc.c
>> +++ b/drivers/gpu/drm/exynos/**exynos_drm_crtc.c
>> @@ -398,7 +398,6 @@ void exynos_drm_crtc_finish_**pageflip(struct
>> drm_device *dev, int crtc)
>> {
>> struct exynos_drm_private *dev_priv = dev->dev_private;
>> struct drm_pending_vblank_event *e, *t;
>> - struct timeval now;
>> unsigned long flags;
>> DRM_DEBUG_KMS("%s\n", __FILE__);
>> @@ -411,14 +410,9 @@ void exynos_drm_crtc_finish_**pageflip(struct
>> drm_device *dev, int crtc)
>> if (crtc != e->pipe)
>> continue;
>> - 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);
>> drm_vblank_put(dev, crtc);
>> + list_del(&e->base.link);
>> + drm_send_vblank_event(dev, -1, e);
>>
>
> I think it's better to add above things before drm_vblank_put is called
> in comparison with prior codes.
>
>
Good point. :) Will resend it.
Thanks,
Inki Dae
> Thanks.
>
> ______________________________**_________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.**org <dri-devel at lists.freedesktop.org>
> http://lists.freedesktop.org/**mailman/listinfo/dri-devel<http://lists.freedesktop.org/mailman/listinfo/dri-devel>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130522/cc347709/attachment.html>
More information about the dri-devel
mailing list