On Tue, Dec 23, 2025 at 10:26:03PM +0200, Dmitry Baryshkov wrote: > On Mon, Dec 22, 2025 at 06:23:58PM +0800, yuanjie yang wrote: > > From: Yuanjie Yang <[email protected]> > > > > DPU version 13.0.0 introduces structural changes including > > register additions, removals, and relocations. > > > > Refactor SSPP-related code to be compatible with DPU 13.0.0 > > modifications. > > > > Co-developed-by: Yongxing Mou <[email protected]> > > Signed-off-by: Yongxing Mou <[email protected]> > > Signed-off-by: Yuanjie Yang <[email protected]> > > --- > > .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 14 +++ > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c | 110 ++++++++++-------- > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h | 52 +++++++++ > > 3 files changed, 130 insertions(+), 46 deletions(-) > > > > 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 336757103b5a..b03fea1b9cbd 100644 > > --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h > > +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h > > @@ -208,6 +208,18 @@ struct dpu_dsc_blk { > > u32 len; > > }; > > > > +/** > > + * struct dpu_sspp_v13_rec_blk - SSPP REC sub-blk information > > + * @name: string name for debug purposes > > + * @base: offset of this sub-block relative to the block offset > > + * @len: register block length of this sub-block > > + */ > > +struct dpu_sspp_v13_rec_blk { > > + char name[DPU_HW_BLK_NAME_LEN]; > > + u32 base; > > + u32 len; > > +}; > > + > > /** > > * enum dpu_qos_lut_usage - define QoS LUT use cases > > */ > > @@ -294,6 +306,8 @@ struct dpu_sspp_sub_blks { > > u32 qseed_ver; > > struct dpu_scaler_blk scaler_blk; > > struct dpu_pp_blk csc_blk; > > + struct dpu_sspp_v13_rec_blk sspp_rec0_blk; > > + struct dpu_sspp_v13_rec_blk sspp_rec1_blk; > > This is a refactoring patch. Don't add fields (and structs) which are > not used inside the patch.
OK, will add in Kaanapali SSPP v13 patch in next patch. Thanks, Yuanjie > > > > const u32 *format_list; > > u32 num_formats; > > Other than that, LGTM. > > -- > With best wishes > Dmitry
