From cab061d263d9072f56f78a725db71221907b26ec Mon Sep 17 00:00:00 2001 From: Damon Ding Date: Thu, 17 Oct 2024 10:41:17 +0800 Subject: [PATCH] drm/rockchip: fbdev: replace rockchip_gem_mmap_buf() by drm_gem_mmap_obj() in .fb_mmp() The rockchip_gem_mmap_buf() will call .mmap() of struct drm_gem_object_funcs twice; One is: rockchip_gem_mmap_buf() -> drm_gem_mmap_obj() -> rockchip_gem_object_funcs.mmap() -> rockchip_drm_gem_object_mmap() The other is: rockchip_gem_mmap_buf()-> rockchip_drm_gem_object_mmap() This will cause the -EBUSY return of vm_map_pages() ultimately. Change-Id: I3de9eb23f81afa437087e2276d16dd8fb4c003d6 Signed-off-by: Damon Ding --- drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c b/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c index bd449be5b9c0..8123df9dd696 100644 --- a/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c +++ b/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c @@ -22,7 +22,7 @@ static int rockchip_fbdev_mmap(struct fb_info *info, struct drm_fb_helper *helper = info->par; struct rockchip_drm_private *private = helper->dev->dev_private; - return rockchip_gem_mmap_buf(private->fbdev_bo, vma); + return drm_gem_mmap_obj(private->fbdev_bo, private->fbdev_bo->size, vma); } static const struct fb_ops rockchip_drm_fbdev_ops = {