From d04e379ac5f07197526d56e2f65a46b2a2666ebe Mon Sep 17 00:00:00 2001 From: Dom Cobley Date: Wed, 3 Dec 2025 17:56:40 +0000 Subject: [PATCH] drm/vc4: plane: Remove dead channels_scaled code This code can never do anything, as channels_scaled can only be 0 or 2. I suspect that the index was intended to be i rather than channel, but that would trigger for scaled RGB planes, and halving lines is not wanted there. Just remove it. Signed-off-by: Dom Cobley --- drivers/gpu/drm/vc4/vc4_plane.c | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c index 7b136e9b9114..58c1f2b4bc68 100644 --- a/drivers/gpu/drm/vc4/vc4_plane.c +++ b/drivers/gpu/drm/vc4/vc4_plane.c @@ -812,11 +812,8 @@ static unsigned int vc4_lbm_channel_size(const struct drm_plane_state *state, unsigned int channel) { const struct drm_format_info *info = state->fb->format; - const struct vc4_plane_state *vc4_state = to_vc4_plane_state(state); - unsigned int channels_scaled = 0; unsigned int components, words, wpc; unsigned int width, lines; - unsigned int i; /* LBM is meant to use the smaller of source or dest width, but there * is a issue with UV scaling that the size required for the second @@ -843,13 +840,6 @@ static unsigned int vc4_lbm_channel_size(const struct drm_plane_state *state, lines = DIV_ROUND_UP(words, 128 / info->hsub); - for (i = 0; i < 2; i++) - if (vc4_state->y_scaling[channel] != VC4_SCALING_NONE) - channels_scaled++; - - if (channels_scaled == 1) - lines = lines / 2; - return lines; }