Re: [PATCH 2/2] drm/msm/dpu: Add SM7150 support

On Fri, 4 Aug 2023 at 00:13, Danila Tikhonov [off-list ref] wrote:

So here too I add new sm7150_vig_sblk_0 and sm7150_vig_sblk_1 with v3lite?

static const struct dpu_sspp_sub_blks sm7150_vig_sblk_0 =
    _VIG_SBLK(5, DPU_SSPP_SCALER_QSEED3LITE);
static const struct dpu_sspp_sub_blks sm7150_vig_sblk_1 =
    _VIG_SBLK(6, DPU_SSPP_SCALER_QSEED3LITE);

Yes, please, close to other sblk variables.

But the priorities should be 4 and 5 (and 1, 2, 3 for DMA).

quoted

+static const struct dpu_sspp_cfg sm7150_sspp[] = {
+ {
+ .name = "sspp_0", .id = SSPP_VIG0,
+ .base = 0x4000, .len = 0x1f0,
+ .features = VIG_SDM845_MASK,
- .sblk = &sm8250_vig_sblk_0, // &sm7150_vig_sblk_0
+ .xin_id = 0,
+ .type = SSPP_TYPE_VIG,
+ .clk_ctrl = DPU_CLK_CTRL_VIG0,
+ }, {
+ .name = "sspp_1", .id = SSPP_VIG1,
+ .base = 0x6000, .len = 0x1f0,
+ .features = VIG_SDM845_MASK,
+ .sblk = &sm8250_vig_sblk_1,  // &sm7150_vig_sblk_1
+ .xin_id = 4,
+ .type = SSPP_TYPE_VIG,
+ .clk_ctrl = DPU_CLK_CTRL_VIG1,
+ }, {

--
Best regards,
Danila

-- 
With best wishes
Dmitry