On 6/24/2024 2:13 PM, Dmitry Baryshkov wrote:
The _dpu_format_get_plane_sizes_linear() already compares pitches of the framebuffer with the calculated pitches. Move the check to the same place, demoting DPU_ERROR to DPU_DEBUG to prevent user from spamming the kernel log. Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> --- drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-)
Not fully convinced about demoting DPU_ERROR to DPU_DEBUG but I think we have had a similar discussion earlier while logging atomic_check failures in the CDM series, so keeping that in mind,
Reviewed-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>