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. > > const u32 *format_list; > u32 num_formats; Other than that, LGTM. -- With best wishes Dmitry
