From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [RFC PATCH 2/2] drm/i915/bxt: Adjusting the error in horizontal timings retrieval Date: Wed, 27 Apr 2016 20:11:18 +0300 Message-ID: <20160427171118.GQ4329@intel.com> References: <1461053894-5058-1-git-send-email-ramalingam.c@intel.com> <1461053894-5058-2-git-send-email-ramalingam.c@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id 86AE26E0A6 for ; Wed, 27 Apr 2016 17:12:16 +0000 (UTC) Content-Disposition: inline In-Reply-To: <1461053894-5058-2-git-send-email-ramalingam.c@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Ramalingam C Cc: jani.nikula@intel.com, intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gVHVlLCBBcHIgMTksIDIwMTYgYXQgMDE6NDg6MTRQTSArMDUzMCwgUmFtYWxpbmdhbSBDIHdy b3RlOgo+IEluIEJYVCBEU0kgdGhlcmUgaXMgbm8gcmVncyBwcm9ncmFtbWVkIHdpdGggZmV3IGhv cml6b250YWwgdGltaW5ncwo+IGluIFBpeGVscyBidXQgdHhieXRlY2xraHMuLiBTbyByZXRyaWV2 YWwgcHJvY2VzcyBhZGRzIHNvbWUKPiBST1VORF9VUCBFUlJPUlMgaW4gdGhlIHByb2Nlc3Mgb2Yg UElYRUxTPD09PnR4Ynl0ZWNsa2hzLgo+IAo+IEFjdHVhbGx5IGhlcmUgZm9yIHRoZSBnaXZlbiBh ZGp1c3RlZF9tb2RlLCB3ZSBhcmUgY2FsY3VsYXRpbmcgdGhlCj4gdmFsdWUgcHJvZ3JhbW1lZCB0 byB0aGUgcG9ydCBhbmQgdGhlbiBiYWNrIHRvIHRoZSBob3Jpem9udGFsIHRpbWluZwo+IHBhcmFt IGluIHBpeGVscy4gVGhpcyBpcyB0aGUgZXhwZWN0ZWQgdmFsdWUgYXQgdGhlIGVuZCBvZiBnZXRf Y29uZmlnLAo+IGluY2x1ZGluZyByb3VuZHVwIGVycm9ycy4gQW5kIGlmIHRoYXQgaXMgc2FtZSBh cyByZXRyaWV2ZWQgdmFsdWUKPiBmcm9tIHBvcnQsIHRoZW4gcmV0cmlldmVkIChIVyBzdGF0ZSkg YWRqdXN0ZWRfbW9kZSdzIGhvcml6b250YWwKPiB0aW1pbmdzIGFyZSBjb3JyZWN0ZWQgdG8gbWF0 Y2ggd2l0aCBTVyBzdGF0ZSB0byBudWxsaWZ5IHRoZSBlcnJvcnMuCj4gCj4gU2lnbmVkLW9mZi1i eTogUmFtYWxpbmdhbSBDIDxyYW1hbGluZ2FtLmNAaW50ZWwuY29tPgo+IC0tLQo+ICBkcml2ZXJz L2dwdS9kcm0vaTkxNS9pbnRlbF9kc2kuYyB8ICAgOTcgKysrKysrKysrKysrKysrKysrKysrKysr KysrKysrKysrKy0tLS0KPiAgMSBmaWxlIGNoYW5nZWQsIDg4IGluc2VydGlvbnMoKyksIDkgZGVs ZXRpb25zKC0pCj4gCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rz aS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHNpLmMKPiBpbmRleCBlMDI1OWU2Li44 YTFiODcyIDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2RzaS5jCj4g KysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHNpLmMKPiBAQCAtNDYsNiArNDYsMTQg QEAgc3RhdGljIGNvbnN0IHN0cnVjdCB7Cj4gIAl9LAo+ICB9Owo+ICAKPiArLyogcmV0dXJuIHBp eGVscyBpbiB0ZXJtcyBvZiB0eGJ5dGVjbGtocyAqLwo+ICtzdGF0aWMgdTE2IHR4Ynl0ZWNsa2hz KHUxNiBwaXhlbHMsIGludCBicHAsIGludCBsYW5lX2NvdW50LAo+ICsJCSAgICAgICB1MTYgYnVy c3RfbW9kZV9yYXRpbykKPiArewo+ICsJcmV0dXJuIERJVl9ST1VORF9VUChESVZfUk9VTkRfVVAo cGl4ZWxzICogYnBwICogYnVyc3RfbW9kZV9yYXRpbywKPiArCQkJCQkgOCAqIDEwMCksIGxhbmVf Y291bnQpOwo+ICt9Cj4gKwo+ICAvKiByZXR1cm4gcGl4ZWxzIGVxdXZhbGVudCB0byB0eGJ5dGVj bGtocyAqLwo+ICBzdGF0aWMgdTE2IHBpeGVsc19mcm9tX3R4Ynl0ZWNsa2hzKHUxNiBjbGtfaHMs IGludCBicHAsIGludCBsYW5lX2NvdW50LAo+ICAJCQl1MTYgYnVyc3RfbW9kZV9yYXRpbykKPiBA QCAtNzc5LDExICs3ODcsMTkgQEAgc3RhdGljIHZvaWQgYnh0X2RzaV9nZXRfcGlwZV9jb25maWco c3RydWN0IGludGVsX2VuY29kZXIgKmVuY29kZXIsCj4gIAlzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0 ZSAqZGV2X3ByaXYgPSBkZXYtPmRldl9wcml2YXRlOwo+ICAJc3RydWN0IGRybV9kaXNwbGF5X21v ZGUgKmFkanVzdGVkX21vZGUgPQo+ICAJCQkJCSZwaXBlX2NvbmZpZy0+YmFzZS5hZGp1c3RlZF9t b2RlOwo+ICsJc3RydWN0IGRybV9kaXNwbGF5X21vZGUgKmFkanVzdGVkX21vZGVfc3c7Cj4gKwlz dHJ1Y3QgaW50ZWxfY3J0YyAqaW50ZWxfY3J0YzsKPiAgCXN0cnVjdCBpbnRlbF9kc2kgKmludGVs X2RzaSA9IGVuY190b19pbnRlbF9kc2koJmVuY29kZXItPmJhc2UpOwo+ICAJdW5zaWduZWQgaW50 IGxhbmVfY291bnQgPSBpbnRlbF9kc2ktPmxhbmVfY291bnQ7Cj4gIAl1bnNpZ25lZCBpbnQgYnBw LCBmbXQ7Cj4gIAllbnVtIHBvcnQgcG9ydDsKPiAgCXUxNiBoYWN0aXZlLCBoZnAsIGhzeW5jLCBo YnAsIHZmcCwgdnN5bmMsIHZicDsKPiArCXUxNiBoZnBfc3csIGhzeW5jX3N3LCBoYnBfc3c7Cj4g Kwl1MTYgY3J0Y19odG90YWxfc3csIGNydGNfaHN5bmNfc3RhcnRfc3csIGNydGNfaHN5bmNfZW5k X3N3LAo+ICsJCQkJY3J0Y19oYmxhbmtfc3RhcnRfc3csIGNydGNfaGJsYW5rX2VuZF9zdzsKPiAr Cj4gKwlpbnRlbF9jcnRjID0gdG9faW50ZWxfY3J0YyhlbmNvZGVyLT5iYXNlLmNydGMpOwo+ICsJ YWRqdXN0ZWRfbW9kZV9zdyA9ICZpbnRlbF9jcnRjLT5jb25maWctPmJhc2UuYWRqdXN0ZWRfbW9k ZTsKPgogCj4gIAkvKgo+ICAJICogQXRsZWFzdCBvbmUgcG9ydCBpcyBhY3RpdmUgYXMgZW5jb2Rl ci0+Z2V0X2NvbmZpZyBjYWxsZWQgb25seSBpZgo+IEBAIC04NDcsOCArODYzLDc5IEBAIHN0YXRp YyB2b2lkIGJ4dF9kc2lfZ2V0X3BpcGVfY29uZmlnKHN0cnVjdCBpbnRlbF9lbmNvZGVyICplbmNv ZGVyLAo+ICAJYWRqdXN0ZWRfbW9kZS0+Y3J0Y192c3luY19lbmQgPSB2c3luYyArIGFkanVzdGVk X21vZGUtPmNydGNfdnN5bmNfc3RhcnQ7Cj4gIAlhZGp1c3RlZF9tb2RlLT5jcnRjX3ZibGFua19z dGFydCA9IGFkanVzdGVkX21vZGUtPmNydGNfdmRpc3BsYXk7Cj4gIAlhZGp1c3RlZF9tb2RlLT5j cnRjX3ZibGFua19lbmQgPSBhZGp1c3RlZF9tb2RlLT5jcnRjX3Z0b3RhbDsKPiAtfQo+ICAKPiAr CS8qCj4gKwkgKiBJbiBCWFQgRFNJIHRoZXJlIGlzIG5vIHJlZ3MgcHJvZ3JhbW1lZCB3aXRoIGZl dyBob3Jpem9udGFsIHRpbWluZ3MKPiArCSAqIGluIFBpeGVscyBidXQgdHhieXRlY2xraHMuLiBT byByZXRyaWV2YWwgcHJvY2VzcyBhZGRzIHNvbWUKPiArCSAqIFJPVU5EX1VQIEVSUk9SUyBpbiB0 aGUgcHJvY2VzcyBvZiBQSVhFTFM8PT0+dHhieXRlY2xraHMuCj4gKwkgKiBBY3R1YWxseSBoZXJl IGZvciB0aGUgZ2l2ZW4gYWRqdXN0ZWRfbW9kZSwgd2UgYXJlIGNhbGN1bGF0aW5nIHRoZQo+ICsJ ICogdmFsdWUgcHJvZ3JhbW1lZCB0byB0aGUgcG9ydCBhbmQgdGhlbiBiYWNrIHRvIHRoZSBob3Jp em9udGFsIHRpbWluZwo+ICsJICogcGFyYW0gaW4gcGl4ZWxzLiBUaGlzIGlzIHRoZSBleHBlY3Rl ZCB2YWx1ZSwgaW5jbHVkaW5nIHJvdW5kdXAgZXJyb3JzCj4gKwkgKiBBbmQgaWYgdGhhdCBpcyBz YW1lIGFzIHJldHJpZXZlZCB2YWx1ZSBmcm9tIHBvcnQsIHRoZW4KPiArCSAqIChIVyBzdGF0ZSkg YWRqdXN0ZWRfbW9kZSdzIGhvcml6b250YWwgdGltaW5ncyBhcmUgY29ycmVjdGVkIHRvCj4gKwkg KiBtYXRjaCB3aXRoIFNXIHN0YXRlIHRvIG51bGxpZnkgdGhlIGVycm9ycy4KPiArCSAqLwo+ICsJ LyogQ2FsY3VsYXRpbmcgdGhlIHZhbHVlIHByb2dyYW1tZWQgdG8gdGhlIFBvcnQgcmVnaXN0ZXIg Ki8KPiArCWhmcF9zdyA9IGFkanVzdGVkX21vZGVfc3ctPmNydGNfaHN5bmNfc3RhcnQgLQo+ICsJ CQkJCWFkanVzdGVkX21vZGVfc3ctPmNydGNfaGRpc3BsYXk7Cj4gKwloc3luY19zdyA9IGFkanVz dGVkX21vZGVfc3ctPmNydGNfaHN5bmNfZW5kIC0KPiArCQkJCQlhZGp1c3RlZF9tb2RlX3N3LT5j cnRjX2hzeW5jX3N0YXJ0Owo+ICsJaGJwX3N3ID0gYWRqdXN0ZWRfbW9kZV9zdy0+Y3J0Y19odG90 YWwgLQo+ICsJCQkJCWFkanVzdGVkX21vZGVfc3ctPmNydGNfaHN5bmNfZW5kOwoKU28gZHVyaW5n IHRoZSBpbml0aWFsIHN0YXRlIHJlYWRvdXQgd2UgZ2V0IHBhc3NlZCBjcnRjLT5jb25maWcgYXMK cGlwZV9jb25maWcsIGFuZCBzbyB3ZSdsbCBlbmQgdXAgY29tcGFyaW5nIHRoZSB0aGluZyB3aXRo IGl0c2VsZi4gVGhhdApzaG91bGQgYmUgZmluZS4gQSBiaXQgb2YgZXh0cmEgY2FyZSBpcyBuZWVk ZWQgdG8gbWFrZSBzdXJlIHdlIGRvbid0IHVzZQphbnl0aGluZyBmcm9tIGNydGMtPmNvbmZpZyBi ZWZvcmUgd2UndmUgZmlsbGVkIGl0IG91dCB3aXRoIHNvbWV0aGluZywKYnV0IGxvb2tzIGxpa2Ug dGhlIGNvZGUgZG9lcyB0aGluZ3MgaW4gdGhlIHJpZ2h0IG9yZGVyIChnaXZlbiB0aGUKcHJldmlv dXMgcGF0Y2ggd2hpY2ggZmlsbHMgb3V0IGFsbCB0aGUgaHRpbWluZ3Mgd2l0aCBzb21ldGhpbmcp LgoKSSB0aGluayB0aGUgYmlnZ2VzdCBpc3N1ZSB3aXRoIHRoaXMgcGF0Y2ggaXMgc2VlaW5nIHRo ZSBmb3Jlc3QgZnJvbSB0aGUKdHJlZXMuIFNvbWUgcmVmYWN0b3Jpbmcgd291bGQgYmUgZ29vZCBz byB0aGF0IHdlJ2QgaGF2ZSBzb21lIGtpbmQgb2YKaHRpbWluZ3Nfe3RvLGZyb219X3R4Ynl0ZWNs a2hzKCkgZnVuY3Rpb25zIGluc3RlYWQgb2YgZHVwbGljYXRpbmcgdGhlCnNhbWUgY29kZSBtdWx0 aXBsZSB0aW1lcy4KClNvIHdoaWxlIGl0IGRvZXMgbG9vayBxdWl0ZSBzdHJhbmdlIHRvIGJlIHVz aW5nIGNydGMtPmNvbmZpZyBpbiB0aGUKLmdldF9jb25maWcoKSBJIHRoaW5rIGl0IHNob3VsZCB3 b3JrIG91dC4KCkFja2VkLWJ5OiBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXgu aW50ZWwuY29tPgoKPiArCj4gKwlpZiAoaW50ZWxfZHNpLT5kdWFsX2xpbmspIHsKPiArCQloZnBf c3cgLz0gMjsKPiArCQloc3luY19zdyAvPSAyOwo+ICsJCWhicF9zdyAvPSAyOwo+ICsJfQo+ICsK PiArCWhmcF9zdyA9IHR4Ynl0ZWNsa2hzKGhmcF9zdywgYnBwLCBsYW5lX2NvdW50LAo+ICsJCQkJ CQlpbnRlbF9kc2ktPmJ1cnN0X21vZGVfcmF0aW8pOwo+ICsJaHN5bmNfc3cgPSB0eGJ5dGVjbGto cyhoc3luY19zdywgYnBwLCBsYW5lX2NvdW50LAo+ICsJCQkgICAgaW50ZWxfZHNpLT5idXJzdF9t b2RlX3JhdGlvKTsKPiArCWhicF9zdyA9IHR4Ynl0ZWNsa2hzKGhicF9zdywgYnBwLCBsYW5lX2Nv dW50LAo+ICsJCQkJCQlpbnRlbF9kc2ktPmJ1cnN0X21vZGVfcmF0aW8pOwo+ICsKPiArCS8qIFJl dmVyc2UgY2FsY3VsYXRpbmcgdGhlIGFkanVzdGVkIG1vZGUgcGFyYW1ldGVycyBmcm9tIHBvcnQg cmVnIHZhbHMqLwo+ICsJaGZwX3N3ID0gcGl4ZWxzX2Zyb21fdHhieXRlY2xraHMoaGZwX3N3LCBi cHAsIGxhbmVfY291bnQsCj4gKwkJCQkJCWludGVsX2RzaS0+YnVyc3RfbW9kZV9yYXRpbyk7Cj4g Kwloc3luY19zdyA9IHBpeGVsc19mcm9tX3R4Ynl0ZWNsa2hzKGhzeW5jX3N3LCBicHAsIGxhbmVf Y291bnQsCj4gKwkJCQkJCWludGVsX2RzaS0+YnVyc3RfbW9kZV9yYXRpbyk7Cj4gKwloYnBfc3cg PSBwaXhlbHNfZnJvbV90eGJ5dGVjbGtocyhoYnBfc3csIGJwcCwgbGFuZV9jb3VudCwKPiArCQkJ CQkJaW50ZWxfZHNpLT5idXJzdF9tb2RlX3JhdGlvKTsKPiArCj4gKwlpZiAoaW50ZWxfZHNpLT5k dWFsX2xpbmspIHsKPiArCQloZnBfc3cgKj0gMjsKPiArCQloc3luY19zdyAqPSAyOwo+ICsJCWhi cF9zdyAqPSAyOwo+ICsJfQo+ICsKPiArCWNydGNfaHRvdGFsX3N3ID0gYWRqdXN0ZWRfbW9kZV9z dy0+Y3J0Y19oZGlzcGxheSArIGhmcF9zdyArCj4gKwkJCQkJCQloc3luY19zdyArIGhicF9zdzsK PiArCWNydGNfaHN5bmNfc3RhcnRfc3cgPSBoZnBfc3cgKyBhZGp1c3RlZF9tb2RlX3N3LT5jcnRj X2hkaXNwbGF5Owo+ICsJY3J0Y19oc3luY19lbmRfc3cgPSBoc3luY19zdyArIGNydGNfaHN5bmNf c3RhcnRfc3c7Cj4gKwljcnRjX2hibGFua19zdGFydF9zdyA9IGFkanVzdGVkX21vZGVfc3ctPmNy dGNfaGRpc3BsYXk7Cj4gKwljcnRjX2hibGFua19lbmRfc3cgPSBjcnRjX2h0b3RhbF9zdzsKPiAr Cj4gKwlpZiAoYWRqdXN0ZWRfbW9kZS0+Y3J0Y19odG90YWwgPT0gY3J0Y19odG90YWxfc3cpCj4g KwkJYWRqdXN0ZWRfbW9kZS0+Y3J0Y19odG90YWwgPSBhZGp1c3RlZF9tb2RlX3N3LT5jcnRjX2h0 b3RhbDsKPiArCj4gKwlpZiAoYWRqdXN0ZWRfbW9kZS0+Y3J0Y19oc3luY19zdGFydCA9PSBjcnRj X2hzeW5jX3N0YXJ0X3N3KQo+ICsJCWFkanVzdGVkX21vZGUtPmNydGNfaHN5bmNfc3RhcnQgPQo+ ICsJCQkJCWFkanVzdGVkX21vZGVfc3ctPmNydGNfaHN5bmNfc3RhcnQ7Cj4gKwo+ICsJaWYgKGFk anVzdGVkX21vZGUtPmNydGNfaHN5bmNfZW5kID09IGNydGNfaHN5bmNfZW5kX3N3KQo+ICsJCWFk anVzdGVkX21vZGUtPmNydGNfaHN5bmNfZW5kID0KPiArCQkJCQlhZGp1c3RlZF9tb2RlX3N3LT5j cnRjX2hzeW5jX2VuZDsKPiArCj4gKwlpZiAoYWRqdXN0ZWRfbW9kZS0+Y3J0Y19oYmxhbmtfc3Rh cnQgPT0gY3J0Y19oYmxhbmtfc3RhcnRfc3cpCj4gKwkJYWRqdXN0ZWRfbW9kZS0+Y3J0Y19oYmxh bmtfc3RhcnQgPQo+ICsJCQkJCWFkanVzdGVkX21vZGVfc3ctPmNydGNfaGJsYW5rX3N0YXJ0Owo+ ICsKPiArCWlmIChhZGp1c3RlZF9tb2RlLT5jcnRjX2hibGFua19lbmQgPT0gY3J0Y19oYmxhbmtf ZW5kX3N3KQo+ICsJCWFkanVzdGVkX21vZGUtPmNydGNfaGJsYW5rX2VuZCA9Cj4gKwkJCQkJYWRq dXN0ZWRfbW9kZV9zdy0+Y3J0Y19oYmxhbmtfZW5kOwo+ICt9Cj4gIAo+ICBzdGF0aWMgdm9pZCBp bnRlbF9kc2lfZ2V0X2NvbmZpZyhzdHJ1Y3QgaW50ZWxfZW5jb2RlciAqZW5jb2RlciwKPiAgCQkJ CSBzdHJ1Y3QgaW50ZWxfY3J0Y19zdGF0ZSAqcGlwZV9jb25maWcpCj4gQEAgLTkxNywxNCArMTAw NCw2IEBAIHN0YXRpYyB1MTYgdHhjbGtlc2ModTMyIGRpdmlkZXIsIHVuc2lnbmVkIGludCB1cykK PiAgCX0KPiAgfQo+ICAKPiAtLyogcmV0dXJuIHBpeGVscyBpbiB0ZXJtcyBvZiB0eGJ5dGVjbGto cyAqLwo+IC1zdGF0aWMgdTE2IHR4Ynl0ZWNsa2hzKHUxNiBwaXhlbHMsIGludCBicHAsIGludCBs YW5lX2NvdW50LAo+IC0JCSAgICAgICB1MTYgYnVyc3RfbW9kZV9yYXRpbykKPiAtewo+IC0JcmV0 dXJuIERJVl9ST1VORF9VUChESVZfUk9VTkRfVVAocGl4ZWxzICogYnBwICogYnVyc3RfbW9kZV9y YXRpbywKPiAtCQkJCQkgOCAqIDEwMCksIGxhbmVfY291bnQpOwo+IC19Cj4gLQo+ICBzdGF0aWMg dm9pZCBzZXRfZHNpX3RpbWluZ3Moc3RydWN0IGRybV9lbmNvZGVyICplbmNvZGVyLAo+ICAJCQkg ICAgY29uc3Qgc3RydWN0IGRybV9kaXNwbGF5X21vZGUgKmFkanVzdGVkX21vZGUpCj4gIHsKPiAt LSAKPiAxLjcuOS41Cj4gX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KPiBJbnRlbC1nZnggbWFpbGluZyBsaXN0Cj4gSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNr dG9wLm9yZwo+IGh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8v aW50ZWwtZ2Z4CgotLSAKVmlsbGUgU3lyasOkbMOkCkludGVsIE9UQwpfX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0Cklu dGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5v cmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK