From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from galahad.ideasonboard.com ([185.26.127.97]:33400 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752512AbeCTLXV (ORCPT ); Tue, 20 Mar 2018 07:23:21 -0400 From: Laurent Pinchart To: Ulrich Hecht Cc: intel-gfx@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org Subject: Re: [PATCH igt 1/8] tests/kms_addfb_basic: skip i915-specific tests on other platforms Date: Tue, 20 Mar 2018 13:24:26 +0200 Message-ID: <1570444.QHh5qsClPT@avalon> In-Reply-To: <1521125144-28614-2-git-send-email-ulrich.hecht+renesas@gmail.com> References: <1521125144-28614-1-git-send-email-ulrich.hecht+renesas@gmail.com> <1521125144-28614-2-git-send-email-ulrich.hecht+renesas@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: Hi Ulrich, Thank you for the patch. On Thursday, 15 March 2018 16:45:37 EET Ulrich Hecht wrote: > Add is_i915_device() requirement to tests using Intel-specific APIs. > > Signed-off-by: Ulrich Hecht Reviewed-by: Laurent Pinchart > --- > tests/kms_addfb_basic.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/tests/kms_addfb_basic.c b/tests/kms_addfb_basic.c > index 7d8852f..cf9ba37 100644 > --- a/tests/kms_addfb_basic.c > +++ b/tests/kms_addfb_basic.c > @@ -104,6 +104,7 @@ static void invalid_tests(int fd) > } > > igt_subtest("clobberred-modifier") { > + igt_require(is_i915_device(fd)); > f.flags = 0; > f.modifier[0] = 0; > gem_set_tiling(fd, gem_bo, I915_TILING_X, 512*4); > @@ -318,6 +319,7 @@ static void size_tests(int fd) > } > > igt_subtest("bo-too-small-due-to-tiling") { > + igt_require(is_i915_device(fd)); > gem_set_tiling(fd, gem_bo_small, I915_TILING_X, 1024*4); > igt_assert(drmIoctl(fd, DRM_IOCTL_MODE_ADDFB2, &f) == -1 && > errno == EINVAL); > @@ -369,6 +371,7 @@ static void addfb25_tests(int fd) > > igt_subtest_group { > igt_fixture { > + igt_require(is_i915_device(fd)); > gem_set_tiling(fd, gem_bo, I915_TILING_X, 1024*4); > igt_require_fb_modifiers(fd); > } -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH igt 1/8] tests/kms_addfb_basic: skip i915-specific tests on other platforms Date: Tue, 20 Mar 2018 13:24:26 +0200 Message-ID: <1570444.QHh5qsClPT@avalon> References: <1521125144-28614-1-git-send-email-ulrich.hecht+renesas@gmail.com> <1521125144-28614-2-git-send-email-ulrich.hecht+renesas@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from galahad.ideasonboard.com (galahad.ideasonboard.com [185.26.127.97]) by gabe.freedesktop.org (Postfix) with ESMTPS id 8468489852 for ; Tue, 20 Mar 2018 11:23:21 +0000 (UTC) In-Reply-To: <1521125144-28614-2-git-send-email-ulrich.hecht+renesas@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Ulrich Hecht Cc: linux-renesas-soc@vger.kernel.org, intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org SGkgVWxyaWNoLAoKVGhhbmsgeW91IGZvciB0aGUgcGF0Y2guCgpPbiBUaHVyc2RheSwgMTUgTWFy Y2ggMjAxOCAxNjo0NTozNyBFRVQgVWxyaWNoIEhlY2h0IHdyb3RlOgo+IEFkZCBpc19pOTE1X2Rl dmljZSgpIHJlcXVpcmVtZW50IHRvIHRlc3RzIHVzaW5nIEludGVsLXNwZWNpZmljIEFQSXMuCj4g Cj4gU2lnbmVkLW9mZi1ieTogVWxyaWNoIEhlY2h0IDx1bHJpY2guaGVjaHQrcmVuZXNhc0BnbWFp bC5jb20+CgpSZXZpZXdlZC1ieTogTGF1cmVudCBQaW5jaGFydCA8bGF1cmVudC5waW5jaGFydEBp ZGVhc29uYm9hcmQuY29tPgoKPiAtLS0KPiAgdGVzdHMva21zX2FkZGZiX2Jhc2ljLmMgfCAzICsr Kwo+ICAxIGZpbGUgY2hhbmdlZCwgMyBpbnNlcnRpb25zKCspCj4gCj4gZGlmZiAtLWdpdCBhL3Rl c3RzL2ttc19hZGRmYl9iYXNpYy5jIGIvdGVzdHMva21zX2FkZGZiX2Jhc2ljLmMKPiBpbmRleCA3 ZDg4NTJmLi5jZjliYTM3IDEwMDY0NAo+IC0tLSBhL3Rlc3RzL2ttc19hZGRmYl9iYXNpYy5jCj4g KysrIGIvdGVzdHMva21zX2FkZGZiX2Jhc2ljLmMKPiBAQCAtMTA0LDYgKzEwNCw3IEBAIHN0YXRp YyB2b2lkIGludmFsaWRfdGVzdHMoaW50IGZkKQo+ICAJfQo+IAo+ICAJaWd0X3N1YnRlc3QoImNs b2JiZXJyZWQtbW9kaWZpZXIiKSB7Cj4gKwkJaWd0X3JlcXVpcmUoaXNfaTkxNV9kZXZpY2UoZmQp KTsKPiAgCQlmLmZsYWdzID0gMDsKPiAgCQlmLm1vZGlmaWVyWzBdID0gMDsKPiAgCQlnZW1fc2V0 X3RpbGluZyhmZCwgZ2VtX2JvLCBJOTE1X1RJTElOR19YLCA1MTIqNCk7Cj4gQEAgLTMxOCw2ICsz MTksNyBAQCBzdGF0aWMgdm9pZCBzaXplX3Rlc3RzKGludCBmZCkKPiAgCX0KPiAKPiAgCWlndF9z dWJ0ZXN0KCJiby10b28tc21hbGwtZHVlLXRvLXRpbGluZyIpIHsKPiArCQlpZ3RfcmVxdWlyZShp c19pOTE1X2RldmljZShmZCkpOwo+ICAJCWdlbV9zZXRfdGlsaW5nKGZkLCBnZW1fYm9fc21hbGws IEk5MTVfVElMSU5HX1gsIDEwMjQqNCk7Cj4gIAkJaWd0X2Fzc2VydChkcm1Jb2N0bChmZCwgRFJN X0lPQ1RMX01PREVfQURERkIyLCAmZikgPT0gLTEgJiYKPiAgCQkJICAgZXJybm8gPT0gRUlOVkFM KTsKPiBAQCAtMzY5LDYgKzM3MSw3IEBAIHN0YXRpYyB2b2lkIGFkZGZiMjVfdGVzdHMoaW50IGZk KQo+IAo+ICAJaWd0X3N1YnRlc3RfZ3JvdXAgewo+ICAJCWlndF9maXh0dXJlIHsKPiArCQkJaWd0 X3JlcXVpcmUoaXNfaTkxNV9kZXZpY2UoZmQpKTsKPiAgCQkJZ2VtX3NldF90aWxpbmcoZmQsIGdl bV9ibywgSTkxNV9USUxJTkdfWCwgMTAyNCo0KTsKPiAgCQkJaWd0X3JlcXVpcmVfZmJfbW9kaWZp ZXJzKGZkKTsKPiAgCQl9CgoKLS0gClJlZ2FyZHMsCgpMYXVyZW50IFBpbmNoYXJ0CgpfX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGlu ZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVl ZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK