drm/moc: Mark legacy fields in drm_driver as such
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 25 Jan 2017 06:26:54 +0000 (07:26 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 26 Jan 2017 09:46:00 +0000 (10:46 +0100)
No point in documenting these, they only confuse.

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170125062657.19270-13-daniel.vetter@ffwll.ch
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
include/drm/drm_drv.h

index 8cb937b..567bc93 100644 (file)
@@ -679,7 +679,6 @@ static struct drm_driver kms_driver = {
            DRIVER_USE_AGP |
            DRIVER_HAVE_IRQ | DRIVER_IRQ_SHARED | DRIVER_GEM |
            DRIVER_PRIME | DRIVER_RENDER | DRIVER_MODESET,
-       .dev_priv_size = 0,
        .load = amdgpu_driver_load_kms,
        .open = amdgpu_driver_open_kms,
        .preclose = amdgpu_driver_preclose_kms,
index 8391135..732e856 100644 (file)
@@ -81,7 +81,6 @@ struct drm_driver {
         * Zero on success, non-zero value on failure.
         */
        int (*load) (struct drm_device *, unsigned long flags);
-       int (*firstopen) (struct drm_device *);
        int (*open) (struct drm_device *, struct drm_file *);
        void (*preclose) (struct drm_device *, struct drm_file *file_priv);
        void (*postclose) (struct drm_device *, struct drm_file *);
@@ -103,9 +102,6 @@ struct drm_driver {
         *
         */
        void (*unload) (struct drm_device *);
-       int (*dma_ioctl) (struct drm_device *dev, void *data, struct drm_file *file_priv);
-       int (*dma_quiescent) (struct drm_device *);
-       int (*context_dtor) (struct drm_device *dev, int context);
        int (*set_busid)(struct drm_device *dev, struct drm_master *master);
 
        /**
@@ -413,13 +409,20 @@ struct drm_driver {
        char *date;
 
        u32 driver_features;
-       int dev_priv_size;
        const struct drm_ioctl_desc *ioctls;
        int num_ioctls;
        const struct file_operations *fops;
 
+       /* Everything below here is for legacy driver, never use! */
+       /* private: */
+
        /* List of devices hanging off this driver with stealth attach. */
        struct list_head legacy_dev_list;
+       int (*firstopen) (struct drm_device *);
+       int (*dma_ioctl) (struct drm_device *dev, void *data, struct drm_file *file_priv);
+       int (*dma_quiescent) (struct drm_device *);
+       int (*context_dtor) (struct drm_device *dev, int context);
+       int dev_priv_size;
 };
 
 extern __printf(6, 7)