Having a macro with 10 arguments doesn't seem like a good idea. It makes it inherently harder to compare the actual structure values. Also this leads to adding macros covering varieties of the block. As it was previously discussed, inline all foo_BLK macros in order to ease performing changes to the catalog data. Major part of the conversion was performed using vim script found at [1]. Then some manual cleanups were applied, like dropping fields set to 0. Dependencies: [2]. Changes since v1: - Rebased on top of msm-next - Dropped dependency on interrupt rework [1] https://pastebin.ubuntu.com/p/26cdW5VpYB/ [2] https://patchwork.freedesktop.org/patch/542142/?series=119220&rev=1 Dmitry Baryshkov (22): drm/msm/dpu: fix sc7280 and sc7180 PINGPONG done interrupts drm/msm/dpu: correct MERGE_3D length drm/msm/dpu: remove unused INTF_NONE interfaces drm/msm: enumerate DSI interfaces drm/msm/dpu: always use MSM_DP/DSI_CONTROLLER_n drm/msm/dpu: simplify peer LM handling drm/msm/dpu: drop dpu_mdss_cfg::mdp_count field drm/msm/dpu: drop enum dpu_mdp and MDP_TOP value drm/msm/dpu: expand .clk_ctrls definitions drm/msm/dpu: drop zero features from dpu_mdp_cfg data drm/msm/dpu: drop zero features from dpu_ctl_cfg data drm/msm/dpu: correct indentation for CTL definitions drm/msm/dpu: inline SSPP_BLK macros drm/msm/dpu: inline DSPP_BLK macros drm/msm/dpu: inline LM_BLK macros drm/msm/dpu: inline DSC_BLK and DSC_BLK_1_2 macros drm/msm/dpu: inline MERGE_3D_BLK macros drm/msm/dpu: inline various PP_BLK_* macros drm/msm/dpu: inline WB_BLK macros drm/msm/dpu: inline INTF_BLK and INTF_BLK_DSI_TE macros drm/msm/dpu: drop empty features mask MERGE_3D_SM8150_MASK drm/msm/dpu: drop empty features mask INTF_SDM845_MASK .../msm/disp/dpu1/catalog/dpu_3_0_msm8998.h | 329 ++++++++---- .../msm/disp/dpu1/catalog/dpu_4_0_sdm845.h | 348 +++++++++---- .../msm/disp/dpu1/catalog/dpu_5_0_sm8150.h | 411 ++++++++++----- .../msm/disp/dpu1/catalog/dpu_5_1_sc8180x.h | 448 +++++++++++----- .../msm/disp/dpu1/catalog/dpu_6_0_sm8250.h | 430 +++++++++++----- .../msm/disp/dpu1/catalog/dpu_6_2_sc7180.h | 180 +++++-- .../msm/disp/dpu1/catalog/dpu_6_3_sm6115.h | 89 +++- .../msm/disp/dpu1/catalog/dpu_6_4_sm6350.h | 188 ++++--- .../msm/disp/dpu1/catalog/dpu_6_5_qcm2290.h | 89 +++- .../msm/disp/dpu1/catalog/dpu_6_9_sm6375.h | 96 ++-- .../msm/disp/dpu1/catalog/dpu_7_0_sm8350.h | 418 ++++++++++----- .../msm/disp/dpu1/catalog/dpu_7_2_sc7280.h | 236 ++++++--- .../msm/disp/dpu1/catalog/dpu_8_0_sc8280xp.h | 484 +++++++++++++----- .../msm/disp/dpu1/catalog/dpu_8_1_sm8450.h | 445 +++++++++++----- .../msm/disp/dpu1/catalog/dpu_9_0_sm8550.h | 467 ++++++++++++----- .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 130 ----- .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 5 +- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_mdss.h | 5 - drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c | 34 +- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.h | 7 +- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 2 +- drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 34 +- drivers/gpu/drm/msm/msm_drv.h | 8 +- 23 files changed, 3320 insertions(+), 1563 deletions(-) -- 2.39.2