From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= <ville.syrjala@linux.intel.com>
Subject: Re: [PATCH 06/10] drm/edid: Fix cea mode aspect ratio
	handling
Date: Thu, 16 Nov 2017 18:23:57 +0200
Message-ID: <20171116162357.GX10981@intel.com>
References: <20171113170427.4150-1-ville.syrjala@linux.intel.com>
 <20171113170427.4150-7-ville.syrjala@linux.intel.com>
 <67ddbed5-421d-d81f-9e0f-7ad4055f3c4e@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
Content-Disposition: inline
In-Reply-To: <67ddbed5-421d-d81f-9e0f-7ad4055f3c4e@intel.com>
List-Unsubscribe: <https://lists.freedesktop.org/mailman/options/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <https://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <https://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Errors-To: intel-gfx-bounces@lists.freedesktop.org
Sender: "Intel-gfx" <intel-gfx-bounces@lists.freedesktop.org>
To: "Sharma, Shashank" <shashank.sharma@intel.com>
Cc: Jose Abreu <Jose.Abreu@synopsys.com>, "Lin, Jia" <lin.a.jia@intel.com>, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Daniel Vetter <daniel.vetter@ffwll.ch>, Nautiyal Ankit <ankit.k.nautiyal@intel.com>
List-Id: dri-devel@lists.freedesktop.org

