On Fri, 4 Aug 2023 at 00:13, Danila Tikhonov <danila@xxxxxxxxxxx> 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). > > > +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