On Mon, Oct 04, 2021 at 09:12:50AM +0000, Simon Ser wrote:
> The KMS documentation doesn't say much about the meaning of each
> content type. Add a reference to the specification defining them.
> 
> Signed-off-by: Simon Ser <cont...@emersion.fr>
> Cc: Emmanuel Gil Peyrot <linkma...@linkmauve.fr>
> Cc: Daniel Vetter <dan...@ffwll.ch>
> Cc: Pekka Paalanen <ppaala...@gmail.com>
> Cc: Ville Syrjala <ville.syrj...@linux.intel.com>
> Cc: Jani Nikula <jani.nik...@linux.intel.com>
> ---
>  drivers/gpu/drm/drm_connector.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c
> index 3bc782b630b9..79d8163686cd 100644
> --- a/drivers/gpu/drm/drm_connector.c
> +++ b/drivers/gpu/drm/drm_connector.c
> @@ -1397,6 +1397,8 @@ 
> EXPORT_SYMBOL(drm_connector_attach_dp_subconnector_property);
>   *   Game:
>   *           Content type is game
>   *
> + *   The meaning of each content type is defined in CTA-861-G table 15.
> + *

A bit annoying to have to refer to an external spec, but copy pasting
the whole thing here seems a bit questionable.

Reviewed-by: Ville Syrjälä <ville.syrj...@linux.intel.com>

>   *   Drivers can set up this property by calling
>   *   drm_connector_attach_content_type_property(). Decoding to
>   *   infoframe values is done through drm_hdmi_avi_infoframe_content_type().
> -- 
> 2.33.0
> 

-- 
Ville Syrjälä
Intel

Reply via email to