drm/msm/dpu: change _dpu_plane_calc_bw() to use u64 to avoid overflow
authorAbhinav Kumar <quic_abhinavk@quicinc.com>
Fri, 8 Sep 2023 01:26:16 +0000 (18:26 -0700)
committerAbhinav Kumar <quic_abhinavk@quicinc.com>
Wed, 20 Sep 2023 23:33:35 +0000 (16:33 -0700)
commit95e681ca3b65e4ce3d2537b47672d787b7d30375
tree9963ebaa52bea17ce18a79bcdcdd242bca4eb64a
parent6a1d4c7976dd1ee7c9f80bc8e62801ec7b1f2f58
drm/msm/dpu: change _dpu_plane_calc_bw() to use u64 to avoid overflow

_dpu_plane_calc_bw() uses integer variables to calculate the bandwidth
used during plane bandwidth calculations. However for high resolution
displays this overflows easily and leads to below errors

[dpu error]crtc83 failed performance check -7

Promote the intermediate variables to u64 to avoid overflow.

changes in v2:
- change to u64 where actually needed in the math

Fixes: c33b7c0389e1 ("drm/msm/dpu: add support for clk and bw scaling for display")
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reported-by: Nia Espera <nespera@igalia.com>
Closes: https://gitlab.freedesktop.org/drm/msm/-/issues/32
Tested-by: Nia Espera <nespera@igalia.com>
Patchwork: https://patchwork.freedesktop.org/patch/556288/
Link: https://lore.kernel.org/r/20230908012616.20654-1-quic_abhinavk@quicinc.com
Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c