Re: [PATCH 1/2] drm/msm: add support for QCM2290 MDSS

[Loic Poulain <loic.poulain@linaro.org>]

Hi Dmitry,

On Tue, 18 Jan 2022 at 19:02, Dmitry Baryshkov
<dmitry.baryshkov@linaro.org> wrote:
>
> On 18/01/2022 18:47, Loic Poulain wrote:
> > Add compatibility for QCM2290 display subsystem, including
> > required entries in DPU hw catalog.
> >
> > Signed-off-by: Loic Poulain <loic.poulain@linaro.org>
> > ---
> >   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 175 ++++++++++++++++++++++++-
> >   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h |   1 +
> >   drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c        |   1 +
> >   drivers/gpu/drm/msm/msm_drv.c                  |   1 +
> >   4 files changed, 177 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
> > index ce6f32a..7fa3fc7 100644
> > --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
> > +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
> > @@ -25,6 +25,8 @@
> >   #define VIG_SM8250_MASK \
[...]
> > +static const struct dpu_vbif_cfg qcm2290_vbif[] = {
> > +     {
> > +     .name = "vbif_0", .id = VBIF_0,
> > +     .base = 0, .len = 0x1040,
> > +     .features = BIT(DPU_VBIF_QOS_REMAP),
> > +     .xin_halt_timeout = 0x4000,
> > +     .qos_rt_tbl = {
> > +             .npriority_lvl = ARRAY_SIZE(sdm845_rt_pri_lvl),
> > +             .priority_lvl = sdm845_rt_pri_lvl,
> > +             },
> > +     .memtype_count = 14,
> > +     .memtype = {3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3},
> > +     },
> > +};
>
> The only difference from sdm845_vbif is the lack of .qos_nrt_tbl. Is
> this on purpose?

Yes, I've not found any info related to non-rt for QCM2290 dpu, but I
assume it would be safe to just use sdm845_vbif here, as the others.

Regards,
Loic