From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH v3 3/8] drm/edid: Fix cea mode aspect ratio handling Date: Wed, 17 Jan 2018 17:29:26 +0200 Message-ID: <20180117152926.GL10981@intel.com> References: <1515738096-16892-1-git-send-email-ankit.k.nautiyal@intel.com> <1515738096-16892-4-git-send-email-ankit.k.nautiyal@intel.com> <9cc4e294-42ac-55de-7a97-34f4b67ebfa1@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id C39796E49B for ; Wed, 17 Jan 2018 15:29:32 +0000 (UTC) Content-Disposition: inline In-Reply-To: <9cc4e294-42ac-55de-7a97-34f4b67ebfa1@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: "Sharma, Shashank" Cc: Jose Abreu , "Lin, Jia" , Akashdeep Sharma , Emil Velikov , dri-devel@lists.freedesktop.org, Daniel Vetter , "Nautiyal, Ankit K" , Jim Bride List-Id: dri-devel@lists.freedesktop.org T24gV2VkLCBKYW4gMTcsIDIwMTggYXQgMDI6MzU6NDBQTSArMDUzMCwgU2hhcm1hLCBTaGFzaGFu ayB3cm90ZToKPiBSZWdhcmRzCj4gCj4gU2hhc2hhbmsKPiAKPiAKPiBPbiAxLzEyLzIwMTggMTE6 NTEgQU0sIE5hdXRpeWFsLCBBbmtpdCBLIHdyb3RlOgo+ID4gRnJvbTogVmlsbGUgU3lyasOkbMOk IDx2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4KPiA+Cj4gPiBjb21taXQgNmRmZmQ0MzFl MjI5ICgiZHJtOiBBZGQgYXNwZWN0IHJhdGlvIHBhcnNpbmcgaW4gRFJNIGxheWVyIikKPiA+IGNh dXNlIHVzIHRvIG5vdCBzZW5kIG91dCBhbnkgVklDcyBpbiB0aGUgQVZJIGluZm9mcmFtZXMuIFRo YXQgY29tbWl0Cj4gPiB3YXMgc2luY2UgcmV2ZXJ0ZWQsIGJ1dCBpZiBhbmQgd2hlbiB3ZSBhZGQg YXNwZWN0IHJhdGlvIGhhbmRpbmcgYmFjawo+ID4gd2UgbmVlZCB0byBiZSBtb3JlIGNhcmVmdWwu Cj4gPgo+ID4gTGV0J3MgaGFuZGxlIHRoaXMgYnkgY29uc2lkZXJpbmcgdGhlIGFzcGVjdCByYXRp byBhcyBhIHJlcXVpcmVtZW50Cj4gPiBmb3IgY2VhIG1vZGUgbWF0Y2hpbmcgb25seSBpZiB0aGUg cGFzc2VkIGluIG1vZGUgYWN0dWFsbHkgaGFzIGEKPiA+IG5vbi16ZXJvIGFzcGVjdCByYXRpbyBm aWVsZC4gVGhpcyB3aWxsIGtlZXAgdXNlcnNwYWNlIHRoYXQgZG9lc24ndAo+ID4gcHJvdmlkZSBh biBhc3BlY3QgcmF0aW8gd29ya2luZyBhcyBiZWZvcmUgYnkgbWF0Y2hpbmcgaXQgdG8gdGhlCj4g PiBmaXJzdCBvdGhlcndpc2UgZXF1YWwgY2VhIG1vZGUuIEFuZCBvbmNlIHVzZXJzcGFjZSBzdGFy dHMgdG8KPiA+IHByb3ZpZGUgdGhlIGFzcGVjdCByYXRpbyBpdCB3aWxsIGJlIGNvbnNpZGVyZCBh IGhhcmQgcmVxdWlyZW1lbnQKPiA+IGZvciB0aGUgbWF0Y2guCj4gPgo+ID4gQWxzbyBjaGFuZ2Ug dGhlIGhkbWkgbW9kZSBtYXRjaGluZyB0byB1c2UgZHJtX21vZGVfbWF0Y2goKSBmb3IKPiA+IGNv bnNpc3RlbmN5LCBidXQgd2UgZG9uJ3QgbWF0Y2ggb24gYXNwZWN0IHJhdGlvIHRoZXJlIHNpbmNl IHRoZQo+ID4gc3BlYyBkb2Vzbid0IGxpc3QgYSBzcGVjaWZpYyBhc3BlY3QgcmF0aW8gZm9yIHRo b3NlIG1vZGVzLgo+ID4KPiA+IENjOiBTaGFzaGFuayBTaGFybWEgPHNoYXNoYW5rLnNoYXJtYUBp bnRlbC5jb20+Cj4gPiBDYzogIkxpbiwgSmlhIiA8bGluLmEuamlhQGludGVsLmNvbT4KPiA+IENj OiBBa2FzaGRlZXAgU2hhcm1hIDxha2FzaGRlZXAuc2hhcm1hQGludGVsLmNvbT4KPiA+IENjOiBK aW0gQnJpZGUgPGppbS5icmlkZUBsaW51eC5pbnRlbC5jb20+Cj4gPiBDYzogSm9zZSBBYnJldSA8 Sm9zZS5BYnJldUBzeW5vcHN5cy5jb20+Cj4gPiBDYzogRGFuaWVsIFZldHRlciA8ZGFuaWVsLnZl dHRlckBmZndsbC5jaD4KPiA+IENjOiBFbWlsIFZlbGlrb3YgPGVtaWwubC52ZWxpa292QGdtYWls LmNvbT4KPiA+IFNpZ25lZC1vZmYtYnk6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBs aW51eC5pbnRlbC5jb20+Cj4gPiAtLS0KPiA+ICAgZHJpdmVycy9ncHUvZHJtL2RybV9lZGlkLmMg fCAxOCArKysrKysrKysrKysrKy0tLS0KPiA+ICAgMSBmaWxlIGNoYW5nZWQsIDE0IGluc2VydGlv bnMoKyksIDQgZGVsZXRpb25zKC0pCj4gPgo+ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2Ry bS9kcm1fZWRpZC5jIGIvZHJpdmVycy9ncHUvZHJtL2RybV9lZGlkLmMKPiA+IGluZGV4IDE4MThh NzEuLjNkNTdjNDEgMTAwNjQ0Cj4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYwo+ ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2RybV9lZGlkLmMKPiA+IEBAIC0yOTA4LDExICsyOTA4 LDE1IEBAIGNlYV9tb2RlX2FsdGVybmF0ZV90aW1pbmdzKHU4IHZpYywgc3RydWN0IGRybV9kaXNw bGF5X21vZGUgKm1vZGUpCj4gPiAgIHN0YXRpYyB1OCBkcm1fbWF0Y2hfY2VhX21vZGVfY2xvY2tf dG9sZXJhbmNlKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICp0b19tYXRjaCwKPiA+ICAg CQkJCQkgICAgIHVuc2lnbmVkIGludCBjbG9ja190b2xlcmFuY2UpCj4gPiAgIHsKPiA+ICsJdW5z aWduZWQgaW50IG1hdGNoX2ZsYWdzID0gRFJNX01PREVfTUFUQ0hfVElNSU5HUyB8IERSTV9NT0RF X01BVENIX0ZMQUdTOwo+ID4gICAJdTggdmljOwo+ID4gICAKPiA+ICAgCWlmICghdG9fbWF0Y2gt PmNsb2NrKQo+ID4gICAJCXJldHVybiAwOwo+ID4gICAKPiA+ICsJaWYgKHRvX21hdGNoLT5waWN0 dXJlX2FzcGVjdF9yYXRpbykKPiA+ICsJCW1hdGNoX2ZsYWdzIHw9IERSTV9NT0RFX01BVENIX0FT UEVDVF9SQVRJTzsKPiA+ICsKPiA+ICAgCWZvciAodmljID0gMTsgdmljIDwgQVJSQVlfU0laRShl ZGlkX2NlYV9tb2Rlcyk7IHZpYysrKSB7Cj4gPiAgIAkJc3RydWN0IGRybV9kaXNwbGF5X21vZGUg Y2VhX21vZGUgPSBlZGlkX2NlYV9tb2Rlc1t2aWNdOwo+ID4gICAJCXVuc2lnbmVkIGludCBjbG9j azEsIGNsb2NrMjsKPiA+IEBAIC0yOTI2LDcgKzI5MzAsNyBAQCBzdGF0aWMgdTggZHJtX21hdGNo X2NlYV9tb2RlX2Nsb2NrX3RvbGVyYW5jZShjb25zdCBzdHJ1Y3QgZHJtX2Rpc3BsYXlfbW9kZSAq dG9fbQo+ID4gICAJCQljb250aW51ZTsKPiA+ICAgCj4gPiAgIAkJZG8gewo+ID4gLQkJCWlmIChk cm1fbW9kZV9lcXVhbF9ub19jbG9ja3Nfbm9fc3RlcmVvKHRvX21hdGNoLCAmY2VhX21vZGUpKQo+ ID4gKwkJCWlmIChkcm1fbW9kZV9tYXRjaCh0b19tYXRjaCwgJmNlYV9tb2RlLCBtYXRjaF9mbGFn cykpCj4gPiAgIAkJCQlyZXR1cm4gdmljOwo+ID4gICAJCX0gd2hpbGUgKGNlYV9tb2RlX2FsdGVy bmF0ZV90aW1pbmdzKHZpYywgJmNlYV9tb2RlKSk7Cj4gPiAgIAl9Cj4gPiBAQCAtMjk0MywxMSAr Mjk0NywxNSBAQCBzdGF0aWMgdTggZHJtX21hdGNoX2NlYV9tb2RlX2Nsb2NrX3RvbGVyYW5jZShj b25zdCBzdHJ1Y3QgZHJtX2Rpc3BsYXlfbW9kZSAqdG9fbQo+ID4gICAgKi8KPiA+ICAgdTggZHJt X21hdGNoX2NlYV9tb2RlKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICp0b19tYXRjaCkK PiA+ICAgewo+ID4gKwl1bnNpZ25lZCBpbnQgbWF0Y2hfZmxhZ3MgPSBEUk1fTU9ERV9NQVRDSF9U SU1JTkdTIHwgRFJNX01PREVfTUFUQ0hfRkxBR1M7Cj4gPiAgIAl1OCB2aWM7Cj4gPiAgIAo+ID4g ICAJaWYgKCF0b19tYXRjaC0+Y2xvY2spCj4gPiAgIAkJcmV0dXJuIDA7Cj4gPiAgIAo+ID4gKwlp ZiAodG9fbWF0Y2gtPnBpY3R1cmVfYXNwZWN0X3JhdGlvKQo+ID4gKwkJbWF0Y2hfZmxhZ3MgfD0g RFJNX01PREVfTUFUQ0hfQVNQRUNUX1JBVElPOwo+ID4gKwo+IEhvdyBhYm91dCBzdGVyZW8gZmxh Z3MgPyBDRUEgbW9kZXMgY2FuIGJlIDNkIGFuZCBpbiB0aGF0IGNhc2Ugd2UgbWlnaHQgCj4gd2Fu dCB0byBhZGQgRFJNX01PREVfTUFUQ0hfM0RfRkxBR1MgaGVyZQoKVGhlcmUgYXJlIG5vIHNlcGFy YXRlIFZJQ3MgZm9yIHN0ZXJlbyB2cy4gbm8gc3RlcmVvLiBTbyBpZ25vcmluZyB0aGUKc3RlcmVv IGZsYWdzIGhlcmUgc2VlbXMgbGlrZSB0aGUgY29ycmVjdCB0aGluZyB0byBkby4KCj4gCj4gLSBT aGFzaGFuawo+ID4gICAJZm9yICh2aWMgPSAxOyB2aWMgPCBBUlJBWV9TSVpFKGVkaWRfY2VhX21v ZGVzKTsgdmljKyspIHsKPiA+ICAgCQlzdHJ1Y3QgZHJtX2Rpc3BsYXlfbW9kZSBjZWFfbW9kZSA9 IGVkaWRfY2VhX21vZGVzW3ZpY107Cj4gPiAgIAkJdW5zaWduZWQgaW50IGNsb2NrMSwgY2xvY2sy Owo+ID4gQEAgLTI5NjEsNyArMjk2OSw3IEBAIHU4IGRybV9tYXRjaF9jZWFfbW9kZShjb25zdCBz dHJ1Y3QgZHJtX2Rpc3BsYXlfbW9kZSAqdG9fbWF0Y2gpCj4gPiAgIAkJCWNvbnRpbnVlOwo+ID4g ICAKPiA+ICAgCQlkbyB7Cj4gPiAtCQkJaWYgKGRybV9tb2RlX2VxdWFsX25vX2Nsb2Nrc19ub19z dGVyZW8odG9fbWF0Y2gsICZjZWFfbW9kZSkpCj4gPiArCQkJaWYgKGRybV9tb2RlX21hdGNoKHRv X21hdGNoLCAmY2VhX21vZGUsIG1hdGNoX2ZsYWdzKSkKPiA+ICAgCQkJCXJldHVybiB2aWM7Cj4g PiAgIAkJfSB3aGlsZSAoY2VhX21vZGVfYWx0ZXJuYXRlX3RpbWluZ3ModmljLCAmY2VhX21vZGUp KTsKPiA+ICAgCX0KPiA+IEBAIC0zMDA4LDYgKzMwMTYsNyBAQCBoZG1pX21vZGVfYWx0ZXJuYXRl X2Nsb2NrKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICpoZG1pX21vZGUpCj4gPiAgIHN0 YXRpYyB1OCBkcm1fbWF0Y2hfaGRtaV9tb2RlX2Nsb2NrX3RvbGVyYW5jZShjb25zdCBzdHJ1Y3Qg ZHJtX2Rpc3BsYXlfbW9kZSAqdG9fbWF0Y2gsCj4gPiAgIAkJCQkJICAgICAgdW5zaWduZWQgaW50 IGNsb2NrX3RvbGVyYW5jZSkKPiA+ICAgewo+ID4gKwl1bnNpZ25lZCBpbnQgbWF0Y2hfZmxhZ3Mg PSBEUk1fTU9ERV9NQVRDSF9USU1JTkdTIHwgRFJNX01PREVfTUFUQ0hfRkxBR1M7Cj4gPiAgIAl1 OCB2aWM7Cj4gPiAgIAo+ID4gICAJaWYgKCF0b19tYXRjaC0+Y2xvY2spCj4gPiBAQCAtMzAyNSw3 ICszMDM0LDcgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9oZG1pX21vZGVfY2xvY2tfdG9sZXJhbmNl KGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICp0b18KPiA+ICAgCQkgICAgYWJzKHRvX21h dGNoLT5jbG9jayAtIGNsb2NrMikgPiBjbG9ja190b2xlcmFuY2UpCj4gPiAgIAkJCWNvbnRpbnVl Owo+ID4gICAKPiA+IC0JCWlmIChkcm1fbW9kZV9lcXVhbF9ub19jbG9ja3Nfbm9fc3RlcmVvKHRv X21hdGNoLCBoZG1pX21vZGUpKQo+ID4gKwkJaWYgKGRybV9tb2RlX21hdGNoKHRvX21hdGNoLCBo ZG1pX21vZGUsIG1hdGNoX2ZsYWdzKSkKPiA+ICAgCQkJcmV0dXJuIHZpYzsKPiA+ICAgCX0KPiA+ ICAgCj4gPiBAQCAtMzA0Miw2ICszMDUxLDcgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9oZG1pX21v ZGVfY2xvY2tfdG9sZXJhbmNlKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICp0b18KPiA+ ICAgICovCj4gPiAgIHN0YXRpYyB1OCBkcm1fbWF0Y2hfaGRtaV9tb2RlKGNvbnN0IHN0cnVjdCBk cm1fZGlzcGxheV9tb2RlICp0b19tYXRjaCkKPiA+ICAgewo+ID4gKwl1bnNpZ25lZCBpbnQgbWF0 Y2hfZmxhZ3MgPSBEUk1fTU9ERV9NQVRDSF9USU1JTkdTIHwgRFJNX01PREVfTUFUQ0hfRkxBR1M7 Cj4gPiAgIAl1OCB2aWM7Cj4gPiAgIAo+ID4gICAJaWYgKCF0b19tYXRjaC0+Y2xvY2spCj4gPiBA QCAtMzA1Nyw3ICszMDY3LDcgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9oZG1pX21vZGUoY29uc3Qg c3RydWN0IGRybV9kaXNwbGF5X21vZGUgKnRvX21hdGNoKQo+ID4gICAKPiA+ICAgCQlpZiAoKEtI WjJQSUNPUyh0b19tYXRjaC0+Y2xvY2spID09IEtIWjJQSUNPUyhjbG9jazEpIHx8Cj4gPiAgIAkJ ICAgICBLSFoyUElDT1ModG9fbWF0Y2gtPmNsb2NrKSA9PSBLSFoyUElDT1MoY2xvY2syKSkgJiYK PiA+IC0JCSAgICBkcm1fbW9kZV9lcXVhbF9ub19jbG9ja3Nfbm9fc3RlcmVvKHRvX21hdGNoLCBo ZG1pX21vZGUpKQo+ID4gKwkJICAgIGRybV9tb2RlX21hdGNoKHRvX21hdGNoLCBoZG1pX21vZGUs IG1hdGNoX2ZsYWdzKSkKPiA+ICAgCQkJcmV0dXJuIHZpYzsKPiA+ICAgCX0KPiA+ICAgCXJldHVy biAwOwoKLS0gClZpbGxlIFN5cmrDpGzDpApJbnRlbCBPVEMKX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2 ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21h aWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==