Hi, On Thu, Feb 13, 2025 at 6:45 AM Maxime Ripard <mrip...@kernel.org> wrote: > > The drm_bridge structure contains an encoder pointer that is widely used > by bridge drivers. This pattern is largely documented as deprecated in > other KMS entities for atomic drivers. > > However, one of the main use of that pointer is done in attach to just > call drm_bridge_attach on the next bridge to add it to the bridge list. > While this dereferences the bridge->encoder pointer, it's effectively > the same encoder the bridge was being attached to. > > We can make it more explicit by adding the encoder the bridge is > attached to to the list of attach parameters. This also removes the need > to dereference bridge->encoder in most drivers. > > Reviewed-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org> > Signed-off-by: Maxime Ripard <mrip...@kernel.org>
>From the point of view of ti-sn65dsi86 and parade-ps8640: Reviewed-by: Douglas Anderson <diand...@chromium.org> Tested-by: Douglas Anderson <diand...@chromium.org>
