intel-gfx.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 0/7] drm/connector: Expose only a properly inited connector
@ 2024-12-11 23:03 Imre Deak
  2024-12-11 23:03 ` [PATCH v3 01/11] drm/connector: Add a way to init/add a connector in separate steps Imre Deak
                   ` (16 more replies)
  0 siblings, 17 replies; 46+ messages in thread
From: Imre Deak @ 2024-12-11 23:03 UTC (permalink / raw)
  To: intel-gfx
  Cc: dri-devel, Rodrigo Vivi, Jani Nikula, Simona Vetter,
	Maxime Ripard, Lyude Paul, Harry Wentland, Leo Li, Wayne Lin,
	Alex Deucher, Karol Herbst, Danilo Krummrich

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


^ permalink raw reply	[flat|nested] 46+ messages in thread

end of thread, other threads:[~2024-12-17 14:46 UTC | newest]

Thread overview: 46+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-11 23:03 [PATCH v3 0/7] drm/connector: Expose only a properly inited connector Imre Deak
2024-12-11 23:03 ` [PATCH v3 01/11] drm/connector: Add a way to init/add a connector in separate steps Imre Deak
2024-12-12  0:33   ` Lyude Paul
2024-12-12 10:16     ` Simona Vetter
2024-12-12 10:04   ` Jani Nikula
2024-12-12 11:45     ` Imre Deak
2024-12-16 12:22   ` Maxime Ripard
2024-12-11 23:03 ` [PATCH v3 02/11] drm/connector: Add FIXME for GETRESOURCES ioctl wrt. uninited connectors Imre Deak
2024-12-12 10:06   ` Jani Nikula
2024-12-13 12:02     ` Imre Deak
2024-12-11 23:03 ` [PATCH v3 03/11] drm/connector: Add deprication notes for drm_connector_register/unregister Imre Deak
2024-12-12 10:10   ` Jani Nikula
2024-12-12 12:02     ` Imre Deak
2024-12-16 14:37       ` Simona Vetter
2024-12-11 23:03 ` [PATCH v3 04/11] drm/dp_mst: Register connectors via drm_connector_dynamic_register() Imre Deak
2024-12-12 10:12   ` Jani Nikula
2024-12-13 12:06     ` Imre Deak
2024-12-16 11:03       ` Jani Nikula
2024-12-16 12:14         ` Imre Deak
2024-12-16 12:23   ` Imre Deak
2024-12-17  8:02     ` Lin, Wayne
2024-12-11 23:03 ` [PATCH v3 05/11] drm/i915/dp_mst: Expose a connector to kernel users after it's properly initialized Imre Deak
2024-12-12 10:13   ` Jani Nikula
2024-12-11 23:03 ` [PATCH v3 06/11] drm/amd/dp_mst: " Imre Deak
2024-12-12 10:14   ` Jani Nikula
2024-12-16 12:45   ` Imre Deak
2024-12-16 18:26     ` Alex Deucher
2024-12-11 23:03 ` [PATCH v3 07/11] drm/nouveau/dp_mst: " Imre Deak
2024-12-12 10:14   ` Jani Nikula
2024-12-11 23:03 ` [PATCH v3 08/11] drm/connector: Warn if a connector is registered/added incorrectly Imre Deak
2024-12-12 10:15   ` Jani Nikula
2024-12-11 23:03 ` [PATCH v3 09/11] drm/tests: Add tests for drm_connector_dynamic_init()/register() Imre Deak
2024-12-16 12:22   ` Maxime Ripard
2024-12-11 23:03 ` [PATCH v3 10/11] drm/i915/dp_mst: Fix error handling while adding a connector Imre Deak
2024-12-12 10:16   ` Jani Nikula
2024-12-11 23:03 ` [PATCH v3 11/11] drm/i915/dp_mst: Use intel_connector vs. drm_connector pointer in intel_dp_mst.c Imre Deak
2024-12-12 10:25   ` Jani Nikula
2024-12-12 12:23     ` Imre Deak
2024-12-11 23:33 ` ✗ Fi.CI.CHECKPATCH: warning for drm/connector: Expose only a properly inited connector Patchwork
2024-12-11 23:33 ` ✗ Fi.CI.SPARSE: " Patchwork
2024-12-11 23:57 ` ✓ i915.CI.BAT: success " Patchwork
2024-12-12  7:33 ` ✗ i915.CI.Full: failure " Patchwork
2024-12-13 15:41   ` Imre Deak
2024-12-16 12:22 ` [PATCH v3 0/7] " Imre Deak
2024-12-16 12:39   ` Maxime Ripard
2024-12-17 14:47 ` Imre Deak

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).