From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Lisovskiy, Stanislav" Subject: Re: [PATCH v13 2/2] drm/i915: Allow "max bpc" property to limit pipe_bpp Date: Fri, 12 Oct 2018 11:13:11 +0000 Message-ID: <26122b7884fa3b008e636b82877a25d49040a5c8.camel@intel.com> References: <20181012072532.25260-1-radhakrishna.sripada@intel.com> <20181012072532.25260-2-radhakrishna.sripada@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20181012072532.25260-2-radhakrishna.sripada@intel.com> Content-Language: en-US Content-ID: <5E6C5E78C7D40C4186B5B5C8CE705A86@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: "Sripada, Radhakrishna" , "intel-gfx@lists.freedesktop.org" Cc: "daniel.vetter@ffwll.ch" , "Kadiyala, Kishore" , "dri-devel@lists.freedesktop.org" , "Vivi, Rodrigo" List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCAyMDE4LTEwLTEyIGF0IDAwOjI1IC0wNzAwLCBSYWRoYWtyaXNobmEgU3JpcGFkYSB3 cm90ZToNCj4gVXNlIHRoZSBuZXdseSBhZGRlZCAibWF4IGJwYyIgY29ubmVjdG9yIHByb3BlcnR5 IHRvIGxpbWl0IHBpcGUgYnBwLg0KPiANCj4gVjM6IFVzZSBkcm1fY29ubmVjdG9yX3N0YXRlIHRv IGFjY2VzcyB0aGUgIm1heCBicGMiIHByb3BlcnR5DQo+IFY0OiBJbml0aWFsaXplIHRoZSBkcm0g cHJvcGVydHksIGFkZCBzdXVwb3J0IHRvIERQKFZpbGxlKQ0KPiBWNTogVXNlIHRoZSBwcm9wZXJ0 eSBpbiB0aGUgY29ubmVjdG9yIGFuZCBmaXggQ0kgZmFpbHVyZShWaWxsZSkNCj4gVjY6IFVzZSB0 aGUgY29yZSBmdW5jdGlvbiB0byBhdHRhY2ggbWF4X2JwYyBwcm9wZXJ0eSwgcmVtb3ZlIHRoZQ0K PiByZWR1bmRhbnQNCj4gICAgIGNsYW1waW5nIG9mIHBpcGUgYnBwIGJhc2VkIG9uIGNvbm5lY3Rv ciBpbmZvDQo+IFY3OiBGaXggQ2hlY2twYXRjaCB3YXJuaW5ncw0KPiBWOTogQ2xlYW51cCBjb25u ZWN0ZWRfc2lua19tYXhfYnBwIGFuZCBmaXggaW5pdGlhbCB2YWx1ZSBpbiBEUChWaWxsZSkNCj4g VjEyOiBGaXggZGVidWcgbWVzc2FnZShWaWxsZSkNCj4gVjEzOiBSZW1vdmUgdGhlIHJlZHVuZGFu dCBjaGVjayBhbmQgc2ltcGxpZnkgdGhlIGNoZWNrIGxvZ2ljKFN0YW4pDQo+IA0KPiBDYzogVmls bGUgU3lyasOkbMOkIDx2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4NCj4gQ2M6IERhbmll bCBWZXR0ZXIgPGRhbmllbC52ZXR0ZXJAZmZ3bGwuY2g+DQo+IENjOiBSb2RyaWdvIFZpdmkgPHJv ZHJpZ28udml2aUBpbnRlbC5jb20+DQo+IENjOiBLaXNob3JlIEthZGl5YWxhIDxraXNob3JlLmth ZGl5YWxhQGludGVsLmNvbT4NCj4gQ2M6IE1hbmFzaSBOYXZhcmUgPG1hbmFzaS5kLm5hdmFyZUBp bnRlbC5jb20+DQo+IENjOiBTdGFuaXNsYXYgTGlzb3Zza2l5IDxzdGFuaXNsYXYubGlzb3Zza2l5 QGludGVsLmNvbT4NCj4gU2lnbmVkLW9mZi1ieTogUmFkaGFrcmlzaG5hIFNyaXBhZGEgPHJhZGhh a3Jpc2huYS5zcmlwYWRhQGludGVsLmNvbT4NCj4gLS0tDQo+ICBkcml2ZXJzL2dwdS9kcm0vaTkx NS9pbnRlbF9kaXNwbGF5LmMgfCA0OSArKysrKysrKysrKysrKysrKysrKy0tLS0NCj4gLS0tLS0t LS0tLS0tDQo+ICBkcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcC5jICAgICAgfCAgNCArKysN Cj4gIGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2hkbWkuYyAgICB8ICA1ICsrKysNCj4gIDMg ZmlsZXMgY2hhbmdlZCwgMzcgaW5zZXJ0aW9ucygrKSwgMjEgZGVsZXRpb25zKC0pDQo+IA0KPiBk aWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jDQo+IGIvZHJp dmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jDQo+IGluZGV4IDk4MGY0ZWE2OGU0OC4u MGYxZjQ2YWJlMmY1IDEwMDY0NA0KPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9k aXNwbGF5LmMNCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jDQo+ IEBAIC0xMDc5MSwzMCArMTA3OTEsMzcgQEAgc3RhdGljIHZvaWQNCj4gaW50ZWxfbW9kZXNldF91 cGRhdGVfY29ubmVjdG9yX2F0b21pY19zdGF0ZShzdHJ1Y3QgZHJtX2RldmljZSAqZGV2KQ0KPiAg CWRybV9jb25uZWN0b3JfbGlzdF9pdGVyX2VuZCgmY29ubl9pdGVyKTsNCj4gIH0NCj4gIA0KPiAt c3RhdGljIHZvaWQNCj4gLWNvbm5lY3RlZF9zaW5rX2NvbXB1dGVfYnBwKHN0cnVjdCBpbnRlbF9j b25uZWN0b3IgKmNvbm5lY3RvciwNCj4gLQkJCSAgIHN0cnVjdCBpbnRlbF9jcnRjX3N0YXRlICpw aXBlX2NvbmZpZykNCj4gK3N0YXRpYyBpbnQNCj4gK2Nvbm5lY3RlZF9zaW5rX21heF9icHAoY29u c3Qgc3RydWN0IGRybV9jb25uZWN0b3Jfc3RhdGUgKmNvbm5fc3RhdGUsDQo+ICsJCSAgICAgICBz dHJ1Y3QgaW50ZWxfY3J0Y19zdGF0ZSAqcGlwZV9jb25maWcpDQo+ICB7DQo+IC0JY29uc3Qgc3Ry dWN0IGRybV9kaXNwbGF5X2luZm8gKmluZm8gPSAmY29ubmVjdG9yLQ0KPiA+YmFzZS5kaXNwbGF5 X2luZm87DQo+IC0JaW50IGJwcCA9IHBpcGVfY29uZmlnLT5waXBlX2JwcDsNCj4gKwlpbnQgYnBw Ow0KPiArCXN0cnVjdCBkcm1fZGlzcGxheV9pbmZvICppbmZvID0gJmNvbm5fc3RhdGUtPmNvbm5l Y3Rvci0NCj4gPmRpc3BsYXlfaW5mbzsNCj4gIA0KPiAtCURSTV9ERUJVR19LTVMoIltDT05ORUNU T1I6JWQ6JXNdIGNoZWNraW5nIGZvciBzaW5rIGJwcA0KPiBjb25zdHJhaW5zXG4iLA0KPiAtCQkg ICAgICBjb25uZWN0b3ItPmJhc2UuYmFzZS5pZCwNCj4gLQkJICAgICAgY29ubmVjdG9yLT5iYXNl Lm5hbWUpOw0KPiAtDQo+IC0JLyogRG9uJ3QgdXNlIGFuIGludmFsaWQgRURJRCBicGMgdmFsdWUg Ki8NCj4gLQlpZiAoaW5mby0+YnBjICE9IDAgJiYgaW5mby0+YnBjICogMyA8IGJwcCkgew0KPiAt CQlEUk1fREVCVUdfS01TKCJjbGFtcGluZyBkaXNwbGF5IGJwcCAod2FzICVkKSB0byBFRElEDQo+ IHJlcG9ydGVkIG1heCBvZiAlZFxuIiwNCj4gLQkJCSAgICAgIGJwcCwgaW5mby0+YnBjICogMyk7 DQo+IC0JCXBpcGVfY29uZmlnLT5waXBlX2JwcCA9IGluZm8tPmJwYyAqIDM7DQo+ICsJc3dpdGNo IChjb25uX3N0YXRlLT5tYXhfYnBjKSB7DQo+ICsJY2FzZSA2IC4uLiA3Og0KPiArCQlicHAgPSA2 ICogMzsNCj4gKwljYXNlIDggLi4uIDk6DQo+ICsJCWJwcCA9IDggKiAzOw0KPiArCQlicmVhazsN Cj4gKwljYXNlIDEwIC4uLiAxMToNCj4gKwkJYnBwID0gMTAgKiAzOw0KPiArCQlicmVhazsNCj4g KwljYXNlIDEyOg0KPiArCQlicHAgPSAxMiAqIDM7DQo+ICsJCWJyZWFrOw0KPiArCWRlZmF1bHQ6 DQo+ICsJCXJldHVybiAtRUlOVkFMOw0KPiAgCX0NCj4gIA0KPiAtCS8qIENsYW1wIGJwcCB0byA4 IG9uIHNjcmVlbnMgd2l0aG91dCBFRElEIDEuNCAqLw0KPiAtCWlmIChpbmZvLT5icGMgPT0gMCAm JiBicHAgPiAyNCkgew0KPiAtCQlEUk1fREVCVUdfS01TKCJjbGFtcGluZyBkaXNwbGF5IGJwcCAo d2FzICVkKSB0bw0KPiBkZWZhdWx0IGxpbWl0IG9mIDI0XG4iLA0KPiAtCQkJICAgICAgYnBwKTsN Cj4gLQkJcGlwZV9jb25maWctPnBpcGVfYnBwID0gMjQ7DQo+ICsJaWYgKGJwcCAhPSBwaXBlX2Nv bmZpZy0+cGlwZV9icHApIHsNCj4gKwkJRFJNX0RFQlVHX0tNUygiTGltaXRpbmcgZGlzcGxheSBi cHAgdG8gJWQgaW5zdGVhZCBvZg0KPiBFZGlkIGJwcCAiDQo+ICsJCQkgICAgICAiJWQsIHJlcXVl c3RlZCBicHAgJWQsIG1heCBwbGF0Zm9ybQ0KPiBicHAgJWRcbiIsIGJwcCwNCj4gKwkJCSAgICAg IDMgKiBpbmZvLT5icGMsIDMgKiBjb25uX3N0YXRlLQ0KPiA+bWF4X3JlcXVlc3RlZF9icGMsDQo+ ICsJCQkgICAgICBwaXBlX2NvbmZpZy0+cGlwZV9icHApOw0KPiArCQlwaXBlX2NvbmZpZy0+cGlw ZV9icHAgPSBicHA7DQo+ICAJfQ0KPiArCXJldHVybiAwOw0KDQpUaGF0IGlzIGRlZmluaXRlbHkg c2ltcGxpZXIuIEhvd2V2ZXIgSSB0aGluayB0aGVyZSBpcyBzdGlsbCBvbmUgdGhpbmc6DQp5b3Ug bm93IGFzc2lnbiBwaXBlX2JwcCB0byBicHAgYWx3YXlzIGlmIHRoZXkgYXJlIG5vdCBlcXVhbC4g SG93ZXZlcg0KYnBwKHdoaWNoIGlzIGxpdGVyYWxseSBtYXhfYnBjKSBpcyBzdXBwb3NlZCB0byBs aW1pdCBwaXBlX2JwcCBmcm9tDQphYm92ZSwgbm90IGZyb20gYmVsb3cuIFdoYXQgaWYgcGlwZV9i cHAgPCBicHAgYWxyZWFkeSwgdGhlbiBjb25kaXRpb24NCndpbGwgc3RpbGwgYmUgZXhlY3V0ZWQg YW5kIHRoZW4geW91J2xsIGdldCBwaXBlX2JwcCBzZXQgdG8gaGlnaGVyIHZhbHVlDQp0aGFuIGl0 IHdhcyBzdXBwb3NlZCB0by4NClNvIEkgZ3Vlc3MgdGhlIGNvbmRpdGlvbiBzaG91bGQgYmUgImlm IChicHAgPCBwaXBlX2NvbmZpZy0+cGlwZV9icHApIg0Kb3Igc2ltcGx5IGRvIHBpcGVfY29uZmln LT5waXBlX2JwcCA9IG1pbihwaXBlX2NvbmZpZy0+cGlwZV9icHAsIGJwcCkgYXMNCkkgc2FpZCBp biBwcmV2aW91cyBlbWFpbC4NCg0KPiAgfQ0KPiAgDQo+ICBzdGF0aWMgaW50DQo+IEBAIC0xMDg0 NSw4ICsxMDg1Miw4IEBAIGNvbXB1dGVfYmFzZWxpbmVfcGlwZV9icHAoc3RydWN0IGludGVsX2Ny dGMNCj4gKmNydGMsDQo+ICAJCWlmIChjb25uZWN0b3Jfc3RhdGUtPmNydGMgIT0gJmNydGMtPmJh c2UpDQo+ICAJCQljb250aW51ZTsNCj4gIA0KPiAtCQljb25uZWN0ZWRfc2lua19jb21wdXRlX2Jw cCh0b19pbnRlbF9jb25uZWN0b3IoY29ubmVjDQo+IHRvciksDQo+IC0JCQkJCSAgIHBpcGVfY29u ZmlnKTsNCj4gKwkJaWYgKGNvbm5lY3RlZF9zaW5rX21heF9icHAoY29ubmVjdG9yX3N0YXRlLA0K PiBwaXBlX2NvbmZpZykgPCAwKQ0KPiArCQkJcmV0dXJuIC1FSU5WQUw7DQo+ICAJfQ0KPiAgDQo+ ICAJcmV0dXJuIGJwcDsNCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVs X2RwLmMNCj4gYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcC5jDQo+IGluZGV4IDEzZmY4 OWJlNmFkNi4uNWY1OTQ0MzBiMmE0IDEwMDY0NA0KPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkx NS9pbnRlbF9kcC5jDQo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2RwLmMNCj4g QEAgLTU3MzUsNiArNTczNSwxMCBAQCBpbnRlbF9kcF9hZGRfcHJvcGVydGllcyhzdHJ1Y3QgaW50 ZWxfZHANCj4gKmludGVsX2RwLCBzdHJ1Y3QgZHJtX2Nvbm5lY3RvciAqY29ubmVjdA0KPiAgCQlp bnRlbF9hdHRhY2hfZm9yY2VfYXVkaW9fcHJvcGVydHkoY29ubmVjdG9yKTsNCj4gIA0KPiAgCWlu dGVsX2F0dGFjaF9icm9hZGNhc3RfcmdiX3Byb3BlcnR5KGNvbm5lY3Rvcik7DQo+ICsJaWYgKEhB U19HTUNIX0RJU1BMQVkoZGV2X3ByaXYpKQ0KPiArCQlkcm1fY29ubmVjdG9yX2F0dGFjaF9tYXhf YnBjX3Byb3BlcnR5KGNvbm5lY3RvciwgNiwNCj4gMTApOw0KPiArCWVsc2UgaWYgKElOVEVMX0dF TihkZXZfcHJpdikgPj0gNSkNCj4gKwkJZHJtX2Nvbm5lY3Rvcl9hdHRhY2hfbWF4X2JwY19wcm9w ZXJ0eShjb25uZWN0b3IsIDYsDQo+IDEyKTsNCj4gIA0KPiAgCWlmIChpbnRlbF9kcF9pc19lZHAo aW50ZWxfZHApKSB7DQo+ICAJCXUzMiBhbGxvd2VkX3NjYWxlcnM7DQo+IGRpZmYgLS1naXQgYS9k cml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9oZG1pLmMNCj4gYi9kcml2ZXJzL2dwdS9kcm0vaTkx NS9pbnRlbF9oZG1pLmMNCj4gaW5kZXggMmM1M2VmYzQ2M2U2Li4zMTU4YWIwODVhMzAgMTAwNjQ0 DQo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2hkbWkuYw0KPiArKysgYi9kcml2 ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9oZG1pLmMNCj4gQEAgLTIxMDMsMTEgKzIxMDMsMTYgQEAg c3RhdGljIGNvbnN0IHN0cnVjdCBkcm1fZW5jb2Rlcl9mdW5jcw0KPiBpbnRlbF9oZG1pX2VuY19m dW5jcyA9IHsNCj4gIHN0YXRpYyB2b2lkDQo+ICBpbnRlbF9oZG1pX2FkZF9wcm9wZXJ0aWVzKHN0 cnVjdCBpbnRlbF9oZG1pICppbnRlbF9oZG1pLCBzdHJ1Y3QNCj4gZHJtX2Nvbm5lY3RvciAqY29u bmVjdG9yKQ0KPiAgew0KPiArCXN0cnVjdCBkcm1faTkxNV9wcml2YXRlICpkZXZfcHJpdiA9IHRv X2k5MTUoY29ubmVjdG9yLT5kZXYpOw0KPiArDQo+ICAJaW50ZWxfYXR0YWNoX2ZvcmNlX2F1ZGlv X3Byb3BlcnR5KGNvbm5lY3Rvcik7DQo+ICAJaW50ZWxfYXR0YWNoX2Jyb2FkY2FzdF9yZ2JfcHJv cGVydHkoY29ubmVjdG9yKTsNCj4gIAlpbnRlbF9hdHRhY2hfYXNwZWN0X3JhdGlvX3Byb3BlcnR5 KGNvbm5lY3Rvcik7DQo+ICAJZHJtX2Nvbm5lY3Rvcl9hdHRhY2hfY29udGVudF90eXBlX3Byb3Bl cnR5KGNvbm5lY3Rvcik7DQo+ICAJY29ubmVjdG9yLT5zdGF0ZS0+cGljdHVyZV9hc3BlY3RfcmF0 aW8gPQ0KPiBIRE1JX1BJQ1RVUkVfQVNQRUNUX05PTkU7DQo+ICsNCj4gKwlpZiAoIUhBU19HTUNI X0RJU1BMQVkoZGV2X3ByaXYpKQ0KPiArCQlkcm1fY29ubmVjdG9yX2F0dGFjaF9tYXhfYnBjX3By b3BlcnR5KGNvbm5lY3RvciwgOCwNCj4gMTIpOw0KPiAgfQ0KPiAgDQo+ICAvKg0KLS0gDQpCZXN0 IFJlZ2FyZHMsDQoNCkxpc292c2tpeSBTdGFuaXNsYXYKX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==