From: Daniel Vetter <daniel@ffwll.ch> To: Rob Clark <robdclark@gmail.com> Cc: "Gustavo Padovan" <gustavo@padovan.org>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, "Gustavo Padovan" <gustavo.padovan@collabora.co.uk>, "David Airlie" <airlied@linux.ie>, "Daniel Vetter" <daniel.vetter@ffwll.ch>, "Ander Conselvan de Oliveira" <conselvan2@gmail.com>, "Ville Syrjälä" <ville.syrjala@linux.intel.com>, "Carlos Palminha" <CARLOS.PALMINHA@synopsys.com>, "Hai Li" <hali@codeaurora.org>, "Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>, "Archit Taneja" <architt@codeaurora.org>, "Daniel Stone" <daniels@collabora.com>, "jilai wang" <jilaiw@codeaurora.org>, "open list" <linux-kernel@vger.kernel.org> Subject: Re: [PATCH 05/14] drm/msm: use drm_crtc_send_vblank_event() Date: Mon, 2 May 2016 17:04:13 +0200 [thread overview] Message-ID: <20160502150413.GS14148@phenom.ffwll.local> (raw) In-Reply-To: <CAF6AEGvNVbJTOkNVxRFBhZkWUZ-i2hUzPsUEZ9aNEKtx-yf4gw@mail.gmail.com> On Thu, Apr 14, 2016 at 04:51:06PM -0400, Rob Clark wrote: > On Thu, Apr 14, 2016 at 1:48 PM, Gustavo Padovan <gustavo@padovan.org> wrote: > > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > > > Replace the legacy drm_send_vblank_event() with the new helper function. > > > > Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > thanks, shall I take this via msm-next for 4.7, I assume? Looks like > no dependency on drm-core patches that haven't already landed? > > Reviewed-by: Rob Clark <robdclark@gmail.com> Applied to drm-misc, thanks. -Daniel > > > --- > > drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c | 2 +- > > drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c > > index e233acf..3a48889 100644 > > --- a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c > > +++ b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c > > @@ -121,7 +121,7 @@ static void complete_flip(struct drm_crtc *crtc, struct drm_file *file) > > if (!file || (event->base.file_priv == file)) { > > mdp4_crtc->event = NULL; > > DBG("%s: send event: %p", mdp4_crtc->name, event); > > - drm_send_vblank_event(dev, mdp4_crtc->id, event); > > + drm_crtc_send_vblank_event(crtc, event); > > } > > } > > spin_unlock_irqrestore(&dev->event_lock, flags); > > diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c > > index 9673b95..ce779d9 100644 > > --- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c > > +++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c > > @@ -149,7 +149,7 @@ static void complete_flip(struct drm_crtc *crtc, struct drm_file *file) > > if (!file || (event->base.file_priv == file)) { > > mdp5_crtc->event = NULL; > > DBG("%s: send event: %p", mdp5_crtc->name, event); > > - drm_send_vblank_event(dev, mdp5_crtc->id, event); > > + drm_crtc_send_vblank_event(crtc, event); > > } > > } > > spin_unlock_irqrestore(&dev->event_lock, flags); > > -- > > 2.5.5 > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Vetter <daniel@ffwll.ch> To: Rob Clark <robdclark@gmail.com> Cc: Daniel Stone <daniels@collabora.com>, Carlos Palminha <CARLOS.PALMINHA@synopsys.com>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, open list <linux-kernel@vger.kernel.org>, Daniel Vetter <daniel.vetter@ffwll.ch>, Gustavo Padovan <gustavo.padovan@collabora.co.uk> Subject: Re: [PATCH 05/14] drm/msm: use drm_crtc_send_vblank_event() Date: Mon, 2 May 2016 17:04:13 +0200 [thread overview] Message-ID: <20160502150413.GS14148@phenom.ffwll.local> (raw) In-Reply-To: <CAF6AEGvNVbJTOkNVxRFBhZkWUZ-i2hUzPsUEZ9aNEKtx-yf4gw@mail.gmail.com> On Thu, Apr 14, 2016 at 04:51:06PM -0400, Rob Clark wrote: > On Thu, Apr 14, 2016 at 1:48 PM, Gustavo Padovan <gustavo@padovan.org> wrote: > > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > > > Replace the legacy drm_send_vblank_event() with the new helper function. > > > > Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > thanks, shall I take this via msm-next for 4.7, I assume? Looks like > no dependency on drm-core patches that haven't already landed? > > Reviewed-by: Rob Clark <robdclark@gmail.com> Applied to drm-misc, thanks. -Daniel > > > --- > > drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c | 2 +- > > drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c > > index e233acf..3a48889 100644 > > --- a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c > > +++ b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c > > @@ -121,7 +121,7 @@ static void complete_flip(struct drm_crtc *crtc, struct drm_file *file) > > if (!file || (event->base.file_priv == file)) { > > mdp4_crtc->event = NULL; > > DBG("%s: send event: %p", mdp4_crtc->name, event); > > - drm_send_vblank_event(dev, mdp4_crtc->id, event); > > + drm_crtc_send_vblank_event(crtc, event); > > } > > } > > spin_unlock_irqrestore(&dev->event_lock, flags); > > diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c > > index 9673b95..ce779d9 100644 > > --- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c > > +++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c > > @@ -149,7 +149,7 @@ static void complete_flip(struct drm_crtc *crtc, struct drm_file *file) > > if (!file || (event->base.file_priv == file)) { > > mdp5_crtc->event = NULL; > > DBG("%s: send event: %p", mdp5_crtc->name, event); > > - drm_send_vblank_event(dev, mdp5_crtc->id, event); > > + drm_crtc_send_vblank_event(crtc, event); > > } > > } > > spin_unlock_irqrestore(&dev->event_lock, flags); > > -- > > 2.5.5 > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2016-05-02 15:04 UTC|newest] Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-04-14 17:48 [PATCH 01/14] drm/amdgpu: use drm_crtc_send_vblank_event() Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 17:48 ` [PATCH 02/14] drm/armada: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 17:48 ` [PATCH 03/14] drm/atmel: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 17:48 ` [PATCH 04/14] drm/i915: " Gustavo Padovan 2016-04-15 7:42 ` Daniel Vetter 2016-04-14 17:48 ` [PATCH 05/14] drm/msm: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 20:51 ` Rob Clark 2016-04-14 20:51 ` Rob Clark 2016-04-19 14:02 ` Gustavo Padovan 2016-04-19 14:02 ` Gustavo Padovan 2016-04-20 11:13 ` Daniel Vetter 2016-04-20 11:13 ` Daniel Vetter 2016-05-02 15:04 ` Daniel Vetter [this message] 2016-05-02 15:04 ` Daniel Vetter 2016-04-14 17:48 ` [PATCH 06/14] drm/qxl: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 17:48 ` [PATCH 07/14] drm/nouveau: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-25 7:01 ` Mario Kleiner 2016-04-25 7:01 ` Mario Kleiner 2016-04-25 20:06 ` Gustavo Padovan 2016-04-14 17:48 ` [PATCH 08/14] drm/radeon: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 17:48 ` [PATCH 09/14] drm/rcar-du: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 21:59 ` Laurent Pinchart 2016-04-14 21:59 ` Laurent Pinchart 2016-04-14 21:59 ` Laurent Pinchart 2016-04-14 17:48 ` [PATCH 10/14] drm/shmobile: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 21:59 ` Laurent Pinchart 2016-04-14 21:59 ` Laurent Pinchart 2016-04-14 21:59 ` Laurent Pinchart 2016-05-02 15:06 ` Daniel Vetter 2016-05-02 15:06 ` Daniel Vetter 2016-05-02 15:06 ` Daniel Vetter 2016-04-14 17:48 ` [PATCH 11/14] drm/tilcdc: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-19 12:56 ` Jyri Sarha 2016-04-19 12:56 ` Jyri Sarha 2016-04-19 14:03 ` Gustavo Padovan 2016-04-19 14:03 ` Gustavo Padovan 2016-05-02 15:05 ` Daniel Vetter 2016-05-02 15:05 ` Daniel Vetter 2016-05-02 18:58 ` Jyri Sarha 2016-05-02 18:58 ` Jyri Sarha 2016-04-14 17:48 ` [PATCH 12/14] drm/udl: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 17:48 ` [PATCH 13/14] drm/virtio: " Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-14 17:48 ` [PATCH 14/14] drm: remove legacy drm_send_vblank_event() Gustavo Padovan 2016-04-14 17:48 ` Gustavo Padovan 2016-04-15 2:55 ` [PATCH 01/14] drm/amdgpu: use drm_crtc_send_vblank_event() Michel Dänzer 2016-04-15 2:55 ` Michel Dänzer 2016-05-02 15:03 ` Daniel Vetter 2016-05-02 15:03 ` Daniel Vetter
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20160502150413.GS14148@phenom.ffwll.local \ --to=daniel@ffwll.ch \ --cc=CARLOS.PALMINHA@synopsys.com \ --cc=airlied@linux.ie \ --cc=architt@codeaurora.org \ --cc=conselvan2@gmail.com \ --cc=daniel.vetter@ffwll.ch \ --cc=daniels@collabora.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=gustavo.padovan@collabora.co.uk \ --cc=gustavo@padovan.org \ --cc=hali@codeaurora.org \ --cc=jilaiw@codeaurora.org \ --cc=linux-kernel@vger.kernel.org \ --cc=maarten.lankhorst@linux.intel.com \ --cc=robdclark@gmail.com \ --cc=ville.syrjala@linux.intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.