[PATCH v3 0/7] drm/connector: Expose only a properly inited connector

Imre Deak imre.deak at intel.com
Wed Dec 11 23:03:17 UTC 2024


This is v3 of [1], with the following changes:

- Instead of the drm_connector_init_core()/drm_connector_add() add
  a drm_connector_dynamic_init()/drm_connector_dynamic_register()
  interface. Adding the connector to the connector list is now part of
  the registration step done by the latter function. (Sima)
- Add kunit tests for the above functions. (Maxime)
- Add a FIXME note for a related issue in the GETRESOURCES ioctl call.
  (Sima)
- Add a note to depricate the use of
  drm_connector_register()/drm_connector_unregister() by drivers for
  static connectors.

Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
Cc: Jani Nikula <jani.nikula at intel.com>
Cc: Simona Vetter <simona at ffwll.ch>
Cc: Maxime Ripard <mripard at kernel.org>
Cc: Lyude Paul <lyude at redhat.com>
Cc: Harry Wentland <harry.wentland at amd.com>
Cc: Leo Li <sunpeng.li at amd.com>
Cc: Wayne Lin <wayne.lin at amd.com>
Cc: Alex Deucher <alexander.deucher at amd.com>
Cc: Karol Herbst <kherbst at redhat.com>
Cc: Danilo Krummrich <dakr at kernel.org>

[1] https://lore.kernel.org/all/20241126161859.1858058-1-imre.deak@intel.com

Imre Deak (11):
  drm/connector: Add a way to init/add a connector in separate steps
  drm/connector: Add FIXME for GETRESOURCES ioctl wrt. uninited
    connectors
  drm/connector: Add deprication notes for
    drm_connector_register/unregister
  drm/dp_mst: Register connectors via drm_connector_dynamic_register()
  drm/i915/dp_mst: Expose a connector to kernel users after it's
    properly initialized
  drm/amd/dp_mst: Expose a connector to kernel users after it's properly
    initialized
  drm/nouveau/dp_mst: Expose a connector to kernel users after it's
    properly initialized
  drm/connector: Warn if a connector is registered/added incorrectly
  drm/tests: Add tests for drm_connector_dynamic_init()/register()
  drm/i915/dp_mst: Fix error handling while adding a connector
  drm/i915/dp_mst: Use intel_connector vs. drm_connector pointer in
    intel_dp_mst.c

 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   |   5 +-
 drivers/gpu/drm/display/drm_dp_mst_topology.c |   2 +-
 drivers/gpu/drm/drm_connector.c               | 168 ++++++-
 drivers/gpu/drm/drm_mode_config.c             |   9 +
 drivers/gpu/drm/i915/display/intel_dp_mst.c   | 206 ++++----
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |   4 +-
 drivers/gpu/drm/tests/drm_connector_test.c    | 463 ++++++++++++++++++
 include/drm/drm_connector.h                   |   6 +
 8 files changed, 722 insertions(+), 141 deletions(-)

-- 
2.44.2



More information about the Intel-gfx mailing list