From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH 24/24] drm/omap: cleanup color space conversion Date: Wed, 28 Feb 2018 12:13:58 +0200 Message-ID: <2831259.8hl8RxvPAe@avalon> References: <1518428694-18018-1-git-send-email-tomi.valkeinen@ti.com> <6564656.5eGqd0f7Ap@avalon> <7d2a7e20-2e76-11c4-8d76-70e53fd33c67@ti.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 [IPv6:2001:4b98:dc2:45:216:3eff:febb:480d]) by gabe.freedesktop.org (Postfix) with ESMTPS id B16BA6E0AF for ; Wed, 28 Feb 2018 10:13:14 +0000 (UTC) In-Reply-To: <7d2a7e20-2e76-11c4-8d76-70e53fd33c67@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Tomi Valkeinen Cc: Peter Ujfalusi , Jyri Sarha , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org SGkgVG9taSwKCk9uIFdlZG5lc2RheSwgMjggRmVicnVhcnkgMjAxOCAxMjowOTozNSBFRVQgVG9t aSBWYWxrZWluZW4gd3JvdGU6Cj4gT24gMjcvMDIvMTggMTY6MDgsIExhdXJlbnQgUGluY2hhcnQg d3JvdGU6Cj4gPiBPbiBNb25kYXksIDEyIEZlYnJ1YXJ5IDIwMTggMTE6NDQ6NTQgRUVUIFRvbWkg VmFsa2VpbmVuIHdyb3RlOgo+ID4+IFRoZSBzZXR1cCBjb2RlIGZvciBjb2xvciBzcGFjZSBjb252 ZXJzaW9uIGlzIGEgYml0IG1lc3N5LiBUaGlzIHBhdGNoCj4gPj4gY2xlYW5zIGl0IHVwLgo+ID4+ IAo+ID4+IEZvciBzb21lIHJlYXNvbiB0aGUgVFJNIHVzZXMgdmFsdWVzIGluIFlDckNiIG9yZGVy LCB3aGljaCBpcyBhbHNvIHVzZWQKPiA+PiBpbiB0aGUgY3VycmVudCBkcml2ZXIsIHdoZXJlYXMg ZXZlcnl3aGVyZSBlbHNlIGl0J3MgWUNiQ3IgKHdoaWNoIGFsc28KPiA+PiBtYXRjaGVzIFlVViBv cmRlcikuIFRoaXMgcGF0Y2ggY2hhbmdlcyB0aGUgdGFibGVzIHRvIHVzZSB0aGUgY29tbW9uCj4g Pj4gb3JkZXIgdG8gYXZvaWQgY29uZnVzaW9uLgo+ID4+IAo+ID4+IFRoZSB0YWJsZXMgYXJlIHNw bGl0IGludG8gc2VwYXJhdGUgbGluZXMsIGFuZCBjb21tZW50cyBhZGRlZCBmb3IKPiA+PiBjbGFy aXR5Lgo+ID4+IAo+ID4+IFdCIGNvbG9yIGNvbnZlcnNpb24gcmVnaXN0ZXJzIGFyZSBzaW1pbGFy IGJ1dCBkaWZmZXJlbnQgdGhhbiBub24tV0IsIGJ1dAo+ID4+IHRoZSBzYW1lIGZ1bmN0aW9uIHdh cyB1c2VkIHRvIHdyaXRlIGJvdGguIEl0IHdvcmtlZCBmaW5lIGJlY2F1c2UgdGhlCj4gPj4gY29l ZiB0YWJsZSB3YXMgYWRqdXN0ZWQgYWNjb3JkaW5nbHksIGJ1dCB0aGF0IHdhcyByYXRoZXIgY29u ZnVzaW5nLiBUaGlzCj4gPj4gcGF0Y2ggYWRkcyBhIHNlcGFyYXRlIGZ1bmN0aW9uIHRvIHdyaXRl IHRoZSBXQiB2YWx1ZXMgc28gdGhhdCB0aGUgY29lZgo+ID4+IHRhYmxlIGNhbiBiZSB3cml0dGVu IGluIGFuIHVuZGVyc3RhbmRhYmxlIHdheS4KPiA+PiAKPiA+PiBTaWduZWQtb2ZmLWJ5OiBUb21p IFZhbGtlaW5lbiA8dG9taS52YWxrZWluZW5AdGkuY29tPgo+ID4+IC0tLQo+ID4+IAo+ID4+ICBk cml2ZXJzL2dwdS9kcm0vb21hcGRybS9kc3MvZGlzcGMuYyB8IDU5ICsrKysrKysrKysrKysrKysr KysrKysrKy0tLS0tLS0KPiA+PiAgMSBmaWxlIGNoYW5nZWQsIDQ0IGluc2VydGlvbnMoKyksIDE1 IGRlbGV0aW9ucygtKQo+ID4+IAo+ID4+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vb21h cGRybS9kc3MvZGlzcGMuYwo+ID4+IGIvZHJpdmVycy9ncHUvZHJtL29tYXBkcm0vZHNzL2Rpc3Bj LmMgaW5kZXggZmYwOWUyYmU0NzBmLi42OTcyNzQzMTdmN2MKPiA+PiAxMDA2NDQKPiA+PiAtLS0g YS9kcml2ZXJzL2dwdS9kcm0vb21hcGRybS9kc3MvZGlzcGMuYwo+ID4+ICsrKyBiL2RyaXZlcnMv Z3B1L2RybS9vbWFwZHJtL2Rzcy9kaXNwYy5jCj4gPj4gQEAgLTM0NSwxMSArMzQ1LDYgQEAgc3Rh dGljIGNvbnN0IHN0cnVjdCB7Cj4gPj4gIAl9LAogPj4gIH07Cj4gPj4gCj4gPj4gLXN0cnVjdCBj b2xvcl9jb252X2NvZWYgewo+ID4+IC0JaW50IHJ5LCByY3IsIHJjYiwgZ3ksIGdjciwgZ2NiLCBi eSwgYmNyLCBiY2I7Cj4gPj4gLQlpbnQgZnVsbF9yYW5nZTsKPiA+PiAtfTsKPiA+PiAtCj4gPj4g IHN0YXRpYyB1bnNpZ25lZCBsb25nIGRpc3BjX2ZjbGtfcmF0ZSh2b2lkKTsKPiA+PiAgc3RhdGlj IHVuc2lnbmVkIGxvbmcgZGlzcGNfY29yZV9jbGtfcmF0ZSh2b2lkKTsKPiA+PiAgc3RhdGljIHVu c2lnbmVkIGxvbmcgZGlzcGNfbWdyX2xjbGtfcmF0ZShlbnVtIG9tYXBfY2hhbm5lbCBjaGFubmVs KTsKPiA+PiBAQCAtODQxLDkgKzgzNiwxOCBAQCBzdGF0aWMgdm9pZCBkaXNwY19vdmxfc2V0X3Nj YWxlX2NvZWYoZW51bQo+ID4+IG9tYXBfcGxhbmVfaWQgcGxhbmUsIGludCBmaXJfaGluYywgfQo+ ID4+IAo+ID4+ICB9Cj4gPj4gCj4gPj4gK3N0cnVjdCBjc2NfY29lZl95dXYycmdiIHsKPiA+PiAr CWludCByeSwgcmNiLCByY3IsIGd5LCBnY2IsIGdjciwgYnksIGJjYiwgYmNyOwo+ID4gCj4gPiBJ ZiB5b3UgbWFkZSB0aGlzIGEgM3gzIG1hdHJpeCB3aXRob3V0IGV4cGxpY2l0IG5hbWVzIGZvciB0 aGUgZmllbGRzIEkKPiA+IHRoaW5rIHlvdSB3b3VsZG4ndCBuZWVkIHR3byBzdHJ1Y3R1cmUgYW5k IHR3byBmdW5jdGlvbnMuCj4gCj4gVGhhdCBpcyB0cnVlLCBidXQgSSB0aGluayBpdCB3b3VsZCBh bHNvIG1ha2UgdGhlIGNvZGUgbW9yZSBkaWZmaWN1bHQgdG8KPiBmb2xsb3cuIEl0J3Mgbm90IGV4 YWN0bHkgb2J2aW91cyB3aGljaCBjb2VmZmljaWVudHMgZ28gd2hlcmUuCgpJIGRvbid0IGV4cGVj dCB0aGlzIGNvZGUgdG8gYmUgb2Z0ZW4gcmVhZCwgYW5kIEkgdGhpbmsgdGhhdCB3aXRoIGFwcHJv cHJpYXRlIApjb21tZW50cyBpbiB0aGUgc291cmNlIGNvZGUgYW5kIHByb3BlciBmb3JtYXR0aW5n IG9mIHRoZSB0YWJsZSwgYXMgeW91IGRvIApiZWxvdywgaXQgc2hvdWxkIGJlIGNsZWFyIGVub3Vn aCwgYnV0IEknbGwgbGVhdmUgaXQgdXAgdG8geW91LiBXZSdyZSByZWFsbHkgCnByb2dyYW1taW5n IGEgM3gzIG1hdHJpeCBpbiB0aGUgaGFyZHdhcmUsIHNvIEkgdGhpbmsgdGhhdCBtb2RlbGluZyBp dCBhcyBhIDN4MyAKbWF0cml4IGluIHRoZSBkcml2ZXIgbWFrZXMgc2Vuc2UgOi0pCgo+ID4+ICsJ Ym9vbCBmdWxsX3JhbmdlOwo+ID4+ICt9Owo+ID4+ICsKPiA+PiArc3RydWN0IGNzY19jb2VmX3Jn YjJ5dXYgewo+ID4+ICsJaW50IHlyLCB5ZywgeWIsIGNiciwgY2JnLCBjYmIsIGNyciwgY3JnLCBj cmI7Cj4gPj4gKwlib29sIGZ1bGxfcmFuZ2U7Cj4gPj4gK307Cj4gPj4gCj4gPj4gIHN0YXRpYyB2 b2lkIGRpc3BjX292bF93cml0ZV9jb2xvcl9jb252X2NvZWYoZW51bSBvbWFwX3BsYW5lX2lkIHBs YW5lLAo+ID4+IAo+ID4+IC0JCWNvbnN0IHN0cnVjdCBjb2xvcl9jb252X2NvZWYgKmN0KQo+ID4+ ICsJCWNvbnN0IHN0cnVjdCBjc2NfY29lZl95dXYycmdiICpjdCkKPiA+PiAKPiA+PiAgewo+ID4+ ICAjZGVmaW5lIENWQUwoeCwgeSkgKEZMRF9WQUwoeCwgMjYsIDE2KSB8IEZMRF9WQUwoeSwgMTAs IDApKQo+ID4+IAo+ID4+IEBAIC04NTMsNyArODU3LDI0IEBAIHN0YXRpYyB2b2lkIGRpc3BjX292 bF93cml0ZV9jb2xvcl9jb252X2NvZWYoZW51bQo+ID4+IG9tYXBfcGxhbmVfaWQgcGxhbmUsIGRp c3BjX3dyaXRlX3JlZyhESVNQQ19PVkxfQ09OVl9DT0VGKHBsYW5lLCAzKSwKPiA+PiBDVkFMKGN0 LT5iY3IsIGN0LT5ieSkpOyBkaXNwY193cml0ZV9yZWcoRElTUENfT1ZMX0NPTlZfQ09FRihwbGFu ZSwgNCksCj4gPj4gQ1ZBTCgwLCBjdC0+YmNiKSk7Cj4gPj4gCj4gPj4gLQlSRUdfRkxEX01PRChE SVNQQ19PVkxfQVRUUklCVVRFUyhwbGFuZSksIGN0LT5mdWxsX3JhbmdlLCAxMSwgMTEpOwo+ID4+ ICsJUkVHX0ZMRF9NT0QoRElTUENfT1ZMX0FUVFJJQlVURVMocGxhbmUpLCAhIWN0LT5mdWxsX3Jh bmdlLCAxMSwgMTEpOwo+ID4gCj4gPiB0cnVlIGFuZCBmYWxzZSBzaG91bGQgYmUgZXF1YWwgdG8g MSBhbmQgMCByZXNwZWN0aXZlbHksIHNvIHRoZSAhIQo+ID4gc2hvdWxkbid0IGJlIG5lZWRlZC4K PiAKPiBZZXAuCj4gCj4gPj4gKwo+ID4+ICsjdW5kZWYgQ1ZBTAo+ID4+ICt9Cj4gPj4gKwo+ID4+ ICtzdGF0aWMgdm9pZCBkaXNwY193Yl93cml0ZV9jb2xvcl9jb252X2NvZWYoY29uc3Qgc3RydWN0 IGNzY19jb2VmX3JnYjJ5dXYKPiA+PiAqY3QpCj4gPj4gK3sKPiA+PiArCWNvbnN0IGVudW0gb21h cF9wbGFuZV9pZCBwbGFuZSA9IE9NQVBfRFNTX1dCOwo+ID4+ICsKPiA+PiArI2RlZmluZSBDVkFM KHgsIHkpIChGTERfVkFMKHgsIDI2LCAxNikgfCBGTERfVkFMKHksIDEwLCAwKSkKPiA+PiArCj4g Pj4gKwlkaXNwY193cml0ZV9yZWcoRElTUENfT1ZMX0NPTlZfQ09FRihwbGFuZSwgMCksIENWQUwo Y3QtPnlnLCAgY3QtPnlyKSk7Cj4gPj4gKwlkaXNwY193cml0ZV9yZWcoRElTUENfT1ZMX0NPTlZf Q09FRihwbGFuZSwgMSksIENWQUwoY3QtPmNyciwgY3QtPnliKSk7Cj4gPj4gKwlkaXNwY193cml0 ZV9yZWcoRElTUENfT1ZMX0NPTlZfQ09FRihwbGFuZSwgMiksIENWQUwoY3QtPmNyYiwgY3QtPmNy ZykpOwo+ID4+ICsJZGlzcGNfd3JpdGVfcmVnKERJU1BDX09WTF9DT05WX0NPRUYocGxhbmUsIDMp LCBDVkFMKGN0LT5jYmcsIGN0LT5jYnIpKTsKPiA+PiArCWRpc3BjX3dyaXRlX3JlZyhESVNQQ19P VkxfQ09OVl9DT0VGKHBsYW5lLCA0KSwgQ1ZBTCgwLCBjdC0+Y2JiKSk7Cj4gPj4gKwo+ID4+ICsJ UkVHX0ZMRF9NT0QoRElTUENfT1ZMX0FUVFJJQlVURVMocGxhbmUpLCAhIWN0LT5mdWxsX3Jhbmdl LCAxMSwgMTEpOwo+ID4+ICAjdW5kZWYgQ1ZBTAo+ID4+ICB9Cj4gPj4gCj4gPj4gQEAgLTg2Miwy MCArODgzLDI4IEBAIHN0YXRpYyB2b2lkIGRpc3BjX3NldHVwX2NvbG9yX2NvbnZfY29lZih2b2lk KQo+ID4+ICB7Cj4gPj4gIAlpbnQgaTsKPiA+PiAgCWludCBudW1fb3ZsID0gZGlzcGNfZ2V0X251 bV9vdmxzKCk7Cj4gPj4gLQljb25zdCBzdHJ1Y3QgY29sb3JfY29udl9jb2VmIGN0YmxfYnQ2MDFf NV9vdmwgPSB7Cj4gPj4gLQkJLyogWVVWIC0+IFJHQiAqLwo+ID4+IC0JCTI5OCwgNDA5LCAwLCAy OTgsIC0yMDgsIC0xMDAsIDI5OCwgMCwgNTE3LCAwLAo+ID4+ICsKPiA+PiArCS8qIFlVViAtPiBS R0IsIElUVS1SIEJULjYwMSwgbGltaXRlZCByYW5nZSAqLwo+ID4+ICsJY29uc3Qgc3RydWN0IGNz Y19jb2VmX3l1djJyZ2IgY29lZnNfeXV2MnJnYl9idDYwMV9saW0gPSB7Cj4gPj4gKwkJMjk4LCAg ICAwLCAgNDA5LAkvKiByeSwgcmNiLCByY3IgKi8KPiA+PiArCQkyOTgsIC0xMDAsIC0yMDgsCS8q IGd5LCBnY2IsIGdjciAqLwo+ID4+ICsJCTI5OCwgIDUxNiwgICAgMCwJLyogYnksIGJjYiwgYmNy ICovCj4gPiAKPiA+IFRoZSA1MTcgdHVybmVkIGludG8gNTE2LCB3YXMgdGhhdCBpbnRlbnRpb25h bCA/Cj4gCj4gR29vZCBwb2ludC4gSXQncyBiZWVuIGEgd2hpbGUsIGJ1dCBJIGRpZCByZWNhbGN1 bGF0ZSB0aGVzZSwgYW5kIHdpdGgKPiByb3VuZGluZywgYWxsIHRoZSBvdGhlciB2YWx1ZXMgbWF0 Y2ggZXhjZXB0IHRoZSA1MTcsIHNvIEkgY2hhbmdlZCBpdAo+IGFjY29yZGluZyB0byBteSBjYWxj dWxhdGlvbnMuCj4gCj4gVGhhdCBzYWlkLCBJJ20gbm90IHN1cmUgaWYgdGhlIG9yaWdpbmFsIGNh bGN1bGF0aW9ucyBhcmUgY29ycmVjdCwgYXMKPiB0aGV5IHNlZW0gdG8gdXNlIDI1NiBhcyBtdWx0 aXBsaWVyLCB3aGljaCB3b3VsZCBub3QgcmVzdWx0IDB4ZmYgZnJvbSAxLjAKPiB2YWx1ZS4gSW4g YW55IGNhc2UsIEkgY2hhbmdlZCB0aGUgc2luZ2xlIHZhbHVlIHRoYXQgc2VlbWVkIHRvIGJlIG9m Zgo+IGZyb20gdGhlIG90aGVyIG9uZXMuCgpJIGhhdmUgbm8gaXNzdWUgd2l0aCB0aGF0LCBidXQg Y291bGQgeW91IHBsZWFzZSBtZW50aW9uIGl0IGluIHRoZSBjb21taXQgCm1lc3NhZ2UgPwoKLS0g ClJlZ2FyZHMsCgpMYXVyZW50IFBpbmNoYXJ0CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0 cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9s aXN0aW5mby9kcmktZGV2ZWwK