]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/i915/dp: Use int for compressed BPP in dsc_compute_link_config()
authorJani Nikula <jani.nikula@intel.com>
Fri, 31 Jan 2025 12:50:04 +0000 (14:50 +0200)
committerJani Nikula <jani.nikula@intel.com>
Mon, 3 Feb 2025 14:31:42 +0000 (16:31 +0200)
Just use ints unless there are actual reasons to do otherwise. Here,
there are not.

Reviewed-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/b8f9aebc4e40afeed3d723f98cae96c9c927a480.1738327620.git.jani.nikula@intel.com
drivers/gpu/drm/i915/display/intel_dp.c

index 22b69c02c13eb5e5ae917f7a2a58a2518db3b9c4..27f066ad4073767b118cc642bd76dae1dd2ee19c 100644 (file)
@@ -1926,7 +1926,7 @@ static bool intel_dp_dsc_supports_format(const struct intel_connector *connector
        return drm_dp_dsc_sink_supports_format(connector->dp.dsc_dpcd, sink_dsc_format);
 }
 
-static bool is_bw_sufficient_for_dsc_config(u16 compressed_bppx16, u32 link_clock,
+static bool is_bw_sufficient_for_dsc_config(int dsc_bpp_x16, u32 link_clock,
                                            u32 lane_count, u32 mode_clock,
                                            enum intel_output_format output_format,
                                            int timeslots)
@@ -1934,7 +1934,7 @@ static bool is_bw_sufficient_for_dsc_config(u16 compressed_bppx16, u32 link_cloc
        u32 available_bw, required_bw;
 
        available_bw = (link_clock * lane_count * timeslots * 16)  / 8;
-       required_bw = compressed_bppx16 * (intel_dp_mode_to_fec_clock(mode_clock));
+       required_bw = dsc_bpp_x16 * (intel_dp_mode_to_fec_clock(mode_clock));
 
        return available_bw > required_bw;
 }
@@ -1942,7 +1942,7 @@ static bool is_bw_sufficient_for_dsc_config(u16 compressed_bppx16, u32 link_cloc
 static int dsc_compute_link_config(struct intel_dp *intel_dp,
                                   struct intel_crtc_state *pipe_config,
                                   const struct link_config_limits *limits,
-                                  u16 compressed_bppx16,
+                                  int dsc_bpp_x16,
                                   int timeslots)
 {
        const struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode;
@@ -1957,7 +1957,7 @@ static int dsc_compute_link_config(struct intel_dp *intel_dp,
                for (lane_count = limits->min_lane_count;
                     lane_count <= limits->max_lane_count;
                     lane_count <<= 1) {
-                       if (!is_bw_sufficient_for_dsc_config(compressed_bppx16, link_rate,
+                       if (!is_bw_sufficient_for_dsc_config(dsc_bpp_x16, link_rate,
                                                             lane_count, adjusted_mode->clock,
                                                             pipe_config->output_format,
                                                             timeslots))