Commit 4ba5a4ad authored by Marijn Suijten's avatar Marijn Suijten Committed by Dmitry Baryshkov

drm/msm/dpu: Remove num_enc from topology struct in favour of num_dsc

Downstream calls this num_enc yet the DSC patches introduced a new
num_dsc struct member, leaving num_enc effectively unused.

Fixes: 7e9cc175 ("drm/msm/disp/dpu1: Add support for DSC in topology")
Signed-off-by: default avatarMarijn Suijten <marijn.suijten@somainline.org>
Reviewed-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Patchwork: https://patchwork.freedesktop.org/patch/515688/
Link: https://lore.kernel.org/r/20221221231943.1961117-7-marijn.suijten@somainline.orgSigned-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
parent 7d9510ef
...@@ -579,19 +579,18 @@ static struct msm_display_topology dpu_encoder_get_topology( ...@@ -579,19 +579,18 @@ static struct msm_display_topology dpu_encoder_get_topology(
topology.num_dspp = topology.num_lm; topology.num_dspp = topology.num_lm;
} }
topology.num_enc = 0;
topology.num_intf = intf_count; topology.num_intf = intf_count;
if (dpu_enc->dsc) { if (dpu_enc->dsc) {
/* In case of Display Stream Compression (DSC), we would use /*
* 2 encoders, 2 layer mixers and 1 interface * In case of Display Stream Compression (DSC), we would use
* 2 DSC encoders, 2 layer mixers and 1 interface
* this is power optimal and can drive up to (including) 4k * this is power optimal and can drive up to (including) 4k
* screens * screens
*/ */
topology.num_enc = 2;
topology.num_dsc = 2; topology.num_dsc = 2;
topology.num_intf = 1;
topology.num_lm = 2; topology.num_lm = 2;
topology.num_intf = 1;
} }
return topology; return topology;
......
...@@ -548,8 +548,8 @@ static int _dpu_rm_populate_requirements( ...@@ -548,8 +548,8 @@ static int _dpu_rm_populate_requirements(
{ {
reqs->topology = req_topology; reqs->topology = req_topology;
DRM_DEBUG_KMS("num_lm: %d num_enc: %d num_intf: %d\n", DRM_DEBUG_KMS("num_lm: %d num_dsc: %d num_intf: %d\n",
reqs->topology.num_lm, reqs->topology.num_enc, reqs->topology.num_lm, reqs->topology.num_dsc,
reqs->topology.num_intf); reqs->topology.num_intf);
return 0; return 0;
......
...@@ -83,14 +83,12 @@ enum msm_event_wait { ...@@ -83,14 +83,12 @@ enum msm_event_wait {
/** /**
* struct msm_display_topology - defines a display topology pipeline * struct msm_display_topology - defines a display topology pipeline
* @num_lm: number of layer mixers used * @num_lm: number of layer mixers used
* @num_enc: number of compression encoder blocks used
* @num_intf: number of interfaces the panel is mounted on * @num_intf: number of interfaces the panel is mounted on
* @num_dspp: number of dspp blocks used * @num_dspp: number of dspp blocks used
* @num_dsc: number of Display Stream Compression (DSC) blocks used * @num_dsc: number of Display Stream Compression (DSC) blocks used
*/ */
struct msm_display_topology { struct msm_display_topology {
u32 num_lm; u32 num_lm;
u32 num_enc;
u32 num_intf; u32 num_intf;
u32 num_dspp; u32 num_dspp;
u32 num_dsc; u32 num_dsc;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment