FBDEV ggtt is not restored correctly, add missing GGTT flag to 
intel_fbdev_fb_alloc to make it work.
This ensures that the global GGTT mapping is always restored on resume. The 
GGTT mapping would
otherwise be created in intel_fb_pin_to_ggtt() by intel_fbdev anyway.

This fixes the fbdev device not working after resume.

Fixes: 67a98f7e27ba ("drm/xe/display: Re-use display vmas when possible")
Signed-off-by: Maarten Lankhorst <d...@lankhorst.se>
Cc: Lucas De Marchi <lucas.demar...@intel.com>
---
 drivers/gpu/drm/xe/display/intel_fbdev_fb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/intel_fbdev_fb.c 
b/drivers/gpu/drm/xe/display/intel_fbdev_fb.c
index ca95fcd098ec7..3a1e505ff1820 100644
--- a/drivers/gpu/drm/xe/display/intel_fbdev_fb.c
+++ b/drivers/gpu/drm/xe/display/intel_fbdev_fb.c
@@ -45,7 +45,7 @@ struct intel_framebuffer *intel_fbdev_fb_alloc(struct 
drm_fb_helper *helper,
                                           NULL, size,
                                           ttm_bo_type_kernel, 
XE_BO_FLAG_SCANOUT |
                                           XE_BO_FLAG_STOLEN |
-                                          XE_BO_FLAG_PINNED);
+                                          XE_BO_FLAG_GGTT | XE_BO_FLAG_PINNED);
                if (!IS_ERR(obj))
                        drm_info(&xe->drm, "Allocated fbdev into stolen\n");
                else
@@ -56,7 +56,7 @@ struct intel_framebuffer *intel_fbdev_fb_alloc(struct 
drm_fb_helper *helper,
                obj = xe_bo_create_pin_map(xe, xe_device_get_root_tile(xe), 
NULL, size,
                                           ttm_bo_type_kernel, 
XE_BO_FLAG_SCANOUT |
                                           
XE_BO_FLAG_VRAM_IF_DGFX(xe_device_get_root_tile(xe)) |
-                                          XE_BO_FLAG_PINNED);
+                                          XE_BO_FLAG_GGTT | XE_BO_FLAG_PINNED);
        }
 
        if (IS_ERR(obj)) {
-- 
2.45.2

Reply via email to