Commit 170ffca8 authored by Marijn Suijten's avatar Marijn Suijten Committed by Dmitry Baryshkov
Browse files

drm/msm/dsi: Remove repeated calculation of slice_per_intf 



slice_per_intf is already computed for intf_width, which holds the same
value as hdisplay.

Fixes: 08802f51 ("drm/msm/dsi: Add support for DSC configuration")
Reviewed-by: default avatarBjorn Andersson <andersson@kernel.org>
Reviewed-by: default avatarKonrad Dybcio <konrad.dybcio@somainline.org>
Reviewed-by: default avatarAbhinav Kumar <quic_abhinavk@quicinc.com>
Reviewed-by: default avatarVinod Koul <vkoul@kernel.org>
Reviewed-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: default avatarMarijn Suijten <marijn.suijten@somainline.org>
Patchwork: https://patchwork.freedesktop.org/patch/508933/
Link: https://lore.kernel.org/r/20221026182824.876933-3-marijn.suijten@somainline.org


Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
parent 01a90798

drivers/gpu/drm/msm/dsi/dsi_host.c

+2 −4
Original line number Diff line number Diff line
@@ -842,7 +842,7 @@ static void dsi_ctrl_config(struct msm_dsi_host *msm_host, bool enable, 
static void dsi_update_dsc_timing(struct msm_dsi_host *msm_host, bool is_cmd_mode, u32 hdisplay)

{

	struct drm_dsc_config *dsc = msm_host->dsc;

	u32 reg, intf_width, reg_ctrl, reg_ctrl2;

	u32 reg, reg_ctrl, reg_ctrl2;

	u32 slice_per_intf, total_bytes_per_intf;

	u32 pkt_per_line;

	u32 bytes_in_slice;
@@ -851,8 +851,7 @@ static void dsi_update_dsc_timing(struct msm_dsi_host *msm_host, bool is_cmd_mod 
	/* first calculate dsc parameters and then program

	 * compress mode registers

	 */

	intf_width = hdisplay;

	slice_per_intf = DIV_ROUND_UP(intf_width, dsc->slice_width);

	slice_per_intf = DIV_ROUND_UP(hdisplay, dsc->slice_width);



	/* If slice_per_pkt is greater than slice_per_intf

	 * then default to 1. This can happen during partial
@@ -861,7 +860,6 @@ static void dsi_update_dsc_timing(struct msm_dsi_host *msm_host, bool is_cmd_mod 
	if (slice_per_intf > dsc->slice_count)

		dsc->slice_count = 1;



	slice_per_intf = DIV_ROUND_UP(hdisplay, dsc->slice_width);

	bytes_in_slice = DIV_ROUND_UP(dsc->slice_width * dsc->bits_per_pixel, 8);



	dsc->slice_chunk_size = bytes_in_slice;