> -----Original Message-----
> From: Grzelak, Michal <[email protected]>
> Sent: Wednesday, June 10, 2026 4:42 AM
> To: [email protected]; [email protected]
> Cc: Grzelak, Michal <[email protected]>; Garg, Nemesa
> <[email protected]>; Ville Syrjälä <[email protected]>
> Subject: [PATCH v1 2/9] drm/i915/scaler: s/i/scaler_id/
> 
> Rename generic iterator into more descriptive one.
> 
> Cc: Nemesa Garg <[email protected]>
> Cc: Ville Syrjälä <[email protected]>
> Signed-off-by: Michał Grzelak <[email protected]>
> ---
>  drivers/gpu/drm/i915/display/skl_scaler.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/skl_scaler.c
> b/drivers/gpu/drm/i915/display/skl_scaler.c
> index 4b456924fdef..0cfa5b7564ba 100644
> --- a/drivers/gpu/drm/i915/display/skl_scaler.c
> +++ b/drivers/gpu/drm/i915/display/skl_scaler.c
> @@ -337,18 +337,18 @@ static int intel_allocate_scaler(struct
> intel_crtc_scaler_state *scaler_state,
>                                bool casf_scaler)
>  {
>       struct intel_display *display = to_intel_display(crtc);
> -     int i;
> +     int scaler_id;
> 
> -     for (i = 0; i < crtc->num_scalers; i++) {
> -             if (scaler_state->scalers[i].in_use)
> +     for (scaler_id = 0; scaler_id < crtc->num_scalers; scaler_id++) {
> +             if (scaler_state->scalers[scaler_id].in_use)
>                       continue;
> 
> -             if (casf_scaler && !scaler_has_casf(display, i))
> +             if (casf_scaler && !scaler_has_casf(display, scaler_id))
>                       continue;
> 
> -             scaler_state->scalers[i].in_use = true;
> +             scaler_state->scalers[scaler_id].in_use = true;
> 
> -             return i;
> +             return scaler_id;
>       }
> 
>       return -1;
> --
LGTM,
Reviewed-by: Nemesa Garg <[email protected]>

> 2.45.2

Reply via email to