Re: [PATCH v3 03/11] drm/connector: Add deprication notes for drm_connector_register/unregister

[Simona Vetter <simona.vetter@ffwll.ch>]

On Thu, Dec 12, 2024 at 02:02:58PM +0200, Imre Deak wrote:
> On Thu, Dec 12, 2024 at 12:10:58PM +0200, Jani Nikula wrote:
> > [...]
> > On Thu, 12 Dec 2024, Imre Deak <imre.deak@intel.com> wrote:
> > > @@ -863,9 +866,14 @@ EXPORT_SYMBOL(drm_connector_dynamic_register);
> > >   * drm_connector_unregister - unregister a connector
> > >   * @connector: the connector to unregister
> > >   *
> > > - * Unregister userspace interfaces for a connector. Only call this for
> > > - * connectors which have been registered explicitly by calling
> > > - * drm_connector_register().
> > > + * Unregister userspace interfaces for a connector. Drivers should call this
> > > + * for dynamic connectors (MST) only, which were registered explicitly by
> > > + * calling drm_connector_dynamic_register(). All other - static - connectors
> > > + * will be unregistered automatically by DRM core and drivers shouldn't call
> > > + * this function for those.
> > 
> > This kind of supports my point about a single
> > drm_connector_register(). There's no
> > drm_connector_dynamic_unregister(). After all the
> > drm_connector_register() calls have been removed, we're left with the
> > asymmetric pair:
> > 
> > - drm_connector_dynamic_register()
> > - drm_connector_unregister()
> > 
> > Then again, all of these should become internal and not for drivers?
> 
> Yes, drm_connector_register() - after this patchset - could be removed
> from drivers. The use of drm_connector_unregister() in drivers for
> static connectors should be also removed, left it for DRM core internal
> use only and exporting drm_connector_dynamic_unregister() for driver
> use (atm only MST).

Yup that's been the plan for a while for static connectors, unfortunately
there's some fun around dp connectors and dp aux specifically that make
the cleanup not entirely a cocci job :-/
-Sima

> 
> > > + *
> > > + * Note: Existing uses of this function in drivers for static connectors
> > > + * should be a nop already and are scheduled to be removed.
> > >   */
> > >  void drm_connector_unregister(struct drm_connector *connector)
> > >  {
> > 
> > -- 
> > Jani Nikula, Intel

-- 
Simona Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch