From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paulo Zanoni Subject: Re: [PATCH 3/3] drm/i915: reorganize the get_cdclk assignment Date: Tue, 21 Feb 2017 16:45:02 -0300 Message-ID: <1487706302.2614.45.camel@intel.com> References: <1487620842-22893-1-git-send-email-paulo.r.zanoni@intel.com> <1487620842-22893-4-git-send-email-paulo.r.zanoni@intel.com> <20170221115150.GE31595@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTPS id E86796E2E0 for ; Tue, 21 Feb 2017 19:45:07 +0000 (UTC) In-Reply-To: <20170221115150.GE31595@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Ville =?ISO-8859-1?Q?Syrj=E4l=E4?= Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org RW0gVGVyLCAyMDE3LTAyLTIxIMOgcyAxMzo1MSArMDIwMCwgVmlsbGUgU3lyasOkbMOkIGVzY3Jl dmV1Ogo+IE9uIE1vbiwgRmViIDIwLCAyMDE3IGF0IDA1OjAwOjQyUE0gLTAzMDAsIFBhdWxvIFph bm9uaSB3cm90ZToKPiA+IAo+ID4gUG9zc2libGUgcHJvYmxlbXMgb2YgdGhlIGN1cnJlbnQgaWYt bGFkZGVyOgo+ID4gwqAgLSBJdCdzIGEgaHVnZSBpZiBsYWRkZXIgd2l0aCBhbG1vc3QgYSBkaWZm ZXJlbnQgY2hlY2sgZm9yIGVhY2ggb2YKPiA+IMKgwqDCoMKgb3VyIHBsYXRmb3Jtcy4KPiA+IMKg IC0gSXQgbWl4ZXMgMyBkaWZmZXJlbnQgdHlwZXMgb2YgY2hlY2tzOiBJU19HRU5YLCBJU19QTEFU Rk9STSBhbmQKPiA+IMKgwqDCoMKgSVNfR1JPVVBfT0ZfUExBVEZPUk1TLgo+ID4gwqAgLSBBcyBk ZW1vbnN0cmF0ZWQgYnkgdGhlIHJlY2VudCBJU19HNFggY29tbWl0LCBpdCdzIG5vdCBlYXN5IHRv Cj4gPiBiZQo+ID4gwqDCoMKgwqBzdXJlIGlmIGEgcGxhdGZvcm0gZG93biBvbiB0aGUgbGlzdCBp c24ndCBhbHNvIGNoZWNrZWQgZWFybGllci4KPiA+IMKgIC0gQXMgZGVtb25zdHJhdGVkIGJ5IHRo ZSBXQVJOIGF0IHRoZSBlbmQsIGl0J3Mgbm90IGVhc3kgdG8gYmUKPiA+IHN1cmUKPiA+IMKgwqDC oMKgaWYgd2UncmUgYWN0dWFsbHkgY2hlY2tpbmcgZm9yIGV2ZXJ5IHNpbmdsZSBwbGF0Zm9ybS4K PiA+IAo+ID4gUG9zc2libGUgYWR2YW50YWdlcyBvZiB0aGUgbmV3IHN3aXRjaCBzdGF0ZW1lbnQ6 Cj4gPiDCoCAtIEl0IG1heSBiZSBlYXNpZXIgZm9yIHRoZSBjb21waWxlciB0byBvcHRpbWl6ZSBz dHVmZiAoSSBkaWRuJ3QKPiA+IMKgwqDCoMKgY2hlY2sgdGhpcyksIGVzcGVjaWFsbHkgc2luY2Ug dGhlIHZhbHVlcyBhcmUgbGFiZWxzIG9mIGFuIGVudW0uCj4gPiDCoCAtIFRoZSBjb21waWxlciB3 aWxsIHRlbGwgdXMgaW4gY2FzZSB3ZSBtaXNzIHNvbWUgcGxhdGZvcm0uCj4gPiDCoCAtIEFsbCBw bGF0Zm9ybXMgYXJlIGV4cGxpY2l0bHkgdGhlcmUgaW5zdGVhZCBvZiBtYXliZSBoaWRkZW4gaW4K PiA+IHNvbWUKPiA+IMKgwqDCoMKgY2hlY2sgZm9yIGEgY2VydGFpbiBncm91cCBvZiBwbGF0Zm9y bXMgc3VjaCBhcyBJU19HRU45X0JDLgo+ID4gCj4gPiBQb3NzaWJsZSBkaXNhZHZhbnRhZ2VzIHdp dGggdGhlIG5ldyBjb2RlOgo+ID4gwqAgLSBBIGZldyBsaW5lcyBiaWdnZXIuCj4gPiAKPiA+IHYy OiBEb24ndCB1bnNvcnQgdGhlIGxpc3QuIE5vdyB0aGUgbGlzdCBhbG1vc3QgbWF0Y2hlcyB0aGUg ZW51bQo+ID4gZGVmaW5pdGlvbiwgd2l0aCB0aGUgZXhjZXB0aW9uIG9mIENIViwgS0JMIGFuZCBH TEssIHdoaWNoIGFyZQo+ID4gbGlzdGVkCj4gPiBhbG9uZyB0aGVpciBwcmVkZWNlc3NvcnMgKFZp bGxlKS4KPiA+IAo+ID4gQ2M6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBsaW51eC5p bnRlbC5jb20+Cj4gPiBTaWduZWQtb2ZmLWJ5OiBQYXVsbyBaYW5vbmkgPHBhdWxvLnIuemFub25p QGludGVsLmNvbT4KPiA+IC0tLQo+ID4gwqBkcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9jZGNs ay5jIHwgMTA1ICsrKysrKysrKysrKysrKysrKysrKysrCj4gPiAtLS0tLS0tLS0tLS0tLQo+ID4g wqAxIGZpbGUgY2hhbmdlZCwgNjYgaW5zZXJ0aW9ucygrKSwgMzkgZGVsZXRpb25zKC0pCj4gPiAK PiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9jZGNsay5jCj4gPiBi L2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2NkY2xrLmMKPiA+IGluZGV4IDk0MmFkZjAuLmUx OTIxZmU3IDEwMDY0NAo+ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfY2RjbGsu Ywo+ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfY2RjbGsuYwo+ID4gQEAgLTE3 NjIsNDkgKzE3NjIsNzYgQEAgdm9pZCBpbnRlbF9pbml0X2NkY2xrX2hvb2tzKHN0cnVjdAo+ID4g ZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3ByaXYpCj4gPiDCoAkJZGV2X3ByaXYtPmRpc3BsYXkuY2Fs Y19jZGNsa19zdGF0ZSA9Cj4gPiBza2xfY2FsY19jZGNsa19zdGF0ZTsKPiA+IMKgCX0KPiA+IMKg Cj4gPiAtCWlmIChJU19HRU45X0JDKGRldl9wcml2KSkKPiA+IC0JCWRldl9wcml2LT5kaXNwbGF5 LmdldF9jZGNsayA9IHNrbF9nZXRfY2RjbGs7Cj4gPiAtCWVsc2UgaWYgKElTX0dFTjlfTFAoZGV2 X3ByaXYpKQo+ID4gLQkJZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0gYnh0X2dldF9jZGNs azsKPiA+IC0JZWxzZSBpZiAoSVNfQlJPQURXRUxMKGRldl9wcml2KSkKPiA+IC0JCWRldl9wcml2 LT5kaXNwbGF5LmdldF9jZGNsayA9IGJkd19nZXRfY2RjbGs7Cj4gPiAtCWVsc2UgaWYgKElTX0hB U1dFTEwoZGV2X3ByaXYpKQo+ID4gLQkJZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0gaHN3 X2dldF9jZGNsazsKPiA+IC0JZWxzZSBpZiAoSVNfVkFMTEVZVklFVyhkZXZfcHJpdikgfHwKPiA+ IElTX0NIRVJSWVZJRVcoZGV2X3ByaXYpKQo+ID4gLQkJZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2Nk Y2xrID0gdmx2X2dldF9jZGNsazsKPiA+IC0JZWxzZSBpZiAoSVNfR0VONihkZXZfcHJpdikgfHwg SVNfSVZZQlJJREdFKGRldl9wcml2KSkKPiA+ICsJc3dpdGNoIChJTlRFTF9JTkZPKGRldl9wcml2 KS0+cGxhdGZvcm0pIHsKPiA+ICsJY2FzZSBJTlRFTF9QTEFURk9STV9VTklOSVRJQUxJWkVEOgo+ ID4gKwkJTUlTU0lOR19DQVNFKElOVEVMX0lORk8oZGV2X3ByaXYpLT5wbGF0Zm9ybSk7Cj4gPiAr CQkvKiBGYWxsIHRocm91Z2guICovCj4gPiArCWNhc2UgSU5URUxfSTgzMDoKPiAKPiBUaGUgZGlm ZiBtaWdodCBjb21lIG91dCBuaWNlciBpZiB5b3UgZGlkbid0IHJldmVyc2UgdGhlIGxpc3QuIFJp Z2h0Cj4gbm93Cj4gaXQncyBxdWl0ZSBoYXJkIHRvIHJlYWQuCgpPbiB0aGUgb3RoZXIgaGFuZCwg dGhlIGxhYmVscyBpbiB0aGUgc3dpdGNoIHN0YXRlbWVudCBtYXRjaCB0aGUgZW51bQpkZWZpbml0 aW9uICh3aXRoIHRoZSBleGNlcHRpb24gb2YgQ0hWL0tCTC9HTEspLiBTbyBJTUhPIHRoaXMgaXMg YW4KImVhc2llciB0byByZXZpZXcgdGhlIHBhdGNoIG5vdyB2cyBlYXNpZXIgdG8gcmV2aWV3IHRo ZSBjb2RlIGxhdGVyIgpwcm9ibGVtLiBTdGlsbCwgSSBjYW4gY2hhbmdlIHRoaXMsIGl0IHdvbid0 IGh1cnQuCgo+IAo+ID4gCj4gPiArCQlkZXZfcHJpdi0+ZGlzcGxheS5nZXRfY2RjbGsgPQo+ID4g Zml4ZWRfMTMzbWh6X2dldF9jZGNsazsKPiA+ICsJCWJyZWFrOwo+ID4gKwljYXNlIElOVEVMX0k4 NDVHOgo+ID4gKwkJZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0KPiA+IGZpeGVkXzIwMG1o el9nZXRfY2RjbGs7Cj4gPiArCQlicmVhazsKPiA+ICsJY2FzZSBJTlRFTF9JODVYOgo+ID4gKwkJ ZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0gaTg1eF9nZXRfY2RjbGs7Cj4gPiArCQlicmVh azsKPiA+ICsJY2FzZSBJTlRFTF9JODY1RzoKPiA+ICsJCWRldl9wcml2LT5kaXNwbGF5LmdldF9j ZGNsayA9Cj4gPiBmaXhlZF8yNjZtaHpfZ2V0X2NkY2xrOwo+ID4gKwkJYnJlYWs7Cj4gPiArCWNh c2UgSU5URUxfSTkxNUc6Cj4gPiArCQlkZXZfcHJpdi0+ZGlzcGxheS5nZXRfY2RjbGsgPQo+ID4g Zml4ZWRfMzMzbWh6X2dldF9jZGNsazsKPiA+ICsJCWJyZWFrOwo+ID4gKwljYXNlIElOVEVMX0k5 MTVHTToKPiA+ICsJCWRldl9wcml2LT5kaXNwbGF5LmdldF9jZGNsayA9IGk5MTVnbV9nZXRfY2Rj bGs7Cj4gPiArCQlicmVhazsKPiA+ICsJY2FzZSBJTlRFTF9JOTQ1RzoKPiA+IMKgCQlkZXZfcHJp di0+ZGlzcGxheS5nZXRfY2RjbGsgPQo+ID4gZml4ZWRfNDAwbWh6X2dldF9jZGNsazsKPiA+IC0J ZWxzZSBpZiAoSVNfR0VONShkZXZfcHJpdikpCj4gPiAtCQlkZXZfcHJpdi0+ZGlzcGxheS5nZXRf Y2RjbGsgPQo+ID4gZml4ZWRfNDUwbWh6X2dldF9jZGNsazsKPiA+IC0JZWxzZSBpZiAoSVNfR000 NShkZXZfcHJpdikpCj4gPiAtCQlkZXZfcHJpdi0+ZGlzcGxheS5nZXRfY2RjbGsgPSBnbTQ1X2dl dF9jZGNsazsKPiA+IC0JZWxzZSBpZiAoSVNfRzQ1KGRldl9wcml2KSkKPiA+ICsJCWJyZWFrOwo+ ID4gKwljYXNlIElOVEVMX0k5NDVHTToKPiA+ICsJCWRldl9wcml2LT5kaXNwbGF5LmdldF9jZGNs ayA9IGk5NDVnbV9nZXRfY2RjbGs7Cj4gPiArCQlicmVhazsKPiA+ICsJY2FzZSBJTlRFTF9HMzM6 Cj4gPiDCoAkJZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0gZzMzX2dldF9jZGNsazsKPiA+ IC0JZWxzZSBpZiAoSVNfSTk2NUdNKGRldl9wcml2KSkKPiA+IC0JCWRldl9wcml2LT5kaXNwbGF5 LmdldF9jZGNsayA9IGk5NjVnbV9nZXRfY2RjbGs7Cj4gPiAtCWVsc2UgaWYgKElTX0k5NjVHKGRl dl9wcml2KSkKPiA+IC0JCWRldl9wcml2LT5kaXNwbGF5LmdldF9jZGNsayA9Cj4gPiBmaXhlZF80 MDBtaHpfZ2V0X2NkY2xrOwo+ID4gLQllbHNlIGlmIChJU19QSU5FVklFVyhkZXZfcHJpdikpCj4g PiArCQlicmVhazsKPiA+ICsJY2FzZSBJTlRFTF9QSU5FVklFVzoKPiA+IMKgCQlkZXZfcHJpdi0+ ZGlzcGxheS5nZXRfY2RjbGsgPSBwbnZfZ2V0X2NkY2xrOwo+ID4gLQllbHNlIGlmIChJU19HMzMo ZGV2X3ByaXYpKQo+ID4gKwkJYnJlYWs7Cj4gPiArCWNhc2UgSU5URUxfSTk2NUc6Cj4gPiArCQlk ZXZfcHJpdi0+ZGlzcGxheS5nZXRfY2RjbGsgPQo+ID4gZml4ZWRfNDAwbWh6X2dldF9jZGNsazsK PiA+ICsJCWJyZWFrOwo+ID4gKwljYXNlIElOVEVMX0k5NjVHTToKPiA+ICsJCWRldl9wcml2LT5k aXNwbGF5LmdldF9jZGNsayA9IGk5NjVnbV9nZXRfY2RjbGs7Cj4gPiArCQlicmVhazsKPiA+ICsJ Y2FzZSBJTlRFTF9HNDU6Cj4gPiDCoAkJZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0gZzMz X2dldF9jZGNsazsKPiA+IC0JZWxzZSBpZiAoSVNfSTk0NUdNKGRldl9wcml2KSkKPiA+IC0JCWRl dl9wcml2LT5kaXNwbGF5LmdldF9jZGNsayA9IGk5NDVnbV9nZXRfY2RjbGs7Cj4gPiAtCWVsc2Ug aWYgKElTX0k5NDVHKGRldl9wcml2KSkKPiA+ICsJCWJyZWFrOwo+ID4gKwljYXNlIElOVEVMX0dN NDU6Cj4gPiArCQlkZXZfcHJpdi0+ZGlzcGxheS5nZXRfY2RjbGsgPSBnbTQ1X2dldF9jZGNsazsK PiA+ICsJCWJyZWFrOwo+ID4gKwljYXNlIElOVEVMX0lST05MQUtFOgo+ID4gKwkJZGV2X3ByaXYt PmRpc3BsYXkuZ2V0X2NkY2xrID0KPiA+IGZpeGVkXzQ1MG1oel9nZXRfY2RjbGs7Cj4gPiArCQli cmVhazsKPiA+ICsJY2FzZSBJTlRFTF9TQU5EWUJSSURHRToKPiA+ICsJY2FzZSBJTlRFTF9JVllC UklER0U6Cj4gPiDCoAkJZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0KPiA+IGZpeGVkXzQw MG1oel9nZXRfY2RjbGs7Cj4gPiAtCWVsc2UgaWYgKElTX0k5MTVHTShkZXZfcHJpdikpCj4gPiAt CQlkZXZfcHJpdi0+ZGlzcGxheS5nZXRfY2RjbGsgPSBpOTE1Z21fZ2V0X2NkY2xrOwo+ID4gLQll bHNlIGlmIChJU19JOTE1RyhkZXZfcHJpdikpCj4gPiAtCQlkZXZfcHJpdi0+ZGlzcGxheS5nZXRf Y2RjbGsgPQo+ID4gZml4ZWRfMzMzbWh6X2dldF9jZGNsazsKPiA+IC0JZWxzZSBpZiAoSVNfSTg2 NUcoZGV2X3ByaXYpKQo+ID4gLQkJZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0KPiA+IGZp eGVkXzI2Nm1oel9nZXRfY2RjbGs7Cj4gPiAtCWVsc2UgaWYgKElTX0k4NVgoZGV2X3ByaXYpKQo+ ID4gLQkJZGV2X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0gaTg1eF9nZXRfY2RjbGs7Cj4gPiAt CWVsc2UgaWYgKElTX0k4NDVHKGRldl9wcml2KSkKPiA+IC0JCWRldl9wcml2LT5kaXNwbGF5Lmdl dF9jZGNsayA9Cj4gPiBmaXhlZF8yMDBtaHpfZ2V0X2NkY2xrOwo+ID4gLQllbHNlIHsgLyogODMw ICovCj4gPiAtCQlXQVJOKCFJU19JODMwKGRldl9wcml2KSwKPiA+IC0JCcKgwqDCoMKgwqAiVW5r bm93biBwbGF0Zm9ybS4gQXNzdW1pbmcgMTMzIE1Iego+ID4gQ0RDTEtcbiIpOwo+ID4gLQkJZGV2 X3ByaXYtPmRpc3BsYXkuZ2V0X2NkY2xrID0KPiA+IGZpeGVkXzEzM21oel9nZXRfY2RjbGs7Cj4g PiArCQlicmVhazsKPiA+ICsJY2FzZSBJTlRFTF9WQUxMRVlWSUVXOgo+ID4gKwljYXNlIElOVEVM X0NIRVJSWVZJRVc6Cj4gPiArCQlkZXZfcHJpdi0+ZGlzcGxheS5nZXRfY2RjbGsgPSB2bHZfZ2V0 X2NkY2xrOwo+ID4gKwkJYnJlYWs7Cj4gPiArCWNhc2UgSU5URUxfSEFTV0VMTDoKPiA+ICsJCWRl dl9wcml2LT5kaXNwbGF5LmdldF9jZGNsayA9IGhzd19nZXRfY2RjbGs7Cj4gPiArCQlicmVhazsK PiA+ICsJY2FzZSBJTlRFTF9CUk9BRFdFTEw6Cj4gPiArCQlkZXZfcHJpdi0+ZGlzcGxheS5nZXRf Y2RjbGsgPSBiZHdfZ2V0X2NkY2xrOwo+ID4gKwkJYnJlYWs7Cj4gPiArCWNhc2UgSU5URUxfU0tZ TEFLRToKPiA+ICsJY2FzZSBJTlRFTF9LQUJZTEFLRToKPiA+ICsJCWRldl9wcml2LT5kaXNwbGF5 LmdldF9jZGNsayA9IHNrbF9nZXRfY2RjbGs7Cj4gPiArCQlicmVhazsKPiA+ICsJY2FzZSBJTlRF TF9CUk9YVE9OOgo+ID4gKwljYXNlIElOVEVMX0dFTUlOSUxBS0U6Cj4gPiArCQlkZXZfcHJpdi0+ ZGlzcGxheS5nZXRfY2RjbGsgPSBieHRfZ2V0X2NkY2xrOwo+ID4gKwkJYnJlYWs7Cj4gPiDCoAl9 Cj4gPiDCoH0KPiA+IC0twqAKPiA+IDIuNy40Cj4gCl9fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxp c3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFu L2xpc3RpbmZvL2ludGVsLWdmeAo=