On Tue, Sep 02, 2025 at 10:22:12PM +0200, Laurent Pinchart wrote:
> > > + struct drm_bridge *bridge = drm_priv_to_bridge(s->obj);
> > > +
> > > + drm_printf(p, "bridge: %s", drm_get_connector_type_name(bridge->type));
> 
> It would be nice to identify the bridge more precisely, but bridges have
> no object ID :-/ The raw pointer may not be very useful. I wonder if we
> should give an object ID to drm_private_obj instances, even if we don't
> expose them to userspace. That's not a topic for this series of course.

I agree, and would have liked to have something a bit more descriptive
indeed. But bridges have neither a name or an ID, so I couldn't come up
with anything better :/

> Reviewed-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>

Thanks!
Maxime

Attachment: signature.asc
Description: PGP signature

Reply via email to