Re: [Intel-gfx] [v9 04/12] drm/i915/display: Attach content type property for LSPCON

From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Uma Shankar <uma.shankar@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [Intel-gfx] [v9 04/12] drm/i915/display: Attach content type property for LSPCON
Date: Wed, 25 Nov 2020 18:29:04 +0200	[thread overview]
Message-ID: <20201125162904.GP6112@intel.com> (raw)
In-Reply-To: <20201103152834.12727-5-uma.shankar@intel.com>

On Tue, Nov 03, 2020 at 08:58:26PM +0530, Uma Shankar wrote:
> Content type is supported on HDMI sink devices. Attached the
> property for the same for LSPCON based devices.
> 
> Signed-off-by: Uma Shankar <uma.shankar@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_dp.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
> index 0ce3204473fa..79a49d1cbb21 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -6736,6 +6736,7 @@ intel_dp_connector_register(struct drm_connector *connector)
>  			drm_object_attach_property(&connector->base,
>  						   connector->dev->mode_config.hdr_output_metadata_property,
>  						   0);
> +		drm_connector_attach_content_type_property(connector);

Doesn't look like you're populating the infoframe until patch 6, so this
looks a bit early. Should probably suck in the infoframe part into this
patch too.

>  	}
>  
>  	return ret;
> -- 
> 2.26.2

-- 
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx