Re: [PATCH v6 00/26] Add support for hot-pluggable DRM bridges

Luca Ceresoli
2025-02-07T09:10:52Z

On Thu, 06 Feb 2025 19:14:15 +0100
Luca Ceresoli <luca.ceresoli@bootlin.com> wrote:

> Patch series overview
> =====================

And I messed up with my patch series overview.

Please ignore all lines from here...

> * 2 Preliminary patches (maybe to be removed as this work progresses):
> - drm/bridge: allow bridges to be informed about added and removed bridges
> - drm/encoder: add drm_encoder_cleanup_from()
>
> * Implement refcounting in the drm_bridge core:
> - drm/bridge: add support for refcounted DRM bridges
> - drm/tests: bridge: add KUnit tests for DRM bridges (init and destroy)
> - drm/bridge: add documentation of refcounted bridges
>
> * Adapt some existing bridges to be refcounted and/or to refcount bridges
> they take a pointer to:
> - drm/bridge: ti-sn65dsi83: use dynamic lifetime management
> - drm/bridge: panel: use dynamic lifetime management
> - drm/bridge: samsung-dsim: use supporting variable for out_bridge
> - drm/bridge: samsung-dsim: refcount the out_bridge
>
> * Add hotplug-bridge (may be removed as this work progresses):
> - drm/bridge: hotplug-bridge: add driver to support hot-pluggable DSI bridges

...to here, as they belong to v5

The lines that follow are the correct ones for v6:

> * Prelimiary, simple cleanups:
> - drm/debugfs: fix printk format for bridge index
> - drm: of: drm_of_find_panel_or_bridge: move misplaced comment
> - drm/bridge: panel: use drm_bridge_is_panel() instead of open code
> - drm/bridge: panel: drm_panel_bridge_remove: warn when called on non-panel bridge
>
> * debugfs improvements to show bridges, part 1:
> - drm/debugfs: add top-level 'bridges' file showing all added bridges
>
> * Restructure panel code to always add a panel_bridge:
> - drm/panel: move all code into bridge/panel.c
> - drm/bridge: panel: forbid initializing a panel with unknown connector type
> - drm/bridge: panel: add a panel_bridge to every panel
> - drm/bridge: move devm_drm_of_get_bridge and drmm_of_get_bridge to drm_bridge.c
>
> * Add new get_bridge variant for drivers supporting non-graph DT:
> - drm/bridge: add devm_drm_of_get_bridge_by_node()
> - drm/bridge: samsung-dsim: use devm_drm_of_get_bridge[_by_node] to find the out_bridge
>
> * Preliminary patches (maybe to be removed as this work progresses):
> - drm/bridge: allow bridges to be informed about added and removed bridges
> - drm/encoder: add drm_encoder_cleanup_from()
>
> * Implement refcounting in the drm_bridge core:
> - drm/bridge: add support for refcounted DRM bridges
> - drm/bridge: devm_drm_of_get_bridge and drmm_of_get_bridge: automatically put the bridge
> - drm/bridge: increment refcount in of_drm_find_bridge()
> - drm/bridge: add devm_drm_put_bridge() and devm_drm_put_and_clear_bridge()
> - drm/bridge: add documentation of refcounted bridges
> - drm/tests: bridge: add KUnit tests for DRM bridges (init and destroy)
>
> * debugfs improvements to show bridges, part 2:
> - drm/debugfs: bridges_show: show refcount
> - drm/bridge: add list of removed refcounted bridges
> - drm/debugfs: show removed bridges
>
> * Adapt some existing bridges to be refcounted and/or to refcount bridges
> they take a pointer to:
> - drm/bridge: samsung-dsim: use refcounting for the out_bridge
> - drm/bridge: panel: use dynamic lifetime management
> - drm/bridge: ti-sn65dsi83: use dynamic lifetime management
>
> * Add hotplug-bridge (may be removed as this work progresses):
> - drm/bridge: hotplug-bridge: add driver to support hot-pluggable DSI bridges (HEAD -> ge/luca/wip)

Apologies for the mess.

Luca

--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering