Re: [PATCH 7/8] drm/display: Move HDMI helpers into display-helper module

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 22 Mar 2022, Thomas Zimmermann <tzimmermann@xxxxxxx> wrote:
> diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h
> index 144c495b99c4..e6e9e4557067 100644
> --- a/include/drm/drm_edid.h
> +++ b/include/drm/drm_edid.h
> @@ -391,33 +391,6 @@ drm_load_edid_firmware(struct drm_connector *connector)
>  
>  bool drm_edid_are_equal(const struct edid *edid1, const struct edid *edid2);
>  
> -int
> -drm_hdmi_avi_infoframe_from_display_mode(struct hdmi_avi_infoframe *frame,
> -					 const struct drm_connector *connector,
> -					 const struct drm_display_mode *mode);
> -int
> -drm_hdmi_vendor_infoframe_from_display_mode(struct hdmi_vendor_infoframe *frame,
> -					    const struct drm_connector *connector,
> -					    const struct drm_display_mode *mode);
> -
> -void
> -drm_hdmi_avi_infoframe_colorimetry(struct hdmi_avi_infoframe *frame,
> -				   const struct drm_connector_state *conn_state);
> -
> -void
> -drm_hdmi_avi_infoframe_bars(struct hdmi_avi_infoframe *frame,
> -			    const struct drm_connector_state *conn_state);
> -
> -void
> -drm_hdmi_avi_infoframe_quant_range(struct hdmi_avi_infoframe *frame,
> -				   const struct drm_connector *connector,
> -				   const struct drm_display_mode *mode,
> -				   enum hdmi_quantization_range rgb_quant_range);
> -
> -int
> -drm_hdmi_infoframe_set_hdr_metadata(struct hdmi_drm_infoframe *frame,
> -				    const struct drm_connector_state *conn_state);
> -
>  /**
>   * drm_eld_mnl - Get ELD monitor name length in bytes.
>   * @eld: pointer to an eld memory structure with mnl set
> @@ -587,6 +560,10 @@ void drm_edid_get_monitor_name(struct edid *edid, char *name,
>  struct drm_display_mode *drm_mode_find_dmt(struct drm_device *dev,
>  					   int hsize, int vsize, int fresh,
>  					   bool rb);
> +
> +u8 drm_match_hdmi_mode(const struct drm_display_mode *to_match);
> +enum hdmi_picture_aspect drm_get_cea_aspect_ratio(const u8 video_code);
> +enum hdmi_picture_aspect drm_get_hdmi_aspect_ratio(const u8 video_code);

I think these were fine as static, but not really great interfaces to
export. There's zero input checking on the vic in the latter, because
internally we could be sure they were fine.

I also wish we could limit the usage to the module you're adding; this
is now available to all drivers which should be discouraged.


BR,
Jani.


-- 
Jani Nikula, Intel Open Source Graphics Center



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux