Messages in this thread | | | Date | Mon, 9 Jan 2023 01:29:15 +0200 | Subject | Re: [PATCH v2 5/8] drm/msm/dpu: Reject topologies for which no DSC blocks are available | From | Dmitry Baryshkov <> |
| |
On 22/12/2022 01:19, Marijn Suijten wrote: > Resource allocation of DSC blocks should behave more like LMs and CTLs > where NULL resources (based on initial hw_blk creation via definitions > in the catalog) are skipped ^1. The current hardcoded mapping of DSC > blocks however means that resource allocation shouldn't succeed at all > when the DSC block on the corresponding index doesn't exist, rather than > searching for the next free block. > > This hardcoded mapping should be loosened separately as DPU 5.0.0 > introduced a crossbar where DSC blocks can be "somewhat" freely bound to > any PP and CTL (in proper pairs). > > ^1: which, on hardware that supports DSC, can happen after a git rebase > ended up moving additions to _dpu_cfg to a different struct which has > the same patch context. > > Fixes: f2803ee91a41 ("drm/msm/disp/dpu1: Add DSC support in RM") > Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org> > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 5 +++++ > 1 file changed, 5 insertions(+)
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
-- With best wishes Dmitry
| |