From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5E6586EB40 for ; Mon, 20 Jan 2020 22:03:29 +0000 (UTC) From: Imre Deak Date: Tue, 21 Jan 2020 00:03:20 +0200 Message-Id: <20200120220320.17092-3-imre.deak@intel.com> In-Reply-To: <20200120220320.17092-1-imre.deak@intel.com> References: <20200120220320.17092-1-imre.deak@intel.com> MIME-Version: 1.0 Subject: [igt-dev] [PATCH i-g-t 3/3] lib/igt_fb: Fix creating FBs on platforms w/o HW detiling List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: igt-dev@lists.freedesktop.org List-ID: On platforms w/o HW detiling don't fail creating the FB due to the expected error from the set_tiling IOCTL. Most of the tests use a cairo surface to draw, which don't depend on HW detiling. Other tests (using lib/igt_draw.c or drawing to the FB directly, like kms_draw_crc, kms_frontbuffer) are failing atm and will need to be fixed separately. Signed-off-by: Imre Deak --- lib/igt_fb.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/lib/igt_fb.c b/lib/igt_fb.c index b5b1750e..ccb76455 100644 --- a/lib/igt_fb.c +++ b/lib/igt_fb.c @@ -944,9 +944,10 @@ static int create_bo_for_fb(struct igt_fb *fb) if (is_i915_device(fd)) { fb->gem_handle = gem_create(fd, fb->size); - gem_set_tiling(fd, fb->gem_handle, - igt_fb_mod_to_tiling(fb->modifier), - fb->strides[0]); + if (gem_has_legacy_hw_tiling(fd)) + gem_set_tiling(fd, fb->gem_handle, + igt_fb_mod_to_tiling(fb->modifier), + fb->strides[0]); } else if (is_vc4_device(fd)) { fb->gem_handle = igt_vc4_create_bo(fd, fb->size); -- 2.23.1 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev