Re: [PATCH 2/2] drm/msm: Drop debug print in _dpu_crtc_setup_lm_bounds()

[abhinavk@codeaurora.org]

On 2020-09-01 14:59, Stephen Boyd wrote:
> This function is called quite often if you have a blinking cursor on 
> the
> screen, hello page flip. Let's drop this debug print here because it
> means enabling the print via the module parameter starts to spam the
> debug console.
> 
> Cc: Abhinav Kumar <abhinavk@codeaurora.org>
> Cc: Jeykumar Sankaran <jsanka@codeaurora.org>
> Cc: Jordan Crouse <jcrouse@codeaurora.org>
> Cc: Sean Paul <seanpaul@chromium.org>
> Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support")
> Signed-off-by: Stephen Boyd <swboyd@chromium.org>
Reviewed-by: Abhinav Kumar <abhinavk@codeaurora.org>
> ---
>  drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> index 74294b5ed93f..2966e488bfd0 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> @@ -421,8 +421,6 @@ static void _dpu_crtc_setup_lm_bounds(struct 
> drm_crtc *crtc,
> 
>  		trace_dpu_crtc_setup_lm_bounds(DRMID(crtc), i, r);
>  	}
> -
> -	drm_mode_debug_printmodeline(adj_mode);
>  }
> 
>  static void _dpu_crtc_get_pcc_coeff(struct drm_crtc_state *state,