On Thu, 2026-11-05 at 16:29 +0100, Maxime Ripard wrote:
> 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: Ruben Wauters <[email protected]>
> ---
> Cc: Ruben Wauters <[email protected]>
> ---
>  drivers/gpu/drm/gud/gud_drv.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/gud/gud_drv.c
> b/drivers/gpu/drm/gud/gud_drv.c
> index 680447f426f3..9ba6d8b94023 100644
> --- a/drivers/gpu/drm/gud/gud_drv.c
> +++ b/drivers/gpu/drm/gud/gud_drv.c
> @@ -641,5 +641,7 @@ static int gud_probe(struct usb_interface *intf,
> const struct usb_device_id *id)
>       }
>  
> -     drm_mode_config_reset(drm);
> +     ret = drm_mode_config_create_initial_state(drm);
> +     if (ret)
> +             return ret;
>       drm_kms_helper_poll_init(drm);
>  

Attachment: signature.asc
Description: This is a digitally signed message part



Reply via email to