drm/sti: Fix up crtc_state->event handling
authorFabien DESSENNE <fabien.dessenne@st.com>
Thu, 12 Jan 2017 16:27:36 +0000 (17:27 +0100)
committerVincent Abriou <vincent.abriou@st.com>
Fri, 10 Feb 2017 08:34:39 +0000 (09:34 +0100)
Use drm-core to handle event.
This is required to be able to use the nonblocking helpers.

Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com>
Acked-by: Vincent Abriou <vincent.abriou@st.com>
drivers/gpu/drm/sti/sti_crtc.c
drivers/gpu/drm/sti/sti_mixer.h

index e992bed98dcba8c7ee441f56709ac5c791fa8fb9..d45a4335df5df92a213be2119eb84d792aa59c3f 100644 (file)
@@ -134,21 +134,6 @@ sti_crtc_mode_set_nofb(struct drm_crtc *crtc)
        sti_crtc_mode_set(crtc, &crtc->state->adjusted_mode);
 }
 
-static void sti_crtc_atomic_begin(struct drm_crtc *crtc,
-                                 struct drm_crtc_state *old_crtc_state)
-{
-       struct sti_mixer *mixer = to_sti_mixer(crtc);
-
-       if (crtc->state->event) {
-               crtc->state->event->pipe = drm_crtc_index(crtc);
-
-               WARN_ON(drm_crtc_vblank_get(crtc) != 0);
-
-               mixer->pending_event = crtc->state->event;
-               crtc->state->event = NULL;
-       }
-}
-
 static void sti_crtc_atomic_flush(struct drm_crtc *crtc,
                                  struct drm_crtc_state *old_crtc_state)
 {
@@ -156,6 +141,8 @@ static void sti_crtc_atomic_flush(struct drm_crtc *crtc,
        struct sti_mixer *mixer = to_sti_mixer(crtc);
        struct sti_compositor *compo = dev_get_drvdata(mixer->dev);
        struct drm_plane *p;
+       struct drm_pending_vblank_event *event;
+       unsigned long flags;
 
        DRM_DEBUG_DRIVER("\n");
 
@@ -220,13 +207,24 @@ static void sti_crtc_atomic_flush(struct drm_crtc *crtc,
                        break;
                }
        }
+
+       event = crtc->state->event;
+       if (event) {
+               crtc->state->event = NULL;
+
+               spin_lock_irqsave(&crtc->dev->event_lock, flags);
+               if (drm_crtc_vblank_get(crtc) == 0)
+                       drm_crtc_arm_vblank_event(crtc, event);
+               else
+                       drm_crtc_send_vblank_event(crtc, event);
+               spin_unlock_irqrestore(&crtc->dev->event_lock, flags);
+       }
 }
 
 static const struct drm_crtc_helper_funcs sti_crtc_helper_funcs = {
        .enable = sti_crtc_enable,
        .disable = sti_crtc_disabling,
        .mode_set_nofb = sti_crtc_mode_set_nofb,
-       .atomic_begin = sti_crtc_atomic_begin,
        .atomic_flush = sti_crtc_atomic_flush,
 };
 
@@ -250,7 +248,6 @@ int sti_crtc_vblank_cb(struct notifier_block *nb,
        struct sti_compositor *compo;
        struct drm_crtc *crtc = data;
        struct sti_mixer *mixer;
-       unsigned long flags;
        struct sti_private *priv;
        unsigned int pipe;
 
@@ -267,14 +264,6 @@ int sti_crtc_vblank_cb(struct notifier_block *nb,
 
        drm_crtc_handle_vblank(crtc);
 
-       spin_lock_irqsave(&crtc->dev->event_lock, flags);
-       if (mixer->pending_event) {
-               drm_crtc_send_vblank_event(crtc, mixer->pending_event);
-               drm_crtc_vblank_put(crtc);
-               mixer->pending_event = NULL;
-       }
-       spin_unlock_irqrestore(&crtc->dev->event_lock, flags);
-
        if (mixer->status == STI_MIXER_DISABLING) {
                struct drm_plane *p;
 
@@ -317,19 +306,12 @@ void sti_crtc_disable_vblank(struct drm_device *drm_dev, unsigned int pipe)
        struct sti_private *priv = drm_dev->dev_private;
        struct sti_compositor *compo = priv->compo;
        struct notifier_block *vtg_vblank_nb = &compo->vtg_vblank_nb[pipe];
-       struct drm_crtc *crtc = &compo->mixer[pipe]->drm_crtc;
        struct sti_vtg *vtg = compo->vtg[pipe];
 
        DRM_DEBUG_DRIVER("\n");
 
        if (sti_vtg_unregister_client(vtg, vtg_vblank_nb))
                DRM_DEBUG_DRIVER("Warning: cannot unregister VTG notifier\n");
-
-       /* free the resources of the pending requests */
-       if (compo->mixer[pipe]->pending_event) {
-               drm_crtc_vblank_put(crtc);
-               compo->mixer[pipe]->pending_event = NULL;
-       }
 }
 
 static int sti_crtc_late_register(struct drm_crtc *crtc)
index 830a3c42d88646dfcc670788f1db7c584dd64e74..e64a00e61049ca23685892a99be7d81018cf1cbb 100644 (file)
@@ -28,7 +28,6 @@ enum sti_mixer_status {
  * @regs: mixer registers
  * @id: id of the mixer
  * @drm_crtc: crtc object link to the mixer
- * @pending_event: set if a flip event is pending on crtc
  * @status: to know the status of the mixer
  */
 struct sti_mixer {
@@ -36,7 +35,6 @@ struct sti_mixer {
        void __iomem *regs;
        int id;
        struct drm_crtc drm_crtc;
-       struct drm_pending_vblank_event *pending_event;
        enum sti_mixer_status status;
 };