From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 07/10] drm/edid: Don't send bogus aspect ratios in AVI infoframes Date: Fri, 17 Nov 2017 13:38:57 +0200 Message-ID: <20171117113857.GD10981@intel.com> References: <20171113170427.4150-1-ville.syrjala@linux.intel.com> <20171113170427.4150-8-ville.syrjala@linux.intel.com> <20171116162600.GY10981@intel.com> <93fa0fa9-34f2-1878-3ff5-310f0d1a35e9@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: <93fa0fa9-34f2-1878-3ff5-310f0d1a35e9@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 , Daniel Vetter , intel-gfx@lists.freedesktop.org, Emil Velikov , dri-devel@lists.freedesktop.org, Nautiyal Ankit List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCBOb3YgMTcsIDIwMTcgYXQgMDg6NTM6NTRBTSArMDUzMCwgU2hhcm1hLCBTaGFzaGFu ayB3cm90ZToKPiBSZWdhcmRzCj4gCj4gU2hhc2hhbmsKPiAKPiAKPiBPbiAxMS8xNi8yMDE3IDk6 NTYgUE0sIFZpbGxlIFN5cmrDpGzDpCB3cm90ZToKPiA+IE9uIFRodSwgTm92IDE2LCAyMDE3IGF0 IDA4OjMxOjM2UE0gKzA1MzAsIFNoYXJtYSwgU2hhc2hhbmsgd3JvdGU6Cj4gPj4gUmVnYXJkcwo+ ID4+Cj4gPj4gU2hhc2hhbmsKPiA+Pgo+ID4+Cj4gPj4gT24gMTEvMTMvMjAxNyAxMDozNCBQTSwg VmlsbGUgU3lyamFsYSB3cm90ZToKPiA+Pj4gRnJvbTogVmlsbGUgU3lyasOkbMOkIDx2aWxsZS5z eXJqYWxhQGxpbnV4LmludGVsLmNvbT4KPiA+Pj4KPiA+Pj4gSWYgdGhlIHVzZXIgbW9kZSB3b3Vs ZCBzcGVjaWZ5IGFuIGFzcGVjdCByYXRpbyBvdGhlciB0aGFuIDQ6MyBvciAxNjo5Cj4gPj4+IHdl IG5vdyBzaWxlbnRseSBpZ25vcmUgaXQuIE1heWJlIGEgYmV0dGVyIGFwcG9yb2FjaCBpcyB0byBy ZXR1cm4gYW4KPiA+Pj4gZXJyb3I/IExldCdzIHRyeSB0aGF0Lgo+ID4+Pgo+ID4+PiBBbHNvIHdl IG11c3QgYmUgY2FyZWZ1bCB0aGF0IHdlIGRvbid0IHRyeSB0byBzZW5kIGlsbGVnYWwgcGljdHVy ZQo+ID4+PiBhc3BlY3QgaW4gdGhlIGluZm9mcmFtZSBhcyBpdCdzIG9ubHkgY2FwYWJsZSBvZiBz aWduYWxsaW5nIG5vbmUsCj4gPj4+IDQ6MywgYW5kIDE2OjkuIEN1cnJlbnRseSB3ZSdyZSBzZW5k aW5nIHRoZXNlIGJvZ3VzIGluZm9mcmFtZXMKPiA+Pj4gd2hlbmV2ZXIgdGhlIGNlYSBtb2RlIHNw ZWNpZmllcyBzb21lIG90aGVyIGFzcGVjdCByYXRpby4KPiA+Pj4KPiA+Pj4gQ2M6IFNoYXNoYW5r IFNoYXJtYSA8c2hhc2hhbmsuc2hhcm1hQGludGVsLmNvbT4KPiA+Pj4gQ2M6IFNlYW4gUGF1bCA8 c2VhbnBhdWxAY2hyb21pdW0ub3JnPgo+ID4+PiBDYzogSm9zZSBBYnJldSA8Sm9zZS5BYnJldUBz eW5vcHN5cy5jb20+Cj4gPj4+IENjOiBEYW5pZWwgVmV0dGVyIDxkYW5pZWwudmV0dGVyQGZmd2xs LmNoPgo+ID4+PiBDYzogRW1pbCBWZWxpa292IDxlbWlsLmwudmVsaWtvdkBnbWFpbC5jb20+Cj4g Pj4+IFNpZ25lZC1vZmYtYnk6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBsaW51eC5p bnRlbC5jb20+Cj4gPj4+IC0tLQo+ID4+PiAgICBkcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYyB8 IDIzICsrKysrKysrKysrKysrKysrLS0tLS0tCj4gPj4+ICAgIDEgZmlsZSBjaGFuZ2VkLCAxNyBp bnNlcnRpb25zKCspLCA2IGRlbGV0aW9ucygtKQo+ID4+Pgo+ID4+PiBkaWZmIC0tZ2l0IGEvZHJp dmVycy9ncHUvZHJtL2RybV9lZGlkLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYwo+ID4+ PiBpbmRleCAwMGFhOThmM2U1NWQuLmJhZmIzZWU0ZWE5NyAxMDA2NDQKPiA+Pj4gLS0tIGEvZHJp dmVycy9ncHUvZHJtL2RybV9lZGlkLmMKPiA+Pj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2RybV9l ZGlkLmMKPiA+Pj4gQEAgLTQ3ODYsNiArNDc4Niw3IEBAIGRybV9oZG1pX2F2aV9pbmZvZnJhbWVf ZnJvbV9kaXNwbGF5X21vZGUoc3RydWN0IGhkbWlfYXZpX2luZm9mcmFtZSAqZnJhbWUsCj4gPj4+ ICAgIAkJCQkJIGNvbnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICptb2RlLAo+ID4+PiAgICAJ CQkJCSBib29sIGlzX2hkbWkyX3NpbmspCj4gPj4+ICAgIHsKPiA+Pj4gKwllbnVtIGhkbWlfcGlj dHVyZV9hc3BlY3QgcGljdHVyZV9hc3BlY3Q7Cj4gPj4+ICAgIAlpbnQgZXJyOwo+ID4+PiAgICAK PiA+Pj4gICAgCWlmICghZnJhbWUgfHwgIW1vZGUpCj4gPj4+IEBAIC00ODI4LDEzICs0ODI5LDIz IEBAIGRybV9oZG1pX2F2aV9pbmZvZnJhbWVfZnJvbV9kaXNwbGF5X21vZGUoc3RydWN0IGhkbWlf YXZpX2luZm9mcmFtZSAqZnJhbWUsCj4gPj4+ICAgIAkgKiBQb3B1bGF0ZSBwaWN0dXJlIGFzcGVj dCByYXRpbyBmcm9tIGVpdGhlcgo+ID4+PiAgICAJICogdXNlciBpbnB1dCAoaWYgc3BlY2lmaWVk KSBvciBmcm9tIHRoZSBDRUEgbW9kZSBsaXN0Lgo+ID4+PiAgICAJICovCj4gPj4+IC0JaWYgKG1v ZGUtPnBpY3R1cmVfYXNwZWN0X3JhdGlvID09IEhETUlfUElDVFVSRV9BU1BFQ1RfNF8zIHx8Cj4g Pj4+IC0JCW1vZGUtPnBpY3R1cmVfYXNwZWN0X3JhdGlvID09IEhETUlfUElDVFVSRV9BU1BFQ1Rf MTZfOSkKPiA+Pj4gLQkJZnJhbWUtPnBpY3R1cmVfYXNwZWN0ID0gbW9kZS0+cGljdHVyZV9hc3Bl Y3RfcmF0aW87Cj4gPj4+IC0JZWxzZSBpZiAoZnJhbWUtPnZpZGVvX2NvZGUgPiAwKQo+ID4+PiAt CQlmcmFtZS0+cGljdHVyZV9hc3BlY3QgPSBkcm1fZ2V0X2NlYV9hc3BlY3RfcmF0aW8oCj4gPj4+ IC0JCQkJCQlmcmFtZS0+dmlkZW9fY29kZSk7Cj4gPj4+ICsJcGljdHVyZV9hc3BlY3QgPSBtb2Rl LT5waWN0dXJlX2FzcGVjdF9yYXRpbzsKPiA+Pj4gKwlpZiAocGljdHVyZV9hc3BlY3QgPT0gSERN SV9QSUNUVVJFX0FTUEVDVF9OT05FKQo+ID4+PiArCQlwaWN0dXJlX2FzcGVjdCA9IGRybV9nZXRf Y2VhX2FzcGVjdF9yYXRpbyhmcmFtZS0+dmlkZW9fY29kZSk7Cj4gPj4gVGhpcyBpcyBzbGlnaHRs eSBnb2luZyBpbiB0aGUgbG9vcC4KPiA+PiAtIER1cmluZyB0aGUgbW9kZXNldCB0aGUgZHJpdmVy IGNhbnQgc3BlY2lmeSB0aGUgYXNwZWN0IHJhdGlvCj4gPj4gaW5mb3JtYXRpb24sIGFzIERSTSBs YXllciBsYWNrcyB0aGlzIHN1cHBvcnQuCj4gPj4gLSBTbyB3ZSBmaWxsIHRoZSBWSUMgZmllbGQs IGJ5IGNvbXBhcmluZyB0aGUgbW9kZSB3aXRoIHRoZQo+ID4+IERSTV9DRUFfTU9ERVNbXSBsaXN0 LiBUaGlzIHdpbGwgcGljayB0aGUgZmlyc3QgbW9kZQo+ID4+ICAgICBhdmFpbGFibGUgaW4gdGhl IGxpc3QgKHJlZ2FyZGxlc3Mgb2YgaXRzIGFzcGVjdCByYXRpbyksIGFuZCBmaWxsIHRoZQo+ID4+ IFZJQywgYXMgd2UgZG9uJ3QgY29uc2lkZXIgYXNwZWN0IHJhdGlvIHdoaWxlIGNvbXBhcmluZyB0 aW1pbmdzLgo+ID4+IC0gQWdhaW4sIG5vdyB3aGlsZSBzZW5kaW5nIHRoZSBhc3BlY3QgcmF0aW8s IHdlIGFyZSBwaWNraW5nIHVwIHRoZSBWSUMsCj4gPj4gd2hpY2ggbWF5IG5vdCBiZSBjb3JyZWN0 Lgo+ID4+Cj4gPj4gU28gaWYgd2UgaGF2ZSA3MjB4NDgwKDQ6MykgYW5kIDcyMHg0ODAoMTY6OSkg aW4gdGhlIGxpc3QsIGFzIDQ6MyBpcwo+ID4+IGZpcnN0IGluIGxpc3QsIHdlIHdpbGwgYWx3YXlz IHBpY2sgNDozIGFzcGVjdCByYXRpby4KPiA+IFllcy4gVGhlIHVzZXIgZGlkbid0IGNhcmUgYWJv dXQgdGhlIGFzcGVjdCByYXRpbyAob3IgcmF0aGVyIGNvdWxkbid0Cj4gPiBzcGVjaWZ5IG9uZSkg c28gd2UganVzdCBwaWNrIG9uZS4gV2hpY2ggaXMgZXhhY3RseSB3aGF0IHdlJ3ZlIGJlZW4KPiA+ IGRvaW5nIGV2ZXIgc2luY2Ugd2Ugc3RhcnRlZCBzZW5kaW5nIHRoZSBWSUMgaW4gdGhlIGluZm9m cmFtZS4KPiBDb3JyZWN0LCBhbmQgd2UgYXJlIGhvcGluZyB0aGF0IHRoaXMgc2hvdWxkIGJlIGJl dHRlciAoaWYgbm90IGZpeGVkKSAKPiB3aXRoIHRoZSBhc3BlY3QgcmF0aW8gc3VwcG9ydAo+IHBh dGNoZXMgKyBEUk0gY2FwLiBJZiB0aGUgdXNlcnNwYWNlIGRvZXNuJ3Qgc2V0IHRoZSBjYXAsIHRo ZW4gYW55d2F5cyAKPiB0aGVyZSBpcyBubyBhc3BlY3QgcmF0aW8KPiBmaWVsZCBhdmFpbGFibGUs IGFuZCBWSUMgd291bGQgYmUgYWx3YXlzIDAsIGFzIHRoaXMgYmVjb21lcyBhIE5vbiBDRUEgbW9k ZS4KPiAKPiBPciBkbyB5b3UgdGhpbmsgaXQgd291bGQgYmUgYSBiZXR0ZXIgaWRlYSB0byBzZW5k IHNvbWUgVklDIGluc3RlYWQgb2YgTm8gCj4gVklDLCB3aGVuIHVzZXJzcGFjZSBkb2Vzbid0Cj4g c2V0IHRoZSBEUk0gY2FwIGZvciBhc3BlY3QgcmF0aW8gPwoKWWVzLiBUaGF0J3MgdGhlIGN1cnJl bnQgYmVoYXZpb3VyLgoKSUlSQyBzb21lIGNyYXBweSBhbXBsaWZpZXJzIGV0Yy4gd2l0aCBIRE1J IHBhc3N0aHJvdWdoIGRvbid0IGV2ZW4gd29yawpjb3JyZWN0bHkgdW5sZXNzIFZJQyBpcyBzcGVj aWZpZWQuIEhlbmNlIHdlIGRvIHdhbnQgdG8gc2VuZCBpdCB3aGVuZXZlcgpwb3NzaWJsZS4KCj4g Cj4gLSBTaGFzaGFuawo+ID4+IC0gU2hhc2hhbmsKPiA+Pj4gICAgCj4gPj4+ICsJLyoKPiA+Pj4g KwkgKiBUaGUgaW5mb2ZyYW1lIGNhbid0IGNvbnZleSBhbnl0aGluZyBidXQgbm9uZSwgNDozCj4g Pj4+ICsJICogYW5kIDE2OjksIHNvIGlmIHRoZSB1c2VyIGhhcyBhc2tlZCBmb3IgYW55dGhpbmcg ZWxzZQo+ID4+PiArCSAqIHdlIGNhbiBvbmx5IHNhdGlzZnkgaXQgYnkgc3BlY2lmeWluZyB0aGUg cmlnaHQgVklDLgo+ID4+PiArCSAqLwo+ID4+PiArCWlmIChwaWN0dXJlX2FzcGVjdCA+IEhETUlf UElDVFVSRV9BU1BFQ1RfMTZfOSkgewo+ID4+PiArCQlpZiAocGljdHVyZV9hc3BlY3QgIT0KPiA+ Pj4gKwkJICAgIGRybV9nZXRfY2VhX2FzcGVjdF9yYXRpbyhmcmFtZS0+dmlkZW9fY29kZSkpCj4g Pj4+ICsJCQlyZXR1cm4gLUVJTlZBTDsKPiA+Pj4gKwkJcGljdHVyZV9hc3BlY3QgPSBIRE1JX1BJ Q1RVUkVfQVNQRUNUX05PTkU7Cj4gPj4+ICsJfQo+ID4+PiArCj4gPj4+ICsJZnJhbWUtPnBpY3R1 cmVfYXNwZWN0ID0gcGljdHVyZV9hc3BlY3Q7Cj4gPj4+ICAgIAlmcmFtZS0+YWN0aXZlX2FzcGVj dCA9IEhETUlfQUNUSVZFX0FTUEVDVF9QSUNUVVJFOwo+ID4+PiAgICAJZnJhbWUtPnNjYW5fbW9k ZSA9IEhETUlfU0NBTl9NT0RFX1VOREVSU0NBTjsKPiA+Pj4gICAgCgotLSAKVmlsbGUgU3lyasOk bMOkCkludGVsIE9UQwpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5v cmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2 ZWwK