On 5/19/26 12:37, Luca Ceresoli wrote:
This bridge is always a tail bridge, i.e. it never needs a following bridge
to complete the pipeline. Add a is_tail func to expose this.

Signed-off-by: Luca Ceresoli <[email protected]>
---
  drivers/gpu/drm/bridge/panel.c | 8 +++++++-
  1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c
index 4978ec98a082..102f987b1235 100644
--- a/drivers/gpu/drm/bridge/panel.c
+++ b/drivers/gpu/drm/bridge/panel.c
@@ -58,6 +58,11 @@ static const struct drm_connector_funcs 
panel_bridge_connector_funcs = {
        .atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
  };
+static bool panel_bridge_is_tail(struct drm_bridge *bridge)
+{
+       return true;
+}
+
  static int panel_bridge_attach(struct drm_bridge *bridge,
                               struct drm_encoder *encoder,
                               enum drm_bridge_attach_flags flags)
@@ -206,6 +211,7 @@ static void panel_bridge_debugfs_init(struct drm_bridge 
*bridge,
  }
static const struct drm_bridge_funcs panel_bridge_bridge_funcs = {
+       .is_tail = panel_bridge_is_tail,
        .attach = panel_bridge_attach,
        .detach = panel_bridge_detach,
        .atomic_pre_enable = panel_bridge_atomic_pre_enable,
@@ -297,7 +303,7 @@ struct drm_bridge *drm_panel_bridge_add_typed(struct 
drm_panel *panel,
        panel_bridge->panel = panel;
panel_bridge->bridge.of_node = panel->dev->of_node;
-       panel_bridge->bridge.ops = DRM_BRIDGE_OP_MODES;
+       panel_bridge->bridge.ops = DRM_BRIDGE_OP_MODES | DRM_BRIDGE_OP_IS_TAIL;
        panel_bridge->bridge.type = connector_type;
        panel_bridge->bridge.pre_enable_prev_first = panel->prepare_prev_first;

This looks reasonable

Reviewed-by: Neil Armstrong <[email protected]>

Thanks,
Neil

Reply via email to