On Fri, Jan 09, 2026 at 10:33:52AM +0800, yuanjiey wrote: > On Thu, Jan 08, 2026 at 03:46:18PM +0200, Dmitry Baryshkov wrote: > > On Thu, Jan 08, 2026 at 04:56:57PM +0800, yuanjie yang wrote: > > > From: Yuanjie Yang <[email protected]> > > > > > > Add support for Kaanapali platform SSPP sub-blocks, which > > > introduce structural changes including register additions, > > > removals, and relocations. Add the new common and rectangle > > > blocks, and update register definitions and handling to > > > ensure compatibility with DPU v13.0. > > > > > > Co-developed-by: Yongxing Mou <[email protected]> > > > Signed-off-by: Yongxing Mou <[email protected]> > > > Signed-off-by: Yuanjie Yang <[email protected]> > > > --- > > > drivers/gpu/drm/msm/Makefile | 1 + > > > .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 14 + > > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c | 14 +- > > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h | 4 + > > > .../gpu/drm/msm/disp/dpu1/dpu_hw_sspp_v13.c | 321 ++++++++++++++++++ > > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.c | 18 + > > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h | 3 + > > > 7 files changed, 371 insertions(+), 4 deletions(-) > > > create mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp_v13.c
> > > > + c->ops.setup_sourceaddress = > > > dpu_hw_sspp_setup_sourceaddress_v13; > > > + c->ops.setup_solidfill = dpu_hw_sspp_setup_solidfill_v13; > > > + c->ops.setup_pe = dpu_hw_sspp_setup_pe_config_v13; > > > + > > > > > /* QOS_QOS_CTRL */ > > > #define QOS_QOS_CTRL_DANGER_SAFE_EN BIT(0) > > > #define QOS_QOS_CTRL_DANGER_VBLANK_MASK GENMASK(5, 4) > > > @@ -475,6 +482,17 @@ void _dpu_hw_setup_qos_lut(struct dpu_hw_blk_reg_map > > > *c, u32 offset, > > > cfg->danger_safe_en ? QOS_QOS_CTRL_DANGER_SAFE_EN : 0); > > > } > > > > > > +void _dpu_hw_setup_qos_lut_v13(struct dpu_hw_blk_reg_map *c, > > > + const struct dpu_hw_qos_cfg *cfg) > > > > Will it be shared with other blocks (DS, WB)? > > yes, _dpu_hw_setup_qos_lut_v13 will be used in WB, in function > dpu_hw_wb_setup_qos_lut_v13. > > So maybe I should drop '_' rename _dpu_hw_setup_qos_lut_v13 --> > dpu_hw_setup_qos_lut_v13 ? Yes, please. As you are going to resend the series, could you please also remove the function from the header? -- With best wishes Dmitry