T24gVGh1LCBOb3YgMTYsIDIwMTcgYXQgMDg6MjE6NDRQTSArMDUzMCwgU2hhcm1hLCBTaGFzaGFu
ayB3cm90ZToKPiBSZWdhcmRzCj4gCj4gU2hhc2hhbmsKPiAKPiAKPiBPbiAxMS8xMy8yMDE3IDEw
OjM0IFBNLCBWaWxsZSBTeXJqYWxhIHdyb3RlOgo+ID4gRnJvbTogVmlsbGUgU3lyasOkbMOkIDx2
aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4KPiA+Cj4gPiBjb21taXQgNmRmZmQ0MzFlMjI5
ICgiZHJtOiBBZGQgYXNwZWN0IHJhdGlvIHBhcnNpbmcgaW4gRFJNIGxheWVyIikKPiA+IGNhdXNl
IHVzIHRvIG5vdCBzZW5kIG91dCBhbnkgVklDcyBpbiB0aGUgQVZJIGluZm9mcmFtZXMuIFRoYXQg
Y29tbWl0Cj4gPiB3YXMgc2luY2UgcmV2ZXJ0ZWQsIGJ1dCBpZiBhbmQgd2hlbiB3ZSBhZGQgYXNw
ZWN0IHJhdGlvIGhhbmRpbmcgYmFjawo+ID4gd2UgbmVlZCB0byBiZSBtb3JlIGNhcmVmdWwuCj4g
Pgo+ID4gTGV0J3MgaGFuZGxlIHRoaXMgYnkgY29uc2lkZXJpbmcgdGhlIGFzcGVjdCByYXRpbyBh
cyBhIHJlcXVpcmVtZW50Cj4gPiBmb3IgY2VhIG1vZGUgbWF0Y2hpbmcgb25seSBpZiB0aGUgcGFz
c2VkIGluIG1vZGUgYWN0dWFsbHkgaGFzIGEKPiA+IG5vbi16ZXJvIGFzcGVjdCByYXRpbyBmaWVs
ZC4gVGhpcyB3aWxsIGtlZXAgdXNlcnNwYWNlIHRoYXQgZG9lc24ndAo+ID4gcHJvdmlkZSBhbiBh
c3BlY3QgcmF0aW8gd29ya2luZyBhcyBiZWZvcmUgYnkgbWF0Y2hpbmcgaXQgdG8gdGhlCj4gPiBm
aXJzdCBvdGhlcndpc2UgZXF1YWwgY2VhIG1vZGUuIEFuZCBvbmNlIHVzZXJzcGFjZSBzdGFydHMg
dG8KPiA+IHByb3ZpZGUgdGhlIGFzcGVjdCByYXRpbyBpdCB3aWxsIGJlIGNvbnNpZGVyZCBhIGhh
cmQgcmVxdWlyZW1lbnQKPiA+IGZvciB0aGUgbWF0Y2guCj4gPgo+ID4gQWxzbyBjaGFuZ2UgdGhl
IGhkbWkgbW9kZSBtYXRjaGluZyB0byB1c2UgZHJtX21vZGVfbWF0Y2goKSBmb3IKPiA+IGNvbnNp
c3RlbmN5LCBidXQgd2UgZG9uJ3QgbWF0Y2ggb24gYXNwZWN0IHJhdGlvIHRoZXJlIHNpbmNlIHRo
ZQo+ID4gc3BlYyBkb2Vzbid0IGxpc3QgYSBzcGVjaWZpYyBhc3BlY3QgcmF0aW8gZm9yIHRob3Nl
IG1vZGVzLgo+ID4KPiA+IENjOiBTaGFzaGFuayBTaGFybWEgPHNoYXNoYW5rLnNoYXJtYUBpbnRl
bC5jb20+Cj4gPiBDYzogIkxpbiwgSmlhIiA8bGluLmEuamlhQGludGVsLmNvbT4KPiA+IENjOiBB
a2FzaGRlZXAgU2hhcm1hIDxha2FzaGRlZXAuc2hhcm1hQGludGVsLmNvbT4KPiA+IENjOiBKaW0g
QnJpZGUgPGppbS5icmlkZUBsaW51eC5pbnRlbC5jb20+Cj4gPiBDYzogSm9zZSBBYnJldSA8Sm9z
ZS5BYnJldUBzeW5vcHN5cy5jb20+Cj4gPiBDYzogRGFuaWVsIFZldHRlciA8ZGFuaWVsLnZldHRl
ckBmZndsbC5jaD4KPiA+IENjOiBFbWlsIFZlbGlrb3YgPGVtaWwubC52ZWxpa292QGdtYWlsLmNv
bT4KPiA+IFNpZ25lZC1vZmYtYnk6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBsaW51
eC5pbnRlbC5jb20+Cj4gPiAtLS0KPiA+ICAgZHJpdmVycy9ncHUvZHJtL2RybV9lZGlkLmMgfCAx
OCArKysrKysrKysrKysrKy0tLS0KPiA+ICAgMSBmaWxlIGNoYW5nZWQsIDE0IGluc2VydGlvbnMo
KyksIDQgZGVsZXRpb25zKC0pCj4gPgo+ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9k
cm1fZWRpZC5jIGIvZHJpdmVycy9ncHUvZHJtL2RybV9lZGlkLmMKPiA+IGluZGV4IDcyMjBiOGY5
YTdlOC4uMDBhYTk4ZjNlNTVkIDEwMDY0NAo+ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2RybV9l
ZGlkLmMKPiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jCj4gPiBAQCAtMjkwMywx
MSArMjkwMywxNSBAQCBjZWFfbW9kZV9hbHRlcm5hdGVfdGltaW5ncyh1OCB2aWMsIHN0cnVjdCBk
cm1fZGlzcGxheV9tb2RlICptb2RlKQo+ID4gICBzdGF0aWMgdTggZHJtX21hdGNoX2NlYV9tb2Rl
X2Nsb2NrX3RvbGVyYW5jZShjb25zdCBzdHJ1Y3QgZHJtX2Rpc3BsYXlfbW9kZSAqdG9fbWF0Y2gs
Cj4gPiAgIAkJCQkJICAgICB1bnNpZ25lZCBpbnQgY2xvY2tfdG9sZXJhbmNlKQo+ID4gICB7Cj4g
PiArCXVuc2lnbmVkIGludCBtYXRjaF9mbGFncyA9IERSTV9NT0RFX01BVENIX1RJTUlOR1MgfCBE
Uk1fTU9ERV9NQVRDSF9GTEFHUzsKPiA+ICAgCXU4IHZpYzsKPiA+ICAgCj4gPiAgIAlpZiAoIXRv
X21hdGNoLT5jbG9jaykKPiA+ICAgCQlyZXR1cm4gMDsKPiA+ICAgCj4gPiArCWlmICh0b19tYXRj
aC0+cGljdHVyZV9hc3BlY3RfcmF0aW8pCj4gPiArCQltYXRjaF9mbGFncyB8PSBEUk1fTU9ERV9N
QVRDSF9BU1BFQ1RfUkFUSU87Cj4gVGhpcyBkb2Vzbid0IGxvb2sgcmlnaHQuIFRoaXMgbWVhbnMg
d2UgYXJlIGV4cGVjdGluZyBhIENFQSBtb2RlIHdpdGhvdXQgCj4gYSBwaWMgYXNwZWN0IHJhdGlv
IGZpZWxkLAo+IHdoaWNoIGlzIGludmFsaWQuCgpObywgaXQncyBwZXJmZWN0bHkgdmFsaWQuIEl0
J3Mgd2hhdCB3ZSBjdXJyZW50bHkgZ2V0IGZyb20gdXNlcnNwYWNlLgoKPiBBbmtpdCBpcyBnb2lu
ZyB0byBwdWJsaXNoIHRoZSBhc3BlY3QgcmF0aW8gcGF0Y2ggCj4gc2VyaWVzIGFnYWluLCB3aXRo
IHByb3BlciBEUk0gY2FwIGFuZCBmbGFncyBjaGVjay4gV291bGQgaXQgYmUKPiBvayBpZiB3ZSBo
YXZlIGEgbG9vayB0aGF0IGhhbmRsaW5nIGZpcnN0ID8KClRoaXMgcGF0Y2ggd2lsbCBiZSBuZWVk
ZWQgYnkgdGhhdCB3b3JrLiBPdGhlcndpc2Ugd2UncmUgZ29pbmcgdG8gc3RvcApzZW5kaW5nIGEg
VklDIGZvciBDRUEgbW9kZXMgd2l0aCBjdXJyZW50IHVzZXJzcGFjZS4KCj4gPiArCj4gPiAgIAlm
b3IgKHZpYyA9IDE7IHZpYyA8IEFSUkFZX1NJWkUoZWRpZF9jZWFfbW9kZXMpOyB2aWMrKykgewo+
ID4gICAJCXN0cnVjdCBkcm1fZGlzcGxheV9tb2RlIGNlYV9tb2RlID0gZWRpZF9jZWFfbW9kZXNb
dmljXTsKPiA+ICAgCQl1bnNpZ25lZCBpbnQgY2xvY2sxLCBjbG9jazI7Cj4gPiBAQCAtMjkyMSw3
ICsyOTI1LDcgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9jZWFfbW9kZV9jbG9ja190b2xlcmFuY2Uo
Y29uc3Qgc3RydWN0IGRybV9kaXNwbGF5X21vZGUgKnRvX20KPiA+ICAgCQkJY29udGludWU7Cj4g
PiAgIAo+ID4gICAJCWRvIHsKPiA+IC0JCQlpZiAoZHJtX21vZGVfZXF1YWxfbm9fY2xvY2tzX25v
X3N0ZXJlbyh0b19tYXRjaCwgJmNlYV9tb2RlKSkKPiA+ICsJCQlpZiAoZHJtX21vZGVfbWF0Y2go
dG9fbWF0Y2gsICZjZWFfbW9kZSwgbWF0Y2hfZmxhZ3MpKQo+ID4gICAJCQkJcmV0dXJuIHZpYzsK
PiA+ICAgCQl9IHdoaWxlIChjZWFfbW9kZV9hbHRlcm5hdGVfdGltaW5ncyh2aWMsICZjZWFfbW9k
ZSkpOwo+ID4gICAJfQo+ID4gQEAgLTI5MzgsMTEgKzI5NDIsMTUgQEAgc3RhdGljIHU4IGRybV9t
YXRjaF9jZWFfbW9kZV9jbG9ja190b2xlcmFuY2UoY29uc3Qgc3RydWN0IGRybV9kaXNwbGF5X21v
ZGUgKnRvX20KPiA+ICAgICovCj4gPiAgIHU4IGRybV9tYXRjaF9jZWFfbW9kZShjb25zdCBzdHJ1
Y3QgZHJtX2Rpc3BsYXlfbW9kZSAqdG9fbWF0Y2gpCj4gPiAgIHsKPiA+ICsJdW5zaWduZWQgaW50
IG1hdGNoX2ZsYWdzID0gRFJNX01PREVfTUFUQ0hfVElNSU5HUyB8IERSTV9NT0RFX01BVENIX0ZM
QUdTOwo+ID4gICAJdTggdmljOwo+ID4gICAKPiA+ICAgCWlmICghdG9fbWF0Y2gtPmNsb2NrKQo+
ID4gICAJCXJldHVybiAwOwo+ID4gICAKPiA+ICsJaWYgKHRvX21hdGNoLT5waWN0dXJlX2FzcGVj
dF9yYXRpbykKPiA+ICsJCW1hdGNoX2ZsYWdzIHw9IERSTV9NT0RFX01BVENIX0FTUEVDVF9SQVRJ
TzsKPiBzYW1lIGhlcmUsIGFuZCBwcm9iYWJseSBpbiBvdGhlciBDRUEgbWF0Y2ggZnVuY3Rpb25z
Lgo+ID4gKwo+ID4gICAJZm9yICh2aWMgPSAxOyB2aWMgPCBBUlJBWV9TSVpFKGVkaWRfY2VhX21v
ZGVzKTsgdmljKyspIHsKPiA+ICAgCQlzdHJ1Y3QgZHJtX2Rpc3BsYXlfbW9kZSBjZWFfbW9kZSA9
IGVkaWRfY2VhX21vZGVzW3ZpY107Cj4gPiAgIAkJdW5zaWduZWQgaW50IGNsb2NrMSwgY2xvY2sy
Owo+ID4gQEAgLTI5NTYsNyArMjk2NCw3IEBAIHU4IGRybV9tYXRjaF9jZWFfbW9kZShjb25zdCBz
dHJ1Y3QgZHJtX2Rpc3BsYXlfbW9kZSAqdG9fbWF0Y2gpCj4gPiAgIAkJCWNvbnRpbnVlOwo+ID4g
ICAKPiA+ICAgCQlkbyB7Cj4gPiAtCQkJaWYgKGRybV9tb2RlX2VxdWFsX25vX2Nsb2Nrc19ub19z
dGVyZW8odG9fbWF0Y2gsICZjZWFfbW9kZSkpCj4gPiArCQkJaWYgKGRybV9tb2RlX21hdGNoKHRv
X21hdGNoLCAmY2VhX21vZGUsIG1hdGNoX2ZsYWdzKSkKPiA+ICAgCQkJCXJldHVybiB2aWM7Cj4g
PiAgIAkJfSB3aGlsZSAoY2VhX21vZGVfYWx0ZXJuYXRlX3RpbWluZ3ModmljLCAmY2VhX21vZGUp
KTsKPiA+ICAgCX0KPiA+IEBAIC0zMDAzLDYgKzMwMTEsNyBAQCBoZG1pX21vZGVfYWx0ZXJuYXRl
X2Nsb2NrKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICpoZG1pX21vZGUpCj4gPiAgIHN0
YXRpYyB1OCBkcm1fbWF0Y2hfaGRtaV9tb2RlX2Nsb2NrX3RvbGVyYW5jZShjb25zdCBzdHJ1Y3Qg
ZHJtX2Rpc3BsYXlfbW9kZSAqdG9fbWF0Y2gsCj4gPiAgIAkJCQkJICAgICAgdW5zaWduZWQgaW50
IGNsb2NrX3RvbGVyYW5jZSkKPiA+ICAgewo+ID4gKwl1bnNpZ25lZCBpbnQgbWF0Y2hfZmxhZ3Mg
PSBEUk1fTU9ERV9NQVRDSF9USU1JTkdTIHwgRFJNX01PREVfTUFUQ0hfRkxBR1M7Cj4gPiAgIAl1
OCB2aWM7Cj4gPiAgIAo+ID4gICAJaWYgKCF0b19tYXRjaC0+Y2xvY2spCj4gPiBAQCAtMzAyMCw3
ICszMDI5LDcgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9oZG1pX21vZGVfY2xvY2tfdG9sZXJhbmNl
KGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICp0b18KPiA+ICAgCQkgICAgYWJzKHRvX21h
dGNoLT5jbG9jayAtIGNsb2NrMikgPiBjbG9ja190b2xlcmFuY2UpCj4gPiAgIAkJCWNvbnRpbnVl
Owo+ID4gICAKPiA+IC0JCWlmIChkcm1fbW9kZV9lcXVhbF9ub19jbG9ja3Nfbm9fc3RlcmVvKHRv
X21hdGNoLCBoZG1pX21vZGUpKQo+ID4gKwkJaWYgKGRybV9tb2RlX21hdGNoKHRvX21hdGNoLCBo
ZG1pX21vZGUsIG1hdGNoX2ZsYWdzKSkKPiA+ICAgCQkJcmV0dXJuIHZpYzsKPiA+ICAgCX0KPiA+
ICAgCj4gPiBAQCAtMzAzNyw2ICszMDQ2LDcgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9oZG1pX21v
ZGVfY2xvY2tfdG9sZXJhbmNlKGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICp0b18KPiA+
ICAgICovCj4gPiAgIHN0YXRpYyB1OCBkcm1fbWF0Y2hfaGRtaV9tb2RlKGNvbnN0IHN0cnVjdCBk
cm1fZGlzcGxheV9tb2RlICp0b19tYXRjaCkKPiA+ICAgewo+ID4gKwl1bnNpZ25lZCBpbnQgbWF0
Y2hfZmxhZ3MgPSBEUk1fTU9ERV9NQVRDSF9USU1JTkdTIHwgRFJNX01PREVfTUFUQ0hfRkxBR1M7
Cj4gPiAgIAl1OCB2aWM7Cj4gPiAgIAo+ID4gICAJaWYgKCF0b19tYXRjaC0+Y2xvY2spCj4gPiBA
QCAtMzA1Miw3ICszMDYyLDcgQEAgc3RhdGljIHU4IGRybV9tYXRjaF9oZG1pX21vZGUoY29uc3Qg
c3RydWN0IGRybV9kaXNwbGF5X21vZGUgKnRvX21hdGNoKQo+ID4gICAKPiA+ICAgCQlpZiAoKEtI
WjJQSUNPUyh0b19tYXRjaC0+Y2xvY2spID09IEtIWjJQSUNPUyhjbG9jazEpIHx8Cj4gPiAgIAkJ
ICAgICBLSFoyUElDT1ModG9fbWF0Y2gtPmNsb2NrKSA9PSBLSFoyUElDT1MoY2xvY2syKSkgJiYK
PiA+IC0JCSAgICBkcm1fbW9kZV9lcXVhbF9ub19jbG9ja3Nfbm9fc3RlcmVvKHRvX21hdGNoLCBo
ZG1pX21vZGUpKQo+ID4gKwkJICAgIGRybV9tb2RlX21hdGNoKHRvX21hdGNoLCBoZG1pX21vZGUs
IG1hdGNoX2ZsYWdzKSkKPiA+ICAgCQkJcmV0dXJuIHZpYzsKPiA+ICAgCX0KPiA+ICAgCXJldHVy
biAwOwo+IC0gU2hhc2hhbmsKCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5n
IGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVk
ZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=