From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Souza, Jose" Subject: Re: [PATCH 04/12] drm/i915/psr: Tie PSR2 support to Y coordinate requirement Date: Thu, 22 Mar 2018 23:16:33 +0000 Message-ID: <1521760406.10092.3.camel@intel.com> References: <20180322214848.28022-1-jose.souza@intel.com> <20180322214848.28022-4-jose.souza@intel.com> <20180322230908.GK2557@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id 668B36E233 for ; Thu, 22 Mar 2018 23:16:35 +0000 (UTC) In-Reply-To: <20180322230908.GK2557@intel.com> Content-Language: en-US Content-ID: <378B62F174A1324DB7C48B34849C713F@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: "Vivi, Rodrigo" Cc: "intel-gfx@lists.freedesktop.org" , "Pandiyan, Dhinakaran" List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCAyMDE4LTAzLTIyIGF0IDE2OjA5IC0wNzAwLCBSb2RyaWdvIFZpdmkgd3JvdGU6DQo+ IE9uIFRodSwgTWFyIDIyLCAyMDE4IGF0IDAyOjQ4OjQwUE0gLTA3MDAsIEpvc8OpIFJvYmVydG8g ZGUgU291emENCj4gd3JvdGU6DQo+ID4gTW92ZSB0byBvbmx5IG9uZSBwbGFjZSB0aGUgc2luayBy ZXF1aXJlbWVudHMgdGhhdCB0aGUgYWN0dWFsIGRyaXZlcg0KPiA+IG5lZWRzIHRvIGVuYWJsZSBQ U1IyLg0KPiA+IA0KPiA+IEFsc28gaW50ZWxfcHNyMl9jb25maWdfdmFsaWQoKSBpcyBjYWxsZWQg ZXZlcnkgdGltZSB0aGUgY3J0YyBjb25maWcNCj4gPiBpcyBjb21wdXRlZCwgd2FzdGluZyBzb21l IHRpbWUgZXZlcnkgdGltZSBpdCB3YXMgY2hlY2tpbmcgZm9yDQo+ID4gWSBjb29yZGluYXRlIHJl cXVpcmVtZW50Lg0KPiA+IA0KPiA+IFRoaXMgYWxsb3cgdXMgdG8gbnVrZSB5X2NvcmRfc3VwcG9y dCBhbmQgc29tZSBvZiBWU0Mgc2V0dXAgY29kZQ0KPiA+IHRoYXQNCj4gPiB3YXMgaGFuZGxpbmcg YSBzY2VuYXJpbyB0aGF0IHdvdWxkIG5ldmVyIGhhcHBlbihQU1IyIHdpdGhvdXQgWQ0KPiA+IGNv b3JkaW5hdGUpLg0KPiA+IA0KPiA+IENjOiBEaGluYWthcmFuIFBhbmRpeWFuIDxkaGluYWthcmFu LnBhbmRpeWFuQGludGVsLmNvbT4NCj4gPiBDYzogUm9kcmlnbyBWaXZpIDxyb2RyaWdvLnZpdmlA aW50ZWwuY29tPg0KPiA+IFNpZ25lZC1vZmYtYnk6IEpvc8OpIFJvYmVydG8gZGUgU291emEgPGpv c2Uuc291emFAaW50ZWwuY29tPg0KPiA+IC0tLQ0KPiA+ICBkcml2ZXJzL2dwdS9kcm0vaTkxNS9p OTE1X2Rydi5oICB8ICAxIC0NCj4gPiAgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcHNyLmMg fCA0NiArKysrKysrKysrKysrKysrKy0tLS0tLS0tLS0tDQo+ID4gLS0tLS0tLS0tLS0tDQo+ID4g IDIgZmlsZXMgY2hhbmdlZCwgMTkgaW5zZXJ0aW9ucygrKSwgMjggZGVsZXRpb25zKC0pDQo+ID4g DQo+ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2LmgNCj4gPiBi L2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2LmgNCj4gPiBpbmRleCA3ZmUwMDUwOWU1MWEu LmNjZTMyNjg2ZmQ3NSAxMDA2NDQNCj4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1 X2Rydi5oDQo+ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9kcnYuaA0KPiA+IEBA IC02MDMsNyArNjAzLDYgQEAgc3RydWN0IGk5MTVfcHNyIHsNCj4gPiAgCXVuc2lnbmVkIGJ1c3lf ZnJvbnRidWZmZXJfYml0czsNCj4gPiAgCWJvb2wgcHNyMl9zdXBwb3J0Ow0KPiA+ICAJYm9vbCBs aW5rX3N0YW5kYnk7DQo+ID4gLQlib29sIHlfY29yZF9zdXBwb3J0Ow0KPiA+ICAJYm9vbCBjb2xv cmltZXRyeV9zdXBwb3J0Ow0KPiA+ICAJYm9vbCBhbHBtOw0KPiA+ICAJYm9vbCBoYXNfaHdfdHJh Y2tpbmc7DQo+ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3Bzci5j DQo+ID4gYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9wc3IuYw0KPiA+IGluZGV4IGQ0NjMy MGE0NTFkOS4uMjNmMzhhYjEwNjM2IDEwMDY0NA0KPiA+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9p OTE1L2ludGVsX3Bzci5jDQo+ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcHNy LmMNCj4gPiBAQCAtOTMsNyArOTMsNyBAQCBzdGF0aWMgdm9pZCBwc3JfYXV4X2lvX3Bvd2VyX3B1 dChzdHJ1Y3QgaW50ZWxfZHANCj4gPiAqaW50ZWxfZHApDQo+ID4gIAlpbnRlbF9kaXNwbGF5X3Bv d2VyX3B1dChkZXZfcHJpdiwNCj4gPiBwc3JfYXV4X2RvbWFpbihpbnRlbF9kcCkpOw0KPiA+ICB9 DQo+ID4gIA0KPiA+IC1zdGF0aWMgYm9vbCBpbnRlbF9kcF9nZXRfeV9jb3JkX3N0YXR1cyhzdHJ1 Y3QgaW50ZWxfZHAgKmludGVsX2RwKQ0KPiA+ICtzdGF0aWMgYm9vbCBpbnRlbF9kcF9nZXRfeV9j b29yZF9yZXF1aXJlZChzdHJ1Y3QgaW50ZWxfZHANCj4gPiAqaW50ZWxfZHApDQo+IA0KPiBuaXA6 IEkgZG9uJ3QgYWdyZWUgdGhhdCByZXF1aXJlZCBpcyBhIGdvb2QgbmFtZSBmb3IgdGhpcyBmdW5j dGlvbg0KPiBjYWxsDQo+IHNvIG1heWJlIHdlIHNob3VsZCBqdXN0IGxlYXZlIHN0YXR1cyBhbmQg bWluaW1pemUgdGhlIGNoYW5nZS4NCg0KVGhpcyBpcyB0aGUgbmFtZSBpbiB0aGUgZURQIHNwZWM6 IFktY29vcmRpbmF0ZSBSZXF1aXJlbWVudCBvZiBTaW5rDQpEZXZpY2Ugb24gUFNSMiBTZWxlY3Rp dmUgVXBkYXRlDQoNCj4gDQo+ID4gIHsNCj4gPiAgCXVpbnQ4X3QgcHNyX2NhcHMgPSAwOw0KPiA+ ICANCj4gPiBAQCAtMTMwLDIyICsxMzAsMjkgQEAgdm9pZCBpbnRlbF9wc3JfaW5pdF9kcGNkKHN0 cnVjdCBpbnRlbF9kcA0KPiA+ICppbnRlbF9kcCkNCj4gPiAgCWRybV9kcF9kcGNkX3JlYWQoJmlu dGVsX2RwLT5hdXgsIERQX1BTUl9TVVBQT1JULCBpbnRlbF9kcC0NCj4gPiA+cHNyX2RwY2QsDQo+ ID4gIAkJCSBzaXplb2YoaW50ZWxfZHAtPnBzcl9kcGNkKSk7DQo+ID4gIA0KPiA+IC0JaWYgKGlu dGVsX2RwLT5wc3JfZHBjZFswXSAmIERQX1BTUl9JU19TVVBQT1JURUQpIHsNCj4gPiArCWlmIChp bnRlbF9kcC0+cHNyX2RwY2RbMF0pIHsNCj4gDQo+IGhtPyB3aHk/DQoNCkEgcGFubmVsIHRoYXQg aXMgUFNSIHZlcnNpb24gMiwgd2lsbCBub3QgaGF2ZSBhbnkgUFNSIGVuYWJsZWQgYXMNCidpbnRl bF9kcC0+cHNyX2RwY2RbMF0gJiBEUF9QU1JfSVNfU1VQUE9SVEVEJyBhbmQgJ2ludGVsX2RwLQ0K PnBzcl9kcGNkWzBdID09IERQX1BTUjJfV0lUSF9ZX0NPT1JEX0lTX1NVUFBPUlRFRCcgd2lsbCBi ZSBmYWxzZS4NCg0KPiANCj4gPiAgCQlkZXZfcHJpdi0+cHNyLnNpbmtfc3VwcG9ydCA9IHRydWU7 DQo+ID4gIAkJRFJNX0RFQlVHX0tNUygiRGV0ZWN0ZWQgRURQIFBTUiBQYW5lbC5cbiIpOw0KPiA+ ICAJfQ0KPiA+ICANCj4gPiAgCWlmIChJTlRFTF9HRU4oZGV2X3ByaXYpID49IDkgJiYNCj4gPiAt CSAgICAoaW50ZWxfZHAtPnBzcl9kcGNkWzBdICYgRFBfUFNSMl9JU19TVVBQT1JURUQpKSB7DQo+ ID4gLQ0KPiA+IC0JCWRldl9wcml2LT5wc3Iuc2lua19zdXBwb3J0ID0gdHJ1ZTsNCj4gPiAtCQlk ZXZfcHJpdi0+cHNyLnBzcjJfc3VwcG9ydCA9IHRydWU7DQo+ID4gKwkgICAgKGludGVsX2RwLT5w c3JfZHBjZFswXSA9PQ0KPiA+IERQX1BTUjJfV0lUSF9ZX0NPT1JEX0lTX1NVUFBPUlRFRCkpIHsN Cj4gPiArCQkvKg0KPiA+ICsJCSAqIEFsbCBwYW5lbHMgdGhhdCBzdXBwb3J0cyBQU1IgdmVyc2lv biAwM2ggKFBTUjINCj4gPiArDQo+ID4gKwkJICogWS1jb29yZGluYXRlKSBjYW4gaGFuZGxlIFkt Y29vcmRpbmF0ZXMgaW4gVlNDDQo+ID4gYnV0IHdlIGFyZQ0KPiA+ICsJCSAqIG9ubHkgc3VyZSB0 aGF0IGl0IGlzIGdvaW5nIHRvIGJlIHVzZWQgd2hlbg0KPiA+IHJlcXVpcmVkIGJ5IHRoZQ0KPiA+ ICsJCSAqIHBhbmVsLiBUaGlzIHdheSBwYW5lbCBpcyBjYXBhYmxlIHRvIGRvDQo+ID4gc2VsZWN0 aXZlIHVwZGF0ZQ0KPiA+ICsJCSAqIHdpdGhvdXQgYSBhdXggZnJhbWUgc3luYy4NCj4gPiArCQkg Kg0KPiA+ICsJCSAqIFRvIHN1cHBvcnQgUFNSIHZlcnNpb24gMDJoIGFuZCBQU1IgdmVyc2lvbiAw M2gNCj4gPiB3aXRob3V0DQo+ID4gKwkJICogWS1jb29yZGluYXRlIHJlcXVpcmVtZW50IHBhbmVs cyB3ZSB3b3VsZCBuZWVkDQo+ID4gdG8gZW5hYmxlDQo+ID4gKwkJICogR1RDIGZpcnN0Lg0KPiA+ ICsJCSAqLw0KPiA+ICsJCWRldl9wcml2LT5wc3IucHNyMl9zdXBwb3J0ID0NCj4gPiBpbnRlbF9k cF9nZXRfeV9jb29yZF9yZXF1aXJlZChpbnRlbF9kcCk7DQo+IA0KPiBvaCBvayBvay4uLiBub3cg SSBzYXcgd2h5IHlvdSBjaGFuZ2VkIHRvICJyZXF1aXJlZCINCj4gDQo+IEJ1dCBteSBxdWVzdGlv biBub3cgaXMuIERvIHdlIHJlYWxseSBuZWVkIHRvIGNoZWNrIHRoaXMgYml0Pw0KPiANCj4gSSBi ZWxpZXZlIHdpdGggeW91ciBjaGFuZ2UgdG8gY2hlY2sgdmVyc2lvbiAwM2ggd2UgZG9uJ3QgbmVl ZCB0bw0KPiBjaGVjaw0KPiB0aGlzIHJlcXVpcmVtZW50IGFueW1vcmUuDQo+IFRoaXMgbG9va3Mg bGlrZSBpbiB0aGUgcGFzdCB3ZSB1c2VkIHRoYXQgdG8gd29ya2Fyb3VuZCBhIHdheSB0bw0KPiBp ZGVudGlmeQ0KPiBpZiB0aGUgcGFuZWwgd2FzIHJlYWxseSB5LWNvb3JkLi4uIGJ1dCBub3cgd2l0 aCAwM2ggdmVyc2lvbiB3ZSBkb24ndA0KPiBuZWVkDQo+IHRvIGNoZWNrIGlmIHBhbmVsIGlzIHJl cXVpcmluZyB0aGUgeS1jb29yZGluYXRpb24gZnJvbSB0aGUgc291cmNlLA0KPiBiZWNhdXNlDQo+ IHdlIGtub3cgd2hhdCBzb3VyY2UgaXMgY2FwYWJsZSBvZi4NCg0KV2Uga25vdyB0aGF0IGlzIGNh cGFibGUgb2YgYnV0IHdlIGRvbid0IGtub3cgaWYgaXMgZ29pbmcgdG8gYmUgdXNlZCBieQ0KdGhl IHBhbm5lbCwgd2hlbiBwYW5uZWxzIHJlcXVpcmUgdGhlIHktY29vcmRpbmF0ZSB3ZSBhcmUgMTAw JSBzdXJlIHRoYXQNCml0IGlzIGdvaW5nIHRvIHVzZS4NCg0KPiANCj4gPiAgCQlEUk1fREVCVUdf S01TKCJQU1IyICVzIG9uIHNpbmsiLA0KPiA+ICAJCQkgICAgICBkZXZfcHJpdi0+cHNyLnBzcjJf c3VwcG9ydCA/DQo+ID4gInN1cHBvcnRlZCIgOiAibm90IHN1cHBvcnRlZCIpOw0KPiA+ICANCj4g PiAgCQlpZiAoZGV2X3ByaXYtPnBzci5wc3IyX3N1cHBvcnQpIHsNCj4gPiAtCQkJZGV2X3ByaXYt PnBzci55X2NvcmRfc3VwcG9ydCA9DQo+ID4gLQkJCQlpbnRlbF9kcF9nZXRfeV9jb3JkX3N0YXR1 cyhpbnRlbF9kDQo+ID4gcCk7DQo+ID4gIAkJCWRldl9wcml2LT5wc3IuY29sb3JpbWV0cnlfc3Vw cG9ydCA9DQo+ID4gIAkJCQlpbnRlbF9kcF9nZXRfY29sb3JpbWV0cnlfc3RhdHVzKGluDQo+ID4g dGVsX2RwKTsNCj4gPiAgCQkJZGV2X3ByaXYtPnBzci5hbHBtID0NCj4gPiBAQCAtMTkxLDE2ICsx OTgsMTIgQEAgc3RhdGljIHZvaWQgaHN3X3Bzcl9zZXR1cF92c2Moc3RydWN0IGludGVsX2RwDQo+ ID4gKmludGVsX2RwLA0KPiA+ICAJCW1lbXNldCgmcHNyX3ZzYywgMCwgc2l6ZW9mKHBzcl92c2Mp KTsNCj4gPiAgCQlwc3JfdnNjLnNkcF9oZWFkZXIuSEIwID0gMDsNCj4gPiAgCQlwc3JfdnNjLnNk cF9oZWFkZXIuSEIxID0gMHg3Ow0KPiA+IC0JCWlmIChkZXZfcHJpdi0+cHNyLmNvbG9yaW1ldHJ5 X3N1cHBvcnQgJiYNCj4gPiAtCQkgICAgZGV2X3ByaXYtPnBzci55X2NvcmRfc3VwcG9ydCkgew0K PiA+ICsJCWlmIChkZXZfcHJpdi0+cHNyLmNvbG9yaW1ldHJ5X3N1cHBvcnQpIHsNCj4gPiAgCQkJ cHNyX3ZzYy5zZHBfaGVhZGVyLkhCMiA9IDB4NTsNCj4gPiAgCQkJcHNyX3ZzYy5zZHBfaGVhZGVy LkhCMyA9IDB4MTM7DQo+ID4gLQkJfSBlbHNlIGlmIChkZXZfcHJpdi0+cHNyLnlfY29yZF9zdXBw b3J0KSB7DQo+ID4gKwkJfSBlbHNlIHsNCj4gPiAgCQkJcHNyX3ZzYy5zZHBfaGVhZGVyLkhCMiA9 IDB4NDsNCj4gPiAgCQkJcHNyX3ZzYy5zZHBfaGVhZGVyLkhCMyA9IDB4ZTsNCj4gPiAtCQl9IGVs c2Ugew0KPiA+IC0JCQlwc3JfdnNjLnNkcF9oZWFkZXIuSEIyID0gMHgzOw0KPiA+IC0JCQlwc3Jf dnNjLnNkcF9oZWFkZXIuSEIzID0gMHhjOw0KPiA+ICAJCX0NCj4gPiAgCX0gZWxzZSB7DQo+ID4g IAkJLyogUHJlcGFyZSBWU0MgcGFja2V0IGFzIHBlciBFRFAgMS4zIHNwZWMsIFRhYmxlDQo+ID4g My4xMCAqLw0KPiA+IEBAIC00NTgsMTUgKzQ2MSw2IEBAIHN0YXRpYyBib29sIGludGVsX3BzcjJf Y29uZmlnX3ZhbGlkKHN0cnVjdA0KPiA+IGludGVsX2RwICppbnRlbF9kcCwNCj4gPiAgCQlyZXR1 cm4gZmFsc2U7DQo+ID4gIAl9DQo+ID4gIA0KPiA+IC0JLyoNCj4gPiAtCSAqIEZJWE1FOmVuYWJs ZSBwc3IyIG9ubHkgZm9yIHktY29yZGluYXRlIHBzcjIgcGFuZWxzDQo+ID4gLQkgKiBBZnRlciBn dGMgaW1wbGVtZW50YXRpb24gLCByZW1vdmUgdGhpcyByZXN0cmljdGlvbi4NCj4gPiAtCSAqLw0K PiA+IC0JaWYgKCFkZXZfcHJpdi0+cHNyLnlfY29yZF9zdXBwb3J0KSB7DQo+ID4gLQkJRFJNX0RF QlVHX0tNUygiUFNSMiBub3QgZW5hYmxlZCwgcGFuZWwgZG9lcyBub3QNCj4gPiBzdXBwb3J0IFkg Y29vcmRpbmF0ZVxuIik7DQo+ID4gLQkJcmV0dXJuIGZhbHNlOw0KPiA+IC0JfQ0KPiA+IC0NCj4g PiAgCXJldHVybiB0cnVlOw0KPiA+ICB9DQo+ID4gIA0KPiA+IEBAIC01NjYsNyArNTYwLDYgQEAg c3RhdGljIHZvaWQgaHN3X3Bzcl9lbmFibGVfc291cmNlKHN0cnVjdA0KPiA+IGludGVsX2RwICpp bnRlbF9kcCwNCj4gPiAgCXN0cnVjdCBkcm1fZGV2aWNlICpkZXYgPSBkaWdfcG9ydC0+YmFzZS5i YXNlLmRldjsNCj4gPiAgCXN0cnVjdCBkcm1faTkxNV9wcml2YXRlICpkZXZfcHJpdiA9IHRvX2k5 MTUoZGV2KTsNCj4gPiAgCWVudW0gdHJhbnNjb2RlciBjcHVfdHJhbnNjb2RlciA9IGNydGNfc3Rh dGUtDQo+ID4gPmNwdV90cmFuc2NvZGVyOw0KPiA+IC0JdTMyIGNoaWNrZW47DQo+ID4gIA0KPiA+ ICAJcHNyX2F1eF9pb19wb3dlcl9nZXQoaW50ZWxfZHApOw0KPiA+ICANCj4gPiBAQCAtNTc3LDkg KzU3MCw4IEBAIHN0YXRpYyB2b2lkIGhzd19wc3JfZW5hYmxlX3NvdXJjZShzdHJ1Y3QNCj4gPiBp bnRlbF9kcCAqaW50ZWxfZHAsDQo+ID4gIAkJaHN3X3Bzcl9zZXR1cF9hdXgoaW50ZWxfZHApOw0K PiA+ICANCj4gPiAgCWlmIChkZXZfcHJpdi0+cHNyLnBzcjJfc3VwcG9ydCkgew0KPiA+IC0JCWNo aWNrZW4gPSBQU1IyX1ZTQ19FTkFCTEVfUFJPR19IRUFERVI7DQo+ID4gLQkJaWYgKGRldl9wcml2 LT5wc3IueV9jb3JkX3N1cHBvcnQpDQo+ID4gLQkJCWNoaWNrZW4gfD0gUFNSMl9BRERfVkVSVElD QUxfTElORV9DT1VOVDsNCj4gPiArCQl1MzIgY2hpY2tlbiA9IFBTUjJfVlNDX0VOQUJMRV9QUk9H X0hFQURFUg0KPiA+ICsJCQkgICAgICB8IFBTUjJfQUREX1ZFUlRJQ0FMX0xJTkVfQ09VTlQ7DQo+ ID4gIAkJSTkxNV9XUklURShDSElDS0VOX1RSQU5TKGNwdV90cmFuc2NvZGVyKSwNCj4gPiBjaGlj a2VuKTsNCj4gPiAgDQo+ID4gIAkJSTkxNV9XUklURShFRFBfUFNSX0RFQlVHLA0KPiA+IC0tIA0K PiA+IDIuMTYuMg0KPiA+IA0KPiA+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fDQo+ID4gSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdA0KPiA+IEludGVsLWdmeEBs aXN0cy5mcmVlZGVza3RvcC5vcmcNCj4gPiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9t YWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeApfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5m cmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0 aW5mby9pbnRlbC1nZngK