Re: [PATCH 22/50] drm/i915/dp: Drop intel_dp parameter from intel_dp_compute_config_link_bpp_limits()

["Govindapillai, Vinod" <vinod.govindapillai@intel.com>]

On Thu, 2025-11-27 at 19:49 +0200, Imre Deak wrote:
> The intel_dp pointer can be deducted from the connector pointer, so
> it's
> enough to pass only connector to
> intel_dp_compute_config_link_bpp_limits(), do so.
> 
> Signed-off-by: Imre Deak <imre.deak@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_dp.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 

Reviewed-by: Vinod Govindapillai <vinod.govindapillai@intel.com>

> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index 801e8fd6b229e..5ad71e697e585 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -2653,13 +2653,13 @@ dsc_throughput_quirk_max_bpp_x16(const struct
> intel_connector *connector,
>   * range, crtc_state and dsc mode. Return true on success.
>   */
>  static bool
> -intel_dp_compute_config_link_bpp_limits(struct intel_dp *intel_dp,
> -					const struct intel_connector
> *connector,
> +intel_dp_compute_config_link_bpp_limits(struct intel_connector
> *connector,
>  					const struct
> intel_crtc_state *crtc_state,
>  					bool dsc,
>  					struct link_config_limits
> *limits)
>  {
> -	struct intel_display *display = to_intel_display(intel_dp);
> +	struct intel_display *display = to_intel_display(connector);
> +	struct intel_dp *intel_dp = intel_attached_dp(connector);
>  	const struct drm_display_mode *adjusted_mode =
>  		&crtc_state->hw.adjusted_mode;
>  	const struct intel_crtc *crtc = to_intel_crtc(crtc_state-
> >uapi.crtc);
> @@ -2831,8 +2831,7 @@ intel_dp_compute_config_limits(struct intel_dp
> *intel_dp,
>  
>  	intel_dp_test_compute_config(intel_dp, crtc_state, limits);
>  
> -	return intel_dp_compute_config_link_bpp_limits(intel_dp,
> -						       connector,
> +	return intel_dp_compute_config_link_bpp_limits(connector,
>  						       crtc_state,
>  						       dsc,
>  						       limits);