From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH] drm: disable deep color when EDID violates spec Date: Tue, 10 Jan 2017 15:39:42 +0200 Message-ID: <8760ln9gpt.fsf@intel.com> References: <20170105234523.5899-1-nicholas.sielicki@gmail.com> <20170110105213.GW31595@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id D254E6E692 for ; Tue, 10 Jan 2017 13:39:45 +0000 (UTC) In-Reply-To: <20170110105213.GW31595@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Ville =?utf-8?B?U3lyasOkbMOk?= , Nicholas Sielicki Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCAxMCBKYW4gMjAxNywgVmlsbGUgU3lyasOkbMOkIDx2aWxsZS5zeXJqYWxhQGxpbnV4 LmludGVsLmNvbT4gd3JvdGU6Cj4gT24gVGh1LCBKYW4gMDUsIDIwMTcgYXQgMDU6NDU6MjNQTSAt MDYwMCwgTmljaG9sYXMgU2llbGlja2kgd3JvdGU6Cj4+IEFzIHBlciB0aGUgSERNSSAxLjMgYW5k IDEuNCBzcGVjLCAiZGVlcCBjb2xvciBtb2RlcyIgYXJlIGNvbG9yIGRlcHRocwo+PiBncmVhdGVy IHRoYW4gMjQgYml0cyBwZXIgcGl4ZWwuIFRoZSBzcGVjIGV4cGxpY2l0bHkgc3RhdGVzLCAiQWxs IERlZXAKPj4gQ29sb3IgbW9kZXMgYXJlIG9wdGlvbmFsIHRob3VnaCBpZiBhbiBIRE1JIFNvdXJj ZSBvciBTaW5rIHN1cHBvcnRzIGFueQo+PiBEZWVwIENvbG9yIG1vZGUsIGl0IHNoYWxsIHN1cHBv cnQgMzYtYml0IG1vZGUuIiAoNi4yLjQgQ29sb3IgRGVwdGgKPj4gUmVxdWlyZW1lbnRzKS4KPj4g Cj4+IEkgY2FtZSBhY3Jvc3MgYSBtb25pdG9yIChBY2VyIFgyMzNIKSB0aGF0IHN1cHBsaWVzIGFu IGlsbGVnYWwgRURJRCB3aGVyZQo+PiBEQ18zMGJpdCBpcyBzZXQgYW5kIERDXzM2Yml0IGlzIG5v dCBzZXQuIFRoZSBlbmQgcmVzdWx0IGlzIGJhZGx5IGdhcmJsZWQKPj4gb3V0cHV0IGJlY2F1c2Ug dGhlIGRyaXZlciBpcyBzZW5kaW5nIDM2QlBQIHdoZW4gdGhlIG1vbml0b3IgY2FuJ3QgaGFuZGxl Cj4+IGl0Lgo+PiAKPj4gTXVjaCBvZiB0aGUgaW50ZWwgaGFyZHdhcmUgaXMgaW5jYXBhYmxlIG9m IG9wZXJhdGluZyBhdCBhbnkKPj4gYml0LXBlci1jb21wb25lbnQgc2V0dGluZyBvdXRzaWRlIG9m IDggb3IgMTIsIGFuZCB0aGUgc3BlYyBzZWVtcyB0bwo+PiBpbXBseSB0aGF0IGlmIGFueSBkZWVw IGNvbG9yIHN1cHBvcnQgaXMgZm91bmQsIHRoZW4gaXQgaXMgYSBzYWZlCj4+IGFzc3VtcHRpb24g dG8gb3BlcmF0ZSBhdCAxMi4KPj4gCj4+IFRoaXMgcGF0Y2ggZW5zdXJlcyB0aGF0IHRoZSBFRElE IGlzIHdpdGhpbiB0aGUgc3BlYyAoc3BlY2lmaWNhbGx5LCB0aGF0Cj4+IERDXzM2Yml0IGlzIHNl dCkgYmVmb3JlIGNvbW1pdHRpbmcgdG8gZ29pbmcgZm9yd2FyZCB3aXRoIGFueSBkZWVwCj4+IGNv bG9ycy4gIFRoZXJlIHdhcyBhbHJlYWR5IGEgY2hlY2sgZm9yIHRoaXMgRURJRCBpbmNvbnNpc3Rl bmN5LCBidXQgaXQKPj4gcmVzdWx0ZWQgb25seSBpbiBhIHdhcm5pbmcgYW5kIGRpZCBub3QgZmFs bC1iYWNrIHRvIHNhZmVyIHNldHRpbmdzLgo+PgoKSSB0aG91Z2h0IHRoZXJlIHdhcyBhIHJlbGF0 ZWQgYnVnemlsbGE/IFdoZXJlJ3MgdGhlIHJlZmVyZW5jZT8KCj4+IENDOiBWaWxsZSBTeXJqw6Rs w6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+PiBTaWduZWQtb2ZmLWJ5OiBOaWNo b2xhcyBTaWVsaWNraSA8bmljaG9sYXMuc2llbGlja2lAZ21haWwuY29tPgo+PiAtLS0KPj4gIGRy aXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jIHwgMzUgKysrKysrKysrKysrKysrKysrKysrKystLS0t LS0tLS0tLS0KPj4gIDEgZmlsZSBjaGFuZ2VkLCAyMyBpbnNlcnRpb25zKCspLCAxMiBkZWxldGlv bnMoLSkKPj4gCj4+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYyBiL2Ry aXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jCj4+IGluZGV4IDMzNmJlMzFmZjNkZS4uNDJjZTNmNTRk MmRjIDEwMDY0NAo+PiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYwo+PiArKysgYi9k cml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYwo+PiBAQCAtMzc3MiwzMCArMzc3MiwzNCBAQCBzdGF0 aWMgdm9pZCBkcm1fcGFyc2VfaGRtaV9kZWVwX2NvbG9yX2luZm8oc3RydWN0IGRybV9jb25uZWN0 b3IgKmNvbm5lY3RvciwKPj4gIHsKPj4gIAlzdHJ1Y3QgZHJtX2Rpc3BsYXlfaW5mbyAqaW5mbyA9 ICZjb25uZWN0b3ItPmRpc3BsYXlfaW5mbzsKPj4gIAl1bnNpZ25lZCBpbnQgZGNfYnBjID0gMDsK Pj4gKwl1OCBtb2RlcyA9IDA7Cj4+ICAKPj4gIAkvKiBIRE1JIHN1cHBvcnRzIGF0IGxlYXN0IDgg YnBjICovCj4+ICAJaW5mby0+YnBjID0gODsKPj4gIAo+PiArCS8qIEVuc3VyZSBhbGwgREMgbW9k ZXMgYXJlIHVuc2V0IGlmIHdlIHJldHVybiBlYXJseSAqLwo+PiArCWluZm8tPmVkaWRfaGRtaV9k Y19tb2RlcyA9IDA7Cj4KPiBDbGVhcmluZyB0aGlzIGluIGRybV9hZGRfZGlzcGxheV9pbmZvKCkg c2hvdWxkIGJlIHN1ZmZpY2llbnQgc2luY2UKPiB0aGlzIGd1eSBkb2Vzbid0IGdldCBjYWxsZWQg ZnJvbSBhbnl3aGVyZSBlbHNlLiBTbyB0aGlzIHBhcnQgY291bGQKPiBiZSBkcm9wcHBlZC4KPgo+ IE90aGVyd2lzZSB0aGlzIGZlZWxzIGxpa2UgYSBkZWNlbnQgd2F5IHRvIGhhbmRsZSB0aGUgcHJv YmxlbS4gV2UKPiBjb3VsZCBvZiBjb3Vyc2UgdHJ5IHRvIHVzZSB0aGUgMTBicGMgKG9yIHdoYXRl dmVyKSBkZWVwIGNvbG9yIG1vZGVzCj4gdGhlIHNpbmsgY2xhaW1zIHRvIHN1cHBvcnQsIGJ1dCBn aXZlbiB0aGF0IHRoZSBwZW9wbGUgZGVzaWduaW5nIHRoZQo+IHRoaW5nIGRpZG4ndCBib3RoZXIg cmVhZGluZyB0aGUgc3BlYywgaXQgc2VlbXMgc2FmZXIgdG8ganVzdCBkaXNhYmxlCj4gZGVlcCBj b2xvciBzdXBwb3J0IGVudGlyZWx5LgoKSG1taC4KClNvIGN1cnJlbnRseSwgdGhlIHNpbmsgaW4g cXVlc3Rpb24gdmlvbGF0ZXMgdGhpcywgIkFsbCBEZWVwIENvbG9yIG1vZGVzCmFyZSBvcHRpb25h bCB0aG91Z2ggaWYgYW4gSERNSSBTb3VyY2Ugb3IgU2luayBzdXBwb3J0cyBhbnkgRGVlcCBDb2xv cgptb2RlLCBpdCBzaGFsbCBzdXBwb3J0IDM2LWJpdCBtb2RlLiIKCkJ1dCBhbHNvIGN1cnJlbnRs eSwgd2UgdmlvbGF0ZSB0aGlzLCAiQW4gSERNSSBTb3VyY2Ugc2hhbGwgbm90IHNlbmQgYW55CkRl ZXAgQ29sb3IgbW9kZSB0byBhIFNpbmsgdGhhdCBkb2VzIG5vdCBpbmRpY2F0ZSBzdXBwb3J0IGZv ciB0aGF0IG1vZGUuIgoKSW5zdGVhZCBvZiBmaXhpbmcgb3VyIHZpb2xhdGlvbiwgdGhpcyBwYXRj aCBwb2ludHMgZmluZ2VycyBhdCB0aGUKdmlvbGF0aW5nIHNpbmtzLCBhbmQgcmVmdXNlcyB0byBw bGF5IGJhbGwgd2l0aCBhbnkgZGVlcCBjb2xvcnMuCgpKdXN0IHRvIGdldCB0aGUgcmVjb3JkIHN0 cmFpZ2h0LCBpcyB0aGF0IGEgZmFpciBhc3Nlc21lbnQ/CgpCUiwKSmFuaS4KCgo+Cj4gUmV2aWV3 ZWQtYnk6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBsaW51eC5pbnRlbC5jb20+Cj4K Pj4gKwo+PiAgCWlmIChjZWFfZGJfcGF5bG9hZF9sZW4oaGRtaSkgPCA2KQo+PiAgCQlyZXR1cm47 Cj4+ICAKPj4gIAlpZiAoaGRtaVs2XSAmIERSTV9FRElEX0hETUlfRENfMzApIHsKPj4gIAkJZGNf YnBjID0gMTA7Cj4+IC0JCWluZm8tPmVkaWRfaGRtaV9kY19tb2RlcyB8PSBEUk1fRURJRF9IRE1J X0RDXzMwOwo+PiArCQltb2RlcyB8PSBEUk1fRURJRF9IRE1JX0RDXzMwOwo+PiAgCQlEUk1fREVC VUcoIiVzOiBIRE1JIHNpbmsgZG9lcyBkZWVwIGNvbG9yIDMwLlxuIiwKPj4gIAkJCSAgY29ubmVj dG9yLT5uYW1lKTsKPj4gIAl9Cj4+ICAKPj4gIAlpZiAoaGRtaVs2XSAmIERSTV9FRElEX0hETUlf RENfMzYpIHsKPj4gIAkJZGNfYnBjID0gMTI7Cj4+IC0JCWluZm8tPmVkaWRfaGRtaV9kY19tb2Rl cyB8PSBEUk1fRURJRF9IRE1JX0RDXzM2Owo+PiArCQltb2RlcyB8PSBEUk1fRURJRF9IRE1JX0RD XzM2Owo+PiAgCQlEUk1fREVCVUcoIiVzOiBIRE1JIHNpbmsgZG9lcyBkZWVwIGNvbG9yIDM2Llxu IiwKPj4gIAkJCSAgY29ubmVjdG9yLT5uYW1lKTsKPj4gIAl9Cj4+ICAKPj4gIAlpZiAoaGRtaVs2 XSAmIERSTV9FRElEX0hETUlfRENfNDgpIHsKPj4gIAkJZGNfYnBjID0gMTY7Cj4+IC0JCWluZm8t PmVkaWRfaGRtaV9kY19tb2RlcyB8PSBEUk1fRURJRF9IRE1JX0RDXzQ4Owo+PiArCQltb2RlcyB8 PSBEUk1fRURJRF9IRE1JX0RDXzQ4Owo+PiAgCQlEUk1fREVCVUcoIiVzOiBIRE1JIHNpbmsgZG9l cyBkZWVwIGNvbG9yIDQ4LlxuIiwKPj4gIAkJCSAgY29ubmVjdG9yLT5uYW1lKTsKPj4gIAl9Cj4+ IEBAIC0zODA2LDkgKzM4MTAsMjQgQEAgc3RhdGljIHZvaWQgZHJtX3BhcnNlX2hkbWlfZGVlcF9j b2xvcl9pbmZvKHN0cnVjdCBkcm1fY29ubmVjdG9yICpjb25uZWN0b3IsCj4+ICAJCXJldHVybjsK Pj4gIAl9Cj4+ICAKPj4gKwkvKgo+PiArCSAqIEFsbCBkZWVwIGNvbG9yIG1vZGVzIGFyZSBvcHRp b25hbCwgYnV0IGlmIGEgc2luayBzdXBwb3J0cyBhbnkgZGVlcAo+PiArCSAqIGNvbG9yIG1vZGUs IGl0IG11c3Qgc3VwcG9ydCAzNi1iaXQgbW9kZS4gSWYgdGhpcyBpcyBmb3VuZCBub3QKPj4gKwkg KiB0byBiZSB0aGUgY2FzZSwgc2luayBpcyBpbiB2aW9sYXRpb24gb2YgSERNSSAxLjMgLyAxLjQg c3BlYyBhbmQgaXQKPj4gKwkgKiBpcyBwcnVkZW50IHRvIGRpc2FibGUgYWxsIGRlZXAgY29sb3Ig bW9kZXMuIFJldHVybiBoZXJlIGJlZm9yZQo+PiArCSAqIGNvbW1pdHRpbmcgYnBjIGFuZCBlZGlk X2hkbWlfZGNfbW9kZXMuCj4+ICsJICovCj4+ICsJaWYgKCEobW9kZXMgJiBEUk1fRURJRF9IRE1J X0RDXzM2KSkgewo+PiArCQlEUk1fREVCVUcoIiVzOiBIRE1JIHNpbmsgc2hvdWxkIGRvIERDXzM2 LCBidXQgZG9lcyBub3QhXG4iLAo+PiArCQkJICBjb25uZWN0b3ItPm5hbWUpOwo+PiArCQlyZXR1 cm47Cj4+ICsJfQo+PiArCj4+ICsKPj4gIAlEUk1fREVCVUcoIiVzOiBBc3NpZ25pbmcgSERNSSBz aW5rIGNvbG9yIGRlcHRoIGFzICVkIGJwYy5cbiIsCj4+ICAJCSAgY29ubmVjdG9yLT5uYW1lLCBk Y19icGMpOwo+PiAgCWluZm8tPmJwYyA9IGRjX2JwYzsKPj4gKwlpbmZvLT5lZGlkX2hkbWlfZGNf bW9kZXMgPSBtb2RlczsKPj4gIAo+PiAgCS8qCj4+ICAJICogRGVlcCBjb2xvciBzdXBwb3J0IG1h bmRhdGVzIFJHQjQ0NCBzdXBwb3J0IGZvciBhbGwgdmlkZW8KPj4gQEAgLTM4MjMsMTUgKzM4NDIs NiBAQCBzdGF0aWMgdm9pZCBkcm1fcGFyc2VfaGRtaV9kZWVwX2NvbG9yX2luZm8oc3RydWN0IGRy bV9jb25uZWN0b3IgKmNvbm5lY3RvciwKPj4gIAkJRFJNX0RFQlVHKCIlczogSERNSSBzaW5rIGRv ZXMgWUNSQ0I0NDQgaW4gZGVlcCBjb2xvci5cbiIsCj4+ICAJCQkgIGNvbm5lY3Rvci0+bmFtZSk7 Cj4+ICAJfQo+PiAtCj4+IC0JLyoKPj4gLQkgKiBTcGVjIHNheXMgdGhhdCBpZiBhbnkgZGVlcCBj b2xvciBtb2RlIGlzIHN1cHBvcnRlZCBhdCBhbGwsCj4+IC0JICogdGhlbiBkZWVwIGNvbG9yIDM2 IGJpdCBtdXN0IGJlIHN1cHBvcnRlZC4KPj4gLQkgKi8KPj4gLQlpZiAoIShoZG1pWzZdICYgRFJN X0VESURfSERNSV9EQ18zNikpIHsKPj4gLQkJRFJNX0RFQlVHKCIlczogSERNSSBzaW5rIHNob3Vs ZCBkbyBEQ18zNiwgYnV0IGRvZXMgbm90IVxuIiwKPj4gLQkJCSAgY29ubmVjdG9yLT5uYW1lKTsK Pj4gLQl9Cj4+ICB9Cj4+ICAKPj4gIHN0YXRpYyB2b2lkCj4+IEBAIC0zODk1LDYgKzM5MDUsNyBA QCBzdGF0aWMgdm9pZCBkcm1fYWRkX2Rpc3BsYXlfaW5mbyhzdHJ1Y3QgZHJtX2Nvbm5lY3RvciAq Y29ubmVjdG9yLAo+PiAgCS8qIGRyaXZlciBmaWd1cmVzIGl0IG91dCBpbiB0aGlzIGNhc2UgKi8K Pj4gIAlpbmZvLT5icGMgPSAwOwo+PiAgCWluZm8tPmNvbG9yX2Zvcm1hdHMgPSAwOwo+PiArCWlu Zm8tPmVkaWRfaGRtaV9kY19tb2RlcyA9IDA7Cj4+ICAJaW5mby0+Y2VhX3JldiA9IDA7Cj4+ICAJ aW5mby0+bWF4X3RtZHNfY2xvY2sgPSAwOwo+PiAgCWluZm8tPmR2aV9kdWFsID0gZmFsc2U7Cj4+ IC0tIAo+PiAyLjExLjAKCi0tIApKYW5pIE5pa3VsYSwgSW50ZWwgT3BlbiBTb3VyY2UgVGVjaG5v bG9neSBDZW50ZXIKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3Jn Cmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs Cg==