On 2023-05-20 03:28:45, Dmitry Baryshkov wrote:
> In dsi_calc_clk_rate_v2() there is no need to call dsi_get_pclk_rate().
> This functions has just been called and it's result is stored at

function (drop -s)
has just been called *inside dsi_calc_pclk()*
it's -> its

> msm_host->pixel_clk_rate. Use this variable directly.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org>

Reviewed-by: Marijn Suijten <marijn.suij...@somainline.org>

> ---
>  drivers/gpu/drm/msm/dsi/dsi_host.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c 
> b/drivers/gpu/drm/msm/dsi/dsi_host.c
> index 961689a255c4..2b257b459974 100644
> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c
> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
> @@ -633,7 +633,7 @@ int dsi_calc_clk_rate_v2(struct msm_dsi_host *msm_host, 
> bool is_bonded_dsi)
>  
>       dsi_calc_pclk(msm_host, is_bonded_dsi);
>  
> -     pclk_bpp = (u64)dsi_get_pclk_rate(msm_host->mode, is_bonded_dsi) * bpp;
> +     pclk_bpp = (u64)msm_host->pixel_clk_rate * bpp;
>       do_div(pclk_bpp, 8);
>       msm_host->src_clk_rate = pclk_bpp;
>  
> -- 
> 2.39.2
> 

Reply via email to