[PATCH 1/2] drm/ast: sil164: Clear EDID if no display is connected

Jocelyn Falempe jfalempe at redhat.com
Tue Oct 15 08:57:26 UTC 2024


On 15/10/2024 08:49, Thomas Zimmermann wrote:
> Do not keep the obsolete EDID around after unplugging the display
> from the connector.

Thanks, it looks good to me.

Reviewed-by: Jocelyn Falempe <jfalempe at redhat.com>

> 
> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
> Fixes: d20c2f846428 ("drm/ast: sil164: Transparently handle BMC support")
> Cc: Thomas Zimmermann <tzimmermann at suse.de>
> Cc: Jocelyn Falempe <jfalempe at redhat.com>
> Cc: Dave Airlie <airlied at redhat.com>
> Cc: dri-devel at lists.freedesktop.org
> ---
>   drivers/gpu/drm/ast/ast_sil164.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/ast/ast_sil164.c b/drivers/gpu/drm/ast/ast_sil164.c
> index 6a72268d2314..be01254dd48a 100644
> --- a/drivers/gpu/drm/ast/ast_sil164.c
> +++ b/drivers/gpu/drm/ast/ast_sil164.c
> @@ -29,6 +29,8 @@ static int ast_sil164_connector_helper_get_modes(struct drm_connector *connector
>   	if (ast_connector->physical_status == connector_status_connected) {
>   		count = drm_connector_helper_get_modes(connector);
>   	} else {
> +		drm_edid_connector_update(connector, NULL);
> +
>   		/*
>   		 * There's no EDID data without a connected monitor. Set BMC-
>   		 * compatible modes in this case. The XGA default resolution



More information about the dri-devel mailing list