Hi, On Thu, Jul 24, 2025 at 04:02:54PM +0800, Damon Ding wrote: > In order to move the parnel/bridge parsing and attachmenet to the > Analogix side, add component struct drm_bridge *bridge to platform > data struct analogix_dp_plat_data. > > The movemenet makes sense because the panel/bridge should logically > be positioned behind the Analogix bridge in the display pipeline. > > Signed-off-by: Damon Ding <[email protected]> > --- > include/drm/bridge/analogix_dp.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/drm/bridge/analogix_dp.h > b/include/drm/bridge/analogix_dp.h > index cf17646c1310..15cb6b706e9f 100644 > --- a/include/drm/bridge/analogix_dp.h > +++ b/include/drm/bridge/analogix_dp.h > @@ -27,6 +27,7 @@ static inline bool is_rockchip(enum analogix_dp_devtype > type) > struct analogix_dp_plat_data { > enum analogix_dp_devtype dev_type; > struct drm_panel *panel; > + struct drm_bridge *bridge; > struct drm_encoder *encoder; > struct drm_connector *connector; > bool skip_connector;
So it's not the analogix_dp bridge, but the bridge after that if any? You should probably change it to next_bridge to avoid any confusion. Maxime
signature.asc
Description: PGP signature
