On Fri, Jul 05, 2024 at 05:52:43PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> Use the more customary name 'height' instead of 'lines'.
>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> ---
> drivers/gpu/drm/i915/display/intel_fbc.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c b/drivers/gpu/drm/i915/display/intel_fbc.c
> index 4a9321f5218f..4d25ebb5ae9d 100644
> --- a/drivers/gpu/drm/i915/display/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/display/intel_fbc.c
> @@ -204,14 +204,14 @@ static unsigned int intel_fbc_cfb_stride(const struct intel_plane_state *plane_s
> static unsigned int intel_fbc_cfb_size(const struct intel_plane_state *plane_state)
> {
> struct intel_display *display = to_intel_display(plane_state->uapi.plane->dev);
> - int lines = drm_rect_height(&plane_state->uapi.src) >> 16;
> + int height = drm_rect_height(&plane_state->uapi.src) >> 16;
>
> if (DISPLAY_VER(display) == 7)
> - lines = min(lines, 2048);
> + height = min(height, 2048);
> else if (DISPLAY_VER(display) >= 8)
> - lines = min(lines, 2560);
> + height = min(height, 2560);
>
> - return lines * intel_fbc_cfb_stride(plane_state);
> + return height * intel_fbc_cfb_stride(plane_state);
> }
>
> static u16 intel_fbc_override_cfb_stride(const struct intel_plane_state *plane_state)
> --
> 2.44.2
>
