On Thu, 2025-11-27 at 19:50 +0200, Imre Deak wrote:
> Rename test_slice_count to slices_per_line for clarity.
> 
> Signed-off-by: Imre Deak <[email protected]>

Reviewed-by: Jouni Högander <[email protected]>

> ---
>  drivers/gpu/drm/i915/display/intel_dp.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index 1d9a130bd4060..650b339fd73bc 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -1036,7 +1036,7 @@ u8 intel_dp_dsc_get_slice_count(const struct
> intel_connector *connector,
>  
>       /* Find the closest match to the valid slice count values */
>       for (i = 0; i < ARRAY_SIZE(valid_dsc_slicecount); i++) {
> -             int test_slice_count = valid_dsc_slicecount[i] *
> num_joined_pipes;
> +             int slices_per_line = valid_dsc_slicecount[i] *
> num_joined_pipes;
>  
>               /*
>                * 3 DSC Slices per pipe need 3 DSC engines, which
> is supported only
> @@ -1046,7 +1046,7 @@ u8 intel_dp_dsc_get_slice_count(const struct
> intel_connector *connector,
>                   (!HAS_DSC_3ENGINES(display) || num_joined_pipes
> != 4))
>                       continue;
>  
> -             if
> (!(drm_dp_dsc_slice_count_to_mask(test_slice_count) &
> +             if
> (!(drm_dp_dsc_slice_count_to_mask(slices_per_line) &
>                     sink_slice_count_mask))
>                       continue;
>  
> @@ -1058,11 +1058,11 @@ u8 intel_dp_dsc_get_slice_count(const struct
> intel_connector *connector,
>               if (num_joined_pipes > 1 && valid_dsc_slicecount[i]
> < 2)
>                       continue;
>  
> -             if (mode_hdisplay % test_slice_count)
> +             if (mode_hdisplay % slices_per_line)
>                       continue;
>  
> -             if (min_slice_count <= test_slice_count)
> -                     return test_slice_count;
> +             if (min_slice_count <= slices_per_line)
> +                     return slices_per_line;
>       }
>  
>       /* Print slice count 1,2,4,..24 if bit#0,1,3,..23 is set in
> the mask. */

Reply via email to