From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Souza, Jose" Subject: Re: [PATCH 1/4] drm/i915/psr: Always wait for idle state when disabling PSR Date: Wed, 10 Oct 2018 01:12:40 +0000 Message-ID: <4370d2bd3aa3984e75ae5fd0390a188753885452.camel@intel.com> References: <20181005233542.2939-1-jose.souza@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTPS id 319536E01D for ; Wed, 10 Oct 2018 01:13:13 +0000 (UTC) In-Reply-To: Content-Language: en-US Content-ID: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: "ville.syrjala@linux.intel.com" , "intel-gfx@lists.freedesktop.org" , "Pandiyan, Dhinakaran" Cc: "Vivi, Rodrigo" List-Id: intel-gfx@lists.freedesktop.org T24gTW9uLCAyMDE4LTEwLTA4IGF0IDE1OjQzIC0wNzAwLCBEaGluYWthcmFuIFBhbmRpeWFuIHdy b3RlOg0KPiBPbiBGcmksIDIwMTgtMTAtMDUgYXQgMTY6MzUgLTA3MDAsIEpvc8OpIFJvYmVydG8g ZGUgU291emEgd3JvdGU6DQo+ID4gSXQgc2hvdWxkIGFsd2F5cyB3YWl0IGZvciBpZGxlIHN0YXRl IHdoZW4gZGlzYWJsaW5nIFBTUiBiZWNhdXNlIFBTUg0KPiA+IGNvdWxkIGJlIGluYWN0aXZlIGR1 ZSBhIGNhbGwgdG8gaW50ZWxfcHNyX2V4aXQoKSBhbmQgd2hpbGUgUFNSIGlzDQo+ID4gc3RpbGwg YmVpbmcgZGlzYWJsZWQgYXN5bmNocm9ub3VzbHkgdXNlcnNwYWNlIGNvdWxkIGNoYW5nZSB0aGUN Cj4gPiBtb2Rlc2V0IGNhdXNpbmcgYSBjYWxsIHRvIHBzcl9kaXNhYmxlKCkgdGhhdCB3aWxsIG5v dCB3YWl0IGZvciBQU1INCj4gPiBpZGxlIGFuZCB0aGVuIFBTUiB3aWxsIGJlIGVuYWJsZWQgYWdh aW4gd2hpbGUgUFNSIGlzIHN0aWxsIG5vdA0KPiA+IGlkbGUuDQo+IEFncmVlZC4NCj4gDQo+ID4g Q2M6IFJvZHJpZ28gVml2aSA8cm9kcmlnby52aXZpQGludGVsLmNvbT4NCj4gPiBDYzogRGhpbmFr YXJhbiBQYW5kaXlhbiA8ZGhpbmFrYXJhbi5wYW5kaXlhbkBpbnRlbC5jb20+DQo+ID4gU2lnbmVk LW9mZi1ieTogSm9zw6kgUm9iZXJ0byBkZSBTb3V6YSA8am9zZS5zb3V6YUBpbnRlbC5jb20+DQo+ ID4gLS0tDQo+ID4gIGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3Bzci5jIHwgNDMgKysrKysr KysrKysrKysrLS0tLS0tLS0tLS0tLQ0KPiA+IC0tDQo+ID4gLS0NCj4gPiAgMSBmaWxlIGNoYW5n ZWQsIDIwIGluc2VydGlvbnMoKyksIDIzIGRlbGV0aW9ucygtKQ0KPiA+IA0KPiA+IGRpZmYgLS1n aXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wc3IuYw0KPiA+IGIvZHJpdmVycy9ncHUv ZHJtL2k5MTUvaW50ZWxfcHNyLmMNCj4gPiBpbmRleCA0MjNjZGY4NDA1OWMuLmNkOWE2MGQxZWZh MSAxMDA2NDQNCj4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wc3IuYw0KPiA+ ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3Bzci5jDQo+ID4gQEAgLTY2MSw0MCAr NjYxLDM3IEBAIHN0YXRpYyB2b2lkDQo+ID4gIGludGVsX3Bzcl9kaXNhYmxlX3NvdXJjZShzdHJ1 Y3QgaW50ZWxfZHAgKmludGVsX2RwKQ0KPiA+ICB7DQo+ID4gIAlzdHJ1Y3QgZHJtX2k5MTVfcHJp dmF0ZSAqZGV2X3ByaXYgPSBkcF90b19pOTE1KGludGVsX2RwKTsNCj4gPiArCWk5MTVfcmVnX3Qg cHNyX3N0YXR1czsNCj4gPiArCXUzMiBwc3Jfc3RhdHVzX21hc2s7DQo+ID4gIA0KPiA+ICAJaWYg KGRldl9wcml2LT5wc3IuYWN0aXZlKSB7DQo+ID4gLQkJaTkxNV9yZWdfdCBwc3Jfc3RhdHVzOw0K PiA+IC0JCXUzMiBwc3Jfc3RhdHVzX21hc2s7DQo+ID4gLQ0KPiA+IC0JCWlmIChkZXZfcHJpdi0+ cHNyLnBzcjJfZW5hYmxlZCkgew0KPiA+IC0JCQlwc3Jfc3RhdHVzID0gRURQX1BTUjJfU1RBVFVT Ow0KPiA+IC0JCQlwc3Jfc3RhdHVzX21hc2sgPSBFRFBfUFNSMl9TVEFUVVNfU1RBVEVfTUFTSzsN Cj4gPiAtDQo+ID4gKwkJaWYgKGRldl9wcml2LT5wc3IucHNyMl9lbmFibGVkKQ0KPiA+ICAJCQlJ OTE1X1dSSVRFKEVEUF9QU1IyX0NUTCwNCj4gPiAtCQkJCSAgIEk5MTVfUkVBRChFRFBfUFNSMl9D VEwpICYNCj4gPiAtCQkJCSAgIH4oRURQX1BTUjJfRU5BQkxFIHwNCj4gPiBFRFBfU1VfVFJBQ0tf RU5BQkxFKSk7DQo+ID4gLQ0KPiA+IC0JCX0gZWxzZSB7DQo+ID4gLQkJCXBzcl9zdGF0dXMgPSBF RFBfUFNSX1NUQVRVUzsNCj4gPiAtCQkJcHNyX3N0YXR1c19tYXNrID0gRURQX1BTUl9TVEFUVVNf U1RBVEVfTUFTSzsNCj4gPiAtDQo+ID4gKwkJCQkgICBJOTE1X1JFQUQoRURQX1BTUjJfQ1RMKSAm DQo+ID4gfkVEUF9QU1IyX0VOQUJMRSk7DQo+IA0KPiBJcyB0aGVyZSBhIHdheSB0byByZXVzZSBw c3JfZXhpdCgpIGFuZCBtb3ZlIHJlc3Qgb2YgdGhlIHN0dWZmIHRvIHRoZQ0KPiBjYWxsZXI/IFdl IHdvbid0IG5vdCBuZWVkIGRpc2FibGVfc291cmNlKCkgaWYgdGhhdCBjYW4gYmUgZG9uZT8NCg0K V2UgY291bGQgbW92ZSB0byB0aGUgY2FsbGVyIGFuZCByZXVzZSBwc3JfZXhpdCgpLCBJIHdpbGwg ZG8gdGhhdCBpbg0KYW5vdGhlciBwYXRjaC4NCg0KPiANCj4gDQo+IA0KPiA+ICsJCWVsc2UNCj4g PiAgCQkJSTkxNV9XUklURShFRFBfUFNSX0NUTCwNCj4gPiAgCQkJCSAgIEk5MTVfUkVBRChFRFBf UFNSX0NUTCkgJg0KPiA+IH5FRFBfUFNSX0VOQUJMRSk7DQo+ID4gLQkJfQ0KPiA+IC0NCj4gPiAt CQkvKiBXYWl0IHRpbGwgUFNSIGlzIGlkbGUgKi8NCj4gPiAtCQlpZiAoaW50ZWxfd2FpdF9mb3Jf cmVnaXN0ZXIoZGV2X3ByaXYsDQo+ID4gLQkJCQkJICAgIHBzcl9zdGF0dXMsDQo+ID4gcHNyX3N0 YXR1c19tYXNrLCAwLA0KPiA+IC0JCQkJCSAgICAyMDAwKSkNCj4gPiAtCQkJRFJNX0VSUk9SKCJU aW1lZCBvdXQgd2FpdGluZyBmb3IgUFNSIElkbGUNCj4gPiBTdGF0ZVxuIik7DQo+ID4gLQ0KPiA+ IC0JCWRldl9wcml2LT5wc3IuYWN0aXZlID0gZmFsc2U7DQo+ID4gIAl9IGVsc2Ugew0KPiA+ICAJ CWlmIChkZXZfcHJpdi0+cHNyLnBzcjJfZW5hYmxlZCkNCj4gPiAgCQkJV0FSTl9PTihJOTE1X1JF QUQoRURQX1BTUjJfQ1RMKSAmDQo+ID4gRURQX1BTUjJfRU5BQkxFKTsNCj4gPiAgCQllbHNlDQo+ ID4gIAkJCVdBUk5fT04oSTkxNV9SRUFEKEVEUF9QU1JfQ1RMKSAmDQo+ID4gRURQX1BTUl9FTkFC TEUpOw0KPiA+ICAJfQ0KPiA+ICsNCj4gPiArCWlmIChkZXZfcHJpdi0+cHNyLnBzcjJfZW5hYmxl ZCkgew0KPiA+ICsJCXBzcl9zdGF0dXMgPSBFRFBfUFNSMl9TVEFUVVM7DQo+ID4gKwkJcHNyX3N0 YXR1c19tYXNrID0gRURQX1BTUjJfU1RBVFVTX1NUQVRFX01BU0s7DQo+ID4gKwl9IGVsc2Ugew0K PiA+ICsJCXBzcl9zdGF0dXMgPSBFRFBfUFNSX1NUQVRVUzsNCj4gPiArCQlwc3Jfc3RhdHVzX21h c2sgPSBFRFBfUFNSX1NUQVRVU19TVEFURV9NQVNLOw0KPiA+ICsJfQ0KPiA+ICsNCj4gPiArCS8q IFdhaXQgdGlsbCBQU1IgaXMgaWRsZSAqLw0KPiA+ICsJaWYgKGludGVsX3dhaXRfZm9yX3JlZ2lz dGVyKGRldl9wcml2LCBwc3Jfc3RhdHVzLA0KPiA+IHBzcl9zdGF0dXNfbWFzaywgMCwNCj4gPiAr CQkJCSAgICAyMDAwKSkNCj4gPiArCQlEUk1fRVJST1IoIlRpbWVkIG91dCB3YWl0aW5nIGZvciBQ U1IgSWRsZSBTdGF0ZVxuIik7DQo+ID4gKw0KPiA+ICsJZGV2X3ByaXYtPnBzci5hY3RpdmUgPSBm YWxzZTsNCj4gPiAgfQ0KPiA+ICANCj4gPiAgc3RhdGljIHZvaWQgaW50ZWxfcHNyX2Rpc2FibGVf bG9ja2VkKHN0cnVjdCBpbnRlbF9kcCAqaW50ZWxfZHApDQo+IA0KPiANCj4gDQpfX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBs aXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVz a3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK