On 07.07.2019 20:07, Laurent Pinchart wrote:
> The drm_display_info structure contains many fields related to HDMI
> sinks, but none that identifies if a sink compliant with CEA-861 (EDID)
> shall be treated as an HDMI sink or a DVI sink. Add such a flag, and
> populate it according to section 8.3.3 ("DVI/HDMI Device
> Discrimination") of the HDMI v1.3 specification.
>
> Signed-off-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>


It looks like it can replace drm_detect_hdmi_monitor usage in most cases.

Reviewed-by: Andrzej Hajda <a.ha...@samsung.com>

 --
Regards
Andrzej



> ---
>  drivers/gpu/drm/drm_edid.c  | 3 +++
>  include/drm/drm_connector.h | 5 +++++
>  2 files changed, 8 insertions(+)
>
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 82a4ceed3fcf..d2e7a5334c3f 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -4559,6 +4559,8 @@ drm_parse_hdmi_vsdb_video(struct drm_connector 
> *connector, const u8 *db)
>       struct drm_display_info *info = &connector->display_info;
>       u8 len = cea_db_payload_len(db);
>  
> +     info->is_hdmi = true;
> +
>       if (len >= 6)
>               info->dvi_dual = db[6] & 1;
>       if (len >= 7)
> @@ -4627,6 +4629,7 @@ drm_reset_display_info(struct drm_connector *connector)
>       info->cea_rev = 0;
>       info->max_tmds_clock = 0;
>       info->dvi_dual = false;
> +     info->is_hdmi = false;
>       info->has_hdmi_infoframe = false;
>       info->rgb_quant_range_selectable = false;
>       memset(&info->hdmi, 0, sizeof(info->hdmi));
> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h
> index ca745d9feaf5..e80ca0d149e5 100644
> --- a/include/drm/drm_connector.h
> +++ b/include/drm/drm_connector.h
> @@ -426,6 +426,11 @@ struct drm_display_info {
>        */
>       bool dvi_dual;
>  
> +     /**
> +      * @is_hdmi: True if the sink is an HDMI device.
> +      */
> +     bool is_hdmi;
> +
>       /**
>        * @has_hdmi_infoframe: Does the sink support the HDMI infoframe?
>        */


_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to