On Wed, 6 Jul 2022 at 19:12, Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx> wrote: > > Some userspace presumes that the first connected connector is the main > display, where it's supposed to display e.g. the login screen. For > laptops, this should be the main panel. > > This patch call drm_helper_move_panel_connectors_to_head() after > drm_bridge_connector_init() to make sure eDP stay at head of > connected connector list. This fixes unexpected corruption happen > at eDP panel if eDP is not placed at head of connected connector > list. > > Changes in v2: > -- move drm_helper_move_panel_connectors_to_head() to > dpu_kms_drm_obj_init() > > Signed-off-by: Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx> > Tested-by: Douglas Anderson <dianders@xxxxxxxxxxxx> > Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx> > Reviewed-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx> Technically this patch was neither Reviewed, nor Tested. Please refrain from posting wrong tags. > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c > index 2b9d931..50ff666 100644 > --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c > +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c > @@ -763,6 +763,8 @@ static int _dpu_kms_drm_obj_init(struct dpu_kms *dpu_kms) > if (ret) > return ret; > > + drm_helper_move_panel_connectors_to_head(dev); > + > num_encoders = 0; > drm_for_each_encoder(encoder, dev) > num_encoders++; -- With best wishes Dmitry
