From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Sharma, Shashank" Subject: Re: [PATCH 07/10] drm/edid: Don't send bogus aspect ratios in AVI infoframes Date: Fri, 17 Nov 2017 08:53:54 +0530 Message-ID: <93fa0fa9-34f2-1878-3ff5-310f0d1a35e9@intel.com> References: <20171113170427.4150-1-ville.syrjala@linux.intel.com> <20171113170427.4150-8-ville.syrjala@linux.intel.com> <20171116162600.GY10981@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20171116162600.GY10981@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= Cc: Jose Abreu , Daniel Vetter , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Nautiyal Ankit List-Id: dri-devel@lists.freedesktop.org UmVnYXJkcwoKU2hhc2hhbmsKCgpPbiAxMS8xNi8yMDE3IDk6NTYgUE0sIFZpbGxlIFN5cmrDpGzD pCB3cm90ZToKPiBPbiBUaHUsIE5vdiAxNiwgMjAxNyBhdCAwODozMTozNlBNICswNTMwLCBTaGFy bWEsIFNoYXNoYW5rIHdyb3RlOgo+PiBSZWdhcmRzCj4+Cj4+IFNoYXNoYW5rCj4+Cj4+Cj4+IE9u IDExLzEzLzIwMTcgMTA6MzQgUE0sIFZpbGxlIFN5cmphbGEgd3JvdGU6Cj4+PiBGcm9tOiBWaWxs ZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+Pj4KPj4+IElmIHRo ZSB1c2VyIG1vZGUgd291bGQgc3BlY2lmeSBhbiBhc3BlY3QgcmF0aW8gb3RoZXIgdGhhbiA0OjMg b3IgMTY6OQo+Pj4gd2Ugbm93IHNpbGVudGx5IGlnbm9yZSBpdC4gTWF5YmUgYSBiZXR0ZXIgYXBw b3JvYWNoIGlzIHRvIHJldHVybiBhbgo+Pj4gZXJyb3I/IExldCdzIHRyeSB0aGF0Lgo+Pj4KPj4+ IEFsc28gd2UgbXVzdCBiZSBjYXJlZnVsIHRoYXQgd2UgZG9uJ3QgdHJ5IHRvIHNlbmQgaWxsZWdh bCBwaWN0dXJlCj4+PiBhc3BlY3QgaW4gdGhlIGluZm9mcmFtZSBhcyBpdCdzIG9ubHkgY2FwYWJs ZSBvZiBzaWduYWxsaW5nIG5vbmUsCj4+PiA0OjMsIGFuZCAxNjo5LiBDdXJyZW50bHkgd2UncmUg c2VuZGluZyB0aGVzZSBib2d1cyBpbmZvZnJhbWVzCj4+PiB3aGVuZXZlciB0aGUgY2VhIG1vZGUg c3BlY2lmaWVzIHNvbWUgb3RoZXIgYXNwZWN0IHJhdGlvLgo+Pj4KPj4+IENjOiBTaGFzaGFuayBT aGFybWEgPHNoYXNoYW5rLnNoYXJtYUBpbnRlbC5jb20+Cj4+PiBDYzogU2VhbiBQYXVsIDxzZWFu cGF1bEBjaHJvbWl1bS5vcmc+Cj4+PiBDYzogSm9zZSBBYnJldSA8Sm9zZS5BYnJldUBzeW5vcHN5 cy5jb20+Cj4+PiBDYzogRGFuaWVsIFZldHRlciA8ZGFuaWVsLnZldHRlckBmZndsbC5jaD4KPj4+ IENjOiBFbWlsIFZlbGlrb3YgPGVtaWwubC52ZWxpa292QGdtYWlsLmNvbT4KPj4+IFNpZ25lZC1v ZmYtYnk6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBsaW51eC5pbnRlbC5jb20+Cj4+ PiAtLS0KPj4+ICAgIGRyaXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jIHwgMjMgKysrKysrKysrKysr KysrKystLS0tLS0KPj4+ICAgIDEgZmlsZSBjaGFuZ2VkLCAxNyBpbnNlcnRpb25zKCspLCA2IGRl bGV0aW9ucygtKQo+Pj4KPj4+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQu YyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jCj4+PiBpbmRleCAwMGFhOThmM2U1NWQuLmJh ZmIzZWU0ZWE5NyAxMDA2NDQKPj4+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9kcm1fZWRpZC5jCj4+ PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2VkaWQuYwo+Pj4gQEAgLTQ3ODYsNiArNDc4Niw3 IEBAIGRybV9oZG1pX2F2aV9pbmZvZnJhbWVfZnJvbV9kaXNwbGF5X21vZGUoc3RydWN0IGhkbWlf YXZpX2luZm9mcmFtZSAqZnJhbWUsCj4+PiAgICAJCQkJCSBjb25zdCBzdHJ1Y3QgZHJtX2Rpc3Bs YXlfbW9kZSAqbW9kZSwKPj4+ICAgIAkJCQkJIGJvb2wgaXNfaGRtaTJfc2luaykKPj4+ICAgIHsK Pj4+ICsJZW51bSBoZG1pX3BpY3R1cmVfYXNwZWN0IHBpY3R1cmVfYXNwZWN0Owo+Pj4gICAgCWlu dCBlcnI7Cj4+PiAgICAKPj4+ICAgIAlpZiAoIWZyYW1lIHx8ICFtb2RlKQo+Pj4gQEAgLTQ4Mjgs MTMgKzQ4MjksMjMgQEAgZHJtX2hkbWlfYXZpX2luZm9mcmFtZV9mcm9tX2Rpc3BsYXlfbW9kZShz dHJ1Y3QgaGRtaV9hdmlfaW5mb2ZyYW1lICpmcmFtZSwKPj4+ICAgIAkgKiBQb3B1bGF0ZSBwaWN0 dXJlIGFzcGVjdCByYXRpbyBmcm9tIGVpdGhlcgo+Pj4gICAgCSAqIHVzZXIgaW5wdXQgKGlmIHNw ZWNpZmllZCkgb3IgZnJvbSB0aGUgQ0VBIG1vZGUgbGlzdC4KPj4+ICAgIAkgKi8KPj4+IC0JaWYg KG1vZGUtPnBpY3R1cmVfYXNwZWN0X3JhdGlvID09IEhETUlfUElDVFVSRV9BU1BFQ1RfNF8zIHx8 Cj4+PiAtCQltb2RlLT5waWN0dXJlX2FzcGVjdF9yYXRpbyA9PSBIRE1JX1BJQ1RVUkVfQVNQRUNU XzE2XzkpCj4+PiAtCQlmcmFtZS0+cGljdHVyZV9hc3BlY3QgPSBtb2RlLT5waWN0dXJlX2FzcGVj dF9yYXRpbzsKPj4+IC0JZWxzZSBpZiAoZnJhbWUtPnZpZGVvX2NvZGUgPiAwKQo+Pj4gLQkJZnJh bWUtPnBpY3R1cmVfYXNwZWN0ID0gZHJtX2dldF9jZWFfYXNwZWN0X3JhdGlvKAo+Pj4gLQkJCQkJ CWZyYW1lLT52aWRlb19jb2RlKTsKPj4+ICsJcGljdHVyZV9hc3BlY3QgPSBtb2RlLT5waWN0dXJl X2FzcGVjdF9yYXRpbzsKPj4+ICsJaWYgKHBpY3R1cmVfYXNwZWN0ID09IEhETUlfUElDVFVSRV9B U1BFQ1RfTk9ORSkKPj4+ICsJCXBpY3R1cmVfYXNwZWN0ID0gZHJtX2dldF9jZWFfYXNwZWN0X3Jh dGlvKGZyYW1lLT52aWRlb19jb2RlKTsKPj4gVGhpcyBpcyBzbGlnaHRseSBnb2luZyBpbiB0aGUg bG9vcC4KPj4gLSBEdXJpbmcgdGhlIG1vZGVzZXQgdGhlIGRyaXZlciBjYW50IHNwZWNpZnkgdGhl IGFzcGVjdCByYXRpbwo+PiBpbmZvcm1hdGlvbiwgYXMgRFJNIGxheWVyIGxhY2tzIHRoaXMgc3Vw cG9ydC4KPj4gLSBTbyB3ZSBmaWxsIHRoZSBWSUMgZmllbGQsIGJ5IGNvbXBhcmluZyB0aGUgbW9k ZSB3aXRoIHRoZQo+PiBEUk1fQ0VBX01PREVTW10gbGlzdC4gVGhpcyB3aWxsIHBpY2sgdGhlIGZp cnN0IG1vZGUKPj4gICAgIGF2YWlsYWJsZSBpbiB0aGUgbGlzdCAocmVnYXJkbGVzcyBvZiBpdHMg YXNwZWN0IHJhdGlvKSwgYW5kIGZpbGwgdGhlCj4+IFZJQywgYXMgd2UgZG9uJ3QgY29uc2lkZXIg YXNwZWN0IHJhdGlvIHdoaWxlIGNvbXBhcmluZyB0aW1pbmdzLgo+PiAtIEFnYWluLCBub3cgd2hp bGUgc2VuZGluZyB0aGUgYXNwZWN0IHJhdGlvLCB3ZSBhcmUgcGlja2luZyB1cCB0aGUgVklDLAo+ PiB3aGljaCBtYXkgbm90IGJlIGNvcnJlY3QuCj4+Cj4+IFNvIGlmIHdlIGhhdmUgNzIweDQ4MCg0 OjMpIGFuZCA3MjB4NDgwKDE2OjkpIGluIHRoZSBsaXN0LCBhcyA0OjMgaXMKPj4gZmlyc3QgaW4g bGlzdCwgd2Ugd2lsbCBhbHdheXMgcGljayA0OjMgYXNwZWN0IHJhdGlvLgo+IFllcy4gVGhlIHVz ZXIgZGlkbid0IGNhcmUgYWJvdXQgdGhlIGFzcGVjdCByYXRpbyAob3IgcmF0aGVyIGNvdWxkbid0 Cj4gc3BlY2lmeSBvbmUpIHNvIHdlIGp1c3QgcGljayBvbmUuIFdoaWNoIGlzIGV4YWN0bHkgd2hh dCB3ZSd2ZSBiZWVuCj4gZG9pbmcgZXZlciBzaW5jZSB3ZSBzdGFydGVkIHNlbmRpbmcgdGhlIFZJ QyBpbiB0aGUgaW5mb2ZyYW1lLgpDb3JyZWN0LCBhbmQgd2UgYXJlIGhvcGluZyB0aGF0IHRoaXMg c2hvdWxkIGJlIGJldHRlciAoaWYgbm90IGZpeGVkKSAKd2l0aCB0aGUgYXNwZWN0IHJhdGlvIHN1 cHBvcnQKcGF0Y2hlcyArIERSTSBjYXAuIElmIHRoZSB1c2Vyc3BhY2UgZG9lc24ndCBzZXQgdGhl IGNhcCwgdGhlbiBhbnl3YXlzIAp0aGVyZSBpcyBubyBhc3BlY3QgcmF0aW8KZmllbGQgYXZhaWxh YmxlLCBhbmQgVklDIHdvdWxkIGJlIGFsd2F5cyAwLCBhcyB0aGlzIGJlY29tZXMgYSBOb24gQ0VB IG1vZGUuCgpPciBkbyB5b3UgdGhpbmsgaXQgd291bGQgYmUgYSBiZXR0ZXIgaWRlYSB0byBzZW5k IHNvbWUgVklDIGluc3RlYWQgb2YgTm8gClZJQywgd2hlbiB1c2Vyc3BhY2UgZG9lc24ndApzZXQg dGhlIERSTSBjYXAgZm9yIGFzcGVjdCByYXRpbyA/CgotIFNoYXNoYW5rCj4+IC0gU2hhc2hhbmsK Pj4+ICAgIAo+Pj4gKwkvKgo+Pj4gKwkgKiBUaGUgaW5mb2ZyYW1lIGNhbid0IGNvbnZleSBhbnl0 aGluZyBidXQgbm9uZSwgNDozCj4+PiArCSAqIGFuZCAxNjo5LCBzbyBpZiB0aGUgdXNlciBoYXMg YXNrZWQgZm9yIGFueXRoaW5nIGVsc2UKPj4+ICsJICogd2UgY2FuIG9ubHkgc2F0aXNmeSBpdCBi eSBzcGVjaWZ5aW5nIHRoZSByaWdodCBWSUMuCj4+PiArCSAqLwo+Pj4gKwlpZiAocGljdHVyZV9h c3BlY3QgPiBIRE1JX1BJQ1RVUkVfQVNQRUNUXzE2XzkpIHsKPj4+ICsJCWlmIChwaWN0dXJlX2Fz cGVjdCAhPQo+Pj4gKwkJICAgIGRybV9nZXRfY2VhX2FzcGVjdF9yYXRpbyhmcmFtZS0+dmlkZW9f Y29kZSkpCj4+PiArCQkJcmV0dXJuIC1FSU5WQUw7Cj4+PiArCQlwaWN0dXJlX2FzcGVjdCA9IEhE TUlfUElDVFVSRV9BU1BFQ1RfTk9ORTsKPj4+ICsJfQo+Pj4gKwo+Pj4gKwlmcmFtZS0+cGljdHVy ZV9hc3BlY3QgPSBwaWN0dXJlX2FzcGVjdDsKPj4+ICAgIAlmcmFtZS0+YWN0aXZlX2FzcGVjdCA9 IEhETUlfQUNUSVZFX0FTUEVDVF9QSUNUVVJFOwo+Pj4gICAgCWZyYW1lLT5zY2FuX21vZGUgPSBI RE1JX1NDQU5fTU9ERV9VTkRFUlNDQU47Cj4+PiAgICAKCl9fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4 QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWls bWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=