From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 5/8] drm/i915: Don't access to crtc->new_config from intel_mode_max_pixclk() Date: Mon, 15 Dec 2014 21:24:23 +0200 Message-ID: <20141215192423.GH10649@intel.com> References: <1418301491-23020-1-git-send-email-ander.conselvan.de.oliveira@intel.com> <1418301491-23020-6-git-send-email-ander.conselvan.de.oliveira@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by gabe.freedesktop.org (Postfix) with ESMTP id E04EA6E2ED for ; Mon, 15 Dec 2014 11:24:39 -0800 (PST) Content-Disposition: inline In-Reply-To: <1418301491-23020-6-git-send-email-ander.conselvan.de.oliveira@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Ander Conselvan de Oliveira Cc: intel-gfx@lists.freedesktop.org, shuang.he@linux.intel.com List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBEZWMgMTEsIDIwMTQgYXQgMDI6Mzg6MDhQTSArMDIwMCwgQW5kZXIgQ29uc2VsdmFu IGRlIE9saXZlaXJhIHdyb3RlOgo+IFNvIHRoYXQgd2UgY2FuIGdldCByaWQgb2YgdGhlIG5ld19j b25maWcgcG9pbnRlciBsYXRlci4KPiAKPiBTaWduZWQtb2ZmLWJ5OiBBbmRlciBDb25zZWx2YW4g ZGUgT2xpdmVpcmEgPGFuZGVyLmNvbnNlbHZhbi5kZS5vbGl2ZWlyYUBpbnRlbC5jb20+Cj4gLS0t Cj4gIGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rpc3BsYXkuYyB8IDMwICsrKysrKysrKysr KysrKysrKysrKystLS0tLS0tLQo+ICAxIGZpbGUgY2hhbmdlZCwgMjIgaW5zZXJ0aW9ucygrKSwg OCBkZWxldGlvbnMoLSkKPiAKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50 ZWxfZGlzcGxheS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jCj4gaW5k ZXggZGE1YWYyMy4uYTlmMzAzNCAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9p bnRlbF9kaXNwbGF5LmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5 LmMKPiBAQCAtNDg5NywyNyArNDg5NywzOSBAQCBzdGF0aWMgaW50IHZhbGxleXZpZXdfY2FsY19j ZGNsayhzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3ByaXYsCj4gIH0KPiAgCj4gIC8qIGNv bXB1dGUgdGhlIG1heCBwaXhlbCBjbG9jayBmb3IgbmV3IGNvbmZpZ3VyYXRpb24gKi8KPiAtc3Rh dGljIGludCBpbnRlbF9tb2RlX21heF9waXhjbGsoc3RydWN0IGRybV9pOTE1X3ByaXZhdGUgKmRl dl9wcml2KQo+ICtzdGF0aWMgaW50IGludGVsX21vZGVfbWF4X3BpeGNsayhzdHJ1Y3QgZHJtX2k5 MTVfcHJpdmF0ZSAqZGV2X3ByaXYsCj4gKwkJCQkgc3RydWN0IGludGVsX2NydGMgKm1vZGVfc2V0 X2NydGMsCj4gKwkJCQkgc3RydWN0IGludGVsX2NydGNfc3RhdGUgKmNydGNfbmV3X3N0YXRlKQo+ ICB7Cj4gIAlzdHJ1Y3QgZHJtX2RldmljZSAqZGV2ID0gZGV2X3ByaXYtPmRldjsKPiAgCXN0cnVj dCBpbnRlbF9jcnRjICppbnRlbF9jcnRjOwo+ICAJaW50IG1heF9waXhjbGsgPSAwOwo+ICsJaW50 IHBpeGNsazsKPiAgCj4gIAlmb3JfZWFjaF9pbnRlbF9jcnRjKGRldiwgaW50ZWxfY3J0Yykgewo+ IC0JCWlmIChpbnRlbF9jcnRjLT5uZXdfZW5hYmxlZCkKPiAtCQkJbWF4X3BpeGNsayA9IG1heCht YXhfcGl4Y2xrLAo+IC0JCQkJCSBpbnRlbF9jcnRjLT5uZXdfY29uZmlnLT5iYXNlLmFkanVzdGVk X21vZGUuY3J0Y19jbG9jayk7Cj4gKwkJaWYgKCFpbnRlbF9jcnRjLT5uZXdfZW5hYmxlZCkKPiAr CQkJY29udGludWU7Cj4gKwo+ICsJCWlmIChpbnRlbF9jcnRjID09IG1vZGVfc2V0X2NydGMpCj4g KwkJCXBpeGNsayA9IGNydGNfbmV3X3N0YXRlLT5iYXNlLmFkanVzdGVkX21vZGUuY3J0Y19jbG9j azsKPiArCQllbHNlCj4gKwkJCXBpeGNsayA9IGludGVsX2NydGMtPmNvbmZpZy0+YmFzZS5hZGp1 c3RlZF9tb2RlLmNydGNfY2xvY2s7Cj4gKwo+ICsJCW1heF9waXhjbGsgPSBtYXgobWF4X3BpeGNs aywgcGl4Y2xrKTsKCkkgdGhpbmsgd2UncmUgbm93IGdvaW5nIGJhY2t3YXJkcy4gV2Ugd2FudCBh dG9taWMgbW9kZXNldHMsIHNvIHdlCm5lZWQgdG8gYmUgYWJsZSB0byBjb21wdXRlIHRoaXMgc3R1 ZmYgd2hlbiBtb2Rlc2V0dGluZyBtdWx0aXBsZSBwaXBlcy4KVGhhdCB3YXMgcHJlY2lzZWx5IHdo eSBJIGFkZGVkIG5ld19jb25maWcuCgo+ICAJfQo+ICAKPiAgCXJldHVybiBtYXhfcGl4Y2xrOwo+ ICB9Cj4gIAo+ICBzdGF0aWMgdm9pZCB2YWxsZXl2aWV3X21vZGVzZXRfZ2xvYmFsX3BpcGVzKHN0 cnVjdCBkcm1fZGV2aWNlICpkZXYsCj4gLQkJCQkJICAgIHVuc2lnbmVkICpwcmVwYXJlX3BpcGVz KQo+ICsJCQkJCSAgICB1bnNpZ25lZCAqcHJlcGFyZV9waXBlcywKPiArCQkJCQkgICAgc3RydWN0 IGludGVsX2NydGMgKm1vZGVfc2V0X2NydGMsCj4gKwkJCQkJICAgIHN0cnVjdCBpbnRlbF9jcnRj X3N0YXRlICpjcnRjX25ld19zdGF0ZSkKPiAgewo+ICAJc3RydWN0IGRybV9pOTE1X3ByaXZhdGUg KmRldl9wcml2ID0gZGV2LT5kZXZfcHJpdmF0ZTsKPiAgCXN0cnVjdCBpbnRlbF9jcnRjICppbnRl bF9jcnRjOwo+IC0JaW50IG1heF9waXhjbGsgPSBpbnRlbF9tb2RlX21heF9waXhjbGsoZGV2X3By aXYpOwo+ICsJaW50IG1heF9waXhjbGsgPQo+ICsJCWludGVsX21vZGVfbWF4X3BpeGNsayhkZXZf cHJpdiwgbW9kZV9zZXRfY3J0YywgY3J0Y19uZXdfc3RhdGUpOwo+ICAKPiAgCWlmICh2YWxsZXl2 aWV3X2NhbGNfY2RjbGsoZGV2X3ByaXYsIG1heF9waXhjbGspID09Cj4gIAkgICAgZGV2X3ByaXYt PnZsdl9jZGNsa19mcmVxKQo+IEBAIC00OTMyLDcgKzQ5NDQsNyBAQCBzdGF0aWMgdm9pZCB2YWxs ZXl2aWV3X21vZGVzZXRfZ2xvYmFsX3BpcGVzKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsCj4gIHN0 YXRpYyB2b2lkIHZhbGxleXZpZXdfbW9kZXNldF9nbG9iYWxfcmVzb3VyY2VzKHN0cnVjdCBkcm1f ZGV2aWNlICpkZXYpCj4gIHsKPiAgCXN0cnVjdCBkcm1faTkxNV9wcml2YXRlICpkZXZfcHJpdiA9 IGRldi0+ZGV2X3ByaXZhdGU7Cj4gLQlpbnQgbWF4X3BpeGNsayA9IGludGVsX21vZGVfbWF4X3Bp eGNsayhkZXZfcHJpdik7Cj4gKwlpbnQgbWF4X3BpeGNsayA9IGludGVsX21vZGVfbWF4X3BpeGNs ayhkZXZfcHJpdiwgTlVMTCwgTlVMTCk7Cj4gIAlpbnQgcmVxX2NkY2xrID0gdmFsbGV5dmlld19j YWxjX2NkY2xrKGRldl9wcml2LCBtYXhfcGl4Y2xrKTsKPiAgCj4gIAlpZiAocmVxX2NkY2xrICE9 IGRldl9wcml2LT52bHZfY2RjbGtfZnJlcSkgewo+IEBAIC0xMDk1Niw3ICsxMDk2OCw5IEBAIHN0 YXRpYyBpbnQgX19pbnRlbF9zZXRfbW9kZShzdHJ1Y3QgZHJtX2NydGMgKmNydGMsCj4gIAkgKiBh ZGp1c3RlZF9tb2RlIGJpdHMgaW4gdGhlIGNydGMgZGlyZWN0bHkuCj4gIAkgKi8KPiAgCWlmIChJ U19WQUxMRVlWSUVXKGRldikpIHsKPiAtCQl2YWxsZXl2aWV3X21vZGVzZXRfZ2xvYmFsX3BpcGVz KGRldiwgJnByZXBhcmVfcGlwZXMpOwo+ICsJCXZhbGxleXZpZXdfbW9kZXNldF9nbG9iYWxfcGlw ZXMoZGV2LCAmcHJlcGFyZV9waXBlcywKPiArCQkJCQkJdG9faW50ZWxfY3J0YyhjcnRjKSwKPiAr CQkJCQkJcGlwZV9jb25maWcpOwo+ICAKPiAgCQkvKiBtYXkgaGF2ZSBhZGRlZCBtb3JlIHRvIHBy ZXBhcmVfcGlwZXMgdGhhbiB3ZSBzaG91bGQgKi8KPiAgCQlwcmVwYXJlX3BpcGVzICY9IH5kaXNh YmxlX3BpcGVzOwo+IC0tIAo+IDEuOS4xCj4gCj4gX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KPiBJbnRlbC1nZnggbWFpbGluZyBsaXN0Cj4gSW50ZWwtZ2Z4 QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwo+IGh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFp bG1hbi9saXN0aW5mby9pbnRlbC1nZngKCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBt YWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMu ZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==