On Tue, Jun 21, 2016 at 03:18:12PM +0200, Daniel Vetter wrote: > On Tue, Jun 21, 2016 at 09:53:44AM +0100, Chris Wilson wrote: > > +void intel_modeset_register(struct drm_i915_private *dev_priv) > > +{ > > + drm_connector_register_all(dev_priv->dev); > > +} > > + > > +void intel_modeset_unregister(struct drm_i915_private *dev_priv) > > +{ > > + drm_connector_unregister_all(dev_priv->dev); > > +} > > Given that this will all disappear again I wouldn't bother with the > wrapper functions. They exist to conform with the intel_modeset_phase(). I thought they helped tell the story of how we are moving the phases around. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx