On Tue, 29 Oct 2024, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote:
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> min_cdclk==0 when intel_vdsc_min_cdclk() calls max_t() on it.
> Drop the redundant max_t().
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/display/intel_cdclk.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
> index 989607c0b35d..d376f7bccf21 100644
> --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
> +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
> @@ -2810,7 +2810,7 @@ static int intel_vdsc_min_cdclk(const struct intel_crtc_state *crtc_state)
> struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
> struct intel_display *display = to_intel_display(crtc);
> int num_vdsc_instances = intel_dsc_get_num_vdsc_instances(crtc_state);
> - int min_cdclk = 0;
> + int min_cdclk;
>
> if (!crtc_state->dsc.compression_enable)
> return 0;
> @@ -2822,8 +2822,7 @@ static int intel_vdsc_min_cdclk(const struct intel_crtc_state *crtc_state)
> * If there 2 VDSC engines, then pixel clock can't be higher than
> * VDSC clock(cdclk) * 2 and so on.
> */
> - min_cdclk = max_t(int, min_cdclk,
> - DIV_ROUND_UP(crtc_state->pixel_rate, num_vdsc_instances));
> + min_cdclk = DIV_ROUND_UP(crtc_state->pixel_rate, num_vdsc_instances);
>
> if (crtc_state->joiner_pipes) {
> int pixel_clock = intel_dp_mode_to_fec_clock(crtc_state->hw.adjusted_mode.clock);
--
Jani Nikula, Intel
