From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH 01/11] drm/i915: take-out common clamping code of fixed16 wrappers Date: Fri, 14 Jul 2017 13:24:29 +0300 Message-ID: <87o9snjoyq.fsf@nikula.org> References: <20170705143154.32132-1-mahesh1.kumar@intel.com> <20170705143154.32132-2-mahesh1.kumar@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTPS id 9F4736E7E5 for ; Fri, 14 Jul 2017 10:19:29 +0000 (UTC) In-Reply-To: <20170705143154.32132-2-mahesh1.kumar@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Mahesh Kumar , intel-gfx@lists.freedesktop.org Cc: paulo.r.zanoni@intel.com, maarten.lankhorst@intel.com List-Id: intel-gfx@lists.freedesktop.org T24gV2VkLCAwNSBKdWwgMjAxNywgTWFoZXNoIEt1bWFyIDxtYWhlc2gxLmt1bWFyQGludGVsLmNv bT4gd3JvdGU6Cj4gRnJvbTogIkt1bWFyLCBNYWhlc2giIDxtYWhlc2gxLmt1bWFyQGludGVsLmNv bT4KPgo+IFRoaXMgcGF0Y2ggY3JlYXRlcyBhIG5ldyBmdW5jdGlvbiBmb3IgY2xhbXBpbmcgdTY0 IHRvIGZpeGVkMTYuCj4gQW5kIG1ha2UgdXNlIG9mIHRoaXMgZnVuY3Rpb24gaW4gb3RoZXIgZml4 ZWQxNiB3cmFwcGVycy4KPgo+IFNpZ25lZC1vZmYtYnk6IE1haGVzaCBLdW1hciA8bWFoZXNoMS5r dW1hckBpbnRlbC5jb20+Cj4gUmV2aWV3ZWQtYnk6IE1hYXJ0ZW4gTGFua2hvcnN0IDxtYWFydGVu Lmxhbmtob3JzdEBsaW51eC5pbnRlbC5jb20+Cj4gLS0tCj4gIGRyaXZlcnMvZ3B1L2RybS9pOTE1 L2k5MTVfZHJ2LmggfCAyOCArKysrKysrKysrKystLS0tLS0tLS0tLS0tLS0tCj4gIDEgZmlsZSBj aGFuZ2VkLCAxMiBpbnNlcnRpb25zKCspLCAxNiBkZWxldGlvbnMoLSkKPgo+IGRpZmYgLS1naXQg YS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X2Rydi5oIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUv aTkxNV9kcnYuaAo+IGluZGV4IDVlNzBmNTcxMWZjOC4uMWI1MjUwNTFiZjVmIDEwMDY0NAo+IC0t LSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2LmgKPiArKysgYi9kcml2ZXJzL2dwdS9k cm0vaTkxNS9pOTE1X2Rydi5oCj4gQEAgLTE2MCw2ICsxNjAsMTQgQEAgc3RhdGljIGlubGluZSB1 aW50X2ZpeGVkXzE2XzE2X3QgbWF4X2ZpeGVkXzE2XzE2KHVpbnRfZml4ZWRfMTZfMTZfdCBtYXgx LAo+ICAJcmV0dXJuIG1heDsKPiAgfQo+ICAKPiArc3RhdGljIGlubGluZSB1aW50X2ZpeGVkXzE2 XzE2X3QgY2xhbXBfdTY0X3RvX2ZpeGVkMTYodWludDY0X3QgdmFsKQo+ICt7Cj4gKwl1aW50X2Zp eGVkXzE2XzE2X3QgZnA7Cj4gKwlXQVJOX09OKHZhbCA+PiAzMik7CgpXQVJOX09OKHZhbCA+IFUz Ml9NQVgpOyBtYWtlcyBtdWNoIG1vcmUgc2Vuc2UuIFNhbWUgYWxsIG92ZXIgdGhlIHBsYWNlLgoK PiArCWZwLnZhbCA9IGNsYW1wX3QodWludDMyX3QsIHZhbCwgMCwgfjApOwoKVGhlIHVzZSBvZiBj bGFtcF90IGhlcmUgaXMgcG9pbnRsZXNzLCBhcyBmcC52YWwgaXMgdWludDMyX3QuIEFzc2lnbiwK cG9zc2libHkgd2l0aCBhIGNhc3QsIGFuZCBiZSBkb25lIHdpdGggaXQuCgo+ICsJcmV0dXJuIGZw Owo+ICt9Cj4gKwo+ICBzdGF0aWMgaW5saW5lIHVpbnQzMl90IGRpdl9yb3VuZF91cF9maXhlZDE2 KHVpbnRfZml4ZWRfMTZfMTZfdCB2YWwsCj4gIAkJCQkJICAgIHVpbnRfZml4ZWRfMTZfMTZfdCBk KQo+ICB7Cj4gQEAgLTE3MCwyNiArMTc4LDIxIEBAIHN0YXRpYyBpbmxpbmUgdWludDMyX3QgbXVs X3JvdW5kX3VwX3UzMl9maXhlZDE2KHVpbnQzMl90IHZhbCwKPiAgCQkJCQkJdWludF9maXhlZF8x Nl8xNl90IG11bCkKPiAgewo+ICAJdWludDY0X3QgaW50ZXJtZWRpYXRlX3ZhbDsKPiAtCXVpbnQz Ml90IHJlc3VsdDsKPiAgCj4gIAlpbnRlcm1lZGlhdGVfdmFsID0gKHVpbnQ2NF90KSB2YWwgKiBt dWwudmFsOwo+ICAJaW50ZXJtZWRpYXRlX3ZhbCA9IERJVl9ST1VORF9VUF9VTEwoaW50ZXJtZWRp YXRlX3ZhbCwgMSA8PCAxNik7Cj4gIAlXQVJOX09OKGludGVybWVkaWF0ZV92YWwgPj4gMzIpOwo+ IC0JcmVzdWx0ID0gY2xhbXBfdCh1aW50MzJfdCwgaW50ZXJtZWRpYXRlX3ZhbCwgMCwgfjApOwo+ IC0JcmV0dXJuIHJlc3VsdDsKPiArCXJldHVybiBjbGFtcF90KHVpbnQzMl90LCBpbnRlcm1lZGlh dGVfdmFsLCAwLCB+MCk7CgpTYW1lIGhlcmUsIHRoZSBmdW5jdGlvbiBpcyB1aW50MzJfdCwganVz dCByZXR1cm4gd2l0aCBhIGNhc3QuCgo+ICB9Cj4gIAo+ICBzdGF0aWMgaW5saW5lIHVpbnRfZml4 ZWRfMTZfMTZfdCBtdWxfZml4ZWQxNih1aW50X2ZpeGVkXzE2XzE2X3QgdmFsLAo+ICAJCQkJCSAg ICAgdWludF9maXhlZF8xNl8xNl90IG11bCkKPiAgewo+ICAJdWludDY0X3QgaW50ZXJtZWRpYXRl X3ZhbDsKPiAtCXVpbnRfZml4ZWRfMTZfMTZfdCBmcDsKPiAgCj4gIAlpbnRlcm1lZGlhdGVfdmFs ID0gKHVpbnQ2NF90KSB2YWwudmFsICogbXVsLnZhbDsKPiAgCWludGVybWVkaWF0ZV92YWwgPSBp bnRlcm1lZGlhdGVfdmFsID4+IDE2Owo+IC0JV0FSTl9PTihpbnRlcm1lZGlhdGVfdmFsID4+IDMy KTsKPiAtCWZwLnZhbCA9IGNsYW1wX3QodWludDMyX3QsIGludGVybWVkaWF0ZV92YWwsIDAsIH4w KTsKPiAtCXJldHVybiBmcDsKPiArCXJldHVybiBjbGFtcF91NjRfdG9fZml4ZWQxNihpbnRlcm1l ZGlhdGVfdmFsKTsKPiAgfQo+ICAKPiAgc3RhdGljIGlubGluZSB1aW50X2ZpeGVkXzE2XzE2X3Qg Zml4ZWRfMTZfMTZfZGl2KHVpbnQzMl90IHZhbCwgdWludDMyX3QgZCkKPiBAQCAtMjAzLDE1ICsy MDYsMTEgQEAgc3RhdGljIGlubGluZSB1aW50X2ZpeGVkXzE2XzE2X3QgZml4ZWRfMTZfMTZfZGl2 KHVpbnQzMl90IHZhbCwgdWludDMyX3QgZCkKPiAgCj4gIHN0YXRpYyBpbmxpbmUgdWludF9maXhl ZF8xNl8xNl90IGZpeGVkXzE2XzE2X2Rpdl91NjQodWludDMyX3QgdmFsLCB1aW50MzJfdCBkKQo+ ICB7Cj4gLQl1aW50X2ZpeGVkXzE2XzE2X3QgcmVzOwo+ICAJdWludDY0X3QgaW50ZXJtX3ZhbDsK PiAgCj4gIAlpbnRlcm1fdmFsID0gKHVpbnQ2NF90KXZhbCA8PCAxNjsKPiAgCWludGVybV92YWwg PSBESVZfUk9VTkRfVVBfVUxMKGludGVybV92YWwsIGQpOwo+IC0JV0FSTl9PTihpbnRlcm1fdmFs ID4+IDMyKTsKPiAtCXJlcy52YWwgPSAodWludDMyX3QpIGludGVybV92YWw7Cj4gLQo+IC0JcmV0 dXJuIHJlczsKPiArCXJldHVybiBjbGFtcF91NjRfdG9fZml4ZWQxNihpbnRlcm1fdmFsKTsKPiAg fQo+ICAKPiAgc3RhdGljIGlubGluZSB1aW50MzJfdCBkaXZfcm91bmRfdXBfdTMyX2ZpeGVkMTYo dWludDMyX3QgdmFsLAo+IEBAIC0yMjksMTIgKzIyOCw5IEBAIHN0YXRpYyBpbmxpbmUgdWludF9m aXhlZF8xNl8xNl90IG11bF91MzJfZml4ZWRfMTZfMTYodWludDMyX3QgdmFsLAo+ICAJCQkJCQkg ICAgIHVpbnRfZml4ZWRfMTZfMTZfdCBtdWwpCj4gIHsKPiAgCXVpbnQ2NF90IGludGVybWVkaWF0 ZV92YWw7Cj4gLQl1aW50X2ZpeGVkXzE2XzE2X3QgZnA7Cj4gIAo+ICAJaW50ZXJtZWRpYXRlX3Zh bCA9ICh1aW50NjRfdCkgdmFsICogbXVsLnZhbDsKPiAtCVdBUk5fT04oaW50ZXJtZWRpYXRlX3Zh bCA+PiAzMik7Cj4gLQlmcC52YWwgPSAodWludDMyX3QpIGludGVybWVkaWF0ZV92YWw7Cj4gLQly ZXR1cm4gZnA7Cj4gKwlyZXR1cm4gY2xhbXBfdTY0X3RvX2ZpeGVkMTYoaW50ZXJtZWRpYXRlX3Zh bCk7Cj4gIH0KPiAgCj4gIHN0YXRpYyBpbmxpbmUgY29uc3QgY2hhciAqeWVzbm8oYm9vbCB2KQoK LS0gCkphbmkgTmlrdWxhLCBJbnRlbCBPcGVuIFNvdXJjZSBUZWNobm9sb2d5IENlbnRlcgpfX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFp bGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5m cmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK