On Thu, Aug 8, 2019 at 7:44 AM Gerd Hoffmann <kra...@redhat.com> wrote: > > > > Gerd Hoffmann (17): > drm/ttm: turn ttm_bo_device.vma_manager into a pointer > drm/ttm: add gem_ttm_bo_device_init() > drm/vram: switch vram helpers to the new gem_ttm_bo_device_init() > drm/qxl: switch qxl to the new gem_ttm_bo_device_init() > drm: add mmap() to drm_gem_object_funcs > drm/shmem: switch shmem helper to drm_gem_object_funcs->mmap > drm/shmem: drop DEFINE_DRM_GEM_SHMEM_FOPS > drm/ttm: factor out ttm_bo_mmap_vma_setup > drm/ttm: add drm_gem_ttm_mmap() > drm/vram: switch vram helper to drm_gem_object_funcs->mmap codepath > drm/vram: drop verify_access > drm: drop DRM_VRAM_MM_FILE_OPERATIONS > drm/qxl: use drm_gem_object_funcs > drm/qxl: drop qxl_ttm_fault > drm/qxl: switch qxl to drm_gem_object_funcs->mmap codepath > drm/qxl: drop verify_access > drm/qxl: use DEFINE_DRM_GEM_FOPS()
Perhaps instead of changing drivers to use DEFINE_DRM_GEM_FOPS, make that the default if .fops is NULL (and perhaps also if the driver sets DRIVER_GEM). That would be in line with other recent rework making various function ptrs optional. Rob _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
