On Fri, Nov 29, 2024 at 04:26:01PM +0200, Imre Deak wrote: > Adding more people from DRM core domain. Any comment, objection to this > change? > > On Tue, Nov 26, 2024 at 06:18:56PM +0200, Imre Deak wrote: > > Atm when the connector is added to the drm_mode_config::connector_list, > > the connector may not be fully initialized yet. This is not a problem > > for user space, which will see the connector only after it's registered > > later, it could be a problem for in-kernel users looking up connectors > > via the above list. It could be, or it actually is a problem? If so, in what situation? > > To resolve the above issue, add a way to separately initialize the DRM > > core specific parts of the connector and add it to the above list. This > > will move adding the connector to the list after the properties on the > > connector have been added, this is ok since these steps don't have a > > dependency. > > > > v2: (Jani) > > - Let initing DDC as well via drm_connector_init_core(). > > - Rename __drm_connector_init to drm_connector_init_core_and_add(). > > > > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> (v1) > > Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx> If we do have an actual problem to solve, we'll need kunit tests too. Maxime
Attachment:
signature.asc
Description: PGP signature
