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, 24 Nov 2017 15:22:29 +0200 Message-ID: <20171124132229.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> <20171117124947.GE10981@intel.com> <5f782a25-1289-ffd3-7d73-289fed577269@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: <5f782a25-1289-ffd3-7d73-289fed577269@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 T24gRnJpLCBOb3YgMjQsIDIwMTcgYXQgMDI6MjY6MDlQTSArMDUzMCwgU2hhcm1hLCBTaGFzaGFu ayB3cm90ZToKPiBSZWdhcmRzCj4gCj4gU2hhc2hhbmsKPiAKPiAKPiBPbiAxMS8xNy8yMDE3IDY6 MTkgUE0sIFZpbGxlIFN5cmrDpGzDpCB3cm90ZToKPiA+IE9uIEZyaSwgTm92IDE3LCAyMDE3IGF0 IDA1OjUwOjExUE0gKzA1MzAsIFNoYXJtYSwgU2hhc2hhbmsgd3JvdGU6Cj4gPj4gUmVnYXJkcwo+ ID4+Cj4gPj4gU2hhc2hhbmsKPiA+Pgo+ID4+Cj4gPj4gT24gMTEvMTcvMjAxNyA1OjA1IFBNLCBW aWxsZSBTeXJqw6Rsw6Qgd3JvdGU6Cj4gPj4+IE9uIEZyaSwgTm92IDE3LCAyMDE3IGF0IDA4OjQ5 OjQ5QU0gKzA1MzAsIFNoYXJtYSwgU2hhc2hhbmsgd3JvdGU6Cj4gPj4+PiBSZWdhcmRzCj4gPj4+ Pgo+ID4+Pj4gU2hhc2hhbmsKPiA+Pj4+Cj4gPj4+Pgo+ID4+Pj4gT24gMTEvMTYvMjAxNyA5OjUz IFBNLCBWaWxsZSBTeXJqw6Rsw6Qgd3JvdGU6Cj4gPj4+Pj4gT24gVGh1LCBOb3YgMTYsIDIwMTcg YXQgMDg6MjE6NDRQTSArMDUzMCwgU2hhcm1hLCBTaGFzaGFuayB3cm90ZToKPiA+Pj4+Pj4gUmVn YXJkcwo+ID4+Pj4+Pgo+ID4+Pj4+PiBTaGFzaGFuawo+ID4+Pj4+Pgo+ID4+Pj4+Pgo+ID4+Pj4+ PiBPbiAxMS8xMy8yMDE3IDEwOjM0IFBNLCBWaWxsZSBTeXJqYWxhIHdyb3RlOgo+ID4+Pj4+Pj4g RnJvbTogVmlsbGUgU3lyasOkbMOkIDx2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4KPiA+ Pj4+Pj4+Cj4gPj4+Pj4+PiBjb21taXQgNmRmZmQ0MzFlMjI5ICgiZHJtOiBBZGQgYXNwZWN0IHJh dGlvIHBhcnNpbmcgaW4gRFJNIGxheWVyIikKPiA+Pj4+Pj4+IGNhdXNlIHVzIHRvIG5vdCBzZW5k IG91dCBhbnkgVklDcyBpbiB0aGUgQVZJIGluZm9mcmFtZXMuIFRoYXQgY29tbWl0Cj4gPj4+Pj4+ PiB3YXMgc2luY2UgcmV2ZXJ0ZWQsIGJ1dCBpZiBhbmQgd2hlbiB3ZSBhZGQgYXNwZWN0IHJhdGlv IGhhbmRpbmcgYmFjawo+ID4+Pj4+Pj4gd2UgbmVlZCB0byBiZSBtb3JlIGNhcmVmdWwuCj4gPj4+ Pj4+Pgo+ID4+Pj4+Pj4gTGV0J3MgaGFuZGxlIHRoaXMgYnkgY29uc2lkZXJpbmcgdGhlIGFzcGVj dCByYXRpbyBhcyBhIHJlcXVpcmVtZW50Cj4gPj4+Pj4+PiBmb3IgY2VhIG1vZGUgbWF0Y2hpbmcg b25seSBpZiB0aGUgcGFzc2VkIGluIG1vZGUgYWN0dWFsbHkgaGFzIGEKPiA+Pj4+Pj4+IG5vbi16 ZXJvIGFzcGVjdCByYXRpbyBmaWVsZC4gVGhpcyB3aWxsIGtlZXAgdXNlcnNwYWNlIHRoYXQgZG9l c24ndAo+ID4+Pj4+Pj4gcHJvdmlkZSBhbiBhc3BlY3QgcmF0aW8gd29ya2luZyBhcyBiZWZvcmUg YnkgbWF0Y2hpbmcgaXQgdG8gdGhlCj4gPj4+Pj4+PiBmaXJzdCBvdGhlcndpc2UgZXF1YWwgY2Vh IG1vZGUuIEFuZCBvbmNlIHVzZXJzcGFjZSBzdGFydHMgdG8KPiA+Pj4+Pj4+IHByb3ZpZGUgdGhl IGFzcGVjdCByYXRpbyBpdCB3aWxsIGJlIGNvbnNpZGVyZCBhIGhhcmQgcmVxdWlyZW1lbnQKPiA+ Pj4+Pj4+IGZvciB0aGUgbWF0Y2guCj4gPj4+Pj4+Pgo+ID4+Pj4+Pj4gQWxzbyBjaGFuZ2UgdGhl IGhkbWkgbW9kZSBtYXRjaGluZyB0byB1c2UgZHJtX21vZGVfbWF0Y2goKSBmb3IKPiA+Pj4+Pj4+ IGNvbnNpc3RlbmN5LCBidXQgd2UgZG9uJ3QgbWF0Y2ggb24gYXNwZWN0IHJhdGlvIHRoZXJlIHNp bmNlIHRoZQo+ID4+Pj4+Pj4gc3BlYyBkb2Vzbid0IGxpc3QgYSBzcGVjaWZpYyBhc3BlY3QgcmF0 aW8gZm9yIHRob3NlIG1vZGVzLgo+ID4+Pj4+Pj4KPiA+Pj4+Pj4+IENjOiBTaGFzaGFuayBTaGFy bWEgPHNoYXNoYW5rLnNoYXJtYUBpbnRlbC5jb20+Cj4gPj4+Pj4+PiBDYzogIkxpbiwgSmlhIiA8 bGluLmEuamlhQGludGVsLmNvbT4KPiA+Pj4+Pj4+IENjOiBBa2FzaGRlZXAgU2hhcm1hIDxha2Fz aGRlZXAuc2hhcm1hQGludGVsLmNvbT4KPiA+Pj4+Pj4+IENjOiBKaW0gQnJpZGUgPGppbS5icmlk ZUBsaW51eC5pbnRlbC5jb20+Cj4gPj4+Pj4+PiBDYzogSm9zZSBBYnJldSA8Sm9zZS5BYnJldUBz eW5vcHN5cy5jb20+Cj4gPj4+Pj4+PiBDYzogRGFuaWVsIFZldHRlciA8ZGFuaWVsLnZldHRlckBm ZndsbC5jaD4KPiA+Pj4+Pj4+IENjOiBFbWlsIFZlbGlrb3YgPGVtaWwubC52ZWxpa292QGdtYWls LmNvbT4KPiA+Pj4+Pj4+IFNpZ25lZC1vZmYtYnk6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3ly amFsYUBsaW51eC5pbnRlbC5jb20+Cj4gPj4+Pj4+PiAtLS0KPiA+Pj4+Pj4+ICAgICAgZHJpdmVy cy9ncHUvZHJtL2RybV9lZGlkLmMgfCAxOCArKysrKysrKysrKysrKy0tLS0KPiA+Pj4+Pj4+ICAg ICAgMSBmaWxlIGNoYW5nZWQsIDE0IGluc2VydGlvbnMoKyksIDQgZGVsZXRpb25zKC0pCj4gPj4+ Pj4+Pgo+ID4+Pj4+Pj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jIGIv ZHJpdmVycy9ncHUvZHJtL2RybV9lZGlkLmMKPiA+Pj4+Pj4+IGluZGV4IDcyMjBiOGY5YTdlOC4u MDBhYTk4ZjNlNTVkIDEwMDY0NAo+ID4+Pj4+Pj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2RybV9l ZGlkLmMKPiA+Pj4+Pj4+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jCj4gPj4+Pj4+ PiBAQCAtMjkwMywxMSArMjkwMywxNSBAQCBjZWFfbW9kZV9hbHRlcm5hdGVfdGltaW5ncyh1OCB2 aWMsIHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICptb2RlKQo+ID4+Pj4+Pj4gICAgICBzdGF0aWMg dTggZHJtX21hdGNoX2NlYV9tb2RlX2Nsb2NrX3RvbGVyYW5jZShjb25zdCBzdHJ1Y3QgZHJtX2Rp c3BsYXlfbW9kZSAqdG9fbWF0Y2gsCj4gPj4+Pj4+PiAgICAgIAkJCQkJICAgICB1bnNpZ25lZCBp bnQgY2xvY2tfdG9sZXJhbmNlKQo+ID4+Pj4+Pj4gICAgICB7Cj4gPj4+Pj4+PiArCXVuc2lnbmVk IGludCBtYXRjaF9mbGFncyA9IERSTV9NT0RFX01BVENIX1RJTUlOR1MgfCBEUk1fTU9ERV9NQVRD SF9GTEFHUzsKPiA+Pj4+Pj4+ICAgICAgCXU4IHZpYzsKPiA+Pj4+Pj4+ICAgICAgCj4gPj4+Pj4+ PiAgICAgIAlpZiAoIXRvX21hdGNoLT5jbG9jaykKPiA+Pj4+Pj4+ICAgICAgCQlyZXR1cm4gMDsK PiA+Pj4+Pj4+ICAgICAgCj4gPj4+Pj4+PiArCWlmICh0b19tYXRjaC0+cGljdHVyZV9hc3BlY3Rf cmF0aW8pCj4gPj4+Pj4+PiArCQltYXRjaF9mbGFncyB8PSBEUk1fTU9ERV9NQVRDSF9BU1BFQ1Rf UkFUSU87Cj4gPj4+Pj4+IFRoaXMgZG9lc24ndCBsb29rIHJpZ2h0LiBUaGlzIG1lYW5zIHdlIGFy ZSBleHBlY3RpbmcgYSBDRUEgbW9kZSB3aXRob3V0Cj4gPj4+Pj4+IGEgcGljIGFzcGVjdCByYXRp byBmaWVsZCwKPiA+Pj4+Pj4gd2hpY2ggaXMgaW52YWxpZC4KPiA+Pj4+PiBObywgaXQncyBwZXJm ZWN0bHkgdmFsaWQuIEl0J3Mgd2hhdCB3ZSBjdXJyZW50bHkgZ2V0IGZyb20gdXNlcnNwYWNlLgo+ ID4+Pj4gWWVwLCBidXQgdGhhdCdzIGR1ZSB0byBtaXNzaW5nIEFzcGVjdCByYXRpbyBoYW5kbGlu ZyBpbiB0aGUgRFJNIGxheWVyLgo+ID4+Pj4gSWYgdGhhdCdzIGZpeGVkLCBhcyBwZXIgdGhlIGxp c3Qgb2YgQ0VBIG1vZGVzLAo+ID4+Pj4gZWFjaCBDRUEgVklDIGNvbnRhaW5zIGFuIGFzcGVjdCBy YXRpbywgd2hpY2ggaXMgYSBwYXJ0IG9mIGl0cyB1bmlxdWUKPiA+Pj4+IGlkZW50aXR5Lgo+ID4+ Pj4KPiA+Pj4+IEkgZ3Vlc3Mgb25jZSB3ZSBoYXZlIHRoZSBhc3BlY3QgcmF0aW8gaGFuZGxpbmcg aW4gRFJNIGxheWVyLCBpdAo+ID4+Pj4gd291bGQvc2hvdWxkIGxvb2sgbGlrZSB0aGlzOgo+ID4+ Pj4gLSBFRElEIGdpdmVzIHlvdSBhbGwgc3VwcG9ydGVkIG1vZGVzLCBpbmNsdWRpbmcgQ0VBIG1v ZGVzIHdpdGggQXNwZWN0IHJhdGlvCj4gPj4+PiAtIFVzZXJzcGNhZSBnZXRzIHRoZSBtb2RlIGlu Zm9ybWF0aW9uLCB3aXRoIGFzcGVjdCByYXRpbyAoZm9yIENFQSBtb2RlcykKPiA+Pj4+IElmICgg VXNlcnNwYWNlIHBpY2tzIG9uZSBvZiB0aGUgQ0VBIG1vZGVzKQo+ID4+Pj4gICAgICAgIC0gc2Vu ZHMgYSBtb2Rlc2V0Cj4gPj4+PiAgICAgICAgLSB3ZSBmaW5kIGEgbWF0Y2hpbmcgQ0VBIFZJQywg Zm91bmQgb25lIGZyb20gbW9kZWRiCj4gPj4+PiAgICAgICAgLSB3ZSBsb2FkIHRoaXMgVklDID0g bm9uemVybyBpbmZvcm1hdGlvbiBpbiBBVkkgSUYgVklDIGZpZWxkLAo+ID4+Pj4gZWxzZQo+ID4+ Pj4gICAgICAgIC0gc2VuZHMgYSBtb2Rlc2V0Cj4gPj4+PiAgICAgICAgLSB3ZSBjb3VsZCBub3Qg ZmluZCBhIG1hdGNoaW5nIENFQSBWSUMsIGFzIGFzcGVjdCByYXRpbyBpcyAwCj4gPj4+PiAgICAg ICAgLSB3ZSBtYWtlIFZJQyBmaWVsZCBpbiBBVkkgSUYgYXMgMAo+ID4+PiBOby4gVGhhdCB3b3Vs ZCBicmVhayBjdXJyZW50IHVzZXJzcGFjZS4KPiA+PiBJIGd1ZXNzIEkgZm9yZ290IHRvIG1ha2Ug aXQgY2xlYXIsIHRoYXQgdXNlcnNwYWNlIHdpbGwgc2V0IHRoZSBjYXAsIG9ubHkKPiA+PiB0aGVu IHdlIHdpbGwgcHJvdmlkZSBhc3BlY3QgcmF0aW8gaW5mb3JtYXRpb24uCj4gPj4gU28gdGhpcyBz aG91bGQgbm90IGJyZWFrIHVzZXJzcGFjZSwgaXNuJ3QgaXQgPwo+ID4+Pj4gVGhpcyBpcyBpbXBv cnRhbnQsIGFzIEhETUkgY29tcGxpYW5jZSB0ZXN0IDctMjcgaW5zcGVjdHMgaWYgdGhlIFZJQwo+ ID4+Pj4gZmllbGQgaW4gdGhlIEFWSSBJRiBpcyBhY2N1cmF0ZS4KPiA+Pj4gQ29tcGxpY2FuY2Ug aXMgc2Vjb25kYXJ5IHRvIG5vdCBicmVha2luZyB0aGluZ3MgdGhhdCB3b3JrLiBBbHNvIEkgZmlu ZAo+ID4+PiBpdCBoYXJkIHRvIHNlZSB3aGF0IHB1cnBvc2UgdGhlcmUgaXMgaW4gaGF2aW5nIGEg Y29tcGxpY2FuY2UgdGVzdCB0aGF0Cj4gPj4+IHNldHMgYSBDRUEgbW9kZXMgdy9vIGFzcGVjdCBy YXRpbyBhbmQgdGhlbiBleHBlY3RzIHRoZSBpbmZvZnJhbWUgdG8gaGF2ZQo+ID4+PiBWSUMgMC4K PiA+PiBBZ2FpbiwgdHlwaWNhbGx5IHRoaXMgaXMgaG93IHRoZXNlIGFuYWx5emVycyBmb3JjZSBt b2Rlc2V0Ogo+ID4+IC0gVGhleSBzZW5kIEVESUQgd2l0aCBvbmx5IG9uZSBtb2RlLCB3aGljaCBp cyB0aGUgdGVzdCBtb2RlLCB3aXRoIGFzcGVjdAo+ID4+IHJhdGlvLgo+ID4+IC0gVGhleSBleHBl Y3QgdGhhdCBWSUMgdG8gYmUgcHJlc2VudCBpbiBBVkkgSUYKPiA+Pj4+IC0gU2hhc2hhbmsKPiA+ Pj4+Pj4gQW5raXQgaXMgZ29pbmcgdG8gcHVibGlzaCB0aGUgYXNwZWN0IHJhdGlvIHBhdGNoCj4g Pj4+Pj4+IHNlcmllcyBhZ2Fpbiwgd2l0aCBwcm9wZXIgRFJNIGNhcCBhbmQgZmxhZ3MgY2hlY2su IFdvdWxkIGl0IGJlCj4gPj4+Pj4+IG9rIGlmIHdlIGhhdmUgYSBsb29rIHRoYXQgaGFuZGxpbmcg Zmlyc3QgPwo+ID4+Pj4+IFRoaXMgcGF0Y2ggd2lsbCBiZSBuZWVkZWQgYnkgdGhhdCB3b3JrLiBP dGhlcndpc2Ugd2UncmUgZ29pbmcgdG8gc3RvcAo+ID4+Pj4+IHNlbmRpbmcgYSBWSUMgZm9yIENF QSBtb2RlcyB3aXRoIGN1cnJlbnQgdXNlcnNwYWNlLgo+ID4+Pj4gSSBndWVzcyB3ZSBzaG91bGQg Zm9yY2UgdXNlcnNwYWNlcyB0byBzdGFydCBib3RoZXJpbmcgYWJvdXQgYXNwZWN0IHJhdGlvCj4g Pj4+PiBmaWVsZCwgcmlnaHQgbm93IHdlCj4gPj4+PiBhcmUgZG9pbmcgdGhpcyBmb3IgV2F5bGFu ZCBiYXNlZCBjb21wb3NpdG9ycywgbWF5IGJlIHdlIHNob3VsZCBleHRlbmQgaXQKPiA+Pj4+IHRv IFggYmFzZWQgdG9vLgo+ID4+PiBZZXMsIEkndmUgYmVlbiBzYXlpbmcgdGhhdCBzb21lb25lIHNo b3VsZCBsb29rIGludG8gZXh0ZW5kaW5nIHRoZSByYW5kcgo+ID4+PiBwcm90b2NvbCB3aXRoIHRo ZSBuZWNlc3NhcnkgYml0cy4gQnV0IHRoYXQgc3RpbGwgZG9lc24ndCBhbGxvdyB1cyB0bwo+ID4+ PiBjaGFuZ2UgdGhlIGN1cnJlbnQgYmVoYXZpb3VyIGFzIG9sZCB1c2Vyc3BhY2Ugd291bGQgYW55 d2F5IGxpbmdlciBhcm91bmQKPiA+Pj4gZm9yIGEgbG9uZyB0aW1lLgo+ID4+IEkgdGhpbmsgY2Fw IHdpbGwgY292ZSB0aGlzIHBhcnQKPiA+IFRoZSBjYXAgaXMgaXJyZWxldmFudCB0byB0aGlzIGRp c2N1c3Npb24uIEl0IHdpbGwgbm90IGJlIHNldCBieSBvbGQKPiA+IHVzZXJzcGFjZSwgaGVuY2Ug aXQgd29uJ3QgY2hhbmdlIGFueXRoaW5nIGZvciBvbGQgdXNlcnNwYWNlLiBNb2Rlcwo+ID4gY29t aW5nIGZyb20gb2xkIHVzZXJzcGFjZSB3aWxsIHN0aWxsIGhhdmUgYXNwZWN0X3JhdGlvPTAuCj4g QWdyZWUsIGFuZCBpbiB0aGF0IGNhc2UsIHdoeSBib3RoZXIgYWJvdXQgYXNwZWN0IHJhdGlvIGZs YWdzIGhlcmUgaW4gCj4gdGhpcyBzZXJpZXMgPyBQcm9iYWJseSBsZXQgdGhpcyBiZSB0YWtlbiBj YXJlIGluIGFzcGVjdCByYXRpbyBzZXJpZXMuCgpJZiBzb21lb25lIHdhbnRzIHRvIHN1Y2sgdGhl c2UgcGF0Y2hlcyBpbnRvIHRoZSBhc3BlY3QgcmF0aW8gc2VyaWVzIHRoYXQKZmluZSBieSBtZS4K CkkgaGFkIHRoZW0gaGVyZSBiZWNhdXNlIHRoZXkgd2VyZSBvbiB0aGUgc2FtZSBicmFuY2ggd2hl cmUgSSBoYWQgdGhlCmZpeCBmb3IgdGhlIGFzcGVjdCByYXRpbyBpbiB0aGUgaW5mb2ZyYW1lLiBB bmQgSSB3YXMgdG9vIGxhenkgdG8KcmVtb3ZlIHRoZW0gZnJvbSB0aGUgc2VyaWVzLCBmaWd1cmlu ZyB0aGV5J3JlIHNlbWktcmVsYXRlZCBhbnl3YXkuCgotLSAKVmlsbGUgU3lyasOkbMOkCkludGVs IE9UQwpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmkt ZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6 Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK