From 4a7a146f1cc6791aaa2f704532f6a3a53332e434 Mon Sep 17 00:00:00 2001 From: Maxime Ripard Date: Mon, 7 Sep 2026 15:23:42 +0200 Subject: [PATCH] drm/bridge: Implement atomic_print_state Bridges have some fields in their state worth printing, but we don't provide an atomic_print_state implementation to show those fields. Provide one. Reviewed-by: Laurent Pinchart Link: https://lore.kernel.org/r/20260907-drm-state-readout-v5-6-5fa1ac7a5148@kernel.org Signed-off-by: Maxime Ripard --- drivers/gpu/drm/drm_bridge.c | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c index 83f1809a5d37..649f6b6ab5c8 100644 --- a/drivers/gpu/drm/drm_bridge.c +++ b/drivers/gpu/drm/drm_bridge.c @@ -549,10 +549,31 @@ drm_bridge_atomic_create_priv_state(struct drm_private_obj *obj) return &state->base; } +static void +drm_bridge_atomic_print_priv_state(struct drm_printer *p, + const struct drm_private_state *s) +{ + const struct drm_bridge_state *state = drm_priv_to_bridge_state(s); + struct drm_bridge *bridge = drm_priv_to_bridge(s->obj); + + if (bridge->of_node) + drm_printf(p, "bridge: %ps (%pOFfc)\n", bridge->funcs, bridge->of_node); + else + drm_printf(p, "bridge: %ps\n", bridge->funcs); + + drm_printf_indent(p, 1, "input bus configuration:"); + drm_printf_indent(p, 2, "code: %04x", state->input_bus_cfg.format); + drm_printf_indent(p, 2, "flags: %08x", state->input_bus_cfg.flags); + drm_printf_indent(p, 1, "output bus configuration:"); + drm_printf_indent(p, 2, "code: %04x", state->output_bus_cfg.format); + drm_printf_indent(p, 2, "flags: %08x", state->output_bus_cfg.flags); +} + static const struct drm_private_state_funcs drm_bridge_priv_state_funcs = { .atomic_create_state = drm_bridge_atomic_create_priv_state, .atomic_duplicate_state = drm_bridge_atomic_duplicate_priv_state, .atomic_destroy_state = drm_bridge_atomic_destroy_priv_state, + .atomic_print_state = drm_bridge_atomic_print_priv_state, }; /**