Re: [PATCH 4/8] drm/i915: Move connector registration to driver registration

Chris Wilson
2016-06-21T13:29:24Z

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