From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 06/10] drm/edid: Fix cea mode aspect ratio handling Date: Fri, 17 Nov 2017 14:49:47 +0200 Message-ID: <20171117124947.GE10981@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> <20171116162357.GX10981@intel.com> <20171117113518.GC10981@intel.com> <4a0a2cee-cc44-6ae5-871e-b706766a283c@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <4a0a2cee-cc44-6ae5-871e-b706766a283c@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 , intel-gfx@lists.freedesktop.org, Emil Velikov , dri-devel@lists.freedesktop.org, Daniel Vetter , Nautiyal Ankit , Jim Bride List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCBOb3YgMTcsIDIwMTcgYXQgMDU6NTA6MTFQTSArMDUzMCwgU2hhcm1hLCBTaGFzaGFu ayB3cm90ZToKPiBSZWdhcmRzCj4gCj4gU2hhc2hhbmsKPiAKPiAKPiBPbiAxMS8xNy8yMDE3IDU6 MDUgUE0sIFZpbGxlIFN5cmrDpGzDpCB3cm90ZToKPiA+IE9uIEZyaSwgTm92IDE3LCAyMDE3IGF0 IDA4OjQ5OjQ5QU0gKzA1MzAsIFNoYXJtYSwgU2hhc2hhbmsgd3JvdGU6Cj4gPj4gUmVnYXJkcwo+ ID4+Cj4gPj4gU2hhc2hhbmsKPiA+Pgo+ID4+Cj4gPj4gT24gMTEvMTYvMjAxNyA5OjUzIFBNLCBW aWxsZSBTeXJqw6Rsw6Qgd3JvdGU6Cj4gPj4+IE9uIFRodSwgTm92IDE2LCAyMDE3IGF0IDA4OjIx OjQ0UE0gKzA1MzAsIFNoYXJtYSwgU2hhc2hhbmsgd3JvdGU6Cj4gPj4+PiBSZWdhcmRzCj4gPj4+ Pgo+ID4+Pj4gU2hhc2hhbmsKPiA+Pj4+Cj4gPj4+Pgo+ID4+Pj4gT24gMTEvMTMvMjAxNyAxMDoz NCBQTSwgVmlsbGUgU3lyamFsYSB3cm90ZToKPiA+Pj4+PiBGcm9tOiBWaWxsZSBTeXJqw6Rsw6Qg PHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+ID4+Pj4+Cj4gPj4+Pj4gY29tbWl0IDZk ZmZkNDMxZTIyOSAoImRybTogQWRkIGFzcGVjdCByYXRpbyBwYXJzaW5nIGluIERSTSBsYXllciIp Cj4gPj4+Pj4gY2F1c2UgdXMgdG8gbm90IHNlbmQgb3V0IGFueSBWSUNzIGluIHRoZSBBVkkgaW5m b2ZyYW1lcy4gVGhhdCBjb21taXQKPiA+Pj4+PiB3YXMgc2luY2UgcmV2ZXJ0ZWQsIGJ1dCBpZiBh bmQgd2hlbiB3ZSBhZGQgYXNwZWN0IHJhdGlvIGhhbmRpbmcgYmFjawo+ID4+Pj4+IHdlIG5lZWQg dG8gYmUgbW9yZSBjYXJlZnVsLgo+ID4+Pj4+Cj4gPj4+Pj4gTGV0J3MgaGFuZGxlIHRoaXMgYnkg Y29uc2lkZXJpbmcgdGhlIGFzcGVjdCByYXRpbyBhcyBhIHJlcXVpcmVtZW50Cj4gPj4+Pj4gZm9y IGNlYSBtb2RlIG1hdGNoaW5nIG9ubHkgaWYgdGhlIHBhc3NlZCBpbiBtb2RlIGFjdHVhbGx5IGhh cyBhCj4gPj4+Pj4gbm9uLXplcm8gYXNwZWN0IHJhdGlvIGZpZWxkLiBUaGlzIHdpbGwga2VlcCB1 c2Vyc3BhY2UgdGhhdCBkb2Vzbid0Cj4gPj4+Pj4gcHJvdmlkZSBhbiBhc3BlY3QgcmF0aW8gd29y a2luZyBhcyBiZWZvcmUgYnkgbWF0Y2hpbmcgaXQgdG8gdGhlCj4gPj4+Pj4gZmlyc3Qgb3RoZXJ3 aXNlIGVxdWFsIGNlYSBtb2RlLiBBbmQgb25jZSB1c2Vyc3BhY2Ugc3RhcnRzIHRvCj4gPj4+Pj4g cHJvdmlkZSB0aGUgYXNwZWN0IHJhdGlvIGl0IHdpbGwgYmUgY29uc2lkZXJkIGEgaGFyZCByZXF1 aXJlbWVudAo+ID4+Pj4+IGZvciB0aGUgbWF0Y2guCj4gPj4+Pj4KPiA+Pj4+PiBBbHNvIGNoYW5n ZSB0aGUgaGRtaSBtb2RlIG1hdGNoaW5nIHRvIHVzZSBkcm1fbW9kZV9tYXRjaCgpIGZvcgo+ID4+ Pj4+IGNvbnNpc3RlbmN5LCBidXQgd2UgZG9uJ3QgbWF0Y2ggb24gYXNwZWN0IHJhdGlvIHRoZXJl IHNpbmNlIHRoZQo+ID4+Pj4+IHNwZWMgZG9lc24ndCBsaXN0IGEgc3BlY2lmaWMgYXNwZWN0IHJh dGlvIGZvciB0aG9zZSBtb2Rlcy4KPiA+Pj4+Pgo+ID4+Pj4+IENjOiBTaGFzaGFuayBTaGFybWEg PHNoYXNoYW5rLnNoYXJtYUBpbnRlbC5jb20+Cj4gPj4+Pj4gQ2M6ICJMaW4sIEppYSIgPGxpbi5h LmppYUBpbnRlbC5jb20+Cj4gPj4+Pj4gQ2M6IEFrYXNoZGVlcCBTaGFybWEgPGFrYXNoZGVlcC5z aGFybWFAaW50ZWwuY29tPgo+ID4+Pj4+IENjOiBKaW0gQnJpZGUgPGppbS5icmlkZUBsaW51eC5p bnRlbC5jb20+Cj4gPj4+Pj4gQ2M6IEpvc2UgQWJyZXUgPEpvc2UuQWJyZXVAc3lub3BzeXMuY29t Pgo+ID4+Pj4+IENjOiBEYW5pZWwgVmV0dGVyIDxkYW5pZWwudmV0dGVyQGZmd2xsLmNoPgo+ID4+ Pj4+IENjOiBFbWlsIFZlbGlrb3YgPGVtaWwubC52ZWxpa292QGdtYWlsLmNvbT4KPiA+Pj4+PiBT aWduZWQtb2ZmLWJ5OiBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwu Y29tPgo+ID4+Pj4+IC0tLQo+ID4+Pj4+ICAgICBkcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYyB8 IDE4ICsrKysrKysrKysrKysrLS0tLQo+ID4+Pj4+ICAgICAxIGZpbGUgY2hhbmdlZCwgMTQgaW5z ZXJ0aW9ucygrKSwgNCBkZWxldGlvbnMoLSkKPiA+Pj4+Pgo+ID4+Pj4+IGRpZmYgLS1naXQgYS9k cml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jCj4g Pj4+Pj4gaW5kZXggNzIyMGI4ZjlhN2U4Li4wMGFhOThmM2U1NWQgMTAwNjQ0Cj4gPj4+Pj4gLS0t IGEvZHJpdmVycy9ncHUvZHJtL2RybV9lZGlkLmMKPiA+Pj4+PiArKysgYi9kcml2ZXJzL2dwdS9k cm0vZHJtX2VkaWQuYwo+ID4+Pj4+IEBAIC0yOTAzLDExICsyOTAzLDE1IEBAIGNlYV9tb2RlX2Fs dGVybmF0ZV90aW1pbmdzKHU4IHZpYywgc3RydWN0IGRybV9kaXNwbGF5X21vZGUgKm1vZGUpCj4g Pj4+Pj4gICAgIHN0YXRpYyB1OCBkcm1fbWF0Y2hfY2VhX21vZGVfY2xvY2tfdG9sZXJhbmNlKGNv bnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICp0b19tYXRjaCwKPiA+Pj4+PiAgICAgCQkJCQkg ICAgIHVuc2lnbmVkIGludCBjbG9ja190b2xlcmFuY2UpCj4gPj4+Pj4gICAgIHsKPiA+Pj4+PiAr CXVuc2lnbmVkIGludCBtYXRjaF9mbGFncyA9IERSTV9NT0RFX01BVENIX1RJTUlOR1MgfCBEUk1f TU9ERV9NQVRDSF9GTEFHUzsKPiA+Pj4+PiAgICAgCXU4IHZpYzsKPiA+Pj4+PiAgICAgCj4gPj4+ Pj4gICAgIAlpZiAoIXRvX21hdGNoLT5jbG9jaykKPiA+Pj4+PiAgICAgCQlyZXR1cm4gMDsKPiA+ Pj4+PiAgICAgCj4gPj4+Pj4gKwlpZiAodG9fbWF0Y2gtPnBpY3R1cmVfYXNwZWN0X3JhdGlvKQo+ ID4+Pj4+ICsJCW1hdGNoX2ZsYWdzIHw9IERSTV9NT0RFX01BVENIX0FTUEVDVF9SQVRJTzsKPiA+ Pj4+IFRoaXMgZG9lc24ndCBsb29rIHJpZ2h0LiBUaGlzIG1lYW5zIHdlIGFyZSBleHBlY3Rpbmcg YSBDRUEgbW9kZSB3aXRob3V0Cj4gPj4+PiBhIHBpYyBhc3BlY3QgcmF0aW8gZmllbGQsCj4gPj4+ PiB3aGljaCBpcyBpbnZhbGlkLgo+ID4+PiBObywgaXQncyBwZXJmZWN0bHkgdmFsaWQuIEl0J3Mg d2hhdCB3ZSBjdXJyZW50bHkgZ2V0IGZyb20gdXNlcnNwYWNlLgo+ID4+IFllcCwgYnV0IHRoYXQn cyBkdWUgdG8gbWlzc2luZyBBc3BlY3QgcmF0aW8gaGFuZGxpbmcgaW4gdGhlIERSTSBsYXllci4K PiA+PiBJZiB0aGF0J3MgZml4ZWQsIGFzIHBlciB0aGUgbGlzdCBvZiBDRUEgbW9kZXMsCj4gPj4g ZWFjaCBDRUEgVklDIGNvbnRhaW5zIGFuIGFzcGVjdCByYXRpbywgd2hpY2ggaXMgYSBwYXJ0IG9m IGl0cyB1bmlxdWUKPiA+PiBpZGVudGl0eS4KPiA+Pgo+ID4+IEkgZ3Vlc3Mgb25jZSB3ZSBoYXZl IHRoZSBhc3BlY3QgcmF0aW8gaGFuZGxpbmcgaW4gRFJNIGxheWVyLCBpdAo+ID4+IHdvdWxkL3No b3VsZCBsb29rIGxpa2UgdGhpczoKPiA+PiAtIEVESUQgZ2l2ZXMgeW91IGFsbCBzdXBwb3J0ZWQg bW9kZXMsIGluY2x1ZGluZyBDRUEgbW9kZXMgd2l0aCBBc3BlY3QgcmF0aW8KPiA+PiAtIFVzZXJz cGNhZSBnZXRzIHRoZSBtb2RlIGluZm9ybWF0aW9uLCB3aXRoIGFzcGVjdCByYXRpbyAoZm9yIENF QSBtb2RlcykKPiA+PiBJZiAoIFVzZXJzcGFjZSBwaWNrcyBvbmUgb2YgdGhlIENFQSBtb2RlcykK PiA+PiAgICAgICAtIHNlbmRzIGEgbW9kZXNldAo+ID4+ICAgICAgIC0gd2UgZmluZCBhIG1hdGNo aW5nIENFQSBWSUMsIGZvdW5kIG9uZSBmcm9tIG1vZGVkYgo+ID4+ICAgICAgIC0gd2UgbG9hZCB0 aGlzIFZJQyA9IG5vbnplcm8gaW5mb3JtYXRpb24gaW4gQVZJIElGIFZJQyBmaWVsZCwKPiA+PiBl bHNlCj4gPj4gICAgICAgLSBzZW5kcyBhIG1vZGVzZXQKPiA+PiAgICAgICAtIHdlIGNvdWxkIG5v dCBmaW5kIGEgbWF0Y2hpbmcgQ0VBIFZJQywgYXMgYXNwZWN0IHJhdGlvIGlzIDAKPiA+PiAgICAg ICAtIHdlIG1ha2UgVklDIGZpZWxkIGluIEFWSSBJRiBhcyAwYQo+ID4gTm8uIFRoYXQgd291bGQg YnJlYWsgY3VycmVudCB1c2Vyc3BhY2UuCj4gSSBndWVzcyBJIGZvcmdvdCB0byBtYWtlIGl0IGNs ZWFyLCB0aGF0IHVzZXJzcGFjZSB3aWxsIHNldCB0aGUgY2FwLCBvbmx5IAo+IHRoZW4gd2Ugd2ls bCBwcm92aWRlIGFzcGVjdCByYXRpbyBpbmZvcm1hdGlvbi4KPiBTbyB0aGlzIHNob3VsZCBub3Qg YnJlYWsgdXNlcnNwYWNlLCBpc24ndCBpdCA/Cj4gPj4gVGhpcyBpcyBpbXBvcnRhbnQsIGFzIEhE TUkgY29tcGxpYW5jZSB0ZXN0IDctMjcgaW5zcGVjdHMgaWYgdGhlIFZJQwo+ID4+IGZpZWxkIGlu IHRoZSBBVkkgSUYgaXMgYWNjdXJhdGUuCj4gPiBDb21wbGljYW5jZSBpcyBzZWNvbmRhcnkgdG8g bm90IGJyZWFraW5nIHRoaW5ncyB0aGF0IHdvcmsuIEFsc28gSSBmaW5kCj4gPiBpdCBoYXJkIHRv IHNlZSB3aGF0IHB1cnBvc2UgdGhlcmUgaXMgaW4gaGF2aW5nIGEgY29tcGxpY2FuY2UgdGVzdCB0 aGF0Cj4gPiBzZXRzIGEgQ0VBIG1vZGVzIHcvbyBhc3BlY3QgcmF0aW8gYW5kIHRoZW4gZXhwZWN0 cyB0aGUgaW5mb2ZyYW1lIHRvIGhhdmUKPiA+IFZJQyAwLgo+IEFnYWluLCB0eXBpY2FsbHkgdGhp cyBpcyBob3cgdGhlc2UgYW5hbHl6ZXJzIGZvcmNlIG1vZGVzZXQ6Cj4gLSBUaGV5IHNlbmQgRURJ RCB3aXRoIG9ubHkgb25lIG1vZGUsIHdoaWNoIGlzIHRoZSB0ZXN0IG1vZGUsIHdpdGggYXNwZWN0 IAo+IHJhdGlvLgo+IC0gVGhleSBleHBlY3QgdGhhdCBWSUMgdG8gYmUgcHJlc2VudCBpbiBBVkkg SUYKPiA+PiAtIFNoYXNoYW5rCj4gPj4+PiBBbmtpdCBpcyBnb2luZyB0byBwdWJsaXNoIHRoZSBh c3BlY3QgcmF0aW8gcGF0Y2gKPiA+Pj4+IHNlcmllcyBhZ2Fpbiwgd2l0aCBwcm9wZXIgRFJNIGNh cCBhbmQgZmxhZ3MgY2hlY2suIFdvdWxkIGl0IGJlCj4gPj4+PiBvayBpZiB3ZSBoYXZlIGEgbG9v ayB0aGF0IGhhbmRsaW5nIGZpcnN0ID8KPiA+Pj4gVGhpcyBwYXRjaCB3aWxsIGJlIG5lZWRlZCBi eSB0aGF0IHdvcmsuIE90aGVyd2lzZSB3ZSdyZSBnb2luZyB0byBzdG9wCj4gPj4+IHNlbmRpbmcg YSBWSUMgZm9yIENFQSBtb2RlcyB3aXRoIGN1cnJlbnQgdXNlcnNwYWNlLgo+ID4+IEkgZ3Vlc3Mg d2Ugc2hvdWxkIGZvcmNlIHVzZXJzcGFjZXMgdG8gc3RhcnQgYm90aGVyaW5nIGFib3V0IGFzcGVj dCByYXRpbwo+ID4+IGZpZWxkLCByaWdodCBub3cgd2UKPiA+PiBhcmUgZG9pbmcgdGhpcyBmb3Ig V2F5bGFuZCBiYXNlZCBjb21wb3NpdG9ycywgbWF5IGJlIHdlIHNob3VsZCBleHRlbmQgaXQKPiA+ PiB0byBYIGJhc2VkIHRvby4KPiA+IFllcywgSSd2ZSBiZWVuIHNheWluZyB0aGF0IHNvbWVvbmUg c2hvdWxkIGxvb2sgaW50byBleHRlbmRpbmcgdGhlIHJhbmRyCj4gPiBwcm90b2NvbCB3aXRoIHRo ZSBuZWNlc3NhcnkgYml0cy4gQnV0IHRoYXQgc3RpbGwgZG9lc24ndCBhbGxvdyB1cyB0bwo+ID4g Y2hhbmdlIHRoZSBjdXJyZW50IGJlaGF2aW91ciBhcyBvbGQgdXNlcnNwYWNlIHdvdWxkIGFueXdh eSBsaW5nZXIgYXJvdW5kCj4gPiBmb3IgYSBsb25nIHRpbWUuCj4gSSB0aGluayBjYXAgd2lsbCBj b3ZlIHRoaXMgcGFydAoKVGhlIGNhcCBpcyBpcnJlbGV2YW50IHRvIHRoaXMgZGlzY3Vzc2lvbi4g SXQgd2lsbCBub3QgYmUgc2V0IGJ5IG9sZAp1c2Vyc3BhY2UsIGhlbmNlIGl0IHdvbid0IGNoYW5n ZSBhbnl0aGluZyBmb3Igb2xkIHVzZXJzcGFjZS4gTW9kZXMKY29taW5nIGZyb20gb2xkIHVzZXJz cGFjZSB3aWxsIHN0aWxsIGhhdmUgYXNwZWN0X3JhdGlvPTAuCgotLSAKVmlsbGUgU3lyasOkbMOk CkludGVsIE9UQwpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f XwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcK aHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK