drm/msm/dpu: move UBWC/memory configuration to separate struct

UBWC and highest bank settings differ slightly between different DPU
units of the same generation, while the dpu_caps and dpu_mdp_cfg are
much more stable. To ease configuration reuse move ubwc_swizzle and
highest_bank_bit data to separate structure.

Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Patchwork: https://patchwork.freedesktop.org/patch/530820/
Link: https://lore.kernel.org/r/20230404130622.509628-7-dmitry.baryshkov@linaro.org
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
index a57be984b44a548eebb4f4ce76b10c37d40c90cd..b73dd9c4b5350748f06a58a32836575936c73627 100644 (file)
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
@@ -323,7 +323,6 @@ static const struct dpu_caps msm8998_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0x7,
        .qseed_type = DPU_SSPP_SCALER_QSEED3,
-       .ubwc_version = DPU_HW_UBWC_VER_10,
        .has_src_split = true,
        .has_dim_layer = true,
        .has_idle_pc = true,
@@ -347,7 +346,6 @@ static const struct dpu_caps sdm845_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0xb,
        .qseed_type = DPU_SSPP_SCALER_QSEED3,
-       .ubwc_version = DPU_HW_UBWC_VER_20,
        .has_src_split = true,
        .has_dim_layer = true,
        .has_idle_pc = true,
@@ -362,7 +360,6 @@ static const struct dpu_caps sc7180_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0x9,
        .qseed_type = DPU_SSPP_SCALER_QSEED4,
-       .ubwc_version = DPU_HW_UBWC_VER_20,
        .has_dim_layer = true,
        .has_idle_pc = true,
        .max_linewidth = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
@@ -373,7 +370,6 @@ static const struct dpu_caps sm6115_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_LINE_WIDTH,
        .max_mixer_blendstages = 0x4,
        .qseed_type = DPU_SSPP_SCALER_QSEED4,
-       .ubwc_version = DPU_HW_UBWC_VER_10,
        .has_dim_layer = true,
        .has_idle_pc = true,
        .max_linewidth = 2160,
@@ -384,7 +380,6 @@ static const struct dpu_caps sm8150_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0xb,
        .qseed_type = DPU_SSPP_SCALER_QSEED3,
-       .ubwc_version = DPU_HW_UBWC_VER_30,
        .has_src_split = true,
        .has_dim_layer = true,
        .has_idle_pc = true,
@@ -399,7 +394,6 @@ static const struct dpu_caps sc8180x_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0xb,
        .qseed_type = DPU_SSPP_SCALER_QSEED3,
-       .ubwc_version = DPU_HW_UBWC_VER_30,
        .has_src_split = true,
        .has_dim_layer = true,
        .has_idle_pc = true,
