From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maarten Lankhorst Subject: Re: [PATCH v4 6/6] drm/i915: Add skl_check_nv12_surface for NV12 Date: Wed, 18 Apr 2018 20:06:57 +0200 Message-ID: <9be96a32-cffa-5517-3363-461e882380be@linux.intel.com> References: <1524024493-23173-1-git-send-email-vidya.srinivas@intel.com> <1524024493-23173-7-git-send-email-vidya.srinivas@intel.com> <20180418153256.GV17795@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by gabe.freedesktop.org (Postfix) with ESMTPS id 083FF89CF8 for ; Wed, 18 Apr 2018 18:06:59 +0000 (UTC) In-Reply-To: <20180418153256.GV17795@intel.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= , Vidya Srinivas Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T3AgMTgtMDQtMTggb20gMTc6MzIgc2NocmVlZiBWaWxsZSBTeXJqw6Rsw6Q6Cj4gT24gV2VkLCBB cHIgMTgsIDIwMTggYXQgMDk6Mzg6MTNBTSArMDUzMCwgVmlkeWEgU3Jpbml2YXMgd3JvdGU6Cj4+ IEZyb206IE1hYXJ0ZW4gTGFua2hvcnN0IDxtYWFydGVuLmxhbmtob3JzdEBsaW51eC5pbnRlbC5j b20+Cj4+Cj4+IFdlIHNraXAgc3JjIHRydW5jdGlvbi9hZGp1c3RtZW50cyBmb3IKPj4gTlYxMiBj YXNlIGFuZCBoYW5kbGUgdGhlIHNpemVzIGRpcmVjdGx5Lgo+PiBXaXRob3V0IHRoaXMsIHBpcGUg ZmlmbyB1bmRlcnJ1bnMgYXJlIHNlZW4gb24gQVBML0tCTC4KPj4KPj4gdjI6IEZvciBOVjEyLCBt YWtpbmcgdGhlIHNyYyBjb29yZGluYXRlcyBtdWx0aXBsaWVyIG9mIDQKPj4KPj4gdjM6IE1vdmlu ZyBhbGwgdGhlIHNyYyBjb29yZHMgaGFuZGxpbmcgY29kZSBmb3IgTlYxMgo+PiB0byBza2xfY2hl Y2tfbnYxMl9zdXJmYWNlCj4+Cj4+IFNpZ25lZC1vZmYtYnk6IE1hYXJ0ZW4gTGFua2hvcnN0IDxt YWFydGVuLmxhbmtob3JzdEBsaW51eC5pbnRlbC5jb20+Cj4+IFNpZ25lZC1vZmYtYnk6IFZpZHlh IFNyaW5pdmFzIDx2aWR5YS5zcmluaXZhc0BpbnRlbC5jb20+Cj4+IC0tLQo+PiAgZHJpdmVycy9n cHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jIHwgMzkgKysrKysrKysrKysrKysrKysrKysrKysr KysrKysrKysrKysrCj4+ICBkcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9zcHJpdGUuYyAgfCAx NSArKysrKysrKysrLS0tLQo+PiAgMiBmaWxlcyBjaGFuZ2VkLCA1MCBpbnNlcnRpb25zKCspLCA0 IGRlbGV0aW9ucygtKQo+Pgo+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50 ZWxfZGlzcGxheS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jCj4+IGlu ZGV4IDkyNTQwMmUuLmI4ZGJhY2EgMTAwNjQ0Cj4+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1 L2ludGVsX2Rpc3BsYXkuYwo+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNw bGF5LmMKPj4gQEAgLTMxMTgsNiArMzExOCw0MiBAQCBzdGF0aWMgaW50IHNrbF9jaGVja19tYWlu X3N1cmZhY2UoY29uc3Qgc3RydWN0IGludGVsX2NydGNfc3RhdGUgKmNydGNfc3RhdGUsCj4+ICAJ cmV0dXJuIDA7Cj4+ICB9Cj4+ICAKPj4gK3N0YXRpYyBpbnQKPj4gK3NrbF9jaGVja19udjEyX3N1 cmZhY2UoY29uc3Qgc3RydWN0IGludGVsX2NydGNfc3RhdGUgKmNydGNfc3RhdGUsCj4+ICsJCSAg ICAgICBzdHJ1Y3QgaW50ZWxfcGxhbmVfc3RhdGUgKnBsYW5lX3N0YXRlKQo+PiArewo+PiArCWlu dCBjcnRjX3gyID0gcGxhbmVfc3RhdGUtPmJhc2UuY3J0Y194ICsgcGxhbmVfc3RhdGUtPmJhc2Uu Y3J0Y193Owo+PiArCWludCBjcnRjX3kyID0gcGxhbmVfc3RhdGUtPmJhc2UuY3J0Y195ICsgcGxh bmVfc3RhdGUtPmJhc2UuY3J0Y19oOwo+PiArCj4+ICsJaWYgKCgocGxhbmVfc3RhdGUtPmJhc2Uu c3JjX3ggPj4gMTYpICUgNCkgIT0gMCB8fAo+PiArCSAgICAoKHBsYW5lX3N0YXRlLT5iYXNlLnNy Y195ID4+IDE2KSAlIDQpICE9IDAgfHwKPj4gKwkgICAgKChwbGFuZV9zdGF0ZS0+YmFzZS5zcmNf dyA+PiAxNikgJSA0KSAhPSAwIHx8Cj4+ICsJICAgICgocGxhbmVfc3RhdGUtPmJhc2Uuc3JjX2gg Pj4gMTYpICUgNCkgIT0gMCkgewo+PiArCQlEUk1fREVCVUdfS01TKCJzcmMgY29vcmRzIG11c3Qg YmUgbXVsdGlwbGUgb2YgNCBmb3IgTlYxMlxuIik7Cj4+ICsJCXJldHVybiAtRUlOVkFMOwo+PiAr CX0KPiBJIGRvbid0IHJlYWxseSBzZWUgd2h5IHdlIHNob3VsZCBjaGVjayB0aGVzZS4gVGhlIGNs aXBwZWQgY29vcmRpbmF0ZXMKPiBhcmUgd2hhdCBtYXR0ZXJzLgoKVG8gcHJvcGFnYXRlIG91ciBs aW1pdHMgdG8gdGhlIHVzZXJzcGFjZS4gSSB0aGluayB3ZSBzaG91bGQgZG8gaXQgZm9yIGFsbCBm b3JtYXRzLApidXQgTlYxMiBpcyB0aGUgZmlyc3QgWVVWIGZvcm1hdCB3ZSBoYXZlIHRlc3RzIGZv ci4gSWYgd2UgY291bGQgd2Ugc2hvdWxkIGRvCnNvbWV0aGluZyBzaW1pbGFyIGZvciB0aGUgb3Ro ZXIgWVVWIGZvcm1hdHMsIGJ1dCB0aGV5IGhhdmUgZGlmZmVyZW50IHJlcXVpcmVtZW50cy4KCklu IGNhc2Ugb2YgTlYxMiB3ZSBkb24ndCBoYXZlIGV4aXN0aW5nIHVzZXJzcGFjZSwgdGhlcmUgd2ls bCBiZSBub3RoaW5nIHRoYXQKYnJlYWtzIGlmIHdlIGVuZm9yY2UgbGltaXRzIGZyb20gdGhlIHN0 YXJ0LgoKPj4gKwo+PiArCS8qIENsaXBwaW5nIHdvdWxkIGNhdXNlIGEgMS0zIHBpeGVsIGdhcCBh dCB0aGUgZWRnZSBvZiB0aGUgc2NyZWVuPyAqLwo+PiArCWlmICgoY3J0Y194MiA+IGNydGNfc3Rh dGUtPnBpcGVfc3JjX3cgJiYgY3J0Y19zdGF0ZS0+cGlwZV9zcmNfdyAlIDQpIHx8Cj4+ICsJICAg IChjcnRjX3kyID4gY3J0Y19zdGF0ZS0+cGlwZV9zcmNfaCAmJiBjcnRjX3N0YXRlLT5waXBlX3Ny Y19oICUgNCkpIHsKPj4gKwkJRFJNX0RFQlVHX0tNUygiSXQncyBub3QgcG9zc2libGUgdG8gY2xp cCAldSwldSB0byAldSwldVxuIiwKPj4gKwkJCSAgICAgIGNydGNfeDIsIGNydGNfeTIsCj4+ICsJ CQkgICAgICBjcnRjX3N0YXRlLT5waXBlX3NyY193LCBjcnRjX3N0YXRlLT5waXBlX3NyY19oKTsK Pj4gKwkJcmV0dXJuIC1FSU5WQUw7Cj4+ICsJfQo+IFdoeSBzaG91bGQgd2UgY2FyZT8gVGhlIGN1 cnJlbnQgY29kZSBhbHJlYWR5IHBsYXlzIGl0IGZhc3QgYW5kIGxvb3NlCj4gYW5kIGFsbG93cyB0 aGUgZHN0IHJlY3RhbmdsZSB0byBzaHJpbmsgdG8gYWNjb21vZGF0ZSB0aGUgaHcgbGltaXRzLgo+ IElmIHdlIHdhbnQgdG8gY2hhbmdlIHRoYXQgd2Ugc2hvdWxkIGNoYW5nZSBpdCB1bml2ZXJzYWxs eS4KClVuZm9ydHVuYXRlbHkgZm9yIHRoZSBvdGhlciBmb3JtYXRzIHdlIGFscmVhZHkgaGF2ZSBh biBleGlzdGluZyB1c2Vyc3BhY2UKKFgub3JnKSB0aGF0IGRvZXNuJ3QgcGVyZm9ybSBhbnkgdmFs aWRhdGlvbi4gV2UgY2FuJ3QgY2hhbmdlIGl0IGZvciB0aGF0LApidXQgd2UgY2FuIHByZXZlbnQg ZnV0dXJlIG1pc3Rha2VzLgoKPj4gKwo+PiArCXBsYW5lX3N0YXRlLT5iYXNlLnNyYy54MSA9Cj4+ ICsJCURJVl9ST1VORF9DTE9TRVNUKHBsYW5lX3N0YXRlLT5iYXNlLnNyYy54MSwgMSA8PCAxOCkg PDwgMTg7Cj4+ICsJcGxhbmVfc3RhdGUtPmJhc2Uuc3JjLngyID0KPj4gKwkJRElWX1JPVU5EX0NM T1NFU1QocGxhbmVfc3RhdGUtPmJhc2Uuc3JjLngyLCAxIDw8IDE4KSA8PCAxODsKPj4gKwlwbGFu ZV9zdGF0ZS0+YmFzZS5zcmMueTEgPQo+PiArCQlESVZfUk9VTkRfQ0xPU0VTVChwbGFuZV9zdGF0 ZS0+YmFzZS5zcmMueTEsIDEgPDwgMTgpIDw8IDE4Owo+PiArCXBsYW5lX3N0YXRlLT5iYXNlLnNy Yy55MiA9Cj4+ICsJCURJVl9ST1VORF9DTE9TRVNUKHBsYW5lX3N0YXRlLT5iYXNlLnNyYy55Miwg MSA8PCAxOCkgPDwgMTg7Cj4gU2luY2UgdGhpcyBjYW4gbm93IGluY3JlYXNlIHRoZSBzaXplIG9m IHRoZSBzb3VyY2UgcmVjdGFuZ2xlIG91cgo+IHNjYWxpbmcgZmFjdG9yIGNoZWNrcyBhcmUgbm8g bG9uZ2VyIDEwMCUgdmFsaWQuIFdlIG1pZ2h0IGVuZCB1cCB3aXRoCj4gYSBzY2FsaW5nIGZhY3Rv ciB0aGF0IGlzIHRvbyBoaWdoLgo+Cj4gSSBkb24ndCByZWFsbHkgbGlrZSBhbnkgb2YgdGhlc2Ug ImxldCdzIG1ha2UgTlYxMiBiZWhhdmUgc3BlY2lhbCIKPiB0cmlja3MuIFdlIHNob3VsZCBtYWtl IHRoZSBjb2RlIGJlaGF2ZSB0aGUgc2FtZSB3YXkgZm9yIGFsbCBwaXhlbAo+IGZvcm1hdHMgaW5z dGVhZCBvZiBhZGRpbmcgZm9ybWF0IHNwZWNpZmljIGhhY2tzLgoKVGhpcyBpcyBub3Qgbml2YWxp ZCBiZWNhdXNlIHdlIHJlc3RyaWN0IHRoZSBvcmlnaW5hbCBzcmMgY29vcmRpbmF0ZXMgdG8gYmUK YSBtdWx0aXBsZSBvZiA0LCB5b3UgY2FuIG9ubHkgY2xpcCB0byBzb21ldGhpbmcgc21hbGxlciwg bm90IHRvIHNvbWV0aGluZwpiaWdnZXIuIDopCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0 cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9s aXN0aW5mby9pbnRlbC1nZngK