[00/22] drm/msm/dpu: another catalog rework

Message ID	20230604203532.1094249-1-dmitry.baryshkov@linaro.org
Headers	show Return-Path: <linux-arm-msm-owner@vger.kernel.org> From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> To: Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>, Abhinav Kumar <quic_abhinavk@quicinc.com>, Marijn Suijten <marijn.suijten@somainline.org> Cc: Stephen Boyd <swboyd@chromium.org>, David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>, Bjorn Andersson <andersson@kernel.org>, linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org Subject: [PATCH 00/22] drm/msm/dpu: another catalog rework Date: Sun, 4 Jun 2023 23:35:10 +0300 Message-Id: <20230604203532.1094249-1-dmitry.baryshkov@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	drm/msm/dpu: another catalog rework \| expand [00/22] drm/msm/dpu: another catalog rework [01/22] drm/msm/dpu: correct MERGE_3D length [02/22] drm/msm/dpu: remove unused INTF_NONE interfaces [03/22] drm/msm: enumerate DSI interfaces [04/22] drm/msm/dpu: always use MSM_DP/DSI_CONTROLLER_n [05/22] drm/msm/dpu: simplify peer LM handling [06/22] drm/msm/dpu: drop dpu_mdss_cfg::mdp_count field [07/22] drm/msm/dpu: drop enum dpu_mdp and MDP_TOP value [08/22] drm/msm/dpu: expand .clk_ctrls definitions [09/22] drm/msm/dpu: drop zero features from dpu_ctl_cfg data [10/22] drm/msm/dpu: correct indentation for CTL definitions [11/22] drm/msm/dpu: drop zero features from dpu_mdp_cfg data [12/22] drm/msm/dpu: inline SSPP_BLK macros [13/22] drm/msm/dpu: inline DSPP_BLK macros [14/22] drm/msm/dpu: inline LM_BLK macros [15/22] drm/msm/dpu: inline DSC_BLK macros [16/22] drm/msm/dpu: inline MERGE_3D_BLK macros [17/22] drm/msm/dpu: inline various PP_BLK_* macros [18/22] drm/msm/dpu: inline WB_BLK macros [19/22] drm/msm/dpu: inline INTF_BLK and INTF_BLK_DSI_TE macros [20/22] drm/msm/dpu: drop empty features mask MERGE_3D_SM8150_MASK [21/22] drm/msm/dpu: drop empty features mask INTF_SDM845_MASK [22/22] drm/msm/dpu: move DPU_PINGPONG_DSC to PINGPONG_SDM845_MASK

Message ID

20230604203532.1094249-1-dmitry.baryshkov@linaro.org

Headers

From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>,
        Abhinav Kumar <quic_abhinavk@quicinc.com>,
        Marijn Suijten <marijn.suijten@somainline.org>
Cc: Stephen Boyd <swboyd@chromium.org>,
        David Airlie <airlied@gmail.com>,
        Daniel Vetter <daniel@ffwll.ch>,
        Bjorn Andersson <andersson@kernel.org>,
        linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
        freedreno@lists.freedesktop.org
Subject: [PATCH 00/22] drm/msm/dpu: another catalog rework
Date: Sun,  4 Jun 2023 23:35:10 +0300
Message-Id: <20230604203532.1094249-1-dmitry.baryshkov@linaro.org>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

Series

drm/msm/dpu: another catalog rework | expand

Message

Dmitry Baryshkov June 4, 2023, 8:35 p.m. UTC

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].

[1] https://pastebin.ubuntu.com/p/mQRhBRXTJs/
[2] https://patchwork.freedesktop.org/series/118836/

Dmitry Baryshkov (22):
  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_ctl_cfg data
  drm/msm/dpu: correct indentation for CTL definitions
  drm/msm/dpu: drop zero features from dpu_mdp_cfg data
  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 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
  drm/msm/dpu: move DPU_PINGPONG_DSC to PINGPONG_SDM845_MASK

 .../msm/disp/dpu1/catalog/dpu_3_0_msm8998.h   | 321 +++++++++----
 .../msm/disp/dpu1/catalog/dpu_4_0_sdm845.h    | 320 +++++++++----
 .../msm/disp/dpu1/catalog/dpu_5_0_sm8150.h    | 410 +++++++++++-----
 .../msm/disp/dpu1/catalog/dpu_5_1_sc8180x.h   | 448 ++++++++++++-----
 .../msm/disp/dpu1/catalog/dpu_6_0_sm8250.h    | 429 ++++++++++++-----
 .../msm/disp/dpu1/catalog/dpu_6_2_sc7180.h    | 177 +++++--
 .../msm/disp/dpu1/catalog/dpu_6_3_sm6115.h    |  87 ++--
 .../msm/disp/dpu1/catalog/dpu_6_5_qcm2290.h   |  87 ++--
 .../msm/disp/dpu1/catalog/dpu_7_0_sm8350.h    | 389 +++++++++++----
 .../msm/disp/dpu1/catalog/dpu_7_2_sc7280.h    | 224 ++++++---
 .../msm/disp/dpu1/catalog/dpu_8_0_sc8280xp.h  | 449 +++++++++++++-----
 .../msm/disp/dpu1/catalog/dpu_8_1_sm8450.h    | 415 +++++++++++-----
 .../msm/disp/dpu1/catalog/dpu_9_0_sm8550.h    | 439 ++++++++++++-----
 .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c    | 134 +-----
 .../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 +-
 21 files changed, 3109 insertions(+), 1315 deletions(-)