From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Sharma, Shashank" Subject: Re: [PATCH 06/10] drm/edid: Fix cea mode aspect ratio handling Date: Fri, 17 Nov 2017 08:49:49 +0530 Message-ID: References: <20171113170427.4150-1-ville.syrjala@linux.intel.com> <20171113170427.4150-7-ville.syrjala@linux.intel.com> <67ddbed5-421d-d81f-9e0f-7ad4055f3c4e@intel.com> <20171116162357.GX10981@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20171116162357.GX10981@intel.com> 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?= Cc: Jose Abreu , "Lin, Jia" , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Daniel Vetter , Nautiyal Ankit List-Id: dri-devel@lists.freedesktop.org UmVnYXJkcwoKU2hhc2hhbmsKCgpPbiAxMS8xNi8yMDE3IDk6NTMgUE0sIFZpbGxlIFN5cmrDpGzD pCB3cm90ZToKPiBPbiBUaHUsIE5vdiAxNiwgMjAxNyBhdCAwODoyMTo0NFBNICswNTMwLCBTaGFy bWEsIFNoYXNoYW5rIHdyb3RlOgo+PiBSZWdhcmRzCj4+Cj4+IFNoYXNoYW5rCj4+Cj4+Cj4+IE9u IDExLzEzLzIwMTcgMTA6MzQgUE0sIFZpbGxlIFN5cmphbGEgd3JvdGU6Cj4+PiBGcm9tOiBWaWxs ZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+Pj4KPj4+IGNvbW1p dCA2ZGZmZDQzMWUyMjkgKCJkcm06IEFkZCBhc3BlY3QgcmF0aW8gcGFyc2luZyBpbiBEUk0gbGF5 ZXIiKQo+Pj4gY2F1c2UgdXMgdG8gbm90IHNlbmQgb3V0IGFueSBWSUNzIGluIHRoZSBBVkkgaW5m b2ZyYW1lcy4gVGhhdCBjb21taXQKPj4+IHdhcyBzaW5jZSByZXZlcnRlZCwgYnV0IGlmIGFuZCB3 aGVuIHdlIGFkZCBhc3BlY3QgcmF0aW8gaGFuZGluZyBiYWNrCj4+PiB3ZSBuZWVkIHRvIGJlIG1v cmUgY2FyZWZ1bC4KPj4+Cj4+PiBMZXQncyBoYW5kbGUgdGhpcyBieSBjb25zaWRlcmluZyB0aGUg YXNwZWN0IHJhdGlvIGFzIGEgcmVxdWlyZW1lbnQKPj4+IGZvciBjZWEgbW9kZSBtYXRjaGluZyBv bmx5IGlmIHRoZSBwYXNzZWQgaW4gbW9kZSBhY3R1YWxseSBoYXMgYQo+Pj4gbm9uLXplcm8gYXNw ZWN0IHJhdGlvIGZpZWxkLiBUaGlzIHdpbGwga2VlcCB1c2Vyc3BhY2UgdGhhdCBkb2Vzbid0Cj4+ PiBwcm92aWRlIGFuIGFzcGVjdCByYXRpbyB3b3JraW5nIGFzIGJlZm9yZSBieSBtYXRjaGluZyBp dCB0byB0aGUKPj4+IGZpcnN0IG90aGVyd2lzZSBlcXVhbCBjZWEgbW9kZS4gQW5kIG9uY2UgdXNl cnNwYWNlIHN0YXJ0cyB0bwo+Pj4gcHJvdmlkZSB0aGUgYXNwZWN0IHJhdGlvIGl0IHdpbGwgYmUg Y29uc2lkZXJkIGEgaGFyZCByZXF1aXJlbWVudAo+Pj4gZm9yIHRoZSBtYXRjaC4KPj4+Cj4+PiBB bHNvIGNoYW5nZSB0aGUgaGRtaSBtb2RlIG1hdGNoaW5nIHRvIHVzZSBkcm1fbW9kZV9tYXRjaCgp IGZvcgo+Pj4gY29uc2lzdGVuY3ksIGJ1dCB3ZSBkb24ndCBtYXRjaCBvbiBhc3BlY3QgcmF0aW8g dGhlcmUgc2luY2UgdGhlCj4+PiBzcGVjIGRvZXNuJ3QgbGlzdCBhIHNwZWNpZmljIGFzcGVjdCBy YXRpbyBmb3IgdGhvc2UgbW9kZXMuCj4+Pgo+Pj4gQ2M6IFNoYXNoYW5rIFNoYXJtYSA8c2hhc2hh bmsuc2hhcm1hQGludGVsLmNvbT4KPj4+IENjOiAiTGluLCBKaWEiIDxsaW4uYS5qaWFAaW50ZWwu Y29tPgo+Pj4gQ2M6IEFrYXNoZGVlcCBTaGFybWEgPGFrYXNoZGVlcC5zaGFybWFAaW50ZWwuY29t Pgo+Pj4gQ2M6IEppbSBCcmlkZSA8amltLmJyaWRlQGxpbnV4LmludGVsLmNvbT4KPj4+IENjOiBK b3NlIEFicmV1IDxKb3NlLkFicmV1QHN5bm9wc3lzLmNvbT4KPj4+IENjOiBEYW5pZWwgVmV0dGVy IDxkYW5pZWwudmV0dGVyQGZmd2xsLmNoPgo+Pj4gQ2M6IEVtaWwgVmVsaWtvdiA8ZW1pbC5sLnZl bGlrb3ZAZ21haWwuY29tPgo+Pj4gU2lnbmVkLW9mZi1ieTogVmlsbGUgU3lyasOkbMOkIDx2aWxs ZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4KPj4+IC0tLQo+Pj4gICAgZHJpdmVycy9ncHUvZHJt L2RybV9lZGlkLmMgfCAxOCArKysrKysrKysrKysrKy0tLS0KPj4+ICAgIDEgZmlsZSBjaGFuZ2Vk LCAxNCBpbnNlcnRpb25zKCspLCA0IGRlbGV0aW9ucygtKQo+Pj4KPj4+IGRpZmYgLS1naXQgYS9k cml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jCj4+ PiBpbmRleCA3MjIwYjhmOWE3ZTguLjAwYWE5OGYzZTU1ZCAxMDA2NDQKPj4+IC0tLSBhL2RyaXZl cnMvZ3B1L2RybS9kcm1fZWRpZC5jCj4+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQu Ywo+Pj4gQEAgLTI5MDMsMTEgKzI5MDMsMTUgQEAgY2VhX21vZGVfYWx0ZXJuYXRlX3RpbWluZ3Mo dTggdmljLCBzdHJ1Y3QgZHJtX2Rpc3BsYXlfbW9kZSAqbW9kZSkKPj4+ICAgIHN0YXRpYyB1OCBk cm1fbWF0Y2hfY2VhX21vZGVfY2xvY2tfdG9sZXJhbmNlKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxh eV9tb2RlICp0b19tYXRjaCwKPj4+ICAgIAkJCQkJICAgICB1bnNpZ25lZCBpbnQgY2xvY2tfdG9s ZXJhbmNlKQo+Pj4gICAgewo+Pj4gKwl1bnNpZ25lZCBpbnQgbWF0Y2hfZmxhZ3MgPSBEUk1fTU9E RV9NQVRDSF9USU1JTkdTIHwgRFJNX01PREVfTUFUQ0hfRkxBR1M7Cj4+PiAgICAJdTggdmljOwo+ Pj4gICAgCj4+PiAgICAJaWYgKCF0b19tYXRjaC0+Y2xvY2spCj4+PiAgICAJCXJldHVybiAwOwo+ Pj4gICAgCj4+PiArCWlmICh0b19tYXRjaC0+cGljdHVyZV9hc3BlY3RfcmF0aW8pCj4+PiArCQlt YXRjaF9mbGFncyB8PSBEUk1fTU9ERV9NQVRDSF9BU1BFQ1RfUkFUSU87Cj4+IFRoaXMgZG9lc24n dCBsb29rIHJpZ2h0LiBUaGlzIG1lYW5zIHdlIGFyZSBleHBlY3RpbmcgYSBDRUEgbW9kZSB3aXRo b3V0Cj4+IGEgcGljIGFzcGVjdCByYXRpbyBmaWVsZCwKPj4gd2hpY2ggaXMgaW52YWxpZC4KPiBO bywgaXQncyBwZXJmZWN0bHkgdmFsaWQuIEl0J3Mgd2hhdCB3ZSBjdXJyZW50bHkgZ2V0IGZyb20g dXNlcnNwYWNlLgpZZXAsIGJ1dCB0aGF0J3MgZHVlIHRvIG1pc3NpbmcgQXNwZWN0IHJhdGlvIGhh bmRsaW5nIGluIHRoZSBEUk0gbGF5ZXIuIApJZiB0aGF0J3MgZml4ZWQsIGFzIHBlciB0aGUgbGlz dCBvZiBDRUEgbW9kZXMsCmVhY2ggQ0VBIFZJQyBjb250YWlucyBhbiBhc3BlY3QgcmF0aW8sIHdo aWNoIGlzIGEgcGFydCBvZiBpdHMgdW5pcXVlIAppZGVudGl0eS4KCkkgZ3Vlc3Mgb25jZSB3ZSBo YXZlIHRoZSBhc3BlY3QgcmF0aW8gaGFuZGxpbmcgaW4gRFJNIGxheWVyLCBpdCAKd291bGQvc2hv dWxkIGxvb2sgbGlrZSB0aGlzOgotIEVESUQgZ2l2ZXMgeW91IGFsbCBzdXBwb3J0ZWQgbW9kZXMs IGluY2x1ZGluZyBDRUEgbW9kZXMgd2l0aCBBc3BlY3QgcmF0aW8KLSBVc2Vyc3BjYWUgZ2V0cyB0 aGUgbW9kZSBpbmZvcm1hdGlvbiwgd2l0aCBhc3BlY3QgcmF0aW8gKGZvciBDRUEgbW9kZXMpCklm ICggVXNlcnNwYWNlIHBpY2tzIG9uZSBvZiB0aGUgQ0VBIG1vZGVzKQogICAgIC0gc2VuZHMgYSBt b2Rlc2V0CiAgICAgLSB3ZSBmaW5kIGEgbWF0Y2hpbmcgQ0VBIFZJQywgZm91bmQgb25lIGZyb20g bW9kZWRiCiAgICAgLSB3ZSBsb2FkIHRoaXMgVklDID0gbm9uemVybyBpbmZvcm1hdGlvbiBpbiBB VkkgSUYgVklDIGZpZWxkLAplbHNlCiAgICAgLSBzZW5kcyBhIG1vZGVzZXQKICAgICAtIHdlIGNv dWxkIG5vdCBmaW5kIGEgbWF0Y2hpbmcgQ0VBIFZJQywgYXMgYXNwZWN0IHJhdGlvIGlzIDAKICAg ICAtIHdlIG1ha2UgVklDIGZpZWxkIGluIEFWSSBJRiBhcyAwCgpUaGlzIGlzIGltcG9ydGFudCwg YXMgSERNSSBjb21wbGlhbmNlIHRlc3QgNy0yNyBpbnNwZWN0cyBpZiB0aGUgVklDIApmaWVsZCBp biB0aGUgQVZJIElGIGlzIGFjY3VyYXRlLgoKLSBTaGFzaGFuawo+PiBBbmtpdCBpcyBnb2luZyB0 byBwdWJsaXNoIHRoZSBhc3BlY3QgcmF0aW8gcGF0Y2gKPj4gc2VyaWVzIGFnYWluLCB3aXRoIHBy b3BlciBEUk0gY2FwIGFuZCBmbGFncyBjaGVjay4gV291bGQgaXQgYmUKPj4gb2sgaWYgd2UgaGF2 ZSBhIGxvb2sgdGhhdCBoYW5kbGluZyBmaXJzdCA/Cj4gVGhpcyBwYXRjaCB3aWxsIGJlIG5lZWRl ZCBieSB0aGF0IHdvcmsuIE90aGVyd2lzZSB3ZSdyZSBnb2luZyB0byBzdG9wCj4gc2VuZGluZyBh IFZJQyBmb3IgQ0VBIG1vZGVzIHdpdGggY3VycmVudCB1c2Vyc3BhY2UuCkkgZ3Vlc3Mgd2Ugc2hv dWxkIGZvcmNlIHVzZXJzcGFjZXMgdG8gc3RhcnQgYm90aGVyaW5nIGFib3V0IGFzcGVjdCByYXRp byAKZmllbGQsIHJpZ2h0IG5vdyB3ZQphcmUgZG9pbmcgdGhpcyBmb3IgV2F5bGFuZCBiYXNlZCBj b21wb3NpdG9ycywgbWF5IGJlIHdlIHNob3VsZCBleHRlbmQgaXQgCnRvIFggYmFzZWQgdG9vLgoK LSBTaGFzaGFuawo+Cj4+PiArCj4+PiAgICAJZm9yICh2aWMgPSAxOyB2aWMgPCBBUlJBWV9TSVpF KGVkaWRfY2VhX21vZGVzKTsgdmljKyspIHsKPj4+ICAgIAkJc3RydWN0IGRybV9kaXNwbGF5X21v ZGUgY2VhX21vZGUgPSBlZGlkX2NlYV9tb2Rlc1t2aWNdOwo+Pj4gICAgCQl1bnNpZ25lZCBpbnQg Y2xvY2sxLCBjbG9jazI7Cj4+PiBAQCAtMjkyMSw3ICsyOTI1LDcgQEAgc3RhdGljIHU4IGRybV9t YXRjaF9jZWFfbW9kZV9jbG9ja190b2xlcmFuY2UoY29uc3Qgc3RydWN0IGRybV9kaXNwbGF5X21v ZGUgKnRvX20KPj4+ICAgIAkJCWNvbnRpbnVlOwo+Pj4gICAgCj4+PiAgICAJCWRvIHsKPj4+IC0J CQlpZiAoZHJtX21vZGVfZXF1YWxfbm9fY2xvY2tzX25vX3N0ZXJlbyh0b19tYXRjaCwgJmNlYV9t b2RlKSkKPj4+ICsJCQlpZiAoZHJtX21vZGVfbWF0Y2godG9fbWF0Y2gsICZjZWFfbW9kZSwgbWF0 Y2hfZmxhZ3MpKQo+Pj4gICAgCQkJCXJldHVybiB2aWM7Cj4+PiAgICAJCX0gd2hpbGUgKGNlYV9t b2RlX2FsdGVybmF0ZV90aW1pbmdzKHZpYywgJmNlYV9tb2RlKSk7Cj4+PiAgICAJfQo+Pj4gQEAg LTI5MzgsMTEgKzI5NDIsMTUgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9jZWFfbW9kZV9jbG9ja190 b2xlcmFuY2UoY29uc3Qgc3RydWN0IGRybV9kaXNwbGF5X21vZGUgKnRvX20KPj4+ICAgICAqLwo+ Pj4gICAgdTggZHJtX21hdGNoX2NlYV9tb2RlKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2Rl ICp0b19tYXRjaCkKPj4+ICAgIHsKPj4+ICsJdW5zaWduZWQgaW50IG1hdGNoX2ZsYWdzID0gRFJN X01PREVfTUFUQ0hfVElNSU5HUyB8IERSTV9NT0RFX01BVENIX0ZMQUdTOwo+Pj4gICAgCXU4IHZp YzsKPj4+ICAgIAo+Pj4gICAgCWlmICghdG9fbWF0Y2gtPmNsb2NrKQo+Pj4gICAgCQlyZXR1cm4g MDsKPj4+ICAgIAo+Pj4gKwlpZiAodG9fbWF0Y2gtPnBpY3R1cmVfYXNwZWN0X3JhdGlvKQo+Pj4g KwkJbWF0Y2hfZmxhZ3MgfD0gRFJNX01PREVfTUFUQ0hfQVNQRUNUX1JBVElPOwo+PiBzYW1lIGhl cmUsIGFuZCBwcm9iYWJseSBpbiBvdGhlciBDRUEgbWF0Y2ggZnVuY3Rpb25zLgo+Pj4gKwo+Pj4g ICAgCWZvciAodmljID0gMTsgdmljIDwgQVJSQVlfU0laRShlZGlkX2NlYV9tb2Rlcyk7IHZpYysr KSB7Cj4+PiAgICAJCXN0cnVjdCBkcm1fZGlzcGxheV9tb2RlIGNlYV9tb2RlID0gZWRpZF9jZWFf bW9kZXNbdmljXTsKPj4+ICAgIAkJdW5zaWduZWQgaW50IGNsb2NrMSwgY2xvY2syOwo+Pj4gQEAg LTI5NTYsNyArMjk2NCw3IEBAIHU4IGRybV9tYXRjaF9jZWFfbW9kZShjb25zdCBzdHJ1Y3QgZHJt X2Rpc3BsYXlfbW9kZSAqdG9fbWF0Y2gpCj4+PiAgICAJCQljb250aW51ZTsKPj4+ICAgIAo+Pj4g ICAgCQlkbyB7Cj4+PiAtCQkJaWYgKGRybV9tb2RlX2VxdWFsX25vX2Nsb2Nrc19ub19zdGVyZW8o dG9fbWF0Y2gsICZjZWFfbW9kZSkpCj4+PiArCQkJaWYgKGRybV9tb2RlX21hdGNoKHRvX21hdGNo LCAmY2VhX21vZGUsIG1hdGNoX2ZsYWdzKSkKPj4+ICAgIAkJCQlyZXR1cm4gdmljOwo+Pj4gICAg CQl9IHdoaWxlIChjZWFfbW9kZV9hbHRlcm5hdGVfdGltaW5ncyh2aWMsICZjZWFfbW9kZSkpOwo+ Pj4gICAgCX0KPj4+IEBAIC0zMDAzLDYgKzMwMTEsNyBAQCBoZG1pX21vZGVfYWx0ZXJuYXRlX2Ns b2NrKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICpoZG1pX21vZGUpCj4+PiAgICBzdGF0 aWMgdTggZHJtX21hdGNoX2hkbWlfbW9kZV9jbG9ja190b2xlcmFuY2UoY29uc3Qgc3RydWN0IGRy bV9kaXNwbGF5X21vZGUgKnRvX21hdGNoLAo+Pj4gICAgCQkJCQkgICAgICB1bnNpZ25lZCBpbnQg Y2xvY2tfdG9sZXJhbmNlKQo+Pj4gICAgewo+Pj4gKwl1bnNpZ25lZCBpbnQgbWF0Y2hfZmxhZ3Mg PSBEUk1fTU9ERV9NQVRDSF9USU1JTkdTIHwgRFJNX01PREVfTUFUQ0hfRkxBR1M7Cj4+PiAgICAJ dTggdmljOwo+Pj4gICAgCj4+PiAgICAJaWYgKCF0b19tYXRjaC0+Y2xvY2spCj4+PiBAQCAtMzAy MCw3ICszMDI5LDcgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9oZG1pX21vZGVfY2xvY2tfdG9sZXJh bmNlKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICp0b18KPj4+ICAgIAkJICAgIGFicyh0 b19tYXRjaC0+Y2xvY2sgLSBjbG9jazIpID4gY2xvY2tfdG9sZXJhbmNlKQo+Pj4gICAgCQkJY29u dGludWU7Cj4+PiAgICAKPj4+IC0JCWlmIChkcm1fbW9kZV9lcXVhbF9ub19jbG9ja3Nfbm9fc3Rl cmVvKHRvX21hdGNoLCBoZG1pX21vZGUpKQo+Pj4gKwkJaWYgKGRybV9tb2RlX21hdGNoKHRvX21h dGNoLCBoZG1pX21vZGUsIG1hdGNoX2ZsYWdzKSkKPj4+ICAgIAkJCXJldHVybiB2aWM7Cj4+PiAg ICAJfQo+Pj4gICAgCj4+PiBAQCAtMzAzNyw2ICszMDQ2LDcgQEAgc3RhdGljIHU4IGRybV9tYXRj aF9oZG1pX21vZGVfY2xvY2tfdG9sZXJhbmNlKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2Rl ICp0b18KPj4+ICAgICAqLwo+Pj4gICAgc3RhdGljIHU4IGRybV9tYXRjaF9oZG1pX21vZGUoY29u c3Qgc3RydWN0IGRybV9kaXNwbGF5X21vZGUgKnRvX21hdGNoKQo+Pj4gICAgewo+Pj4gKwl1bnNp Z25lZCBpbnQgbWF0Y2hfZmxhZ3MgPSBEUk1fTU9ERV9NQVRDSF9USU1JTkdTIHwgRFJNX01PREVf TUFUQ0hfRkxBR1M7Cj4+PiAgICAJdTggdmljOwo+Pj4gICAgCj4+PiAgICAJaWYgKCF0b19tYXRj aC0+Y2xvY2spCj4+PiBAQCAtMzA1Miw3ICszMDYyLDcgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9o ZG1pX21vZGUoY29uc3Qgc3RydWN0IGRybV9kaXNwbGF5X21vZGUgKnRvX21hdGNoKQo+Pj4gICAg Cj4+PiAgICAJCWlmICgoS0haMlBJQ09TKHRvX21hdGNoLT5jbG9jaykgPT0gS0haMlBJQ09TKGNs b2NrMSkgfHwKPj4+ICAgIAkJICAgICBLSFoyUElDT1ModG9fbWF0Y2gtPmNsb2NrKSA9PSBLSFoy UElDT1MoY2xvY2syKSkgJiYKPj4+IC0JCSAgICBkcm1fbW9kZV9lcXVhbF9ub19jbG9ja3Nfbm9f c3RlcmVvKHRvX21hdGNoLCBoZG1pX21vZGUpKQo+Pj4gKwkJICAgIGRybV9tb2RlX21hdGNoKHRv X21hdGNoLCBoZG1pX21vZGUsIG1hdGNoX2ZsYWdzKSkKPj4+ICAgIAkJCXJldHVybiB2aWM7Cj4+ PiAgICAJfQo+Pj4gICAgCXJldHVybiAwOwo+PiAtIFNoYXNoYW5rCgpfX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0Cklu dGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5v cmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK