[Freedreno] [PATCH v3 0/7] drm/msm: clean up the hw catalog init

[Dmitry Baryshkov]

On 02/06/2022 23:24, Abhinav Kumar wrote:
> 
> 
> On 6/2/2022 1:22 PM, Dmitry Baryshkov wrote:
>> Replace superfluous cfg_init functions, which just assign a static
>> config to the struct dpu_mdss_cfg, with static instances of struct
>> dpu_mdss_cfg.
>>
>> Changes since v2:
>>   - Add DPU_MIXER_COMBINED_ALPHA to sc7180's mixer features mask (noted
>>     by Abhinav).
>>
>> Changes since v1:
>>   - Turn catalog->perf and catalog->dma_cfg to be pointers, otherwise
>>     clang complains that they are not constant.
>>
> 
> I see you have dropped the R-bs from all the changes in V2.
> Can you please add them back?

Yes, I missed that in v3. Should be fixed in v4.

> 
>> Dmitry Baryshkov (7):
>>    drm/msm/dpu: use feature bit for LM combined alpha check
>>    drm/msm/dpu: move VBIF_XINL_QOS_LVL_REMAP size to hw_catalog
>>    drm/msm/dpu: remove hwversion field from data structures
>>    drm/msm/dpu: change catalog->perf to be a const pointer
>>    drm/msm/dpu: change catalog->dma_cfg to be a const pointer
>>    drm/msm/dpu: constify struct dpu_mdss_cfg
>>    drm/msm/dpu: make dpu hardware catalog static const
>>
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c |  24 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.h |   4 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c   |   4 +-
>>   .../drm/msm/disp/dpu1/dpu_encoder_phys_wb.c   |  12 +-
>>   .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c    | 502 ++++++++----------
>>   .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h    |  20 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c    |   1 -
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_dsc.c    |   5 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_dsc.h    |   2 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_dspp.c   |   1 -
>>   .../gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.c |   5 +-
>>   .../gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.h |   2 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c   |   1 -
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c     |   7 +-
>>   .../gpu/drm/msm/disp/dpu1/dpu_hw_merge3d.c    |   1 -
>>   .../gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.c   |   1 -
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c   |   5 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h   |   4 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c    |   1 -
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h   |   2 -
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_vbif.c   |   5 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_wb.c     |   1 -
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c       |   6 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h       |   2 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c     |  20 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c        |   2 +-
>>   drivers/gpu/drm/msm/disp/dpu1/dpu_rm.h        |   2 +-
>>   27 files changed, 283 insertions(+), 359 deletions(-)
>>


-- 
With best wishes
Dmitry