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 [thread overview] Message-ID: <20230604203532.1094249-1-dmitry.baryshkov@linaro.org> (raw) 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(-) -- 2.39.2
WARNING: multiple messages have this Message-ID (diff)
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: freedreno@lists.freedesktop.org, linux-arm-msm@vger.kernel.org, Bjorn Andersson <andersson@kernel.org>, dri-devel@lists.freedesktop.org, Stephen Boyd <swboyd@chromium.org> Subject: [PATCH 00/22] drm/msm/dpu: another catalog rework Date: Sun, 4 Jun 2023 23:35:10 +0300 [thread overview] Message-ID: <20230604203532.1094249-1-dmitry.baryshkov@linaro.org> (raw) 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(-) -- 2.39.2
next reply other threads:[~2023-06-04 20:35 UTC|newest] Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-06-04 20:35 Dmitry Baryshkov [this message] 2023-06-04 20:35 ` [PATCH 00/22] drm/msm/dpu: another catalog rework Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 01/22] drm/msm/dpu: correct MERGE_3D length Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 02/22] drm/msm/dpu: remove unused INTF_NONE interfaces Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 03/22] drm/msm: enumerate DSI interfaces Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 04/22] drm/msm/dpu: always use MSM_DP/DSI_CONTROLLER_n Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 05/22] drm/msm/dpu: simplify peer LM handling Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 06/22] drm/msm/dpu: drop dpu_mdss_cfg::mdp_count field Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 07/22] drm/msm/dpu: drop enum dpu_mdp and MDP_TOP value Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 08/22] drm/msm/dpu: expand .clk_ctrls definitions Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 09/22] drm/msm/dpu: drop zero features from dpu_ctl_cfg data Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 10/22] drm/msm/dpu: correct indentation for CTL definitions Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 11/22] drm/msm/dpu: drop zero features from dpu_mdp_cfg data Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 12/22] drm/msm/dpu: inline SSPP_BLK macros Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 13/22] drm/msm/dpu: inline DSPP_BLK macros Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 14/22] drm/msm/dpu: inline LM_BLK macros Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 15/22] drm/msm/dpu: inline DSC_BLK macros Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 16/22] drm/msm/dpu: inline MERGE_3D_BLK macros Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 17/22] drm/msm/dpu: inline various PP_BLK_* macros Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 18/22] drm/msm/dpu: inline WB_BLK macros Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 19/22] drm/msm/dpu: inline INTF_BLK and INTF_BLK_DSI_TE macros Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 20/22] drm/msm/dpu: drop empty features mask MERGE_3D_SM8150_MASK Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 21/22] drm/msm/dpu: drop empty features mask INTF_SDM845_MASK Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov 2023-06-04 20:35 ` [PATCH 22/22] drm/msm/dpu: move DPU_PINGPONG_DSC to PINGPONG_SDM845_MASK Dmitry Baryshkov 2023-06-04 20:35 ` Dmitry Baryshkov
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20230604203532.1094249-1-dmitry.baryshkov@linaro.org \ --to=dmitry.baryshkov@linaro.org \ --cc=airlied@gmail.com \ --cc=andersson@kernel.org \ --cc=daniel@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=freedreno@lists.freedesktop.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=marijn.suijten@somainline.org \ --cc=quic_abhinavk@quicinc.com \ --cc=robdclark@gmail.com \ --cc=sean@poorly.run \ --cc=swboyd@chromium.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.