From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH] drm: disable deep color when EDID violates spec Date: Tue, 10 Jan 2017 15:18:04 +0200 Message-ID: <20170110131804.GA31595@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 mga05.intel.com (mga05.intel.com [192.55.52.43]) by gabe.freedesktop.org (Postfix) with ESMTPS id B2D546E2E7 for ; Tue, 10 Jan 2017 13:18:08 +0000 (UTC) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Ernst =?iso-8859-1?Q?Sj=F6strand?= Cc: Nicholas Sielicki , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCBKYW4gMTAsIDIwMTcgYXQgMTI6MzM6MzVQTSArMDEwMCwgRXJuc3QgU2rDtnN0cmFu ZCB3cm90ZToKPiBJc24ndCAxMGJwYyB2ZXJ5IGNvbW1vbiBhbW9uZyBtb25pdG9ycywgYW5kIDEy YnBjIHZlcnkgcmFyZT8gT3IgbWF5YmUgSSdtCj4gY29uZnVzaW5nIHRoZSB0cmFuc3BvcnQgbGF5 ZXIgd2l0aCB0aGUgcHJlc2VudGF0aW9uIGNhcGFiaWxpdGllcy4uLj8KPiBIZXJlIGFyZSAyMDEg bW9uaXRvcnMgdGhhdCBjbGFpbSAxMGJwYzoKPiBodHRwOi8vcHJpY2VzcHkuY28udWsvY2F0ZWdv cnkucGhwP2w9czMwMDg1OTQzNCZvPWVnXzQwMSNwcm9kbGlzdGEKCkkgc3VwcG9zZSB0aGF0IHJl ZmVycyB0byB0aGUgcGFuZWw/IE5vdCBzdXJlLgoKPiAKPiBSZWdhcmRzCj4gLy9Fcm5zdAo+IAo+ IDIwMTctMDEtMTAgMTE6NTIgR01UKzAxOjAwIFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFs YUBsaW51eC5pbnRlbC5jb20+Ogo+IAo+ID4gT24gVGh1LCBKYW4gMDUsIDIwMTcgYXQgMDU6NDU6 MjNQTSAtMDYwMCwgTmljaG9sYXMgU2llbGlja2kgd3JvdGU6Cj4gPiA+IEFzIHBlciB0aGUgSERN SSAxLjMgYW5kIDEuNCBzcGVjLCAiZGVlcCBjb2xvciBtb2RlcyIgYXJlIGNvbG9yIGRlcHRocwo+ ID4gPiBncmVhdGVyIHRoYW4gMjQgYml0cyBwZXIgcGl4ZWwuIFRoZSBzcGVjIGV4cGxpY2l0bHkg c3RhdGVzLCAiQWxsIERlZXAKPiA+ID4gQ29sb3IgbW9kZXMgYXJlIG9wdGlvbmFsIHRob3VnaCBp ZiBhbiBIRE1JIFNvdXJjZSBvciBTaW5rIHN1cHBvcnRzIGFueQo+ID4gPiBEZWVwIENvbG9yIG1v ZGUsIGl0IHNoYWxsIHN1cHBvcnQgMzYtYml0IG1vZGUuIiAoNi4yLjQgQ29sb3IgRGVwdGgKPiA+ ID4gUmVxdWlyZW1lbnRzKS4KPiA+ID4KPiA+ID4gSSBjYW1lIGFjcm9zcyBhIG1vbml0b3IgKEFj ZXIgWDIzM0gpIHRoYXQgc3VwcGxpZXMgYW4gaWxsZWdhbCBFRElEIHdoZXJlCj4gPiA+IERDXzMw Yml0IGlzIHNldCBhbmQgRENfMzZiaXQgaXMgbm90IHNldC4gVGhlIGVuZCByZXN1bHQgaXMgYmFk bHkgZ2FyYmxlZAo+ID4gPiBvdXRwdXQgYmVjYXVzZSB0aGUgZHJpdmVyIGlzIHNlbmRpbmcgMzZC UFAgd2hlbiB0aGUgbW9uaXRvciBjYW4ndCBoYW5kbGUKPiA+ID4gaXQuCj4gPiA+Cj4gPiA+IE11 Y2ggb2YgdGhlIGludGVsIGhhcmR3YXJlIGlzIGluY2FwYWJsZSBvZiBvcGVyYXRpbmcgYXQgYW55 Cj4gPiA+IGJpdC1wZXItY29tcG9uZW50IHNldHRpbmcgb3V0c2lkZSBvZiA4IG9yIDEyLCBhbmQg dGhlIHNwZWMgc2VlbXMgdG8KPiA+ID4gaW1wbHkgdGhhdCBpZiBhbnkgZGVlcCBjb2xvciBzdXBw b3J0IGlzIGZvdW5kLCB0aGVuIGl0IGlzIGEgc2FmZQo+ID4gPiBhc3N1bXB0aW9uIHRvIG9wZXJh dGUgYXQgMTIuCj4gPiA+Cj4gPiA+IFRoaXMgcGF0Y2ggZW5zdXJlcyB0aGF0IHRoZSBFRElEIGlz IHdpdGhpbiB0aGUgc3BlYyAoc3BlY2lmaWNhbGx5LCB0aGF0Cj4gPiA+IERDXzM2Yml0IGlzIHNl dCkgYmVmb3JlIGNvbW1pdHRpbmcgdG8gZ29pbmcgZm9yd2FyZCB3aXRoIGFueSBkZWVwCj4gPiA+ IGNvbG9ycy4gIFRoZXJlIHdhcyBhbHJlYWR5IGEgY2hlY2sgZm9yIHRoaXMgRURJRCBpbmNvbnNp c3RlbmN5LCBidXQgaXQKPiA+ID4gcmVzdWx0ZWQgb25seSBpbiBhIHdhcm5pbmcgYW5kIGRpZCBu b3QgZmFsbC1iYWNrIHRvIHNhZmVyIHNldHRpbmdzLgo+ID4gPgo+ID4gPiBDQzogVmlsbGUgU3ly asOkbMOkIDx2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4KPiA+ID4gU2lnbmVkLW9mZi1i eTogTmljaG9sYXMgU2llbGlja2kgPG5pY2hvbGFzLnNpZWxpY2tpQGdtYWlsLmNvbT4KPiA+ID4g LS0tCj4gPiA+ICBkcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYyB8IDM1ICsrKysrKysrKysrKysr KysrKysrKysrLS0tLS0tLS0tLS0tCj4gPiA+ICAxIGZpbGUgY2hhbmdlZCwgMjMgaW5zZXJ0aW9u cygrKSwgMTIgZGVsZXRpb25zKC0pCj4gPiA+Cj4gPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dw dS9kcm0vZHJtX2VkaWQuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jCj4gPiA+IGluZGV4 IDMzNmJlMzFmZjNkZS4uNDJjZTNmNTRkMmRjIDEwMDY0NAo+ID4gPiAtLS0gYS9kcml2ZXJzL2dw dS9kcm0vZHJtX2VkaWQuYwo+ID4gPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYwo+ ID4gPiBAQCAtMzc3MiwzMCArMzc3MiwzNCBAQCBzdGF0aWMgdm9pZCBkcm1fcGFyc2VfaGRtaV9k ZWVwX2NvbG9yX2luZm8oc3RydWN0Cj4gPiBkcm1fY29ubmVjdG9yICpjb25uZWN0b3IsCj4gPiA+ ICB7Cj4gPiA+ICAgICAgIHN0cnVjdCBkcm1fZGlzcGxheV9pbmZvICppbmZvID0gJmNvbm5lY3Rv ci0+ZGlzcGxheV9pbmZvOwo+ID4gPiAgICAgICB1bnNpZ25lZCBpbnQgZGNfYnBjID0gMDsKPiA+ ID4gKyAgICAgdTggbW9kZXMgPSAwOwo+ID4gPgo+ID4gPiAgICAgICAvKiBIRE1JIHN1cHBvcnRz IGF0IGxlYXN0IDggYnBjICovCj4gPiA+ICAgICAgIGluZm8tPmJwYyA9IDg7Cj4gPiA+Cj4gPiA+ ICsgICAgIC8qIEVuc3VyZSBhbGwgREMgbW9kZXMgYXJlIHVuc2V0IGlmIHdlIHJldHVybiBlYXJs eSAqLwo+ID4gPiArICAgICBpbmZvLT5lZGlkX2hkbWlfZGNfbW9kZXMgPSAwOwo+ID4KPiA+IENs ZWFyaW5nIHRoaXMgaW4gZHJtX2FkZF9kaXNwbGF5X2luZm8oKSBzaG91bGQgYmUgc3VmZmljaWVu dCBzaW5jZQo+ID4gdGhpcyBndXkgZG9lc24ndCBnZXQgY2FsbGVkIGZyb20gYW55d2hlcmUgZWxz ZS4gU28gdGhpcyBwYXJ0IGNvdWxkCj4gPiBiZSBkcm9wcHBlZC4KPiA+Cj4gPiBPdGhlcndpc2Ug dGhpcyBmZWVscyBsaWtlIGEgZGVjZW50IHdheSB0byBoYW5kbGUgdGhlIHByb2JsZW0uIFdlCj4g PiBjb3VsZCBvZiBjb3Vyc2UgdHJ5IHRvIHVzZSB0aGUgMTBicGMgKG9yIHdoYXRldmVyKSBkZWVw IGNvbG9yIG1vZGVzCj4gPiB0aGUgc2luayBjbGFpbXMgdG8gc3VwcG9ydCwgYnV0IGdpdmVuIHRo YXQgdGhlIHBlb3BsZSBkZXNpZ25pbmcgdGhlCj4gPiB0aGluZyBkaWRuJ3QgYm90aGVyIHJlYWRp bmcgdGhlIHNwZWMsIGl0IHNlZW1zIHNhZmVyIHRvIGp1c3QgZGlzYWJsZQo+ID4gZGVlcCBjb2xv ciBzdXBwb3J0IGVudGlyZWx5Lgo+ID4KPiA+IFJldmlld2VkLWJ5OiBWaWxsZSBTeXJqw6Rsw6Qg PHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+ID4KPiA+ID4gKwo+ID4gPiAgICAgICBp ZiAoY2VhX2RiX3BheWxvYWRfbGVuKGhkbWkpIDwgNikKPiA+ID4gICAgICAgICAgICAgICByZXR1 cm47Cj4gPiA+Cj4gPiA+ICAgICAgIGlmIChoZG1pWzZdICYgRFJNX0VESURfSERNSV9EQ18zMCkg ewo+ID4gPiAgICAgICAgICAgICAgIGRjX2JwYyA9IDEwOwo+ID4gPiAtICAgICAgICAgICAgIGlu Zm8tPmVkaWRfaGRtaV9kY19tb2RlcyB8PSBEUk1fRURJRF9IRE1JX0RDXzMwOwo+ID4gPiArICAg ICAgICAgICAgIG1vZGVzIHw9IERSTV9FRElEX0hETUlfRENfMzA7Cj4gPiA+ICAgICAgICAgICAg ICAgRFJNX0RFQlVHKCIlczogSERNSSBzaW5rIGRvZXMgZGVlcCBjb2xvciAzMC5cbiIsCj4gPiA+ ICAgICAgICAgICAgICAgICAgICAgICAgIGNvbm5lY3Rvci0+bmFtZSk7Cj4gPiA+ICAgICAgIH0K PiA+ID4KPiA+ID4gICAgICAgaWYgKGhkbWlbNl0gJiBEUk1fRURJRF9IRE1JX0RDXzM2KSB7Cj4g PiA+ICAgICAgICAgICAgICAgZGNfYnBjID0gMTI7Cj4gPiA+IC0gICAgICAgICAgICAgaW5mby0+ ZWRpZF9oZG1pX2RjX21vZGVzIHw9IERSTV9FRElEX0hETUlfRENfMzY7Cj4gPiA+ICsgICAgICAg ICAgICAgbW9kZXMgfD0gRFJNX0VESURfSERNSV9EQ18zNjsKPiA+ID4gICAgICAgICAgICAgICBE Uk1fREVCVUcoIiVzOiBIRE1JIHNpbmsgZG9lcyBkZWVwIGNvbG9yIDM2LlxuIiwKPiA+ID4gICAg ICAgICAgICAgICAgICAgICAgICAgY29ubmVjdG9yLT5uYW1lKTsKPiA+ID4gICAgICAgfQo+ID4g Pgo+ID4gPiAgICAgICBpZiAoaGRtaVs2XSAmIERSTV9FRElEX0hETUlfRENfNDgpIHsKPiA+ID4g ICAgICAgICAgICAgICBkY19icGMgPSAxNjsKPiA+ID4gLSAgICAgICAgICAgICBpbmZvLT5lZGlk X2hkbWlfZGNfbW9kZXMgfD0gRFJNX0VESURfSERNSV9EQ180ODsKPiA+ID4gKyAgICAgICAgICAg ICBtb2RlcyB8PSBEUk1fRURJRF9IRE1JX0RDXzQ4Owo+ID4gPiAgICAgICAgICAgICAgIERSTV9E RUJVRygiJXM6IEhETUkgc2luayBkb2VzIGRlZXAgY29sb3IgNDguXG4iLAo+ID4gPiAgICAgICAg ICAgICAgICAgICAgICAgICBjb25uZWN0b3ItPm5hbWUpOwo+ID4gPiAgICAgICB9Cj4gPiA+IEBA IC0zODA2LDkgKzM4MTAsMjQgQEAgc3RhdGljIHZvaWQgZHJtX3BhcnNlX2hkbWlfZGVlcF9jb2xv cl9pbmZvKHN0cnVjdAo+ID4gZHJtX2Nvbm5lY3RvciAqY29ubmVjdG9yLAo+ID4gPiAgICAgICAg ICAgICAgIHJldHVybjsKPiA+ID4gICAgICAgfQo+ID4gPgo+ID4gPiArICAgICAvKgo+ID4gPiAr ICAgICAgKiBBbGwgZGVlcCBjb2xvciBtb2RlcyBhcmUgb3B0aW9uYWwsIGJ1dCBpZiBhIHNpbmsg c3VwcG9ydHMgYW55Cj4gPiBkZWVwCj4gPiA+ICsgICAgICAqIGNvbG9yIG1vZGUsIGl0IG11c3Qg c3VwcG9ydCAzNi1iaXQgbW9kZS4gSWYgdGhpcyBpcyBmb3VuZCBub3QKPiA+ID4gKyAgICAgICog dG8gYmUgdGhlIGNhc2UsIHNpbmsgaXMgaW4gdmlvbGF0aW9uIG9mIEhETUkgMS4zIC8gMS40IHNw ZWMgYW5kCj4gPiBpdAo+ID4gPiArICAgICAgKiBpcyBwcnVkZW50IHRvIGRpc2FibGUgYWxsIGRl ZXAgY29sb3IgbW9kZXMuIFJldHVybiBoZXJlIGJlZm9yZQo+ID4gPiArICAgICAgKiBjb21taXR0 aW5nIGJwYyBhbmQgZWRpZF9oZG1pX2RjX21vZGVzLgo+ID4gPiArICAgICAgKi8KPiA+ID4gKyAg ICAgaWYgKCEobW9kZXMgJiBEUk1fRURJRF9IRE1JX0RDXzM2KSkgewo+ID4gPiArICAgICAgICAg ICAgIERSTV9ERUJVRygiJXM6IEhETUkgc2luayBzaG91bGQgZG8gRENfMzYsIGJ1dCBkb2VzIG5v dCFcbiIsCj4gPiA+ICsgICAgICAgICAgICAgICAgICAgICAgIGNvbm5lY3Rvci0+bmFtZSk7Cj4g PiA+ICsgICAgICAgICAgICAgcmV0dXJuOwo+ID4gPiArICAgICB9Cj4gPiA+ICsKPiA+ID4gKwo+ ID4gPiAgICAgICBEUk1fREVCVUcoIiVzOiBBc3NpZ25pbmcgSERNSSBzaW5rIGNvbG9yIGRlcHRo IGFzICVkIGJwYy5cbiIsCj4gPiA+ICAgICAgICAgICAgICAgICBjb25uZWN0b3ItPm5hbWUsIGRj X2JwYyk7Cj4gPiA+ICAgICAgIGluZm8tPmJwYyA9IGRjX2JwYzsKPiA+ID4gKyAgICAgaW5mby0+ ZWRpZF9oZG1pX2RjX21vZGVzID0gbW9kZXM7Cj4gPiA+Cj4gPiA+ICAgICAgIC8qCj4gPiA+ICAg ICAgICAqIERlZXAgY29sb3Igc3VwcG9ydCBtYW5kYXRlcyBSR0I0NDQgc3VwcG9ydCBmb3IgYWxs IHZpZGVvCj4gPiA+IEBAIC0zODIzLDE1ICszODQyLDYgQEAgc3RhdGljIHZvaWQgZHJtX3BhcnNl X2hkbWlfZGVlcF9jb2xvcl9pbmZvKHN0cnVjdAo+ID4gZHJtX2Nvbm5lY3RvciAqY29ubmVjdG9y LAo+ID4gPiAgICAgICAgICAgICAgIERSTV9ERUJVRygiJXM6IEhETUkgc2luayBkb2VzIFlDUkNC NDQ0IGluIGRlZXAgY29sb3IuXG4iLAo+ID4gPiAgICAgICAgICAgICAgICAgICAgICAgICBjb25u ZWN0b3ItPm5hbWUpOwo+ID4gPiAgICAgICB9Cj4gPiA+IC0KPiA+ID4gLSAgICAgLyoKPiA+ID4g LSAgICAgICogU3BlYyBzYXlzIHRoYXQgaWYgYW55IGRlZXAgY29sb3IgbW9kZSBpcyBzdXBwb3J0 ZWQgYXQgYWxsLAo+ID4gPiAtICAgICAgKiB0aGVuIGRlZXAgY29sb3IgMzYgYml0IG11c3QgYmUg c3VwcG9ydGVkLgo+ID4gPiAtICAgICAgKi8KPiA+ID4gLSAgICAgaWYgKCEoaGRtaVs2XSAmIERS TV9FRElEX0hETUlfRENfMzYpKSB7Cj4gPiA+IC0gICAgICAgICAgICAgRFJNX0RFQlVHKCIlczog SERNSSBzaW5rIHNob3VsZCBkbyBEQ18zNiwgYnV0IGRvZXMgbm90IVxuIiwKPiA+ID4gLSAgICAg ICAgICAgICAgICAgICAgICAgY29ubmVjdG9yLT5uYW1lKTsKPiA+ID4gLSAgICAgfQo+ID4gPiAg fQo+ID4gPgo+ID4gPiAgc3RhdGljIHZvaWQKPiA+ID4gQEAgLTM4OTUsNiArMzkwNSw3IEBAIHN0 YXRpYyB2b2lkIGRybV9hZGRfZGlzcGxheV9pbmZvKHN0cnVjdAo+ID4gZHJtX2Nvbm5lY3RvciAq Y29ubmVjdG9yLAo+ID4gPiAgICAgICAvKiBkcml2ZXIgZmlndXJlcyBpdCBvdXQgaW4gdGhpcyBj YXNlICovCj4gPiA+ICAgICAgIGluZm8tPmJwYyA9IDA7Cj4gPiA+ICAgICAgIGluZm8tPmNvbG9y X2Zvcm1hdHMgPSAwOwo+ID4gPiArICAgICBpbmZvLT5lZGlkX2hkbWlfZGNfbW9kZXMgPSAwOwo+ ID4gPiAgICAgICBpbmZvLT5jZWFfcmV2ID0gMDsKPiA+ID4gICAgICAgaW5mby0+bWF4X3RtZHNf Y2xvY2sgPSAwOwo+ID4gPiAgICAgICBpbmZvLT5kdmlfZHVhbCA9IGZhbHNlOwo+ID4gPiAtLQo+ ID4gPiAyLjExLjAKPiA+Cj4gPiAtLQo+ID4gVmlsbGUgU3lyasOkbMOkCj4gPiBJbnRlbCBPVEMK PiA+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCj4gPiBk cmktZGV2ZWwgbWFpbGluZyBsaXN0Cj4gPiBkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3Jn Cj4gPiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1k ZXZlbAo+ID4KCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJp LWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9y Zy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=