* @get_crtc: return CRTC that the encoder is currently attached to
   * @detect: connection status detection
   * @disable: disable encoder when not in use (overrides DPMS off)
+  * @enable: enable encoder
 + * @atomic_check: check for validity of an atomic update
   *
   * The helper operations are called by the mid-layer CRTC helper.
+  *
+  * Note that with atomic helpers @dpms, @prepare and @commit hooks are
+  * deprecated. Used @enable and @disable instead exclusively.
+  *
+  * With legacy crtc helpers there's a big semantic difference between @disable
+  * and the other hooks: @disable also needs to release any resources acquired in
+  * @mode_set (like shared PLLs).
   */
  struct drm_encoder_helper_funcs {
        void (*dpms)(struct drm_encoder *encoder, int mode);
        /* detect for DAC style encoders */
        enum drm_connector_status (*detect)(struct drm_encoder *encoder,
                                            struct drm_connector *connector);
-       /* disable encoder when not in use - more explicit than dpms off */
        void (*disable)(struct drm_encoder *encoder);
++
+       void (*enable)(struct drm_encoder *encoder);
 +
 +      /* atomic helpers */
 +      int (*atomic_check)(struct drm_encoder *encoder,
 +                          struct drm_crtc_state *crtc_state,
 +                          struct drm_connector_state *conn_state);
  };
  
  /**