On Thu, Feb 13, 2025 at 03:43:51PM +0100, Maxime Ripard wrote: > Other entities (drm_connector.crtc, drm_encoder.crtc, etc.) have > pointer to other currently bound entities. They are all considered > relevant only for non-atomic drivers, and generally perceived as > deprecated in favour of the equivalent pointers in the atomic states. I think there is a significant difference between mentioned fields and drm_bridge.encoder: the former fields are variable and can change. The latter one is static and set at the bridge attachment time. Nevertheless I think it is a good idea to deprecate it. > > It used to be useful because we didn't have alternatives, but now that > we do, let's make sure it's done for drm_bridge.encoder too. > > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > Signed-off-by: Maxime Ripard <mripard@xxxxxxxxxx> > --- > include/drm/drm_bridge.h | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) -- With best wishes Dmitry
