Hi Emil
Am 22.05.20 um 20:11 schrieb Emil Velikov:
Hi Thomas,
On Fri, 22 May 2020 at 14:53, Thomas Zimmermann tzimmermann@suse.de wrote:
The kirin driver uses the default implementation for CMA functions; except for the .dumb_create callback. The __DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults and .dumb_create in struct drm_driver. All remaining operations are provided by CMA GEM object functions.
Signed-off-by: Thomas Zimmermann tzimmermann@suse.de
drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c b/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c index c339e632522a9..b1ffd7d43e562 100644 --- a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c +++ b/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c @@ -921,17 +921,7 @@ DEFINE_DRM_GEM_CMA_FOPS(ade_fops); static struct drm_driver ade_driver = { .driver_features = DRIVER_GEM | DRIVER_MODESET | DRIVER_ATOMIC, .fops = &ade_fops,
.gem_free_object_unlocked = drm_gem_cma_free_object,.gem_vm_ops = &drm_gem_cma_vm_ops,.dumb_create = drm_gem_cma_dumb_create_internal,This doesn't seem right. The _internal documentation explicitly says that this should _not_ be used as .dumb_create. Instead drivers should use it to implement their callback.
Since it yields the same result as drm_gem_cma_dumb_create we can use the default macro below.
I noticed this and thought that the driver authors probably had their reasons. Changing the driver to the default macro is probably still a good idea.
Best regards Thomas
Weather to the .dumb_create in separate patch, or squash it here - I'll leave to you. In case of the latter, please mentioned it in the commit message.
-Emil _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel