On 03/07/2023 05:01, Abhinav Kumar wrote:
On 6/19/2023 2:25 PM, Dmitry Baryshkov wrote:
There is always a single MDP TOP block. Drop the mdp_count field and
stop declaring dpu_mdp_cfg instances as arrays.
Tested-by: Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
---
The change drops mdp_count and stops using the array which is fine and I
will support that.
But looking at the pattern I saw while using core_revision, both
DPU_MDP_VSYNC_SEL and DPU_MDP_AUDIO_SELECT can also be dropped from the
catalog in favor of using core_revision.
Hence for that, I request you not to stop passing dpu_mdss_cfg to
dpu_hw_mdptop_init as that has the necessary information of core_revision.
Sure, I'll restore it. Please note, however, that it might be better to
pass struct dpu_caps instead of the full struct dpu_mdss_cfg.
.../msm/disp/dpu1/catalog/dpu_3_0_msm8998.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_4_0_sdm845.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_5_0_sm8150.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_5_1_sc8180x.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_6_0_sm8250.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_6_2_sc7180.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_6_3_sm6115.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_6_4_sm6350.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_6_5_qcm2290.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_6_9_sm6375.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_7_0_sm8350.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_7_2_sc7280.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_8_0_sc8280xp.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_8_1_sm8450.h | 7 +---
.../msm/disp/dpu1/catalog/dpu_9_0_sm8550.h | 7 +---
.../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 1 -
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c | 38 +++----------------
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.h | 8 ++--
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 4 +-
19 files changed, 41 insertions(+), 115 deletions(-)
--
With best wishes
Dmitry
