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

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?

•   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?