drm_bridge_atomic_print_priv_state() currently identifies bridges by
printing the symbol name of bridge->funcs using the %ps format
specifier. This produces output like a raw function pointer name, which
is not particularly user-friendly for debugging.

Since bridges now carry a human-readable name, switch to printing
bridge->name instead. This makes the state readout output more
consistent and easier to interpret.

Signed-off-by: Maxime Ripard <[email protected]>
---
 drivers/gpu/drm/drm_bridge.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c
index d474d6020411..447fe6d333b3 100644
--- a/drivers/gpu/drm/drm_bridge.c
+++ b/drivers/gpu/drm/drm_bridge.c
@@ -561,13 +561,13 @@ drm_bridge_atomic_print_priv_state(struct drm_printer *p,
 {
        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);
+               drm_printf(p, "bridge: %s (%pOFfc)\n", bridge->name, 
bridge->of_node);
        else
-               drm_printf(p, "bridge: %ps\n", bridge->funcs);
+               drm_printf(p, "bridge: %s\n", bridge->name);
 
        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:");

-- 
2.55.0

Reply via email to