Pantherlake intergrated graphics can have the XE_GGTT_PTEDM bit set
in the BIOS-initialised framebuffer PTE. The new validation rejects
this and fails to create the initial frambuffer, resulting in static.

Remove the too-strict validation.

Signed-off-by: Andrew Zhou <[email protected]>
Tested-by: Andrew Zhou <[email protected]>
Tested-on: Intel Core Ultra X7 358H, integrated graphics, B0 stepping
---
 drivers/gpu/drm/xe/display/xe_initial_plane.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c 
b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index 0f86b7303..8c95dc4e6 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -85,7 +85,6 @@ initial_plane_bo(struct xe_device *xe,
                            &phys_base);
        } else {
                struct ttm_resource_manager *stolen;
-               u64 pte;
 
                stolen = ttm_manager_type(&xe->ttm, XE_PL_STOLEN);
                if (!stolen) {
@@ -93,13 +92,6 @@ initial_plane_bo(struct xe_device *xe,
                        return NULL;
                }
 
-               pte = xe_ggtt_read_pte(tile0->mem.ggtt, base);
-
-               if (is_pte_local(pte) != need_pte_local(xe)) {
-                       drm_err(&xe->drm, "Initial plane PTE has bad local 
memory bit\n");
-                       return NULL;
-               }
-
                phys_base = base;
                flags |= XE_BO_FLAG_STOLEN;
 
-- 
2.55.0

Reply via email to