From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ramalingam C Subject: Re: [RFC PATCH 2/2] drm/i915/bxt: Adjusting the error in horizontal timings retrieval Date: Thu, 21 Apr 2016 10:11:32 +0530 Message-ID: <571859FC.90801@intel.com> References: <1461053894-5058-1-git-send-email-ramalingam.c@intel.com> <1461053894-5058-2-git-send-email-ramalingam.c@intel.com> <20160420132647.GH2510@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTP id 04E166EBB6 for ; Thu, 21 Apr 2016 04:50:44 +0000 (UTC) In-Reply-To: <20160420132647.GH2510@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: jani.nikula@intel.com, intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org Ck9uIFdlZG5lc2RheSAyMCBBcHJpbCAyMDE2IDA2OjU2IFBNLCBEYW5pZWwgVmV0dGVyIHdyb3Rl Ogo+IE9uIFR1ZSwgQXByIDE5LCAyMDE2IGF0IDAxOjQ4OjE0UE0gKzA1MzAsIFJhbWFsaW5nYW0g QyB3cm90ZToKPj4gSW4gQlhUIERTSSB0aGVyZSBpcyBubyByZWdzIHByb2dyYW1tZWQgd2l0aCBm ZXcgaG9yaXpvbnRhbCB0aW1pbmdzCj4+IGluIFBpeGVscyBidXQgdHhieXRlY2xraHMuLiBTbyBy ZXRyaWV2YWwgcHJvY2VzcyBhZGRzIHNvbWUKPj4gUk9VTkRfVVAgRVJST1JTIGluIHRoZSBwcm9j ZXNzIG9mIFBJWEVMUzw9PT50eGJ5dGVjbGtocy4KPj4KPj4gQWN0dWFsbHkgaGVyZSBmb3IgdGhl IGdpdmVuIGFkanVzdGVkX21vZGUsIHdlIGFyZSBjYWxjdWxhdGluZyB0aGUKPj4gdmFsdWUgcHJv Z3JhbW1lZCB0byB0aGUgcG9ydCBhbmQgdGhlbiBiYWNrIHRvIHRoZSBob3Jpem9udGFsIHRpbWlu Zwo+PiBwYXJhbSBpbiBwaXhlbHMuIFRoaXMgaXMgdGhlIGV4cGVjdGVkIHZhbHVlIGF0IHRoZSBl bmQgb2YgZ2V0X2NvbmZpZywKPj4gaW5jbHVkaW5nIHJvdW5kdXAgZXJyb3JzLiBBbmQgaWYgdGhh dCBpcyBzYW1lIGFzIHJldHJpZXZlZCB2YWx1ZQo+PiBmcm9tIHBvcnQsIHRoZW4gcmV0cmlldmVk IChIVyBzdGF0ZSkgYWRqdXN0ZWRfbW9kZSdzIGhvcml6b250YWwKPj4gdGltaW5ncyBhcmUgY29y cmVjdGVkIHRvIG1hdGNoIHdpdGggU1cgc3RhdGUgdG8gbnVsbGlmeSB0aGUgZXJyb3JzLgo+Pgo+ PiBTaWduZWQtb2ZmLWJ5OiBSYW1hbGluZ2FtIEMgPHJhbWFsaW5nYW0uY0BpbnRlbC5jb20+Cj4g SSBkaWRuJ3QgY2hlY2sgdGhlIGRldGFpbHMsIGJ1dCBJIHRoaW5rIHRoaXMgaXMgdGhlIGZpeCB3 ZSB3YW50LiBTdGlsbAo+IG5lZWRzIGluLWRlcHRoIHJldmlldywgYW5kIHNvbWUgdGVzdGluZyBv biBhIGZldyBkaWZmZXJlbnQgZHNpIGJ4dAo+IG1hY2hpbmVzIGp1c3QgdG8gbWFrZSBzdXJlIEkg dGhpbmssIGJ1dCBvbiBib3RoIHBhdGNoZXM6Cj4KPiBBY2tlZC1ieTogRGFuaWVsIFZldHRlciA8 ZGFuaWVsLnZldHRlckBmZndsbC5jaD4KRGFuaWVsIGFuZCBKYW5pLAoKSSBoYXZlIHRlc3RlZCB0 aGlzIG9uIEJYVC1UIEIxIGJvYXJkIHdpdGggVGlhbm1hIHBhbmVsLiBUaGlzIGlzIHdvcmtpbmcg CmZpbmUgdy5yLnQgUElQRV9DT05GSUcgQ09NUEFSRS4KTm90IHN1cmUgd2hldGhlciBpIGNhbiBn aXZlIHRlc3RlZC1ieSBmb3IgbXkgb3duIHBhdGNoLiA6KQoKLS1SYW0KPgo+IENoZWVycywgRGFu aWVsCj4+IC0tLQo+PiAgIGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2RzaS5jIHwgICA5NyAr KysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrLS0tLQo+PiAgIDEgZmlsZSBjaGFuZ2Vk LCA4OCBpbnNlcnRpb25zKCspLCA5IGRlbGV0aW9ucygtKQo+Pgo+PiBkaWZmIC0tZ2l0IGEvZHJp dmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHNpLmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRl bF9kc2kuYwo+PiBpbmRleCBlMDI1OWU2Li44YTFiODcyIDEwMDY0NAo+PiAtLS0gYS9kcml2ZXJz L2dwdS9kcm0vaTkxNS9pbnRlbF9kc2kuYwo+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9p bnRlbF9kc2kuYwo+PiBAQCAtNDYsNiArNDYsMTQgQEAgc3RhdGljIGNvbnN0IHN0cnVjdCB7Cj4+ ICAgCX0sCj4+ICAgfTsKPj4gICAKPj4gKy8qIHJldHVybiBwaXhlbHMgaW4gdGVybXMgb2YgdHhi eXRlY2xraHMgKi8KPj4gK3N0YXRpYyB1MTYgdHhieXRlY2xraHModTE2IHBpeGVscywgaW50IGJw cCwgaW50IGxhbmVfY291bnQsCj4+ICsJCSAgICAgICB1MTYgYnVyc3RfbW9kZV9yYXRpbykKPj4g K3sKPj4gKwlyZXR1cm4gRElWX1JPVU5EX1VQKERJVl9ST1VORF9VUChwaXhlbHMgKiBicHAgKiBi dXJzdF9tb2RlX3JhdGlvLAo+PiArCQkJCQkgOCAqIDEwMCksIGxhbmVfY291bnQpOwo+PiArfQo+ PiArCj4+ICAgLyogcmV0dXJuIHBpeGVscyBlcXV2YWxlbnQgdG8gdHhieXRlY2xraHMgKi8KPj4g ICBzdGF0aWMgdTE2IHBpeGVsc19mcm9tX3R4Ynl0ZWNsa2hzKHUxNiBjbGtfaHMsIGludCBicHAs IGludCBsYW5lX2NvdW50LAo+PiAgIAkJCXUxNiBidXJzdF9tb2RlX3JhdGlvKQo+PiBAQCAtNzc5 LDExICs3ODcsMTkgQEAgc3RhdGljIHZvaWQgYnh0X2RzaV9nZXRfcGlwZV9jb25maWcoc3RydWN0 IGludGVsX2VuY29kZXIgKmVuY29kZXIsCj4+ICAgCXN0cnVjdCBkcm1faTkxNV9wcml2YXRlICpk ZXZfcHJpdiA9IGRldi0+ZGV2X3ByaXZhdGU7Cj4+ICAgCXN0cnVjdCBkcm1fZGlzcGxheV9tb2Rl ICphZGp1c3RlZF9tb2RlID0KPj4gICAJCQkJCSZwaXBlX2NvbmZpZy0+YmFzZS5hZGp1c3RlZF9t b2RlOwo+PiArCXN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICphZGp1c3RlZF9tb2RlX3N3Owo+PiAr CXN0cnVjdCBpbnRlbF9jcnRjICppbnRlbF9jcnRjOwo+PiAgIAlzdHJ1Y3QgaW50ZWxfZHNpICpp bnRlbF9kc2kgPSBlbmNfdG9faW50ZWxfZHNpKCZlbmNvZGVyLT5iYXNlKTsKPj4gICAJdW5zaWdu ZWQgaW50IGxhbmVfY291bnQgPSBpbnRlbF9kc2ktPmxhbmVfY291bnQ7Cj4+ICAgCXVuc2lnbmVk IGludCBicHAsIGZtdDsKPj4gICAJZW51bSBwb3J0IHBvcnQ7Cj4+ICAgCXUxNiBoYWN0aXZlLCBo ZnAsIGhzeW5jLCBoYnAsIHZmcCwgdnN5bmMsIHZicDsKPj4gKwl1MTYgaGZwX3N3LCBoc3luY19z dywgaGJwX3N3Owo+PiArCXUxNiBjcnRjX2h0b3RhbF9zdywgY3J0Y19oc3luY19zdGFydF9zdywg Y3J0Y19oc3luY19lbmRfc3csCj4+ICsJCQkJY3J0Y19oYmxhbmtfc3RhcnRfc3csIGNydGNfaGJs YW5rX2VuZF9zdzsKPj4gKwo+PiArCWludGVsX2NydGMgPSB0b19pbnRlbF9jcnRjKGVuY29kZXIt PmJhc2UuY3J0Yyk7Cj4+ICsJYWRqdXN0ZWRfbW9kZV9zdyA9ICZpbnRlbF9jcnRjLT5jb25maWct PmJhc2UuYWRqdXN0ZWRfbW9kZTsKPj4gICAKPj4gICAJLyoKPj4gICAJICogQXRsZWFzdCBvbmUg cG9ydCBpcyBhY3RpdmUgYXMgZW5jb2Rlci0+Z2V0X2NvbmZpZyBjYWxsZWQgb25seSBpZgo+PiBA QCAtODQ3LDggKzg2Myw3OSBAQCBzdGF0aWMgdm9pZCBieHRfZHNpX2dldF9waXBlX2NvbmZpZyhz dHJ1Y3QgaW50ZWxfZW5jb2RlciAqZW5jb2RlciwKPj4gICAJYWRqdXN0ZWRfbW9kZS0+Y3J0Y192 c3luY19lbmQgPSB2c3luYyArIGFkanVzdGVkX21vZGUtPmNydGNfdnN5bmNfc3RhcnQ7Cj4+ICAg CWFkanVzdGVkX21vZGUtPmNydGNfdmJsYW5rX3N0YXJ0ID0gYWRqdXN0ZWRfbW9kZS0+Y3J0Y192 ZGlzcGxheTsKPj4gICAJYWRqdXN0ZWRfbW9kZS0+Y3J0Y192YmxhbmtfZW5kID0gYWRqdXN0ZWRf bW9kZS0+Y3J0Y192dG90YWw7Cj4+IC19Cj4+ICAgCj4+ICsJLyoKPj4gKwkgKiBJbiBCWFQgRFNJ IHRoZXJlIGlzIG5vIHJlZ3MgcHJvZ3JhbW1lZCB3aXRoIGZldyBob3Jpem9udGFsIHRpbWluZ3MK Pj4gKwkgKiBpbiBQaXhlbHMgYnV0IHR4Ynl0ZWNsa2hzLi4gU28gcmV0cmlldmFsIHByb2Nlc3Mg YWRkcyBzb21lCj4+ICsJICogUk9VTkRfVVAgRVJST1JTIGluIHRoZSBwcm9jZXNzIG9mIFBJWEVM Uzw9PT50eGJ5dGVjbGtocy4KPj4gKwkgKiBBY3R1YWxseSBoZXJlIGZvciB0aGUgZ2l2ZW4gYWRq dXN0ZWRfbW9kZSwgd2UgYXJlIGNhbGN1bGF0aW5nIHRoZQo+PiArCSAqIHZhbHVlIHByb2dyYW1t ZWQgdG8gdGhlIHBvcnQgYW5kIHRoZW4gYmFjayB0byB0aGUgaG9yaXpvbnRhbCB0aW1pbmcKPj4g KwkgKiBwYXJhbSBpbiBwaXhlbHMuIFRoaXMgaXMgdGhlIGV4cGVjdGVkIHZhbHVlLCBpbmNsdWRp bmcgcm91bmR1cCBlcnJvcnMKPj4gKwkgKiBBbmQgaWYgdGhhdCBpcyBzYW1lIGFzIHJldHJpZXZl ZCB2YWx1ZSBmcm9tIHBvcnQsIHRoZW4KPj4gKwkgKiAoSFcgc3RhdGUpIGFkanVzdGVkX21vZGUn cyBob3Jpem9udGFsIHRpbWluZ3MgYXJlIGNvcnJlY3RlZCB0bwo+PiArCSAqIG1hdGNoIHdpdGgg U1cgc3RhdGUgdG8gbnVsbGlmeSB0aGUgZXJyb3JzLgo+PiArCSAqLwo+PiArCS8qIENhbGN1bGF0 aW5nIHRoZSB2YWx1ZSBwcm9ncmFtbWVkIHRvIHRoZSBQb3J0IHJlZ2lzdGVyICovCj4+ICsJaGZw X3N3ID0gYWRqdXN0ZWRfbW9kZV9zdy0+Y3J0Y19oc3luY19zdGFydCAtCj4+ICsJCQkJCWFkanVz dGVkX21vZGVfc3ctPmNydGNfaGRpc3BsYXk7Cj4+ICsJaHN5bmNfc3cgPSBhZGp1c3RlZF9tb2Rl X3N3LT5jcnRjX2hzeW5jX2VuZCAtCj4+ICsJCQkJCWFkanVzdGVkX21vZGVfc3ctPmNydGNfaHN5 bmNfc3RhcnQ7Cj4+ICsJaGJwX3N3ID0gYWRqdXN0ZWRfbW9kZV9zdy0+Y3J0Y19odG90YWwgLQo+ PiArCQkJCQlhZGp1c3RlZF9tb2RlX3N3LT5jcnRjX2hzeW5jX2VuZDsKPj4gKwo+PiArCWlmIChp bnRlbF9kc2ktPmR1YWxfbGluaykgewo+PiArCQloZnBfc3cgLz0gMjsKPj4gKwkJaHN5bmNfc3cg Lz0gMjsKPj4gKwkJaGJwX3N3IC89IDI7Cj4+ICsJfQo+PiArCj4+ICsJaGZwX3N3ID0gdHhieXRl Y2xraHMoaGZwX3N3LCBicHAsIGxhbmVfY291bnQsCj4+ICsJCQkJCQlpbnRlbF9kc2ktPmJ1cnN0 X21vZGVfcmF0aW8pOwo+PiArCWhzeW5jX3N3ID0gdHhieXRlY2xraHMoaHN5bmNfc3csIGJwcCwg bGFuZV9jb3VudCwKPj4gKwkJCSAgICBpbnRlbF9kc2ktPmJ1cnN0X21vZGVfcmF0aW8pOwo+PiAr CWhicF9zdyA9IHR4Ynl0ZWNsa2hzKGhicF9zdywgYnBwLCBsYW5lX2NvdW50LAo+PiArCQkJCQkJ aW50ZWxfZHNpLT5idXJzdF9tb2RlX3JhdGlvKTsKPj4gKwo+PiArCS8qIFJldmVyc2UgY2FsY3Vs YXRpbmcgdGhlIGFkanVzdGVkIG1vZGUgcGFyYW1ldGVycyBmcm9tIHBvcnQgcmVnIHZhbHMqLwo+ PiArCWhmcF9zdyA9IHBpeGVsc19mcm9tX3R4Ynl0ZWNsa2hzKGhmcF9zdywgYnBwLCBsYW5lX2Nv dW50LAo+PiArCQkJCQkJaW50ZWxfZHNpLT5idXJzdF9tb2RlX3JhdGlvKTsKPj4gKwloc3luY19z dyA9IHBpeGVsc19mcm9tX3R4Ynl0ZWNsa2hzKGhzeW5jX3N3LCBicHAsIGxhbmVfY291bnQsCj4+ ICsJCQkJCQlpbnRlbF9kc2ktPmJ1cnN0X21vZGVfcmF0aW8pOwo+PiArCWhicF9zdyA9IHBpeGVs c19mcm9tX3R4Ynl0ZWNsa2hzKGhicF9zdywgYnBwLCBsYW5lX2NvdW50LAo+PiArCQkJCQkJaW50 ZWxfZHNpLT5idXJzdF9tb2RlX3JhdGlvKTsKPj4gKwo+PiArCWlmIChpbnRlbF9kc2ktPmR1YWxf bGluaykgewo+PiArCQloZnBfc3cgKj0gMjsKPj4gKwkJaHN5bmNfc3cgKj0gMjsKPj4gKwkJaGJw X3N3ICo9IDI7Cj4+ICsJfQo+PiArCj4+ICsJY3J0Y19odG90YWxfc3cgPSBhZGp1c3RlZF9tb2Rl X3N3LT5jcnRjX2hkaXNwbGF5ICsgaGZwX3N3ICsKPj4gKwkJCQkJCQloc3luY19zdyArIGhicF9z dzsKPj4gKwljcnRjX2hzeW5jX3N0YXJ0X3N3ID0gaGZwX3N3ICsgYWRqdXN0ZWRfbW9kZV9zdy0+ Y3J0Y19oZGlzcGxheTsKPj4gKwljcnRjX2hzeW5jX2VuZF9zdyA9IGhzeW5jX3N3ICsgY3J0Y19o c3luY19zdGFydF9zdzsKPj4gKwljcnRjX2hibGFua19zdGFydF9zdyA9IGFkanVzdGVkX21vZGVf c3ctPmNydGNfaGRpc3BsYXk7Cj4+ICsJY3J0Y19oYmxhbmtfZW5kX3N3ID0gY3J0Y19odG90YWxf c3c7Cj4+ICsKPj4gKwlpZiAoYWRqdXN0ZWRfbW9kZS0+Y3J0Y19odG90YWwgPT0gY3J0Y19odG90 YWxfc3cpCj4+ICsJCWFkanVzdGVkX21vZGUtPmNydGNfaHRvdGFsID0gYWRqdXN0ZWRfbW9kZV9z dy0+Y3J0Y19odG90YWw7Cj4+ICsKPj4gKwlpZiAoYWRqdXN0ZWRfbW9kZS0+Y3J0Y19oc3luY19z dGFydCA9PSBjcnRjX2hzeW5jX3N0YXJ0X3N3KQo+PiArCQlhZGp1c3RlZF9tb2RlLT5jcnRjX2hz eW5jX3N0YXJ0ID0KPj4gKwkJCQkJYWRqdXN0ZWRfbW9kZV9zdy0+Y3J0Y19oc3luY19zdGFydDsK Pj4gKwo+PiArCWlmIChhZGp1c3RlZF9tb2RlLT5jcnRjX2hzeW5jX2VuZCA9PSBjcnRjX2hzeW5j X2VuZF9zdykKPj4gKwkJYWRqdXN0ZWRfbW9kZS0+Y3J0Y19oc3luY19lbmQgPQo+PiArCQkJCQlh ZGp1c3RlZF9tb2RlX3N3LT5jcnRjX2hzeW5jX2VuZDsKPj4gKwo+PiArCWlmIChhZGp1c3RlZF9t b2RlLT5jcnRjX2hibGFua19zdGFydCA9PSBjcnRjX2hibGFua19zdGFydF9zdykKPj4gKwkJYWRq dXN0ZWRfbW9kZS0+Y3J0Y19oYmxhbmtfc3RhcnQgPQo+PiArCQkJCQlhZGp1c3RlZF9tb2RlX3N3 LT5jcnRjX2hibGFua19zdGFydDsKPj4gKwo+PiArCWlmIChhZGp1c3RlZF9tb2RlLT5jcnRjX2hi bGFua19lbmQgPT0gY3J0Y19oYmxhbmtfZW5kX3N3KQo+PiArCQlhZGp1c3RlZF9tb2RlLT5jcnRj X2hibGFua19lbmQgPQo+PiArCQkJCQlhZGp1c3RlZF9tb2RlX3N3LT5jcnRjX2hibGFua19lbmQ7 Cj4+ICt9Cj4+ICAgCj4+ICAgc3RhdGljIHZvaWQgaW50ZWxfZHNpX2dldF9jb25maWcoc3RydWN0 IGludGVsX2VuY29kZXIgKmVuY29kZXIsCj4+ICAgCQkJCSBzdHJ1Y3QgaW50ZWxfY3J0Y19zdGF0 ZSAqcGlwZV9jb25maWcpCj4+IEBAIC05MTcsMTQgKzEwMDQsNiBAQCBzdGF0aWMgdTE2IHR4Y2xr ZXNjKHUzMiBkaXZpZGVyLCB1bnNpZ25lZCBpbnQgdXMpCj4+ICAgCX0KPj4gICB9Cj4+ICAgCj4+ IC0vKiByZXR1cm4gcGl4ZWxzIGluIHRlcm1zIG9mIHR4Ynl0ZWNsa2hzICovCj4+IC1zdGF0aWMg dTE2IHR4Ynl0ZWNsa2hzKHUxNiBwaXhlbHMsIGludCBicHAsIGludCBsYW5lX2NvdW50LAo+PiAt CQkgICAgICAgdTE2IGJ1cnN0X21vZGVfcmF0aW8pCj4+IC17Cj4+IC0JcmV0dXJuIERJVl9ST1VO RF9VUChESVZfUk9VTkRfVVAocGl4ZWxzICogYnBwICogYnVyc3RfbW9kZV9yYXRpbywKPj4gLQkJ CQkJIDggKiAxMDApLCBsYW5lX2NvdW50KTsKPj4gLX0KPj4gLQo+PiAgIHN0YXRpYyB2b2lkIHNl dF9kc2lfdGltaW5ncyhzdHJ1Y3QgZHJtX2VuY29kZXIgKmVuY29kZXIsCj4+ICAgCQkJICAgIGNv bnN0IHN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICphZGp1c3RlZF9tb2RlKQo+PiAgIHsKPj4gLS0g Cj4+IDEuNy45LjUKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3Jn Cmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4 Cg==