Re: [PATCH 3/3] drm/msm: don't store created planes, connectors and encoders

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 6 Apr 2022 at 07:03, Stephen Boyd <swboyd@xxxxxxxxxxxx> wrote:
>
> Quoting Dmitry Baryshkov (2022-04-05 16:50:59)
> > diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
> > index d661debb50f1..ee3093890d97 100644
> > --- a/drivers/gpu/drm/msm/msm_drv.h
> > +++ b/drivers/gpu/drm/msm/msm_drv.h
> > @@ -184,23 +184,14 @@ struct msm_drm_private {
> >
> >         struct workqueue_struct *wq;
> >
> > -       unsigned int num_planes;
> > -       struct drm_plane *planes[MAX_PLANES];
>
> Can we get rid of MAX_PLANES?

Sure, I'll drop all these defines in v2.

>
> > -
> >         unsigned int num_crtcs;
> >         struct drm_crtc *crtcs[MAX_CRTCS];
> >
> >         struct msm_drm_thread event_thread[MAX_CRTCS];
> >
> > -       unsigned int num_encoders;
> > -       struct drm_encoder *encoders[MAX_ENCODERS];
>
> And MAX_ENCODERS?
>
> > -
> >         unsigned int num_bridges;
> >         struct drm_bridge *bridges[MAX_BRIDGES];
> >
> > -       unsigned int num_connectors;
> > -       struct drm_connector *connectors[MAX_CONNECTORS];
>
> And MAX_CONNECTORS?



-- 
With best wishes
Dmitry



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux