[Intel-gfx] [PATCH 07/26] drm/amdgpu: Use drm_fb_helper_fill_info

[Daniel Vetter]

Should not cause any changes.

Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
Cc: Alex Deucher <alexander.deuc...@amd.com>
Cc: "Christian König" <christian.koe...@amd.com>
Cc: Samuel Li <samuel...@amd.com>
Cc: "Michel Dänzer" <michel.daen...@amd.com>
Cc: Huang Rui <ray.hu...@amd.com>
Cc: Junwei Zhang <jerry.zh...@amd.com>
Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
Cc: Daniel Stone <dani...@collabora.com>
Cc: Shirish S <shiris...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c | 24 ++++++++----------------
 1 file changed, 8 insertions(+), 16 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
index 24890d8f9ee4..dd37170af42b 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
@@ -49,12 +49,11 @@
 static int
 amdgpufb_open(struct fb_info *info, int user)
 {
-       struct amdgpu_fbdev *rfbdev = info->par;
-       struct amdgpu_device *adev = rfbdev->adev;
-       int ret = pm_runtime_get_sync(adev->ddev->dev);
+       struct drm_fb_helper *fb_helper = info->par;
+       int ret = pm_runtime_get_sync(fb_helper->dev->dev);
        if (ret < 0 && ret != -EACCES) {
-               pm_runtime_mark_last_busy(adev->ddev->dev);
-               pm_runtime_put_autosuspend(adev->ddev->dev);
+               pm_runtime_mark_last_busy(fb_helper->dev->dev);
+               pm_runtime_put_autosuspend(fb_helper->dev->dev);
                return ret;
        }
        return 0;
@@ -63,11 +62,10 @@ amdgpufb_open(struct fb_info *info, int user)
 static int
 amdgpufb_release(struct fb_info *info, int user)
 {
-       struct amdgpu_fbdev *rfbdev = info->par;
-       struct amdgpu_device *adev = rfbdev->adev;
+       struct drm_fb_helper *fb_helper = info->par;
 
-       pm_runtime_mark_last_busy(adev->ddev->dev);
-       pm_runtime_put_autosuspend(adev->ddev->dev);
+       pm_runtime_mark_last_busy(fb_helper->dev->dev);
+       pm_runtime_put_autosuspend(fb_helper->dev->dev);
        return 0;
 }
 
@@ -233,8 +231,6 @@ static int amdgpufb_create(struct drm_fb_helper *helper,
                goto out;
        }
 
-       info->par = rfbdev;
-
        ret = amdgpu_display_framebuffer_init(adev->ddev, &rfbdev->rfb,
                                              &mode_cmd, gobj);
        if (ret) {
@@ -247,10 +243,6 @@ static int amdgpufb_create(struct drm_fb_helper *helper,
        /* setup helper */
        rfbdev->helper.fb = fb;
 
-       strcpy(info->fix.id, "amdgpudrmfb");
-
-       drm_fb_helper_fill_fix(info, fb->pitches[0], fb->format->depth);
-
        info->fbops = &amdgpufb_ops;
 
        tmp = amdgpu_bo_gpu_offset(abo) - adev->gmc.vram_start;
@@ -259,7 +251,7 @@ static int amdgpufb_create(struct drm_fb_helper *helper,
        info->screen_base = amdgpu_bo_kptr(abo);
        info->screen_size = amdgpu_bo_size(abo);
 
-       drm_fb_helper_fill_var(info, &rfbdev->helper, sizes->fb_width, 
sizes->fb_height);
+       drm_fb_helper_fill_info(info, &rfbdev->helper);
 
        /* setup aperture base/size for vesafb takeover */
        info->apertures->ranges[0].base = adev->ddev->mode_config.fb_base;
-- 
2.20.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx