From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH v4 2/4] drm: Add aspect ratio parsing in DRM layer Date: Mon, 17 Oct 2016 17:06:12 +0300 Message-ID: <20161017140612.GR4329@intel.com> References: <1476705880-15600-1-git-send-email-shashank.sharma@intel.com> <1476705880-15600-3-git-send-email-shashank.sharma@intel.com> <20161017123159.GP4329@intel.com> <4244f1c9-e6cb-3fbb-c119-cd667fb4570d@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: <4244f1c9-e6cb-3fbb-c119-cd667fb4570d@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: "Sharma, Shashank" Cc: Jose.Abreu@synopsys.com, Daniel Vetter , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, tomi.valkeinen@ti.com, daniel.vetter@intel.com List-Id: intel-gfx@lists.freedesktop.org T24gTW9uLCBPY3QgMTcsIDIwMTYgYXQgMDc6MTA6NDJQTSArMDUzMCwgU2hhcm1hLCBTaGFzaGFu ayB3cm90ZToKPiBSZWdhcmRzCj4gCj4gU2hhc2hhbmsKPiAKPiAKPiBPbiAxMC8xNy8yMDE2IDY6 MDEgUE0sIFZpbGxlIFN5cmrDpGzDpCB3cm90ZToKPiA+IE9uIE1vbiwgT2N0IDE3LCAyMDE2IGF0 IDA1OjM0OjM4UE0gKzA1MzAsIFNoYXNoYW5rIFNoYXJtYSB3cm90ZToKPiA+PiBDdXJyZW50IERS TSBsYXllciBmdW5jdGlvbnMgZG9uJ3QgcGFyc2UgYXNwZWN0IHJhdGlvIGluZm9ybWF0aW9uCj4g Pj4gd2hpbGUgY29udmVydGluZyBhIHVzZXIgbW9kZS0+a2VybmVsIG1vZGUgb3IgdmljZSB2ZXJz YS4gVGhpcwo+ID4+IGNhdXNlcyBtb2Rlc2V0IHRvIHBpY2sgbW9kZSB3aXRoIHdyb25nIGFzcGVj dCByYXRpbywgZXZlbnR1YWxseQo+ID4+IGNhdXNpbmcgZmFpbHVyZXMgaW4gSERNSSBjb21wbGlh bmNlIHRlc3QgY2FzZXMsIGR1ZSB0byB3cm9uZyBWSUMuCj4gPj4KPiA+PiBUaGlzIHBhdGNoIGFk ZHMgYXNwZWN0IHJhdGlvIGluZm9ybWF0aW9uIGluIERSTSdzIG1vZGUgY29udmVyc2lvbgo+ID4+ IGFuZCBtb2RlIGNvbXBhcmlzaW9uIGZ1bmN0aW9ucywgdG8gbWFrZSBzdXJlIGtlcm5lbCBwaWNr cyBtb2RlCj4gPj4gd2l0aCByaWdodCBhc3BlY3QgcmF0aW8gKGFzIHBlciB0aGUgVklDKS4KPiA+ Pgo+ID4+IFNpZ25lZC1vZmYtYnk6IFNoYXNoYW5rIFNoYXJtYSA8c2hhc2hhbmsuc2hhcm1hQGlu dGVsLmNvbT4KPiA+PiBTaWduZWQtb2ZmLWJ5OiBMaW4sIEppYSA8bGluLmEuamlhQGludGVsLmNv bT4KPiA+PiBTaWduZWQtb2ZmLWJ5OiBBa2FzaGRlZXAgU2hhcm1hIDxha2FzaGRlZXAuc2hhcm1h QGludGVsLmNvbT4KPiA+PiBSZXZpZXdlZC1ieTogSmltIEJyaWRlIDxqaW0uYnJpZGVAbGludXgu aW50ZWwuY29tPgo+ID4+IFJldmlld2VkLWJ5OiBKb3NlIEFicmV1IDxKb3NlLkFicmV1QHN5bm9w c3lzLmNvbT4KPiA+PiBDYzogRGFuaWVsIFZldHRlciA8ZGFuaWVsLnZldHRlckBmZndsbC5jaD4K PiA+PiBDYzogRW1pbCBWZWxpa292IDxlbWlsLmwudmVsaWtvdkBnbWFpbC5jb20+Cj4gPj4KPiA+ PiBWMjogQWRkcmVzc2VkIHJldmlldyBjb21tZW50cyBmcm9tIFNlYW46Cj4gPj4gLSBGaXggc3Bl bGxpbmdzL3R5cG8KPiA+PiAtIE5vIG5lZWQgdG8gaGFuZGxlIGFzcGVjdCByYXRpbyBub25lCj4g Pj4gLSBBZGQgYSBicmVhaywgZm9yIGRlZmF1bHQgY2FzZSB0b28KPiA+PiBWMzogUmViYXNlCj4g Pj4gVjQ6IEFkZGVkIHItYiBmcm9tIEpvc2UKPiA+PiAtLS0KPiA+PiAgIGRyaXZlcnMvZ3B1L2Ry bS9kcm1fbW9kZXMuYyB8IDMxICsrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysKPiA+PiAg IDEgZmlsZSBjaGFuZ2VkLCAzMSBpbnNlcnRpb25zKCspCj4gPj4KPiA+PiBkaWZmIC0tZ2l0IGEv ZHJpdmVycy9ncHUvZHJtL2RybV9tb2Rlcy5jIGIvZHJpdmVycy9ncHUvZHJtL2RybV9tb2Rlcy5j Cj4gPj4gaW5kZXggNTNmMDdhYy4uZmRlOTI3YSAxMDA2NDQKPiA+PiAtLS0gYS9kcml2ZXJzL2dw dS9kcm0vZHJtX21vZGVzLmMKPiA+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX21vZGVzLmMK PiA+PiBAQCAtOTk3LDYgKzk5Nyw3IEBAIGJvb2wgZHJtX21vZGVfZXF1YWxfbm9fY2xvY2tzX25v X3N0ZXJlbyhjb25zdCBzdHJ1Y3QgZHJtX2Rpc3BsYXlfbW9kZSAqbW9kZTEsCj4gPj4gICAJICAg IG1vZGUxLT52c3luY19lbmQgPT0gbW9kZTItPnZzeW5jX2VuZCAmJgo+ID4+ICAgCSAgICBtb2Rl MS0+dnRvdGFsID09IG1vZGUyLT52dG90YWwgJiYKPiA+PiAgIAkgICAgbW9kZTEtPnZzY2FuID09 IG1vZGUyLT52c2NhbiAmJgo+ID4+ICsJICAgIG1vZGUxLT5waWN0dXJlX2FzcGVjdF9yYXRpbyA9 PSBtb2RlMi0+cGljdHVyZV9hc3BlY3RfcmF0aW8gJiYKPiA+PiAgIAkgICAgKG1vZGUxLT5mbGFn cyAmIH5EUk1fTU9ERV9GTEFHXzNEX01BU0spID09Cj4gPj4gICAJICAgICAobW9kZTItPmZsYWdz ICYgfkRSTV9NT0RFX0ZMQUdfM0RfTUFTSykpCj4gPj4gICAJCXJldHVybiB0cnVlOwo+ID4+IEBA IC0xNDk5LDYgKzE1MDAsMjEgQEAgdm9pZCBkcm1fbW9kZV9jb252ZXJ0X3RvX3Vtb2RlKHN0cnVj dCBkcm1fbW9kZV9tb2RlaW5mbyAqb3V0LAo+ID4+ICAgCW91dC0+dnJlZnJlc2ggPSBpbi0+dnJl ZnJlc2g7Cj4gPj4gICAJb3V0LT5mbGFncyA9IGluLT5mbGFnczsKPiA+PiAgIAlvdXQtPnR5cGUg PSBpbi0+dHlwZTsKPiA+PiArCW91dC0+ZmxhZ3MgJj0gfkRSTV9NT0RFX0ZMQUdfUElDX0FSX01B U0s7Cj4gPj4gKwo+ID4+ICsJc3dpdGNoIChpbi0+cGljdHVyZV9hc3BlY3RfcmF0aW8pIHsKPiA+ PiArCWNhc2UgSERNSV9QSUNUVVJFX0FTUEVDVF80XzM6Cj4gPj4gKwkJb3V0LT5mbGFncyB8PSBE Uk1fTU9ERV9GTEFHX1BJQ19BUl80XzM7Cj4gPj4gKwkJYnJlYWs7Cj4gPj4gKwljYXNlIEhETUlf UElDVFVSRV9BU1BFQ1RfMTZfOToKPiA+PiArCQlvdXQtPmZsYWdzIHw9IERSTV9NT0RFX0ZMQUdf UElDX0FSXzE2Xzk7Cj4gPj4gKwkJYnJlYWs7Cj4gPj4gKwljYXNlIEhETUlfUElDVFVSRV9BU1BF Q1RfUkVTRVJWRUQ6Cj4gPj4gKwlkZWZhdWx0Ogo+ID4+ICsJCW91dC0+ZmxhZ3MgfD0gRFJNX01P REVfRkxBR19QSUNfQVJfTk9ORTsKPiA+PiArCQlicmVhazsKPiA+PiArCX0KPiA+PiArCj4gPj4g ICAJc3RybmNweShvdXQtPm5hbWUsIGluLT5uYW1lLCBEUk1fRElTUExBWV9NT0RFX0xFTik7Cj4g Pj4gICAJb3V0LT5uYW1lW0RSTV9ESVNQTEFZX01PREVfTEVOLTFdID0gMDsKPiA+PiAgIH0KPiA+ PiBAQCAtMTU0NCw2ICsxNTYwLDIxIEBAIGludCBkcm1fbW9kZV9jb252ZXJ0X3Vtb2RlKHN0cnVj dCBkcm1fZGlzcGxheV9tb2RlICpvdXQsCj4gPj4gICAJc3RybmNweShvdXQtPm5hbWUsIGluLT5u YW1lLCBEUk1fRElTUExBWV9NT0RFX0xFTik7Cj4gPj4gICAJb3V0LT5uYW1lW0RSTV9ESVNQTEFZ X01PREVfTEVOLTFdID0gMDsKPiA+PiAgIAo+ID4+ICsJLyogQ2xlYXJpbmcgcGljdHVyZSBhc3Bl Y3QgcmF0aW8gYml0cyBmcm9tIG91dCBmbGFncyAqLwo+ID4+ICsJb3V0LT5mbGFncyAmPSB+RFJN X01PREVfRkxBR19QSUNfQVJfTUFTSzsKPiA+PiArCj4gPj4gKwlzd2l0Y2ggKGluLT5mbGFncyAm IERSTV9NT0RFX0ZMQUdfUElDX0FSX01BU0spIHsKPiA+PiArCWNhc2UgRFJNX01PREVfRkxBR19Q SUNfQVJfNF8zOgo+ID4+ICsJCW91dC0+cGljdHVyZV9hc3BlY3RfcmF0aW8gfD0gSERNSV9QSUNU VVJFX0FTUEVDVF80XzM7Cj4gPj4gKwkJYnJlYWs7Cj4gPj4gKwljYXNlIERSTV9NT0RFX0ZMQUdf UElDX0FSXzE2Xzk6Cj4gPj4gKwkJb3V0LT5waWN0dXJlX2FzcGVjdF9yYXRpbyB8PSBIRE1JX1BJ Q1RVUkVfQVNQRUNUXzE2Xzk7Cj4gPj4gKwkJYnJlYWs7Cj4gPj4gKwlkZWZhdWx0Ogo+ID4+ICsJ CW91dC0+cGljdHVyZV9hc3BlY3RfcmF0aW8gPSBIRE1JX1BJQ1RVUkVfQVNQRUNUX05PTkU7Cj4g Pj4gKwkJYnJlYWs7Cj4gPj4gKwl9Cj4gPiBIbW0uIFNvIG5vdyB3ZSBoYXZlIHRoZSBtb2RlIGFz cGVjdCByYXRpbyBpbmZyb21hdGlvbiBkdXBsaWNhdGVkCj4gPiBpbiB0d28gZGlmZmVyZW50IHBs YWNlcy4gTm90IHN1cmUgdGhhdCB3b24ndCBsZWFkIHRvIHNvbWUgY29uZnVzaW9uLgo+IEluIGRy bSBsYXllciwgdGhpcyBpcyB0aGUgb25seSBwbGFjZS4gQWN0dWFsbHkgdGlsbCBub3csIERSTSBs YXllciBkaW50IAo+IGhhdmUgdGhlIHN1cHBvcnQgZm9yIGFzcGVjdCByYXRpbyBhdCBhbGwuIFRo aXMgd2FzIGNhdXNpbmcKPiBIRE1JIGNvbXBsaWFuY2UgdGVzdHMgbGlrZSA3LTI3IGZhaWwsIHdo aWNoIGV4cGVjdCBhIHBhcnRpY3VsYXIgdW5pcXVlIAo+IFZJQyBpbiBBVkkgaW5mb2ZyYW1lIG9u IG1vZGVzZXQuCj4gCj4gSSBoYXZlIGdpdmVuIHNvbWUgZGV0YWlscyBhYm91dCB0aGlzIGluIGNv dmVyLWxldHRlci4KPiA+IEFsdGhvdWdoIHdlIGRvIHdhbnQgdGhlIHVzZXIgdG8gYmUgYWJsZSB0 byBvdmVycmlkZSB2aWEgdGhlIHByb3BlcnR5IEkKPiA+IHN1cHBvc2UsIHNvIHdlJ2QgaGF2ZSB0 byBjaGFuZ2UgdGhhdCAoKyB0aGUgaW5mb3JmcmFtZSBjb2RlKSB0bwo+ID4gbG9vayBhdCB0aGUg bW9kZSBmbGFncyBpbnN0ZWFkIGlmIHdlIHdvdWxkIGVsaW1pbmF0ZQo+ID4gJ3BpY3R1cmVfYXNw ZWN0X3JhdGlvJy4KPiBJIGhhZCBhIHNtYWxsIGRpc2N1c3Npb24gb24gdGhpcyB3aXRoIERhbmll bCwgYW5kIHdlIGRpc2N1c3NlZCB0aGF0IGl0IAo+IGRvZXNudCBtYWtlIHNlbnNlIHRvIG92ZXJy aWRlIGp1c3QgdGhlIGFzcGVjdCByYXRpbyBpZiB0aGUgbW9uaXRvciAKPiBkb2Vzbid0IGV2ZW4g c3VwcG9ydCB0aGF0IGFzcGVjdCByYXRpby4KPiBBbmQgY3VycmVudGx5IHRoZSB3YXMgaG93IHRo aXMgcHJvcGVydHkgaXMgaW1wbGVtZW50ZWQgaXMsIHdlIGp1c3QgCj4gb3ZlcnJpZGUgdGhlIGFz cGVjdCByYXRpbyB3aXRob3V0IGFueSB2YWxpZGl0eSBjaGVjay4KPiAKPiBOb3cgYXMgd2UgaGF2 ZSBhbGwgdGhlIHN1cHBvcnRlZCBhc3BlY3QgcmF0aW8gYWRkZWQgcHJvcGVybHkgaW4gdGhlIG1v ZGUgCj4gaW5mbyBpdHNlbGYsIHdlIG5lZWQgbm90IHRvIGhhdmUgdGhpcyBwcm9wZXJ0eSBhdCBh bGwsIFNvIERhbmllbCAKPiBzdWdnZXN0ZWQgbWUgdG8gcmVtb3ZlIHRoYXQgcHJvcGVydHkgdG9v Lgo+ID4KPiA+IEFuZCB0aGF0IGJyaW5ncyBtZSB0byB0aGUgb3RoZXIgcG9pbnQuIEF0IGxlYXN0 IGk5MTUgd2lsbCBzaW1wbHkKPiA+IG92ZXJyaWRlIHRoZSBtb2RlJ3MgYXNwZWN0IHJhdGlvbiB3 aXRoIHRoZSBwcm9wZXJ0eS4gU28gdGhpcyBsb29rcyBsaWtlCj4gPiBhIGJpZyBuby1vcCBmb3Ig bm93IG9uIGk5MTUuCj4gWWVzLCBUaGlzIGlzIGEgYnVnIGluIEk5MTUuIFdoZW4gSSBwdWJsaXNo ZWQgdGhlIGZpcnN0IHZlcnNpb24gb2YgdGhpcyAKPiBzZXJpZXMsIEkgaGFkIG9uZSBwYXRjaCwg d2hpY2ggd2FzIG92ZXJyaWRpbmcgdGhlIHZhbHVlIG9ubHkgd2hlbiB0aGUgCj4gcHJvcGVydHkg aXMgc2V0Lgo+IFRoaXMgc2hvdWxkIGJlIHRoZSByaWdodCBjYXNlLiBBbmQgdGhlbiBEYW5pZWwg c3VnZ2VzdGVkIHRvIHJlbW92ZSB0aGUgCj4gcHJvcGVydHkgYWxsIHRvZ2V0aGVyIChhbmQgaXQg bWFrZXMgc2Vuc2UgYXMgd2UgaGF2ZSBwcm9wZXIgYXNwZWN0IAo+IHJhdGlvcyBpbiBtb2RlIGlu Zm9ybWF0aW9uCj4gaXRzZWxmKSBTbyBJIGtlcHQgdGhhdCBwYXRjaCBzZXBhcmF0ZSwgdG8gYmUg bWVyZ2VkIHNlcGFyYXRlbHkuCgpZZWFoLCByZW1vdmluZyB0aGUgcHJvcGVydHkgZW50aXJlbHkg c2VlbXMgbGlrZSBhbiBPS2lzaCBzb2x1dGlvbiBzaW5jZQp1c2Vyc3BhY2UgY2FuIGp1c3Qgc3R1 ZmYgdGhhdCBpbmZvcm1hdGlvbiBpbnRvIHRoZSBtb2RlIGZsYWdzIGluc3RlYWQuClRoZSBvbmx5 IHNsaWdodCBjb25jZXJuIGlzIHRoYXQgc29tZW9uZSdzIHNldHVwIG1pZ2h0IGRlcGVuZCBvbiB0 aGUKcHJvcGVydHksIGFuZCBub3cgd2UncmUgcmVtb3ZpbmcgaXQuCgo+ID4gICBJdCdzIGxvb2tp bmcgbGlrZSB3ZSBtaWdodCBuZWVkIGEgbmV3Cj4gPiBwcm9wZXR0eSB2YWx1ZSB0byBkaWZmZXJl bnRpYXRlIGJldHdlZW4gImF1dG8iIGFuZCAibm9uZSIgZm9yIHJlYWwuCj4gTm93IHdlIGhhdmUg dGhlIGV4YWN0IGFzcGVjdCByYXRpbyBnaXZlbiBieSBtb25pdG9yIEVESUQsIHNvIElNSE8gaXQg Cj4gd291bGQgYmUgYmV0dGVyIGlmIHdlIGNhbiBqdXN0IGhhdmUgcmVhbCBhc3BlY3QuCgpXZWxs LCAicmVhbCIgaXNuJ3QgcXVpdGUgdGhlIHJpZ2h0IHRlcm0uIEl0J3Mgc3RpbGwgYSB1c2VyIHNw ZWNpZmllZAp0aGluZywgd2hpY2ggbWVhbnMgdGhlIHVzZXIgY2FuIGFzayBmb3Igc29tZWh0aW5n IHRoZSBkaXNwbGF5IGRpZG4ndApldmVuIGFkdmVydGlzZS4gV2hpY2ggaXMgZmluZSBhcyBzdWNo LCBob3dldmVyIHNpbmNlIHRoZSBBVkkgaW5mb2ZyYW1lCmxhY2tzIHRoZSBjYXBhYmlsaXR5IHRv IHRyYW5zbWl0IHRoZXNlIG5ldyBhc3BlY3QgcmF0aW9zIHdlIGhhdmUgYSBiaXQKb2YgcHJvYmxl bSBvbiBvdXIgaGFuZHMgaWYgdGhlIHVzZXIgYXNrcyBmb3Igc29tZXRoaW5nIHdlIGNhbid0IGV2 ZW4KdGVsbCB0aGUgZGlzcGxheSB0byBkby4gSSBndWVzcyB3ZSB3b3VsZCBqdXN0IG5lZWQgdG8g cmV0dXJuIGFuIGVycm9yCnRvIHRoZSB1c2VyIGluIHRoYXQgY2FzZS4KCi0tIApWaWxsZSBTeXJq w6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9w Lm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVs LWdmeAo=