Hi Maxime, Le jeudi 05 novembre 2026 à 16:29 +0100, Maxime Ripard a écrit : > drm_mode_config_reset() has a dual role: it creates the initial KMS > state at probe time, and resets hardware and software state during > suspend/resume cycles. > > Since the two roles need different behaviors (notably for objects like > drm_private_obj that must persist across suspend/resume), > drm_mode_config_create_initial_state() was introduced to handle only > the initial state creation at probe time. > > Switch to drm_mode_config_create_initial_state(). > > Signed-off-by: Maxime Ripard <[email protected]>
Acked-by: Paul Cercueil <[email protected]> Cheers, -Paul > --- > Cc: Paul Cercueil <[email protected]> > Cc: [email protected] > --- > drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c > b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c > index 625ec76cf04d..63198fd59498 100644 > --- a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c > +++ b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c > @@ -1355,5 +1355,7 @@ static int ingenic_drm_bind(struct device *dev, bool > has_components) > } > > - drm_mode_config_reset(drm); > + ret = drm_mode_config_create_initial_state(drm); > + if (ret) > + goto err_drvdata; > > ret = clk_prepare_enable(priv->pix_clk);
