Nouveau - Jan 2025 - [PATCH] drm/nouveau: Do not override forced connector status

Keep user-forced connector status even if it cannot be programmed. Same
behavior as for the rest of the drivers.

Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
---
This patch is in preparation of
https://patchwork.freedesktop.org/series/139879/. The series improves
internal handling of the connector status. That first requires fixes in
several drivers; including nouveau.
---
 drivers/gpu/drm/nouveau/nouveau_connector.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.c
b/drivers/gpu/drm/nouveau/nouveau_connector.c
index 6fb9719d721f7..1b10c6c12f468 100644
--- a/drivers/gpu/drm/nouveau/nouveau_connector.c
+++ b/drivers/gpu/drm/nouveau/nouveau_connector.c
@@ -775,7 +775,6 @@ nouveau_connector_force(struct drm_connector *connector)
 	if (!nv_encoder) {
 		NV_ERROR(drm, "can't find encoder to force %s on!\n",
 			 connector->name);
-		connector->status = connector_status_disconnected;
 		return;
 	}
 
-- 
2.47.1

Ping. Are there any comments on this patch?

Am 14.01.25 um 10:57 schrieb Thomas Zimmermann:
> Keep user-forced connector status even if it cannot be programmed. Same
> behavior as for the rest of the drivers.
>
> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
> ---
> This patch is in preparation of
> https://patchwork.freedesktop.org/series/139879/. The series improves
> internal handling of the connector status. That first requires fixes in
> several drivers; including nouveau.
> ---
>   drivers/gpu/drm/nouveau/nouveau_connector.c | 1 -
>   1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.c
b/drivers/gpu/drm/nouveau/nouveau_connector.c
> index 6fb9719d721f7..1b10c6c12f468 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_connector.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_connector.c
> @@ -775,7 +775,6 @@ nouveau_connector_force(struct drm_connector
*connector)
>   	if (!nv_encoder) {
>   		NV_ERROR(drm, "can't find encoder to force %s on!\n",
>   			 connector->name);
> -		connector->status = connector_status_disconnected;
>   		return;
>   	}
>   
-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)