[PATCH 1/3] drm/i915/dp: Fix colorimetry detection

[Hogander, Jouni]

On Wed, 2024-09-18 at 22:04 +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> intel_dp_init_connector() is no place for detecting stuff via
> DPCD (except perhaps for eDP). Move the colorimetry stuff into
> a more appropriate place.
> 
> Cc: Jouni Högander <jouni.hogander at intel.com>
> Fixes: 00076671a648 ("drm/i915/display: Move colorimetry_support from
> intel_psr to intel_dp")
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

Reviewed-by: Jouni Högander <jouni.hogander at intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_dp.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index 19f78432cc8f..f44951d8a0f8 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -4082,6 +4082,9 @@ intel_edp_init_dpcd(struct intel_dp *intel_dp,
> struct intel_connector *connector
>                          drm_dp_is_branch(intel_dp->dpcd));
>         intel_init_dpcd_quirks(intel_dp, &intel_dp->desc.ident);
>  
> +       intel_dp->colorimetry_support =
> +               intel_dp_get_colorimetry_status(intel_dp);
> +
>         /*
>          * Read the eDP display control registers.
>          *
> @@ -4195,6 +4198,9 @@ intel_dp_get_dpcd(struct intel_dp *intel_dp)
>  
>                 intel_init_dpcd_quirks(intel_dp, &intel_dp-
> >desc.ident);
>  
> +               intel_dp->colorimetry_support =
> +                       intel_dp_get_colorimetry_status(intel_dp);
> +
>                 intel_dp_update_sink_caps(intel_dp);
>         }
>  
> @@ -6946,9 +6952,6 @@ intel_dp_init_connector(struct
> intel_digital_port *dig_port,
>                                     "HDCP init failed, skipping.\n");
>         }
>  
> -       intel_dp->colorimetry_support =
> -               intel_dp_get_colorimetry_status(intel_dp);
> -
>         intel_dp->frl.is_trained = false;
>         intel_dp->frl.trained_rate_gbps = 0;
>