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.

Reviewed-by: Maarten Lankhorst <[email protected]>
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 ac65ac941691..e237366199a7 100644
--- a/drivers/gpu/drm/drm_bridge.c
+++ b/drivers/gpu/drm/drm_bridge.c
@@ -570,7 +570,7 @@ drm_bridge_atomic_print_priv_state(struct drm_printer *p,
 
        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:");

-- 
2.55.0

Reply via email to