@@ -414,7 +408,6 @@ static const struct dpu_caps sc8280xp_dpu_caps = {
        .max_mixer_width = 2560,
        .max_mixer_blendstages = 11,
        .qseed_type = DPU_SSPP_SCALER_QSEED4,
-       .ubwc_version = DPU_HW_UBWC_VER_40,
        .has_src_split = true,
        .has_dim_layer = true,
        .has_idle_pc = true,
@@ -427,7 +420,6 @@ static const struct dpu_caps sm8250_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0xb,
        .qseed_type = DPU_SSPP_SCALER_QSEED4,
-       .ubwc_version = DPU_HW_UBWC_VER_40,
        .has_src_split = true,
        .has_dim_layer = true,
        .has_idle_pc = true,
@@ -440,7 +432,6 @@ static const struct dpu_caps sm8350_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0xb,
        .qseed_type = DPU_SSPP_SCALER_QSEED4,
-       .ubwc_version = DPU_HW_UBWC_VER_40,
        .has_src_split = true,
        .has_dim_layer = true,
        .has_idle_pc = true,
@@ -453,7 +444,6 @@ static const struct dpu_caps sm8450_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0xb,
        .qseed_type = DPU_SSPP_SCALER_QSEED4,
-       .ubwc_version = DPU_HW_UBWC_VER_40,
        .has_src_split = true,
        .has_dim_layer = true,
        .has_idle_pc = true,
@@ -466,7 +456,6 @@ static const struct dpu_caps sm8550_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0xb,
        .qseed_type = DPU_SSPP_SCALER_QSEED4,
-       .ubwc_version = DPU_HW_UBWC_VER_40,
        .has_src_split = true,
        .has_dim_layer = true,
        .has_idle_pc = true,
@@ -479,19 +468,86 @@ static const struct dpu_caps sc7280_dpu_caps = {
        .max_mixer_width = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
        .max_mixer_blendstages = 0x7,
        .qseed_type = DPU_SSPP_SCALER_QSEED4,
-       .ubwc_version = DPU_HW_UBWC_VER_30,
        .has_dim_layer = true,
        .has_idle_pc = true,
        .max_linewidth = 2400,
        .pixel_ram_size = DEFAULT_PIXEL_RAM_SIZE,
 };
 
+static const struct dpu_ubwc_cfg msm8998_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_10,
+       .highest_bank_bit = 0x2,
+};
+
+static const struct dpu_ubwc_cfg qcm2290_ubwc_cfg = {
+       .highest_bank_bit = 0x2,
+};
+
+static const struct dpu_ubwc_cfg sdm845_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_20,
+       .highest_bank_bit = 0x2,
+};
+
+static const struct dpu_ubwc_cfg sc7180_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_20,
+       .highest_bank_bit = 0x3,
+};
+
+static const struct dpu_ubwc_cfg sm6115_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_10,
+       .highest_bank_bit = 0x1,
+       .ubwc_swizzle = 0x7,
+};
+
+static const struct dpu_ubwc_cfg sm8150_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_30,
+       .highest_bank_bit = 0x2,
+};
+
+static const struct dpu_ubwc_cfg sc8180x_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_30,
+       .highest_bank_bit = 0x3,
+};
+
+static const struct dpu_ubwc_cfg sc8280xp_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_40,
+       .highest_bank_bit = 2,
+       .ubwc_swizzle = 6,
+};
+
+static const struct dpu_ubwc_cfg sm8250_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_40,
+       .highest_bank_bit = 0x3, /* TODO: 2 for LP_DDR4 */
+       .ubwc_swizzle = 0x6,
+};
+
+static const struct dpu_ubwc_cfg sm8350_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_40,
+       .highest_bank_bit = 0x3, /* TODO: 2 for LP_DDR4 */
+};
+
+static const struct dpu_ubwc_cfg sm8450_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_40,
+       .highest_bank_bit = 0x3, /* TODO: 2 for LP_DDR4 */
+       .ubwc_swizzle = 0x6,
+};
+
+static const struct dpu_ubwc_cfg sm8550_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_40,
+       .highest_bank_bit = 0x3, /* TODO: 2 for LP_DDR4 */
+};
+
+static const struct dpu_ubwc_cfg sc7280_ubwc_cfg = {
+       .ubwc_version = DPU_HW_UBWC_VER_30,
+       .highest_bank_bit = 0x1,
+       .ubwc_swizzle = 0x6,
+};
+
 static const struct dpu_mdp_cfg msm8998_mdp[] = {
        {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x458,
        .features = 0,
-       .highest_bank_bit = 0x2,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                        .reg_off = 0x2AC, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_VIG1] = {
@@ -520,7 +576,6 @@ static const struct dpu_mdp_cfg sdm845_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x45C,
        .features = BIT(DPU_MDP_AUDIO_SELECT),
-       .highest_bank_bit = 0x2,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                        .reg_off = 0x2AC, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_VIG1] = {
@@ -545,7 +600,6 @@ static const struct dpu_mdp_cfg sc7180_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x494,
        .features = 0,
-       .highest_bank_bit = 0x3,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                .reg_off = 0x2AC, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_DMA0] = {
@@ -564,7 +618,6 @@ static const struct dpu_mdp_cfg sc8180x_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x45C,
        .features = BIT(DPU_MDP_AUDIO_SELECT),
-       .highest_bank_bit = 0x3,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                        .reg_off = 0x2AC, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_VIG1] = {
@@ -589,8 +642,6 @@ static const struct dpu_mdp_cfg sm6115_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x494,
        .features = 0,
-       .highest_bank_bit = 0x1,
-       .ubwc_swizzle = 0x7,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                .reg_off = 0x2ac, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_DMA0] = {
@@ -603,8 +654,6 @@ static const struct dpu_mdp_cfg sm8250_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x494,
        .features = 0,
-       .highest_bank_bit = 0x3, /* TODO: 2 for LP_DDR4 */
-       .ubwc_swizzle = 0x6,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                        .reg_off = 0x2AC, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_VIG1] = {
@@ -633,7 +682,6 @@ static const struct dpu_mdp_cfg sm8350_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x494,
        .features = 0,
-       .highest_bank_bit = 0x3, /* TODO: 2 for LP_DDR4 */
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                        .reg_off = 0x2ac, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_VIG1] = {
@@ -660,8 +708,6 @@ static const struct dpu_mdp_cfg sm8450_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x494,
        .features = BIT(DPU_MDP_PERIPH_0_REMOVED),
-       .highest_bank_bit = 0x3, /* TODO: 2 for LP_DDR4 */
-       .ubwc_swizzle = 0x6,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                        .reg_off = 0x2AC, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_VIG1] = {
@@ -687,8 +733,6 @@ static const struct dpu_mdp_cfg sc7280_mdp[] = {
        {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x2014,
-       .highest_bank_bit = 0x1,
-       .ubwc_swizzle = 0x6,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                .reg_off = 0x2AC, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_DMA0] = {
@@ -705,8 +749,6 @@ static const struct dpu_mdp_cfg sc8280xp_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x494,
        .features = BIT(DPU_MDP_PERIPH_0_REMOVED),
-       .highest_bank_bit = 2,
-       .ubwc_swizzle = 6,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = { .reg_off = 0x2ac, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_VIG1] = { .reg_off = 0x2b4, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_VIG2] = { .reg_off = 0x2bc, .bit_off = 0},
@@ -724,8 +766,6 @@ static const struct dpu_mdp_cfg sm8550_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0, .len = 0x494,
        .features = BIT(DPU_MDP_PERIPH_0_REMOVED),
-       .highest_bank_bit = 0x3, /* TODO: 2 for LP_DDR4 */
-       .ubwc_swizzle = 0x6,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                        .reg_off = 0x4330, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_VIG1] = {
@@ -756,7 +796,6 @@ static const struct dpu_mdp_cfg qcm2290_mdp[] = {
        .name = "top_0", .id = MDP_TOP,
        .base = 0x0, .len = 0x494,
        .features = 0,
-       .highest_bank_bit = 0x2,
        .clk_ctrls[DPU_CLK_CTRL_VIG0] = {
                .reg_off = 0x2AC, .bit_off = 0},
        .clk_ctrls[DPU_CLK_CTRL_DMA0] = {
@@ -2530,6 +2569,7 @@ static const struct dpu_perf_cfg qcm2290_perf_data = {
 
 static const struct dpu_mdss_cfg msm8998_dpu_cfg = {
        .caps = &msm8998_dpu_caps,
+       .ubwc = &msm8998_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(msm8998_mdp),
        .mdp = msm8998_mdp,
        .ctl_count = ARRAY_SIZE(msm8998_ctl),
@@ -2553,6 +2593,7 @@ static const struct dpu_mdss_cfg msm8998_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sdm845_dpu_cfg = {
        .caps = &sdm845_dpu_caps,
+       .ubwc = &sdm845_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sdm845_mdp),
        .mdp = sdm845_mdp,
        .ctl_count = ARRAY_SIZE(sdm845_ctl),
@@ -2577,6 +2618,7 @@ static const struct dpu_mdss_cfg sdm845_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sc7180_dpu_cfg = {
        .caps = &sc7180_dpu_caps,
+       .ubwc = &sc7180_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sc7180_mdp),
        .mdp = sc7180_mdp,
        .ctl_count = ARRAY_SIZE(sc7180_ctl),
@@ -2603,6 +2645,7 @@ static const struct dpu_mdss_cfg sc7180_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sm6115_dpu_cfg = {
        .caps = &sm6115_dpu_caps,
+       .ubwc = &sm6115_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sm6115_mdp),
        .mdp = sm6115_mdp,
        .ctl_count = ARRAY_SIZE(qcm2290_ctl),
@@ -2625,6 +2668,7 @@ static const struct dpu_mdss_cfg sm6115_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sm8150_dpu_cfg = {
        .caps = &sm8150_dpu_caps,
+       .ubwc = &sm8150_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sdm845_mdp),
        .mdp = sdm845_mdp,
        .ctl_count = ARRAY_SIZE(sm8150_ctl),
@@ -2653,6 +2697,7 @@ static const struct dpu_mdss_cfg sm8150_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sc8180x_dpu_cfg = {
        .caps = &sc8180x_dpu_caps,
+       .ubwc = &sc8180x_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sc8180x_mdp),
        .mdp = sc8180x_mdp,
        .ctl_count = ARRAY_SIZE(sm8150_ctl),
@@ -2677,6 +2722,7 @@ static const struct dpu_mdss_cfg sc8180x_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sc8280xp_dpu_cfg = {
        .caps = &sc8280xp_dpu_caps,
+       .ubwc = &sc8280xp_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sc8280xp_mdp),
        .mdp = sc8280xp_mdp,
        .ctl_count = ARRAY_SIZE(sc8280xp_ctl),
@@ -2703,6 +2749,7 @@ static const struct dpu_mdss_cfg sc8280xp_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sm8250_dpu_cfg = {
        .caps = &sm8250_dpu_caps,
+       .ubwc = &sm8250_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sm8250_mdp),
        .mdp = sm8250_mdp,
        .ctl_count = ARRAY_SIZE(sm8150_ctl),
@@ -2733,6 +2780,7 @@ static const struct dpu_mdss_cfg sm8250_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sm8350_dpu_cfg = {
        .caps = &sm8350_dpu_caps,
+       .ubwc = &sm8350_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sm8350_mdp),
        .mdp = sm8350_mdp,
        .ctl_count = ARRAY_SIZE(sm8350_ctl),
@@ -2759,6 +2807,7 @@ static const struct dpu_mdss_cfg sm8350_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sm8450_dpu_cfg = {
        .caps = &sm8450_dpu_caps,
+       .ubwc = &sm8450_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sm8450_mdp),
        .mdp = sm8450_mdp,
        .ctl_count = ARRAY_SIZE(sm8450_ctl),
@@ -2785,6 +2834,7 @@ static const struct dpu_mdss_cfg sm8450_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sm8550_dpu_cfg = {
        .caps = &sm8550_dpu_caps,
+       .ubwc = &sm8550_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sm8550_mdp),
        .mdp = sm8550_mdp,
        .ctl_count = ARRAY_SIZE(sm8550_ctl),
@@ -2811,6 +2861,7 @@ static const struct dpu_mdss_cfg sm8550_dpu_cfg = {
 
 static const struct dpu_mdss_cfg sc7280_dpu_cfg = {
        .caps = &sc7280_dpu_caps,
+       .ubwc = &sc7280_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(sc7280_mdp),
        .mdp = sc7280_mdp,
        .ctl_count = ARRAY_SIZE(sc7280_ctl),
@@ -2833,6 +2884,7 @@ static const struct dpu_mdss_cfg sc7280_dpu_cfg = {
 
 static const struct dpu_mdss_cfg qcm2290_dpu_cfg = {
        .caps = &qcm2290_dpu_caps,
+       .ubwc = &qcm2290_ubwc_cfg,
        .mdp_count = ARRAY_SIZE(qcm2290_mdp),
        .mdp = qcm2290_mdp,
        .ctl_count = ARRAY_SIZE(qcm2290_ctl),

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h
index fe360fb7c77be27709a027704a743ca7b368eb31..00c3c67dd26732ebcb2477d78f9ec978f874aaa2 100644 (file)
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h
@@ -400,7 +400,6 @@ struct dpu_rotation_cfg {
  * @max_mixer_blendstages max layer mixer blend stages or
  *                       supported z order
  * @qseed_type         qseed2 or qseed3 support.
- * @ubwc_version       UBWC feature version (0x0 for not supported)
  * @has_src_split      source split feature status
  * @has_dim_layer      dim layer feature status
  * @has_idle_pc        indicate if idle power collapse feature is supported
@@ -414,7 +413,6 @@ struct dpu_caps {
        u32 max_mixer_width;
        u32 max_mixer_blendstages;
        u32 qseed_type;
-       u32 ubwc_version;
        bool has_src_split;
        bool has_dim_layer;
        bool has_idle_pc;
@@ -543,15 +541,24 @@ struct dpu_clk_ctrl_reg {
  * @id:                index identifying this block
  * @base:              register base offset to mdss
  * @features           bit mask identifying sub-blocks/features
- * @highest_bank_bit:  UBWC parameter
- * @ubwc_swizzle:      ubwc default swizzle setting
  * @clk_ctrls          clock control register definition
  */
 struct dpu_mdp_cfg {
        DPU_HW_BLK_INFO;
+       struct dpu_clk_ctrl_reg clk_ctrls[DPU_CLK_CTRL_MAX];
+};
+
+/**
+ * struct dpu_ubwc_cfg - UBWC and memory configuration
+ *
+ * @ubwc_version       UBWC feature version (0x0 for not supported)
+ * @highest_bank_bit:  UBWC parameter
+ * @ubwc_swizzle:      ubwc default swizzle setting
+ */
+struct dpu_ubwc_cfg {
+       u32 ubwc_version;
        u32 highest_bank_bit;
        u32 ubwc_swizzle;
-       struct dpu_clk_ctrl_reg clk_ctrls[DPU_CLK_CTRL_MAX];
 };
 
 /* struct dpu_ctl_cfg : MDP CTL instance info
@@ -853,6 +860,8 @@ struct dpu_perf_cfg {
 struct dpu_mdss_cfg {
        const struct dpu_caps *caps;
 
+       const struct dpu_ubwc_cfg *ubwc;
+
        u32 mdp_count;
        const struct dpu_mdp_cfg *mdp;
 

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c
index 6e5b62f3276f955dc7e4c3f142c0d8d2b850bdc5..cf70a9bd1034f08654e9e27afcce752136db0ca7 100644 (file)
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c
@@ -307,25 +307,25 @@ static void dpu_hw_sspp_setup_format(struct dpu_sw_pipe *pipe,
                src_format |= (fmt->fetch_mode & 3) << 30; /*FRAME_FORMAT */
                DPU_REG_WRITE(c, SSPP_FETCH_CONFIG,
                        DPU_FETCH_CONFIG_RESET_VALUE |
-                       ctx->mdp->highest_bank_bit << 18);
-               switch (ctx->catalog->caps->ubwc_version) {
+                       ctx->ubwc->highest_bank_bit << 18);
+               switch (ctx->ubwc->ubwc_version) {
                case DPU_HW_UBWC_VER_10:
                        fast_clear = fmt->alpha_enable ? BIT(31) : 0;
                        DPU_REG_WRITE(c, SSPP_UBWC_STATIC_CTRL,
-                                       fast_clear | (ctx->mdp->ubwc_swizzle & 0x1) |
+                                       fast_clear | (ctx->ubwc->ubwc_swizzle & 0x1) |
                                        BIT(8) |
-                                       (ctx->mdp->highest_bank_bit << 4));
+                                       (ctx->ubwc->highest_bank_bit << 4));
                        break;
                case DPU_HW_UBWC_VER_20:
                        fast_clear = fmt->alpha_enable ? BIT(31) : 0;
                        DPU_REG_WRITE(c, SSPP_UBWC_STATIC_CTRL,
-                                       fast_clear | (ctx->mdp->ubwc_swizzle) |
-                                       (ctx->mdp->highest_bank_bit << 4));
+                                       fast_clear | (ctx->ubwc->ubwc_swizzle) |
+                                       (ctx->ubwc->highest_bank_bit << 4));
                        break;
                case DPU_HW_UBWC_VER_30:
                        DPU_REG_WRITE(c, SSPP_UBWC_STATIC_CTRL,
-                                       BIT(30) | (ctx->mdp->ubwc_swizzle) |
-                                       (ctx->mdp->highest_bank_bit << 4));
+                                       BIT(30) | (ctx->ubwc->ubwc_swizzle) |
+                                       (ctx->ubwc->highest_bank_bit << 4));
                        break;
                case DPU_HW_UBWC_VER_40:
                        DPU_REG_WRITE(c, SSPP_UBWC_STATIC_CTRL,
@@ -813,7 +813,7 @@ struct dpu_hw_sspp *dpu_hw_sspp_init(enum dpu_sspp idx,
 
        /* Assign ops */
        hw_pipe->catalog = catalog;
-       hw_pipe->mdp = &catalog->mdp[0];
+       hw_pipe->ubwc = catalog->ubwc;
        hw_pipe->idx = idx;
        hw_pipe->cap = cfg;
        _setup_layer_ops(hw_pipe, hw_pipe->cap->features);

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h
index e73d6ac863ad5bf1487e89edc71d2d015f4b60e7..74b98b6b3bc3a19151ae0834079d0237847686d9 100644 (file)
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h
@@ -342,7 +342,7 @@ struct dpu_hw_sspp_ops {
  * @base: hardware block base structure
  * @hw: block hardware details
  * @catalog: back pointer to catalog
- * @mdp: pointer to associated mdp portion of the catalog
+ * @ubwc: ubwc configuration data
  * @idx: pipe index
  * @cap: pointer to layer_cfg
  * @ops: pointer to operations possible for this pipe
@@ -351,7 +351,7 @@ struct dpu_hw_sspp {
        struct dpu_hw_blk base;
        struct dpu_hw_blk_reg_map hw;
        const struct dpu_mdss_cfg *catalog;
-       const struct dpu_mdp_cfg *mdp;
+       const struct dpu_ubwc_cfg *ubwc;
 
        /* Pipe */
        enum dpu_sspp idx;